Octavia params not being reflected in its confg files

Bug #1805345 reported by Carlos Goncalves
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
Undecided
Carlos Goncalves

Bug Description

Recent change [1] in puppet-octavia deprecated some parameters in class octavia::worker and moved them to a new class octavia::controller. A later change [2] also removed the include of the controller class so because of that we have to include it in the Octavia services manifest files in puppet-tripleo.

[1] https://review.openstack.org/#/c/612395/
[2] https://review.openstack.org/#/c/619013/

tags: added: rocky-backport-potential
tags: added: queens-backport-potential
Changed in tripleo:
assignee: nobody → Carlos Goncalves (cgoncalves)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-tripleo (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.openstack.org/622498

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-tripleo (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/622500

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.openstack.org/623171

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/623174

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

Reviewed: https://review.openstack.org/620118
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=43ffaed334e0cdf34329972085ffadd7a2fa1ba0
Submitter: Zuul
Branch: master

commit 43ffaed334e0cdf34329972085ffadd7a2fa1ba0
Author: Carlos Goncalves <email address hidden>
Date: Mon Nov 26 18:39:21 2018 +0100

    Include octavia::controller

    Commits https://review.openstack.org/#/c/612395/ and
    https://review.openstack.org/#/c/619013/ deprecated some
    parameters in class octavia::worker and moved them to the new class
    octavia::controller. The latter commit also removed the include of the
    controller class so because of that we have to include it in the Octavia
    services manifest files in puppet-tripleo.

    Partial-Bug: #1805345

    Change-Id: I0256e125de367615c21343683ed5311521d8337c

tags: added: in-stable-rocky
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-tripleo (stable/rocky)

Reviewed: https://review.openstack.org/622498
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=fbbca5c0ac43192d2afa0ef16d6ccdc02b8e81e0
Submitter: Zuul
Branch: stable/rocky

commit fbbca5c0ac43192d2afa0ef16d6ccdc02b8e81e0
Author: Carlos Goncalves <email address hidden>
Date: Mon Nov 26 18:39:21 2018 +0100

    Include octavia::controller

    Commits https://review.openstack.org/#/c/612395/ and
    https://review.openstack.org/#/c/619013/ deprecated some
    parameters in class octavia::worker and moved them to the new class
    octavia::controller. The latter commit also removed the include of the
    controller class so because of that we have to include it in the Octavia
    services manifest files in puppet-tripleo.

    Partial-Bug: #1805345

    Change-Id: I0256e125de367615c21343683ed5311521d8337c
    (cherry picked from commit 43ffaed334e0cdf34329972085ffadd7a2fa1ba0)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-tripleo (stable/queens)

Reviewed: https://review.openstack.org/622500
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=8e57bf869d06ba7f889934e0a23545c05b535520
Submitter: Zuul
Branch: stable/queens

commit 8e57bf869d06ba7f889934e0a23545c05b535520
Author: Carlos Goncalves <email address hidden>
Date: Mon Nov 26 18:39:21 2018 +0100

    Include octavia::controller

    Commits https://review.openstack.org/#/c/612395/ and
    https://review.openstack.org/#/c/619013/ deprecated some
    parameters in class octavia::worker and moved them to the new class
    octavia::controller. The latter commit also removed the include of the
    controller class so because of that we have to include it in the Octavia
    services manifest files in puppet-tripleo.

    Partial-Bug: #1805345

    Change-Id: I0256e125de367615c21343683ed5311521d8337c
    (cherry picked from commit 43ffaed334e0cdf34329972085ffadd7a2fa1ba0)
    (cherry picked from commit fbbca5c0ac43192d2afa0ef16d6ccdc02b8e81e0)

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (master)

Reviewed: https://review.openstack.org/620106
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=3c8e736981393cabe3517ced44355af292a93453
Submitter: Zuul
Branch: master

commit 3c8e736981393cabe3517ced44355af292a93453
Author: Carlos Goncalves <email address hidden>
Date: Mon Nov 26 17:36:15 2018 +0100

    Fix Octavia hieradata keys

    Commit https://review.openstack.org/#/c/612395/ deprecated some
    parameters in class octavia::worker and moved them to the new class
    octavia::controller.

    Closes-Bug: #1805345
    Depends-On: https://review.openstack.org/#/c/620118/

    Change-Id: I53264dc6cbb75165484c3b1f6ddbee45dc4e0206

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 10.3.0

This issue was fixed in the openstack/tripleo-heat-templates 10.3.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (stable/rocky)

Reviewed: https://review.openstack.org/623171
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=6e0ff00b04e18f38be4e62823d0b1b33a5ec3221
Submitter: Zuul
Branch: stable/rocky

commit 6e0ff00b04e18f38be4e62823d0b1b33a5ec3221
Author: Carlos Goncalves <email address hidden>
Date: Mon Nov 26 17:36:15 2018 +0100

    Fix Octavia hieradata keys

    Commit https://review.openstack.org/#/c/612395/ deprecated some
    parameters in class octavia::worker and moved them to the new class
    octavia::controller.

    Conflicts:
        puppet/services/octavia-api.yaml

    Closes-Bug: #1805345
    Depends-On: https://review.openstack.org/#/c/620118/

    Change-Id: I53264dc6cbb75165484c3b1f6ddbee45dc4e0206
    (cherry picked from commit 3c8e736981393cabe3517ced44355af292a93453)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (stable/queens)

Reviewed: https://review.openstack.org/623174
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=e60a19a3f04a9b9d60e5ade6f044e68a2e907b89
Submitter: Zuul
Branch: stable/queens

commit e60a19a3f04a9b9d60e5ade6f044e68a2e907b89
Author: Carlos Goncalves <email address hidden>
Date: Mon Nov 26 17:36:15 2018 +0100

    Fix Octavia hieradata keys

    Commit https://review.openstack.org/#/c/612395/ deprecated some
    parameters in class octavia::worker and moved them to the new class
    octavia::controller.

    Closes-Bug: #1805345
    Depends-On: https://review.openstack.org/#/c/620118/

    Change-Id: I53264dc6cbb75165484c3b1f6ddbee45dc4e0206
    (cherry picked from commit 3c8e736981393cabe3517ced44355af292a93453)
    (cherry picked from commit a60d8d4bb30694fadc61b75279414c6d50368835)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 8.3.0

This issue was fixed in the openstack/tripleo-heat-templates 8.3.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 9.3.0

This issue was fixed in the openstack/tripleo-heat-templates 9.3.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.