Browse Source

Fix cases where child classes override OpenShiftCLI values

Jason DeTiberus 8 years ago
parent
commit
51f8852ed1

+ 1 - 1
roles/lib_openshift/library/oadm_manage_node.py

@@ -1296,7 +1296,7 @@ class ManageNode(OpenShiftCLI):
                  config,
                  verbose=False):
         ''' Constructor for ManageNode '''
-        super(ManageNode, self).__init__(None, config.kubeconfig)
+        super(ManageNode, self).__init__(None, kubeconfig=config.kubeconfig, verbose=verbose)
         self.config = config
 
     def evacuate(self):

+ 1 - 4
roles/lib_openshift/library/oc_edit.py

@@ -1314,13 +1314,10 @@ class Edit(OpenShiftCLI):
                  separator='.',
                  verbose=False):
         ''' Constructor for OpenshiftOC '''
-        super(Edit, self).__init__(namespace, kubeconfig)
-        self.namespace = namespace
+        super(Edit, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose)
         self.kind = kind
         self.name = resource_name
-        self.kubeconfig = kubeconfig
         self.separator = separator
-        self.verbose = verbose
 
     def get(self):
         '''return a secret by name '''

+ 1 - 4
roles/lib_openshift/library/oc_env.py

@@ -1630,13 +1630,10 @@ class OCEnv(OpenShiftCLI):
                  kubeconfig='/etc/origin/master/admin.kubeconfig',
                  verbose=False):
         ''' Constructor for OpenshiftOC '''
-        super(OCEnv, self).__init__(namespace, kubeconfig)
+        super(OCEnv, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose)
         self.kind = kind
         self.name = resource_name
-        self.namespace = namespace
         self.env_vars = env_vars
-        self.kubeconfig = kubeconfig
-        self.verbose = verbose
         self._resource = None
 
     @property

+ 1 - 3
roles/lib_openshift/library/oc_label.py

@@ -1294,11 +1294,9 @@ class OCLabel(OpenShiftCLI):
                  selector=None,
                  verbose=False):
         ''' Constructor for OCLabel '''
-        super(OCLabel, self).__init__(namespace, kubeconfig)
+        super(OCLabel, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose)
         self.name = name
-        self.namespace = namespace
         self.kind = kind
-        self.kubeconfig = kubeconfig
         self.labels = labels
         self._curr_labels = None
         self.selector = selector

+ 1 - 4
roles/lib_openshift/library/oc_obj.py

@@ -1296,14 +1296,11 @@ class OCObject(OpenShiftCLI):
                  verbose=False,
                  all_namespaces=False):
         ''' Constructor for OpenshiftOC '''
-        super(OCObject, self).__init__(namespace, kubeconfig,
+        super(OCObject, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose,
                                        all_namespaces=all_namespaces)
         self.kind = kind
-        self.namespace = namespace
         self.name = rname
         self.selector = selector
-        self.kubeconfig = kubeconfig
-        self.verbose = verbose
 
     def get(self):
         '''return a kind by name '''

+ 1 - 4
roles/lib_openshift/library/oc_process.py

@@ -1286,14 +1286,11 @@ class OCProcess(OpenShiftCLI):
                  tdata=None,
                  verbose=False):
         ''' Constructor for OpenshiftOC '''
-        super(OCProcess, self).__init__(namespace, kubeconfig)
-        self.namespace = namespace
+        super(OCProcess, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose)
         self.name = tname
         self.data = tdata
         self.params = params
         self.create = create
-        self.kubeconfig = kubeconfig
-        self.verbose = verbose
         self._template = None
 
     @property

+ 1 - 2
roles/lib_openshift/library/oc_route.py

@@ -1461,9 +1461,8 @@ class OCRoute(OpenShiftCLI):
                  config,
                  verbose=False):
         ''' Constructor for OCVolume '''
-        super(OCRoute, self).__init__(config.namespace, config.kubeconfig)
+        super(OCRoute, self).__init__(config.namespace, kubeconfig=config.kubeconfig, verbose=verbose)
         self.config = config
-        self.namespace = config.namespace
         self._route = None
 
     @property

+ 1 - 4
roles/lib_openshift/library/oc_scale.py

@@ -1629,13 +1629,10 @@ class OCScale(OpenShiftCLI):
                  kubeconfig='/etc/origin/master/admin.kubeconfig',
                  verbose=False):
         ''' Constructor for OCScale '''
