diff options
author | Andrew Butcher <abutcher@afrolegs.com> | 2017-03-09 11:41:06 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-03-09 11:41:06 -0500 |
commit | 573e91d03d7dccd79a03774c6889c2c2753836b1 (patch) | |
tree | 06db9fb260c9f3d0c96d18e4d59d92ee3b14ec14 /roles/lib_openshift/library/oc_serviceaccount_secret.py | |
parent | 20aa8f8919d335042568cb3e6b75aa8fd7a490bb (diff) | |
parent | ea6f73440c47ae3c24852f11542e715b3765d9d1 (diff) | |
download | openshift-573e91d03d7dccd79a03774c6889c2c2753836b1.tar.gz openshift-573e91d03d7dccd79a03774c6889c2c2753836b1.tar.bz2 openshift-573e91d03d7dccd79a03774c6889c2c2753836b1.tar.xz openshift-573e91d03d7dccd79a03774c6889c2c2753836b1.zip |
Merge pull request #3590 from kwoodson/router_registry_unit_tests
Fix to OpenshiftCLIConfig to support an sorted keys. Removed redundant get.
Diffstat (limited to 'roles/lib_openshift/library/oc_serviceaccount_secret.py')
-rw-r--r-- | roles/lib_openshift/library/oc_serviceaccount_secret.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/roles/lib_openshift/library/oc_serviceaccount_secret.py b/roles/lib_openshift/library/oc_serviceaccount_secret.py index c058c555b..925a5a088 100644 --- a/roles/lib_openshift/library/oc_serviceaccount_secret.py +++ b/roles/lib_openshift/library/oc_serviceaccount_secret.py @@ -1338,10 +1338,11 @@ class OpenShiftCLIConfig(object): def stringify(self): ''' return the options hash as cli params in a string ''' rval = [] - for key, data in self.config_options.items(): + for key in sorted(self.config_options.keys()): + data = self.config_options[key] if data['include'] \ and (data['value'] or isinstance(data['value'], int)): - rval.append('--%s=%s' % (key.replace('_', '-'), data['value'])) + rval.append('--{}={}'.format(key.replace('_', '-'), data['value'])) return rval |