policy classes parameters documentation incorrect
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
puppet-glance |
Fix Released
|
Low
|
Mike Dorman | ||
puppet-neutron |
Fix Released
|
Low
|
Mike Dorman | ||
puppet-nova |
Fix Released
|
Low
|
Mike Dorman |
Bug Description
Policy configuration through ::nova::policy (and similarly for the other services' modules) does not work if you use the $policies parameter format as is documented in the comments.
The comments on the ::nova::policy class indicate that the policies hash should be in this format:
{
'nova-
'context_
},
'nova-default' => {
'default' => 'rule:admin_
}
}
... which is given directly to create_resources.
However, in order for this to work properly, the hash given to create_resources should look like:
{
'nova-
'key' => 'context_is_admin',
'value' => 'true'
},
'nova-default' => {
'key' => 'default',
'value' => 'rule:admin_
}
}
If the user is providing a hash to ::nova::policy of the correct format, then all will be well. But if the user is providing a format as indicated in the comments, then the Puppet run will fail with:
Error: Invalid parameter context_is_admin on Openstacklib:
description: | updated |
summary: |
- policy configuration does not work with Puppet future parser + policy classes parameters documentation incorrect |
description: | updated |
Changed in puppet-glance: | |
importance: | Undecided → Low |
Changed in puppet-neutron: | |
importance: | Undecided → Low |
Changed in puppet-nova: | |
importance: | Undecided → Low |
Changed in puppet-glance: | |
milestone: | none → 6.0.0 |
Changed in puppet-neutron: | |
milestone: | none → 6.0.0 |
Changed in puppet-nova: | |
milestone: | none → 6.0.0 |
Changed in puppet-glance: | |
status: | Fix Committed → Fix Released |
Changed in puppet-neutron: | |
status: | Fix Committed → Fix Released |
Changed in puppet-nova: | |
status: | Fix Committed → Fix Released |
Changed in puppet-glance: | |
status: | Fix Released → Fix Committed |
Changed in puppet-neutron: | |
status: | Fix Released → Fix Committed |
Changed in puppet-nova: | |
status: | Fix Released → Fix Committed |
Changed in puppet-glance: | |
status: | Fix Committed → Fix Released |
Changed in puppet-nova: | |
status: | Fix Committed → Fix Released |
Changed in puppet-neutron: | |
status: | Fix Committed → Fix Released |
Fix proposed to branch: master /review. openstack. org/146635
Review: https:/