-        super(OCScale, self).__init__(namespace, kubeconfig)
+        super(OCScale, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose)
         self.kind = kind
         self.replicas = replicas
         self.name = resource_name
-        self.namespace = namespace
-        self.kubeconfig = kubeconfig
-        self.verbose = verbose
         self._resource = None
 
     @property

+ 1 - 4
roles/lib_openshift/library/oc_secret.py

@@ -1418,12 +1418,9 @@ class OCSecret(OpenShiftCLI):
                  kubeconfig='/etc/origin/master/admin.kubeconfig',
                  verbose=False):
         ''' Constructor for OpenshiftOC '''
-        super(OCSecret, self).__init__(namespace, kubeconfig)
-        self.namespace = namespace
+        super(OCSecret, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose)
         self.name = secret_name
-        self.kubeconfig = kubeconfig
         self.decode = decode
-        self.verbose = verbose
 
     def get(self):
         '''return a secret by name '''

+ 1 - 2
roles/lib_openshift/library/oc_serviceaccount.py

@@ -1396,9 +1396,8 @@ class OCServiceAccount(OpenShiftCLI):
                  config,
                  verbose=False):
         ''' Constructor for OCVolume '''
-        super(OCServiceAccount, self).__init__(config.namespace, config.kubeconfig)
+        super(OCServiceAccount, self).__init__(config.namespace, kubeconfig=config.kubeconfig, verbose=verbose)
         self.config = config
-        self.namespace = config.namespace
         self.service_account = None
 
     def exists(self):

+ 1 - 1
roles/lib_openshift/library/oc_serviceaccount_secret.py

@@ -1391,7 +1391,7 @@ class OCServiceAccountSecret(OpenShiftCLI):
     kind = 'sa'
     def __init__(self, config, verbose=False):
         ''' Constructor for OpenshiftOC '''
-        super(OCServiceAccountSecret, self).__init__(config.namespace, config.kubeconfig)
+        super(OCServiceAccountSecret, self).__init__(config.namespace, kubeconfig=config.kubeconfig, verbose=verbose)
         self.config = config
         self.verbose = verbose
         self._service_account = None

+ 1 - 1
roles/lib_openshift/src/class/oadm_manage_node.py

@@ -23,7 +23,7 @@ class ManageNode(OpenShiftCLI):
                  config,
                  verbose=False):
         ''' Constructor for ManageNode '''
-        super(ManageNode, self).__init__(None, config.kubeconfig)
+        super(ManageNode, self).__init__(None, kubeconfig=config.kubeconfig, verbose=verbose)
         self.config = config
 
     def evacuate(self):

+ 1 - 4
roles/lib_openshift/src/class/oc_edit.py

@@ -13,13 +13,10 @@ class Edit(OpenShiftCLI):
                  separator='.',
                  verbose=False):
         ''' Constructor for OpenshiftOC '''
-        super(Edit, self).__init__(namespace, kubeconfig)
-        self.namespace = namespace
+        super(Edit, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose)
         self.kind = kind
         self.name = resource_name
-        self.kubeconfig = kubeconfig
         self.separator = separator
-        self.verbose = verbose
 
     def get(self):
         '''return a secret by name '''

+ 1 - 4
roles/lib_openshift/src/class/oc_env.py

@@ -21,13 +21,10 @@ class OCEnv(OpenShiftCLI):
                  kubeconfig='/etc/origin/master/admin.kubeconfig',
                  verbose=False):
         ''' Constructor for OpenshiftOC '''
-        super(OCEnv, self).__init__(namespace, kubeconfig)
+        super(OCEnv, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose)
         self.kind = kind
         self.name = resource_name
-        self.namespace = namespace
         self.env_vars = env_vars
-        self.kubeconfig = kubeconfig
-        self.verbose = verbose
         self._resource = None
 
     @property

+ 1 - 3
roles/lib_openshift/src/class/oc_label.py

@@ -17,11 +17,9 @@ class OCLabel(OpenShiftCLI):
                  selector=None,
                  verbose=False):
         ''' Constructor for OCLabel '''
-        super(OCLabel, self).__init__(namespace, kubeconfig)
+        super(OCLabel, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose)
         self.name = name
-        self.namespace = namespace
         self.kind = kind
