openstack/oslo.policy
Code Issues Proposed changes

Merge "Fixes is_admin type from StrOpt to BoolOpt."

Browse Source
This commit is contained in:
Zuul 2019-01-18 01:32:51 +00:00 committed by Gerrit Code Review
commit ce31c0fce9
1 changed files with 7 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
oslo_policy/shell.py
View File

@ -95,13 +95,13 @@ def main():
conf.register_cli_opt(cfg.StrOpt( conf.register_cli_opt(cfg.StrOpt(
'policy', 'policy',
required=True, required=True,
help='path to a policy file')) help='path to a policy file.'))
conf.register_cli_opt(cfg.StrOpt( conf.register_cli_opt(cfg.StrOpt(
'access', 'access',
required=True, required=True,
help='path to a file containing OpenStack Identity API ' help='path to a file containing OpenStack Identity API '
'access info in JSON format')) 'access info in JSON format.'))
conf.register_cli_opt(cfg.StrOpt( conf.register_cli_opt(cfg.StrOpt(
'target', 'target',
@ -110,18 +110,16 @@ def main():
conf.register_cli_opt(cfg.StrOpt( conf.register_cli_opt(cfg.StrOpt(
'rule', 'rule',
help='rule to test')) help='rule to test.'))
conf.register_cli_opt(cfg.StrOpt( conf.register_cli_opt(cfg.BoolOpt(
'is_admin', 'is_admin',
help='set is_admin=True on the credentials used for the evaluation', help='set is_admin=True on the credentials used for the evaluation.',
default="")) default=False))
conf() conf()
is_admin = conf.is_admin.lower() == "true" tool(conf.policy, conf.access, conf.rule, conf.is_admin, conf.target)
tool(conf.policy, conf.access, conf.rule, is_admin, conf.target)
if __name__ == "__main__": if __name__ == "__main__":