[RFE] Centralize config options in Ironic

Bug #1561100 reported by Ramamani Yeleswarapu
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Wishlist
Ramamani Yeleswarapu

Bug Description

There is a need for centralizing config options in Ironic because we have lists of config options and registering options mixed with source code in regular files in Ironic. This can be done similar to what is done/being done in Nova.

Reference: http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/centralize-config-options.html

A series of patches have been submitted for centralizing the config options: https://review.openstack.org/#/c/295946/

NOTE:
This work is dependent on [RFE] https://bugs.launchpad.net/ironic/+bug/1564195 and its corresponding patch: https://review.openstack.org/#/c/247331/, which focuses on adopting oslo-config-generator to generate sample config files.

Tags: rfe-approved
Changed in ironic:
assignee: nobody → Ramamani Yeleswarapu (ramamani-yeleswarapu)
status: New → In Progress
Revision history for this message
Jim Rollenhagen (jim-rollenhagen) wrote :

I'm not sure this needs a spec, as we've talked about doing something similar recently. However, I do want to discuss this with the team at the next ironic meeting before approving it.

As part of this work, I'd like to first see us switch to using oslo-config-generator, before we move the config options around.

Revision history for this message
Jim Rollenhagen (jim-rollenhagen) wrote :

This is the patch moving to oslo-config-generator, by the way. https://review.openstack.org/#/c/247331/

Changed in ironic:
assignee: Ramamani Yeleswarapu (ramamani-yeleswarapu) → Tan Lin (tan-lin-good)
Changed in ironic:
assignee: Tan Lin (tan-lin-good) → Ramamani Yeleswarapu (ramamani-yeleswarapu)
Revision history for this message
Jim Rollenhagen (jim-rollenhagen) wrote :
Revision history for this message
John L. Villalovos (happycamp) wrote :

There is also a cross-project spec on this topic:
https://review.openstack.org/#/c/295543/

Revision history for this message
Jim Rollenhagen (jim-rollenhagen) wrote :

We've agreed we'll hold off on approving this until the aforementioned cross-project spec merges. I don't expect it to deviate much from nova, so please feel free to continue working on the code :)

Revision history for this message
Jim Rollenhagen (jim-rollenhagen) wrote :

This user story is also relevant :) https://review.openstack.org/#/c/291406/7

Changed in ironic:
assignee: Ramamani Yeleswarapu (ramamani-yeleswarapu) → Tan Lin (tan-lin-good)
Changed in ironic:
assignee: Tan Lin (tan-lin-good) → Ramamani Yeleswarapu (ramamani-yeleswarapu)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Changed in ironic:
assignee: Ramamani Yeleswarapu (ramamani-yeleswarapu) → Tan Lin (tan-lin-good)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Changed in ironic:
assignee: Tan Lin (tan-lin-good) → Ramamani Yeleswarapu (ramamani-yeleswarapu)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Changed in ironic:
importance: Undecided → Wishlist
description: updated
Revision history for this message
Jim Rollenhagen (jim-rollenhagen) wrote :

Talked with Dmitry and Lucas, decided this doesn't need to wait on c-p spec. That spec focuses on "categorize, document better, improve the sample file", with an optional "centralize". So I think we can move forward with the centralization.

tags: added: rfe-approved
removed: rfe
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

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

Reviewed: https://review.openstack.org/295946
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=5f59a264c5342336618c19c5b27cb1c2d1ad8bb7
Submitter: Jenkins
Branch: master

commit 5f59a264c5342336618c19c5b27cb1c2d1ad8bb7
Author: Ramamani Yeleswarapu <email address hidden>
Date: Tue Mar 22 09:24:41 2016 -0700

    Centralize config options - [console]

    Nova style refactor of config options in Ironic.

    Partial-Bug: #1561100
    Change-Id: Id3ed2021ec2f236eb501c3b2a3d4e1cf5b19062a

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/301376
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=921bca203f9a0984cb0e21f34cc0a222ea606206
Submitter: Jenkins
Branch: master

commit 921bca203f9a0984cb0e21f34cc0a222ea606206
Author: Ramamani Yeleswarapu <email address hidden>
Date: Tue Mar 22 09:24:41 2016 -0700

    Centralize config options - [cimc]

    Nova style refactor of config options in Ironic.

    Change-Id: Id64f4919d70293fa61226651a3ec40257d91e73e
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/301382
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=73c89c0407cec4735a2ffde43b64af2efa8cd0c1
Submitter: Jenkins
Branch: master

