RFE: Ability to add parameters to db sync commands

Bug #1472740 reported by Mike Dorman
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-ceilometer
Fix Released
Undecided
Nate Potter
puppet-cinder
Fix Released
Undecided
Nate Potter
puppet-designate
Fix Released
Undecided
Nate Potter
puppet-glance
Fix Released
Undecided
Nate Potter
puppet-heat
Fix Released
Undecided
Nate Potter
puppet-ironic
Fix Released
Undecided
Vasyl Saienko
puppet-keystone
Fix Released
Undecided
Mike Dorman
puppet-neutron
Fix Released
Undecided
Nate Potter
puppet-nova
Fix Released
Undecided
Nate Potter
puppet-sahara
Fix Released
Undecided
Nate Potter

Bug Description

We have a use case where keystone has multiple configuration files, so there are multiple --config-file parameters on the command line.

However, this is problematic for the db sync command, as in some cases we also need to add more --config-file parameters to that command.

Would like to have a way to customize additional parameters to pass to the db sync commands for the various projects.

Mike Dorman (mdorman-m)
Changed in puppet-keystone:
assignee: nobody → Mike Dorman (mdorman-m)
Changed in puppet-keystone:
status: New → In Progress
Revision history for this message
Mike Dorman (mdorman-m) wrote :

Need to wait for the db_sync topic patches to merge:
https://review.openstack.org/#/q/status:open+branch:master+topic:db_sync,n,z

Vasyl Saienko (vsaienko)
Changed in puppet-ironic:
assignee: nobody → Vasyl Saienko (vsaienko)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-ironic (master)

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

Changed in puppet-ironic:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-keystone (master)

Reviewed: https://review.openstack.org/199714
Committed: https://git.openstack.org/cgit/openstack/puppet-keystone/commit/?id=ea12fb2d54ff22b4029112910a742316f1189c91
Submitter: Jenkins
Branch: master

commit ea12fb2d54ff22b4029112910a742316f1189c91
Author: Mike Dorman <email address hidden>
Date: Wed Jul 8 13:38:00 2015 -0600

    Allow customization of db sync command line

    Add $extra_params parameter to keystone::db::sync class to allow
    end-users to add command line parameters to the db_sync command.

    Change-Id: I3965a4d0d4974e7ee925c99207d4d6d4510a517f
    Closes-bug: 1472740

Changed in puppet-keystone:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-ironic (master)

Reviewed: https://review.openstack.org/221245
Committed: https://git.openstack.org/cgit/openstack/puppet-ironic/commit/?id=b28cafe7183ddae7963f1436d5271bb8025c02f9
Submitter: Jenkins
Branch: master

commit b28cafe7183ddae7963f1436d5271bb8025c02f9
Author: vsaienko <email address hidden>
Date: Tue Sep 8 13:44:00 2015 +0300

    Allow customization of db sync command line

     Add parameter to ironic::db::sync class to allow
     end-users to add command line parameters to the db_sync command.

    Change-Id: I8cf3b47a99e828a338aad617d43272725c9b7419
    Partial-bug: 1472740

Vasyl Saienko (vsaienko)
Changed in puppet-ironic:
status: In Progress → Fix Committed
Nate Potter (ntpttr)
Changed in puppet-cinder:
assignee: nobody → Nate Potter (ntpttr)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-cinder (master)

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

Changed in puppet-cinder:
status: New → In Progress
Nate Potter (ntpttr)
Changed in puppet-glance:
assignee: nobody → Nate Potter (ntpttr)
Changed in puppet-nova:
assignee: nobody → Nate Potter (ntpttr)
Changed in puppet-neutron:
assignee: nobody → Nate Potter (ntpttr)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-glance (master)

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

Changed in puppet-glance:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-nova (master)

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

Changed in puppet-nova:
status: New → In Progress
Nate Potter (ntpttr)
Changed in puppet-ceilometer:
assignee: nobody → Nate Potter (ntpttr)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-ceilometer (master)

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

Changed in puppet-ceilometer:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-sahara (master)

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

Changed in puppet-sahara:
assignee: nobody → Nate Potter (ntpttr)
status: New → In Progress
Nate Potter (ntpttr)
Changed in puppet-neutron:
status: New → In Progress
Changed in puppet-heat:
assignee: nobody → Nate Potter (ntpttr)
Changed in puppet-designate:
assignee: nobody → Nate Potter (ntpttr)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-heat (master)

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

Changed in puppet-heat:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-designate (master)

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

Changed in puppet-designate:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-ceilometer (master)

Reviewed: https://review.openstack.org/241276
Committed: https://git.openstack.org/cgit/openstack/puppet-ceilometer/commit/?id=fa33bb0ad6e2481146d4341725556e6bdab1938b
Submitter: Jenkins
Branch: master

commit fa33bb0ad6e2481146d4341725556e6bdab1938b
Author: Nate Potter <email address hidden>
Date: Tue Nov 3 16:22:01 2015 +0000

    Allow customization of db sync command line

    Add parameter to ceilometer::db::sync class to allow end
    users to add command line parameters to the db sync command.

    Change-Id: I664a822ef2962d5cb32bef7573e13f0b57ee6057
    Partial-bug: #1472740

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

Reviewed: https://review.openstack.org/240963
Committed: https://git.openstack.org/cgit/openstack/puppet-cinder/commit/?id=8a1e5f96b38c0a2e7e93b78b388adbba9a8ae06e
Submitter: Jenkins
Branch: master