-        self.kubeconfig = kubeconfig
         self.labels = labels
         self._curr_labels = None
         self.selector = selector

+ 1 - 4
roles/lib_openshift/src/class/oc_obj.py

@@ -16,14 +16,11 @@ class OCObject(OpenShiftCLI):
                  verbose=False,
                  all_namespaces=False):
         ''' Constructor for OpenshiftOC '''
-        super(OCObject, self).__init__(namespace, kubeconfig,
+        super(OCObject, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose,
                                        all_namespaces=all_namespaces)
         self.kind = kind
-        self.namespace = namespace
         self.name = rname
         self.selector = selector
-        self.kubeconfig = kubeconfig
-        self.verbose = verbose
 
     def get(self):
         '''return a kind by name '''

+ 1 - 4
roles/lib_openshift/src/class/oc_process.py

@@ -17,14 +17,11 @@ class OCProcess(OpenShiftCLI):
                  tdata=None,
                  verbose=False):
         ''' Constructor for OpenshiftOC '''
-        super(OCProcess, self).__init__(namespace, kubeconfig)
-        self.namespace = namespace
+        super(OCProcess, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose)
         self.name = tname
         self.data = tdata
         self.params = params
         self.create = create
-        self.kubeconfig = kubeconfig
-        self.verbose = verbose
         self._template = None
 
     @property

+ 1 - 2
roles/lib_openshift/src/class/oc_route.py

@@ -11,9 +11,8 @@ class OCRoute(OpenShiftCLI):
                  config,
                  verbose=False):
         ''' Constructor for OCVolume '''
-        super(OCRoute, self).__init__(config.namespace, config.kubeconfig)
+        super(OCRoute, self).__init__(config.namespace, kubeconfig=config.kubeconfig, verbose=verbose)
         self.config = config
-        self.namespace = config.namespace
         self._route = None
 
     @property

+ 1 - 4
roles/lib_openshift/src/class/oc_scale.py

@@ -15,13 +15,10 @@ class OCScale(OpenShiftCLI):
                  kubeconfig='/etc/origin/master/admin.kubeconfig',
                  verbose=False):
         ''' Constructor for OCScale '''
-        super(OCScale, self).__init__(namespace, kubeconfig)
+        super(OCScale, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose)
         self.kind = kind
         self.replicas = replicas
         self.name = resource_name
-        self.namespace = namespace
-        self.kubeconfig = kubeconfig
-        self.verbose = verbose
         self._resource = None
 
     @property

+ 1 - 4
roles/lib_openshift/src/class/oc_secret.py

@@ -17,12 +17,9 @@ class OCSecret(OpenShiftCLI):
                  kubeconfig='/etc/origin/master/admin.kubeconfig',
                  verbose=False):
         ''' Constructor for OpenshiftOC '''
-        super(OCSecret, self).__init__(namespace, kubeconfig)
-        self.namespace = namespace
+        super(OCSecret, self).__init__(namespace, kubeconfig=kubeconfig, verbose=verbose)
         self.name = secret_name
-        self.kubeconfig = kubeconfig
         self.decode = decode
-        self.verbose = verbose
 
     def get(self):
         '''return a secret by name '''

+ 1 - 2
roles/lib_openshift/src/class/oc_serviceaccount.py

@@ -12,9 +12,8 @@ class OCServiceAccount(OpenShiftCLI):
                  config,
                  verbose=False):
         ''' Constructor for OCVolume '''
-        super(OCServiceAccount, self).__init__(config.namespace, config.kubeconfig)
+        super(OCServiceAccount, self).__init__(config.namespace, kubeconfig=config.kubeconfig, verbose=verbose)
         self.config = config
-        self.namespace = config.namespace
         self.service_account = None
 
     def exists(self):

+ 1 - 1
roles/lib_openshift/src/class/oc_serviceaccount_secret.py

@@ -7,7 +7,7 @@ class OCServiceAccountSecret(OpenShiftCLI):
     kind = 'sa'
     def __init__(self, config, verbose=False):
         ''' Constructor for OpenshiftOC '''
-        super(OCServiceAccountSecret, self).__init__(config.namespace, config.kubeconfig)
+        super(OCServiceAccountSecret, self).__init__(config.namespace, kubeconfig=config.kubeconfig, verbose=verbose)
         self.config = config
         self.verbose = verbose
         self._service_account = None