commit 73c89c0407cec4735a2ffde43b64af2efa8cd0c1
Author: Ramamani Yeleswarapu <email address hidden>
Date: Mon Apr 4 16:01:45 2016 -0700

    Centralize config options - [cisco_ucs]

    Nova style refactor of config options in Ironic.

    Change-Id: I61e906e79a7c26b35e60a72ead9331b6b8f3ebe7
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/301399
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=c0743e42b77484558b1de856439bc4d13765b5f1
Submitter: Jenkins
Branch: master

commit c0743e42b77484558b1de856439bc4d13765b5f1
Author: Ramamani Yeleswarapu <email address hidden>
Date: Mon Apr 4 16:30:16 2016 -0700

    Centralize config options - [conductor]

    Nova style refactor of config options in Ironic.

    Change-Id: Ib04214490e5c796da019c5d7199ace9f131aaa4d
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/301815
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=f38565e7517779ffae2edf52e667b5ea8d99c434
Submitter: Jenkins
Branch: master

commit f38565e7517779ffae2edf52e667b5ea8d99c434
Author: Ramamani Yeleswarapu <email address hidden>
Date: Tue Apr 5 09:35:56 2016 -0700

    Centralize config options - [database]

    Nova style refactor of config options in Ironic.

    Change-Id: Ic5e2148565edf926c4279b44e9f262a46847b4d0
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/301830
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=522fa1c93ee48a70c7a1d491d734563c3b532f77
Submitter: Jenkins
Branch: master

commit 522fa1c93ee48a70c7a1d491d734563c3b532f77
Author: Ramamani Yeleswarapu <email address hidden>
Date: Tue Apr 5 10:08:18 2016 -0700

    Centralize config options - [dhcp]

    Nova style refactor of config options in Ironic.

    Change-Id: Ie801d2f2c1fe6b52e99a491b88f45e658f15ef1c
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/301856
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=0c50ee3d26ab1ac66452aec63a9efe6e9cb8f2d0
Submitter: Jenkins
Branch: master

commit 0c50ee3d26ab1ac66452aec63a9efe6e9cb8f2d0
Author: Ramamani Yeleswarapu <email address hidden>
Date: Tue Apr 5 11:13:55 2016 -0700

    Centralize config options - [iboot]

    Nova style refactor of config options in Ironic.

    Change-Id: Id472e88bd550088c8b0d4f3ed7e7c04b05fa8b8a
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/301947
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=ba30be8238701a2949d19b119a5f582a738c95bc
Submitter: Jenkins
Branch: master

commit ba30be8238701a2949d19b119a5f582a738c95bc
Author: Ramamani Yeleswarapu <email address hidden>
Date: Tue Apr 5 15:10:19 2016 -0700

    Centralize config options - [ilo]

    Nova style refactor of config options in Ironic.

    Change-Id: I6a84ba4ba1a7721c03efaf682b94b89e935da4a4
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/301955
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=fffd74176940003e6d815138ea2c33a655e7118b
Submitter: Jenkins
Branch: master

commit fffd74176940003e6d815138ea2c33a655e7118b
Author: Ramamani Yeleswarapu <email address hidden>
Date: Tue Apr 5 15:37:42 2016 -0700

    Centralize config options - [inspector]

    Nova style refactor of config options in Ironic.

    Change-Id: I34bc2e5fb54661389d62fa51dc05f72022f482a6
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/301967
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=700ad0567b0f93bbb3c527093156e7c4f09f9402
Submitter: Jenkins
Branch: master

commit 700ad0567b0f93bbb3c527093156e7c4f09f9402
Author: Ramamani Yeleswarapu <email address hidden>
Date: Tue Apr 5 16:12:02 2016 -0700

    Centralize config options - [ipmi]

    Nova style refactor of config options in Ironic.

    Change-Id: I3ca346cca5161baeaefc1f976ae5ec38c0122eab
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/302476
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=038d600b8e8286500dadcbaa07d2f7116433b063
Submitter: Jenkins
Branch: master

commit 038d600b8e8286500dadcbaa07d2f7116433b063
Author: Ramamani Yeleswarapu <email address hidden>
Date: Wed Apr 6 16:33:33 2016 -0700

    Centralize config options - [irmc]

    Nova style refactor of config options in Ironic.

    Change-Id: Ia5d4e49c0a8d2de47d34ef9f0b4dfa740c500efa
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/303093
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=d6a9f08051531b18aa9f89afe9e9d9dcbdc94139
Submitter: Jenkins
Branch: master

commit d6a9f08051531b18aa9f89afe9e9d9dcbdc94139
Author: Ramamani Yeleswarapu <email address hidden>
Date: Thu Apr 7 15:00:25 2016 -0700

    Centralize config options - [keystone]

    Nova style refactor of config options in Ironic.

    Change-Id: Ic95322f3a0c7bc36ed7f8def0c1aba7f4e813882
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/303103
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=de43a3308435f1d28e06f4cbf1f52ef3eed02ad0
Submitter: Jenkins
Branch: master