commit 8a1e5f96b38c0a2e7e93b78b388adbba9a8ae06e
Author: Nate Potter <email address hidden>
Date: Mon Nov 2 18:31:54 2015 +0000

    Allow customization of db sync command line

    Add parameter to cinder::db::sync class to allow
    end users to add command line parameters to the db sync command.

    Change-Id: I34d9de89aae007d6b9f8644a8ba936bc0cf0c04c
    Partial-bug: #1472740

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

Reviewed: https://review.openstack.org/241351
Committed: https://git.openstack.org/cgit/openstack/puppet-designate/commit/?id=f5c5f88eed4706922b49da7c1b2b4fd33e3176c5
Submitter: Jenkins
Branch: master

commit f5c5f88eed4706922b49da7c1b2b4fd33e3176c5
Author: Nate Potter <email address hidden>
Date: Tue Nov 3 19:54:31 2015 +0000

    Allow customization of db sync command line

    Add parameter to designate::db::sync class to allow end
    users to add command line parameters to the db sync command.

    Change-Id: Iae224e97d91ee24e0aae1b79330f0906ff516508
    Partial-bug: #1472740

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

Reviewed: https://review.openstack.org/241003
Committed: https://git.openstack.org/cgit/openstack/puppet-glance/commit/?id=9babb26d31bc25477bb5223bc749901064ff57bb
Submitter: Jenkins
Branch: master

commit 9babb26d31bc25477bb5223bc749901064ff57bb
Author: Nate Potter <email address hidden>
Date: Mon Nov 2 22:04:35 2015 +0000

    Allow customization of db sync command line

    Add parameter to glance::db::sync class to allow end
    users to add command line parameters to the db sync command.

    Change-Id: I043dd9560f7d5eb7bc18721e759e5f61186f5cf7
    Partial-bug: #1472740

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

Reviewed: https://review.openstack.org/241012
Committed: https://git.openstack.org/cgit/openstack/puppet-neutron/commit/?id=091116d509a431e8141e7fbd589657293289acdc
Submitter: Jenkins
Branch: master

commit 091116d509a431e8141e7fbd589657293289acdc
Author: Nate Potter <email address hidden>
Date: Mon Nov 2 22:34:06 2015 +0000

    Allow customization of db sync command line

    Add $extra_params parameter to neutron::db::sync class to
    allow end users to add command line parameters to db sync command.

    Change-Id: Ideeeb70115bcec8526444c142b54890707bfbf61
    Partial-bug: #1472740

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

Reviewed: https://review.openstack.org/241290
Committed: https://git.openstack.org/cgit/openstack/puppet-sahara/commit/?id=b63d071d62fa307570e0d19aa486f63d04eb9dfd
Submitter: Jenkins
Branch: master

commit b63d071d62fa307570e0d19aa486f63d04eb9dfd
Author: Nate Potter <email address hidden>
Date: Tue Nov 3 16:41:30 2015 +0000

    Allow customization of db sync command line

    Add parameter to sahara::db::sync class to allow end
    users to add command line parameters to the db sync command.

    Change-Id: I14c177f3df9cd541408ee033638be21fa6e2fb6a
    Partial-bug: #1472740

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

Reviewed: https://review.openstack.org/241009
Committed: https://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=1b965fd70ba0cf8e4d7ed051e5c8dc790f9a2225
Submitter: Jenkins
Branch: master

commit 1b965fd70ba0cf8e4d7ed051e5c8dc790f9a2225
Author: Nate Potter <email address hidden>
Date: Mon Nov 2 22:19:21 2015 +0000

    Allow customization of db sync command line

    Add parameter to nova::db::sync class to allow end
    users to add command line parameters to the db sync command.

    Change-Id: I89dea475495d5903d953ff4492ad2e0f12a293c1
    Partial-bug: #1472740

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

Reviewed: https://review.openstack.org/241299
Committed: https://git.openstack.org/cgit/openstack/puppet-heat/commit/?id=4d4430efd68a75d29554e5d98db02735f2d0823b
Submitter: Jenkins
Branch: master

commit 4d4430efd68a75d29554e5d98db02735f2d0823b
Author: Nate Potter <email address hidden>
Date: Tue Nov 3 17:07:19 2015 +0000

    Allow customization of db sync command line

    Add parameter to heat::db::sync to allow end
    users to add command line parameters to the db sync command.

    Change-Id: Id9de1022481fbeedc3db78a3dd2e3c5f2a962e10
    Partial-bug: #1472740

Changed in puppet-ironic:
status: Fix Committed → Fix Released
Changed in puppet-keystone:
status: Fix Committed → Fix Released
Changed in puppet-glance:
status: In Progress → Fix Released
Changed in puppet-nova:
status: In Progress → Fix Released
Changed in puppet-ceilometer:
status: In Progress → Fix Released
Changed in puppet-sahara:
status: In Progress → Fix Released
Changed in puppet-heat:
status: In Progress → Fix Released
Changed in puppet-designate:
status: In Progress → Fix Released
Revision history for this message
Dan Williams (adventsol) wrote :

It looks like from comment 13 and 16 that both puppet-cinder and puppet-neutron now have the fixes merged. This ticket just needs closing.

Changed in puppet-neutron:
status: In Progress → Fix Released
Changed in puppet-cinder:
status: In Progress → Fix Released
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.