openstack overcloud export always uses "passwords" from plan-environment.yaml

Bug #1891388 reported by James Slagle
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
James Slagle

Bug Description

The "passwords" key from plan-environment.yaml is actually the generated passwords. This results in openstack overcloud export always exporting the generated passwords instead of any user specified ones.

The export command needs to look at parameter_defaults from plan-environment.yaml to get the user defined values, and fall back to passwords if none was specified.

Changed in tripleo:
status: New → In Progress
importance: Undecided → High
assignee: nobody → James Slagle (james-slagle)
milestone: none → victoria-3
tags: added: train-backport-potential ussuri-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-tripleoclient (master)

Fix proposed to branch: master
Review: https://review.opendev.org/745958

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-tripleoclient (master)

Reviewed: https://review.opendev.org/745958
Committed: https://git.openstack.org/cgit/openstack/python-tripleoclient/commit/?id=ced221406b70c348453c82247c9274fb11c77dd3
Submitter: Zuul
Branch: master

commit ced221406b70c348453c82247c9274fb11c77dd3
Author: James Slagle <email address hidden>
Date: Wed Aug 12 16:07:03 2020 -0400

    Use user set password values in overcloud export

    The overcloud export command needs to look at both the
    parameter_defaults and passwords keys from plan-environment.yaml.
    passwords contains only the generated password values, while
    parameter_defaults contains any user set values. If a value is not set
    in parameter_defaults, then passwords can be used.

    Change-Id: I6b1b8bd1e7800720f6bbbe418c1f83b2f710fb48
    Closes-Bug: #1891388
    Signed-off-by: James Slagle <email address hidden>

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-tripleoclient (stable/ussuri)

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/746143

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-tripleoclient (stable/train)

Fix proposed to branch: stable/train
Review: https://review.opendev.org/746160

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-tripleoclient (stable/ussuri)

Reviewed: https://review.opendev.org/746143
Committed: https://git.openstack.org/cgit/openstack/python-tripleoclient/commit/?id=31aeb48e7063d32f83d903a3a0eb8d287f76a861
Submitter: Zuul
Branch: stable/ussuri

commit 31aeb48e7063d32f83d903a3a0eb8d287f76a861
Author: James Slagle <email address hidden>
Date: Wed Aug 12 16:07:03 2020 -0400

    Use user set password values in overcloud export

    The overcloud export command needs to look at both the
    parameter_defaults and passwords keys from plan-environment.yaml.
    passwords contains only the generated password values, while
    parameter_defaults contains any user set values. If a value is not set
    in parameter_defaults, then passwords can be used.

    Change-Id: I6b1b8bd1e7800720f6bbbe418c1f83b2f710fb48
    Closes-Bug: #1891388
    Signed-off-by: James Slagle <email address hidden>
    (cherry picked from commit ced221406b70c348453c82247c9274fb11c77dd3)

tags: added: in-stable-ussuri
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-tripleoclient (stable/train)

Reviewed: https://review.opendev.org/746160
Committed: https://git.openstack.org/cgit/openstack/python-tripleoclient/commit/?id=c15d0d0ba0b3de7e78a9d9a4b1c88b8f497b97df
Submitter: Zuul
Branch: stable/train

commit c15d0d0ba0b3de7e78a9d9a4b1c88b8f497b97df
Author: James Slagle <email address hidden>
Date: Wed Aug 12 16:07:03 2020 -0400

    Use user set password values in overcloud export

    The overcloud export command needs to look at both the
    parameter_defaults and passwords keys from plan-environment.yaml.
    passwords contains only the generated password values, while
    parameter_defaults contains any user set values. If a value is not set
    in parameter_defaults, then passwords can be used.

    Change-Id: I6b1b8bd1e7800720f6bbbe418c1f83b2f710fb48
    Closes-Bug: #1891388
    Signed-off-by: James Slagle <email address hidden>
    (cherry picked from commit ced221406b70c348453c82247c9274fb11c77dd3)

tags: added: in-stable-train
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/python-tripleoclient 12.4.0

This issue was fixed in the openstack/python-tripleoclient 12.4.0 release.

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.