commit de43a3308435f1d28e06f4cbf1f52ef3eed02ad0
Author: Ramamani Yeleswarapu <email address hidden>
Date: Thu Apr 7 16:02:34 2016 -0700

    Centralize config options - [oneview]

    Nova style refactor of config options in Ironic.

    Change-Id: I47c6dea0a255b335c5f8001acfdaf152d55bd994
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/303113
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=31e80ab9bd474c7b3333efde02fd2838919fa369
Submitter: Jenkins
Branch: master

commit 31e80ab9bd474c7b3333efde02fd2838919fa369
Author: Ramamani Yeleswarapu <email address hidden>
Date: Thu Apr 7 16:33:43 2016 -0700

    Centralize config options - [seamicro]

    Nova style refactor of config options in Ironic.

    Change-Id: Ia8e38f448253b58ed63edd1c6aa64da324578cda
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/303644
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=c3d8918f975f5d24284350e5d663bb9c2810708f
Submitter: Jenkins
Branch: master

commit c3d8918f975f5d24284350e5d663bb9c2810708f
Author: Ramamani Yeleswarapu <email address hidden>
Date: Fri Apr 8 15:27:59 2016 -0700

    Centralize config options - [snmp]

    Nova style refactor of config options in Ironic.

    Change-Id: I2a803ced4080c5f3358b9ac939d4864151cca5b6
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/303649
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=b05491433fbd9d6bfde9817ce7d3b60f773f4d87
Submitter: Jenkins
Branch: master

commit b05491433fbd9d6bfde9817ce7d3b60f773f4d87
Author: Ramamani Yeleswarapu <email address hidden>
Date: Fri Apr 8 15:39:54 2016 -0700

    Centralize config options - [ssh]

    Nova style refactor of config options in Ironic.

    Change-Id: I1b3e1f6d05d9fbea125933f6a410e4907fb0fadb
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/303653
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=4938c8ec63c155ff91c414966d522703c233f2bb
Submitter: Jenkins
Branch: master

commit 4938c8ec63c155ff91c414966d522703c233f2bb
Author: Ramamani Yeleswarapu <email address hidden>
Date: Fri Apr 8 15:53:07 2016 -0700

    Centralize config options - [swift]

    Nova style refactor of config options in Ironic.

    Change-Id: If2539aa019e9f2e03f74c9e32da50b9fad6f611d
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/303655
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=28f74a8c9c0f954851ce71bcde9605850c17dbba
Submitter: Jenkins
Branch: master

commit 28f74a8c9c0f954851ce71bcde9605850c17dbba
Author: Ramamani Yeleswarapu <email address hidden>
Date: Fri Apr 8 16:05:09 2016 -0700

    Centralize config options - [virtualbox]

    Nova style refactor of config options in Ironic.

    Change-Id: I9c7b1ea0c4c35eaacc4655d605993297242f0c0a
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/304829
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=dbdd01bda219285fd5a9a564eb7e136157413188
Submitter: Jenkins
Branch: master

commit dbdd01bda219285fd5a9a564eb7e136157413188
Author: Ramamani Yeleswarapu <email address hidden>
Date: Tue Apr 12 12:25:50 2016 -0700

    Centralize config options - [glance]

    Nova style refactor of config options in Ironic.

    Change-Id: If91ab834f32f571e9c669ec5888bfc8de03bb2e0
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/304838
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=af78314694c6d91d0f33c226581d49c2a327abbc
Submitter: Jenkins
Branch: master

commit af78314694c6d91d0f33c226581d49c2a327abbc
Author: Ramamani Yeleswarapu <email address hidden>
Date: Tue Apr 12 12:54:00 2016 -0700

    Centralize config options - [neutron]

    Nova style refactor of config options in Ironic.

    Change-Id: I5963f08dd2d476ebf242851b6b4647916393d648
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/309070
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=6f0aea966b49ec4c1470d18e91fb2bf95cfb5963
Submitter: Jenkins
Branch: master

commit 6f0aea966b49ec4c1470d18e91fb2bf95cfb5963
Author: Ramamani Yeleswarapu <email address hidden>
Date: Fri Apr 15 15:51:42 2016 -0700

    Centralize config options - [DEFAULT]

    Nova style refactor of config options in Ironic.

    Change-Id: I6baebf1bbc829238441ddd1399f6487fad33a15e
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/309186
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=2de6e17cf202b01e7134ffd94a16d0ed4a69745b
Submitter: Jenkins
Branch: master

