tripleoclient - constants.OVERCLOUD_NETWORKS_FILE not used when jinja2 rendering templates

Bug #1934010 reported by Harald Jensås
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Harald Jensås

Bug Description

The default in THT is used, instead of the default in tripleoclient when no 'parsed_args.networks_file is not used. The default in THT/tools/process-templates.py is used instead. This worked differently prior to Ic9db8d852fbfa50f4ac574ebe524c1fb1d85c588, when the tripleo constant was used.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-tripleoclient (master)
Changed in tripleo:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-tripleoclient (master)

Reviewed: https://review.opendev.org/c/openstack/python-tripleoclient/+/798744
Committed: https://opendev.org/openstack/python-tripleoclient/commit/a586cb7795cf301c968fafa2e6fd9e5201220977
Submitter: "Zuul (22348)"
Branch: master

commit a586cb7795cf301c968fafa2e6fd9e5201220977
Author: Harald Jensås <email address hidden>
Date: Tue Jun 29 21:26:18 2021 +0200

    Honor role and networks file constants

    Prior to change Ic9db8d852fbfa50f4ac574ebe524c1fb1d85c588
    the tripleoclient constants OVERCLOUD_ROLES_FILE and
    OVERCLOUD_NETWORKS_FILE was used as the defaults.

    This was changed to use the defaults in process-templates.py

    This restores the previous behaviour, honoring the tripleoclient
    constants.

    Closes-Bug: #1934010
    Change-Id: Ie133a6718ca50f30fee7a48fc83ff0e89d84d2dc

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

Fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/python-tripleoclient/+/799056

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

Reviewed: https://review.opendev.org/c/openstack/python-tripleoclient/+/799056
Committed: https://opendev.org/openstack/python-tripleoclient/commit/2ac670653640630c83d204ca53744553ef799390
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 2ac670653640630c83d204ca53744553ef799390
Author: Harald Jensås <email address hidden>
Date: Tue Jun 29 21:26:18 2021 +0200

    Honor role and networks file constants

    Prior to change Ic9db8d852fbfa50f4ac574ebe524c1fb1d85c588
    the tripleoclient constants OVERCLOUD_ROLES_FILE and
    OVERCLOUD_NETWORKS_FILE was used as the defaults.

    This was changed to use the defaults in process-templates.py

    This restores the previous behaviour, honoring the tripleoclient
    constants.

    Closes-Bug: #1934010
    Change-Id: Ie133a6718ca50f30fee7a48fc83ff0e89d84d2dc
    (cherry picked from commit a586cb7795cf301c968fafa2e6fd9e5201220977)

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/python-tripleoclient 17.1.0

This issue was fixed in the openstack/python-tripleoclient 17.1.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.