Add a CI check for hiera_array in puppet
Bug #1499803 reported by
Matthew Mosesohn
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Fuel for OpenStack |
Fix Released
|
Low
|
Bartłomiej Piotrowski |
Bug Description
We had several commits in the 7.0 cycle that tried to introduce hiera_array lookups, causing strange results when working with plugins that override service role lists. At the current moment, no services require a hiera_array merge, but several arrays in hiera are sensitive to calls with hiera_array. The current list is:
corosync_roles
memcache_roles
mongo_roles
heat_roles
sahara_roles
We should add a voting CI job to watch for undesirable calls to hiera_array('KEY') in fuel-library
Changed in fuel: | |
milestone: | none → 8.0 |
importance: | Undecided → Low |
Changed in fuel: | |
assignee: | nobody → Fuel CI team (fuel-ci) |
tags: | added: feature |
Changed in fuel: | |
status: | New → Confirmed |
tags: | added: non-release |
Changed in fuel: | |
status: | Confirmed → Triaged |
Changed in fuel: | |
assignee: | Fuel Library Team (fuel-library) → Bartłomiej Piotrowski (bpiotrowski) |
tags: | added: area-library |
Changed in fuel: | |
status: | Triaged → In Progress |
Changed in fuel: | |
assignee: | Bartłomiej Piotrowski (bpiotrowski) → Alex Schultz (alex-schultz) |
Changed in fuel: | |
assignee: | Alex Schultz (alex-schultz) → Bartłomiej Piotrowski (bpiotrowski) |
Changed in fuel: | |
assignee: | Bartłomiej Piotrowski (bpiotrowski) → Alex Schultz (alex-schultz) |
Changed in fuel: | |
assignee: | Alex Schultz (alex-schultz) → Bartłomiej Piotrowski (bpiotrowski) |
tags: | added: on-verification |
description: | updated |
To post a comment you must log in.
Please provide initial implementation for the test and how it might look on CI.
Ideally - just send a patch to the repository, similar to https:/ /review. fuel-infra. org/#/c/ 5540/