Validation: missing check for HostnameMap

Bug #1740396 reported by Cédric Jeanneret deactivated
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Incomplete
Medium
Unassigned

Bug Description

Dear Stackers,

When we use a static HostnameMap hash in an environment file, it would be great to have a validation that ensures we do not messed that hash.

In order to do so, it should ensure the uniqueness of tuples and ensure this kind of entries aren't present:
    overcloud-controller-0: controller-0
    overcloud-controller-1: controller-1
    overcloud-controller-2: controller-1 <- that one is wrong

Preventing users to make typo and errors in that hash will prevent breaking the whole heat stack when we want to add new nodes (computes, for example).

Thank you!

Cheers,

C.

Changed in tripleo:
milestone: none → rocky-1
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Cédric Jeanneret deactivated (cjeanneret-c2c-deactivated) wrote :

Should affect https://github.com/openstack/tripleo-validations/tree/master/validations tree

but on that matter, I'm clueless, as I really don't know what to do… validations seem to be dedicated workflows described in yaml - checking the hash would require some kind of script language I guess… Any hint would be good :).

Revision history for this message
Zane Bitter (zaneb) wrote :

When people blacklist failed nodes they typically update the hostname map with dummy names like so:

    overcloud-compute-0: compute-0
    overcloud-compute-1: UNUSED
    overcloud-compute-2: UNUSED
    overcloud-compute-3: compute-3

Enforcing uniqueness at the parameter level in Heat (as https://review.openstack.org/#/c/530903/ seems to be preparing the ground for) will break existing deployments using this pattern.

Changed in tripleo:
milestone: rocky-1 → rocky-2
Changed in tripleo:
milestone: rocky-2 → rocky-3
Changed in tripleo:
milestone: rocky-3 → rocky-rc1
Changed in tripleo:
milestone: rocky-rc1 → stein-1
Changed in tripleo:
milestone: stein-1 → stein-2
Changed in tripleo:
milestone: stein-2 → stein-3
Changed in tripleo:
milestone: stein-3 → train-1
Changed in tripleo:
milestone: train-1 → train-2
Changed in tripleo:
milestone: train-2 → train-3
Changed in tripleo:
milestone: train-3 → ussuri-1
Changed in tripleo:
milestone: ussuri-1 → ussuri-2
wes hayutin (weshayutin)
Changed in tripleo:
milestone: ussuri-2 → ussuri-3
wes hayutin (weshayutin)
Changed in tripleo:
milestone: ussuri-3 → ussuri-rc3
wes hayutin (weshayutin)
Changed in tripleo:
status: Triaged → Incomplete
wes hayutin (weshayutin)
Changed in tripleo:
milestone: ussuri-rc3 → victoria-1
Changed in tripleo:
milestone: victoria-1 → victoria-3
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.