Inconsistent package_ensure parameter name
Bug Description
The parameter name used to enforce package state is inconsistent across manifests and modules.
I found the following occurrences of each names (found in repositories in the openstack/
* ensure_package = 33
* package_ensure = 141
I suggest we settle on package_ensure since it's already widely used.
This means the following manifests will need to be updated accordingly:
* aodh::ensure_
* barbican:
* cloudkitty:
* gnocchi:
* gnocchi:
* nova::api:
* nova::cells:
* nova::cert:
* nova::compute:
* nova::conductor
* nova::consoleau
* nova::ensure_
* nova::network:
* nova::objectsto
* nova::scheduler
* nova::serialpro
* nova::spicehtml
* nova::vncproxy:
* nova::compute:
* nova::metadata:
* nova::wsgi:
* octavia:
* panko::
* qdr::ensure_package
* qdr::install:
* rally::
* tempest:
* trove::
* trove::
* trove::
* trove::
* vitrage:
* watcher:
Changed in puppet-aodh: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-barbican: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-cloudkitty: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-gnocchi: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-nova: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-octavia: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-panko: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-rally: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-tempest: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-trove: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-vitrage: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-watcher: | |
status: | New → Confirmed |
Changed in puppet-cinder: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-ceph: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-designate: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-swift: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-glance: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-glare: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-horizon: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-ironic: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-keystone: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-monasca: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-neutron: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-openstacklib: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-oslo: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in puppet-watcher: | |
importance: | Undecided → Medium |
Changed in puppet-panko: | |
assignee: | nobody → Michael Glaser (mikeg451) |
Changed in puppet-rally: | |
assignee: | nobody → Michael Glaser (mikeg451) |
Changed in puppet-panko: | |
assignee: | Michael Glaser (mikeg451) → nobody |
Changed in puppet-rally: | |
assignee: | Michael Glaser (mikeg451) → nobody |
Changed in puppet-panko: | |
status: | In Progress → Confirmed |
Changed in puppet-rally: | |
status: | In Progress → Confirmed |
Changed in puppet-octavia: | |
assignee: | nobody → Brent Eagles (beagles) |
Changed in puppet-neutron: | |
assignee: | nobody → Brent Eagles (beagles) |
assignee: | Brent Eagles (beagles) → nobody |
Changed in puppet-octavia: | |
assignee: | Brent Eagles (beagles) → nobody |
Changed in puppet-aodh: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-barbican: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-ceph: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-aodh: | |
status: | Confirmed → In Progress |
Changed in puppet-cinder: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-cloudkitty: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-designate: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-glance: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-glare: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-gnocchi: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-horizon: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-ironic: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-keystone: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-monasca: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-neutron: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-octavia: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-nova: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-panko: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-rally: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-tempest: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-trove: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-vitrage: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-watcher: | |
assignee: | nobody → zhongshengping (chdzsp) |
Changed in puppet-ceph: | |
status: | Confirmed → Fix Released |
Changed in puppet-openstacklib: | |
assignee: | nobody → zhongshengping (chdzsp) |
status: | Confirmed → Fix Released |
Changed in puppet-oslo: | |
assignee: | nobody → zhongshengping (chdzsp) |
status: | Confirmed → Fix Released |
Changed in puppet-swift: | |
assignee: | nobody → zhongshengping (chdzsp) |
status: | Confirmed → Fix Released |
Changed in puppet-cinder: | |
status: | Confirmed → Fix Released |
Changed in puppet-glance: | |
status: | Confirmed → Fix Released |
Changed in puppet-glare: | |
status: | Confirmed → Fix Released |
Changed in puppet-horizon: | |
status: | Confirmed → Fix Released |
Changed in puppet-ironic: | |
status: | Confirmed → Fix Released |
Changed in puppet-keystone: | |
status: | Confirmed → Fix Released |
Changed in puppet-monasca: | |
status: | Confirmed → Fix Released |
Changed in puppet-neutron: | |
status: | Confirmed → Fix Released |
Changed in puppet-nova: | |
status: | Confirmed → Fix Released |
Fix proposed to branch: master /review. openstack. org/441947
Review: https:/