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