commit 0c6040ef6b85a77abd277a9e63be614877ca7c6a
Author: Ben Nemec <email address hidden>
Date: Tue Feb 28 00:02:16 2017 +0000
Handle both - and _ forms of deprecated opts
While config file opts cannot have a - in their name, CLI opts can
and the current deprecation code doesn't handle that case properly.
Additionally, because oslo.config has some magic to translate one
to the other it is possible we have existing users relying on the
wrong behavior. For this reason, let's add both forms of the opt
when adding deprecated opts. This will allow both to work as
expected.
The config generator is updated to ignore the - form of deprecated
opts since they won't work in a config file anyway.
I'm unsure how to filter the extra opts out of the help text, but
since both forms will work in that case it's a minor cosmetic
issue.
Reviewed: https:/ /review. openstack. org/438752 /git.openstack. org/cgit/ openstack/ oslo.config/ commit/ ?id=0c6040ef6b8 5a77abd277a9e63 be614877ca7c6a
Committed: https:/
Submitter: Jenkins
Branch: master
commit 0c6040ef6b85a77 abd277a9e63be61 4877ca7c6a
Author: Ben Nemec <email address hidden>
Date: Tue Feb 28 00:02:16 2017 +0000
Handle both - and _ forms of deprecated opts
While config file opts cannot have a - in their name, CLI opts can
and the current deprecation code doesn't handle that case properly.
Additionally, because oslo.config has some magic to translate one
to the other it is possible we have existing users relying on the
wrong behavior. For this reason, let's add both forms of the opt
when adding deprecated opts. This will allow both to work as
expected.
The config generator is updated to ignore the - form of deprecated
opts since they won't work in a config file anyway.
I'm unsure how to filter the extra opts out of the help text, but
since both forms will work in that case it's a minor cosmetic
issue.
Change-Id: I4d2584b33d97fe 00a427b992682d9 eac391725a2
Closes-Bug: 1279973