commit 2de6e17cf202b01e7134ffd94a16d0ed4a69745b
Author: Ramamani Yeleswarapu <email address hidden>
Date: Thu Apr 21 14:33:26 2016 -0700

    Centralize config options - [api]

    Nova style refactor of config options in Ironic.

    Change-Id: I67ebeca9a4efb6065d2b4f0a7bcf5830b0cc5e3f
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/309206
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=a351fd6a4d68864ad3655f00c779ffad0be04cdd
Submitter: Jenkins
Branch: master

commit a351fd6a4d68864ad3655f00c779ffad0be04cdd
Author: Ramamani Yeleswarapu <email address hidden>
Date: Thu Apr 21 15:34:34 2016 -0700

    Centralize config options - [deploy]

    Nova style refactor of config options in Ironic.

    Change-Id: Ic303847415d5f3564510a8f1b1a40168b8a165f8
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/309603
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=46fcfb3199254107df1dfe8295f8ad1e09624c64
Submitter: Jenkins
Branch: master

commit 46fcfb3199254107df1dfe8295f8ad1e09624c64
Author: Ramamani Yeleswarapu <email address hidden>
Date: Fri Apr 22 14:37:24 2016 -0700

    Centralize config options - [agent]

    Nova style refactor of config options in Ironic.

    Change-Id: I97d3a616ca8b68a74d4ad546f81f3a94ce4bad2a
    Partial-Bug: #1561100

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

Related fix proposed to branch: master
Review: https://review.openstack.org/354012

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

Reviewed: https://review.openstack.org/354012
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=58fea0ab509c96aa236e4522059ad78c48484b33
Submitter: Jenkins
Branch: master

commit 58fea0ab509c96aa236e4522059ad78c48484b33
Author: Dao Cong Tien <email address hidden>
Date: Thu Aug 11 19:17:14 2016 +0700

    Remove CONF.import_opt() from api/controllers/v1/node.py

    Since all configs moved to ironic/conf dir, the CONF.import_opt()
    located in ironic/api/controllers/v1/node.py is now incorrect.

    Change-Id: I852baae089c6778a0696995589b864e395e7f036
    Related-Bug: #1561100

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

Reviewed: https://review.openstack.org/309612
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=91af2530f10c65dd9892fc414652f1ecc4ee9b67
Submitter: Jenkins
Branch: master

commit 91af2530f10c65dd9892fc414652f1ecc4ee9b67
Author: Ramamani Yeleswarapu <email address hidden>
Date: Fri Apr 22 15:45:34 2016 -0700

    Centralize config options - [pxe]

    Nova style refactor of config options in Ironic.

    Change-Id: I99f954dcc0097fd62b8c6b538bd1577a09e71a35
    Partial-Bug: #1561100

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

Reviewed: https://review.openstack.org/353907
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=531bca4f6247e07573db09207150d3b37350d3cd
Submitter: Jenkins
Branch: master

commit 531bca4f6247e07573db09207150d3b37350d3cd
Author: Dao Cong Tien <email address hidden>
Date: Thu Aug 11 15:47:17 2016 +0700

    Remove incorrect CONF.import_opt() from test_ipmitool.py

    Since all configs moved to ironic/conf dir, the CONF.import_opt()
    located in test_ipmitool.py is now incorrect.

    Related-Bug: #1561100

    Change-Id: Ia1e4b947d46b212cf81759dc6a4a7d7ceb435caa

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

Reviewed: https://review.openstack.org/309615
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=f609e3a0d7112777566a68174561c9cdac6f97c3
Submitter: Jenkins
Branch: master

commit f609e3a0d7112777566a68174561c9cdac6f97c3
Author: Ramamani Yeleswarapu <email address hidden>
Date: Fri Apr 22 16:03:36 2016 -0700

    Centralize config options - [iscsi]

    Nova style refactor of config options in Ironic.

    Change-Id: I238af0a0d92073cdf62b5303550e5ada25a69655
    Partial-Bug: #1561100

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/330160
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=0791fa7c96b7ae6e568fa9fd3386c8975a55a9b3
Submitter: Jenkins
Branch: master

commit 0791fa7c96b7ae6e568fa9fd3386c8975a55a9b3
Author: Ramamani Yeleswarapu <email address hidden>
Date: Wed Jun 15 11:51:28 2016 -0700

    Centralize Config Options - patch merge, cleanup

    Nova style refactor of config options in Ironic.

    - Merge cimc, cisco_ucs options into one file: conf/cisco_ucs.py
    - Remove unnecessary import_opts
    - Move [amt] group options to conf/amt.py
    - Renamed conf/cisco_ucs.py as conf/cisco.py
    - Removed netconf.py.

    Change-Id: I3c3f5d21de26a9d23ce766c24674f09999f610f5
    Closes-Bug: #1561100

Changed in ironic:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.