tempest::tempest_config_file is not fully honored

Bug #2043210 reported by Takashi Kajinami
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-tempest
Fix Released
Medium
Takashi Kajinami

Bug Description

The tempest class provides the tempest_config_file to customize path of the tempest.conf .

However this parameter is not honored properly.

 - When install_from_source is true, this parameter is not at all userd

 - When install_from_source is false, this parameter is used by tempest_conf resource but
   is not used by the other setters.

Changed in puppet-tempest:
assignee: nobody → Takashi Kajinami (kajinamit)
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-tempest (master)
Changed in puppet-tempest:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-tempest (master)

Reviewed: https://review.opendev.org/c/openstack/puppet-tempest/+/900621
Committed: https://opendev.org/openstack/puppet-tempest/commit/dbef30e09b89317500c0c6c4cd24d3de38ac2bf5
Submitter: "Zuul (22348)"
Branch: master

commit dbef30e09b89317500c0c6c4cd24d3de38ac2bf5
Author: Takashi Kajinami <email address hidden>
Date: Fri Nov 10 23:28:55 2023 +0900

    Deprecate confusing tempest_config_file

    The tempest_config_file parameter is supposed to determine the path of
    tempest.conf but this is not working completely.

    * This parameter is ignored when install_from_source is true

    * This parameter affects tempest_conf when install_from_source but
      some resources like tempest_flavor_id_setter uses a fixed path.

    This deprecates the parameter to avoid complexity. Especially the 2nd
    behavior is quite confusing and requires users to set two independent
    parameters with "consistent" values.

    Closes-Bug: #2043210
    Change-Id: Ica5e77dff0d76250835b9187e292c2384d02b683

Changed in puppet-tempest:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-tempest 24.0.0

This issue was fixed in the openstack/puppet-tempest 24.0.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to puppet-tempest (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/puppet-tempest/+/916572

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to puppet-tempest (master)

Reviewed: https://review.opendev.org/c/openstack/puppet-tempest/+/916572
Committed: https://opendev.org/openstack/puppet-tempest/commit/e82a668c4028c28c1aa49ecd703a968552aba8dc
Submitter: "Zuul (22348)"
Branch: master

commit e82a668c4028c28c1aa49ecd703a968552aba8dc
Author: Takashi Kajinami <email address hidden>
Date: Sat Apr 20 23:19:57 2024 +0900

    magnum: Deprecate tempest_config_file

    ... because we deprecated the same parameter of the tempest class.

    Related-Bug: #2043210
    Change-Id: I87c1db9a9d5297da4f5a716faa44f0228203893d

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.