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_objectvalidator.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_objectvalidator.py')
| -rw-r--r-- | roles/lib_openshift/library/oc_objectvalidator.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/roles/lib_openshift/library/oc_objectvalidator.py b/roles/lib_openshift/library/oc_objectvalidator.py index 5a966fa93..1d0e4c876 100644 --- a/roles/lib_openshift/library/oc_objectvalidator.py +++ b/roles/lib_openshift/library/oc_objectvalidator.py @@ -1297,10 +1297,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 |