octavia::db parameter missing in parent

Bug #1632667 reported by Fabian Zimmermann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-octavia
Fix Released
Undecided
Alex Schultz

Bug Description

::octavia is including ::octavia::db without any possibility to configure the database_* settings.

Well, I see two possibilities

1. Add something like "manage_octavia_db" to enable/disable the "include octavia::db"
2. Add all params of "octavia::db" to ::octavia

I will submit a short patch for (1), but would also submit a suitable patch for (2) if someone prefers this solution.

 Fabian

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

Fix proposed to branch: master
Review: https://review.openstack.org/385843

Changed in puppet-octavia:
assignee: nobody → Fabian Zimmermann (dev-faz)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/422031

Changed in puppet-octavia:
assignee: Fabian Zimmermann (dev-faz) → Brent Eagles (beagles)
Changed in puppet-octavia:
assignee: Brent Eagles (beagles) → Alex Schultz (alex-schultz)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-octavia (master)

Reviewed: https://review.openstack.org/422031
Committed: https://git.openstack.org/cgit/openstack/puppet-octavia/commit/?id=76a2df9e7290ff3d2455e145be87c33f9049be82
Submitter: Jenkins
Branch: master

commit 76a2df9e7290ff3d2455e145be87c33f9049be82
Author: Brent Eagles <email address hidden>
Date: Wed Jan 18 11:56:11 2017 -0330

    Move db include from ::octavia to ::octavia::api

    The database configuration for octavia isn't required by all services so
    moving it out of ::octavia and into ::octavia::api.

    Closes-Bug: #1632667
    Change-Id: I7a782a1378b94ac001419e331a2e48a8559a6565

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

This issue was fixed in the openstack/puppet-octavia 10.2.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on puppet-octavia (master)

Change abandoned by Mohammed Naser (<email address hidden>) on branch: master
Review: https://review.openstack.org/385843

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.