Browse Source

Merge pull request #8448 from jcantrill/fix_oc_adm_rolebinding

fix  The error was: KeyError: 'userNames'
Vadim Rutkovsky 6 years ago
parent
commit
0a94b84f6c

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

@@ -2096,7 +2096,7 @@ class PolicyUser(OpenShiftCLI):
 
         for binding in bindings:
             if binding['roleRef']['name'] == self.config.config_options['name']['value'] and \
-                    binding['userNames'] is not None and \
+                    'userNames' in binding and binding['userNames'] is not None and \
                     self.config.config_options['user']['value'] in binding['userNames']:
                 self.role_binding = binding
                 return True

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

@@ -108,7 +108,7 @@ class PolicyUser(OpenShiftCLI):
 
         for binding in bindings:
             if binding['roleRef']['name'] == self.config.config_options['name']['value'] and \
-                    binding['userNames'] is not None and \
+                    'userNames' in binding and binding['userNames'] is not None and \
                     self.config.config_options['user']['value'] in binding['userNames']:
                 self.role_binding = binding
                 return True