[openstack-dev] [ironic] this week's priorities and subteam reports

Yeleswarapu, Ramamani ramamani.yeleswarapu at intel.com
Mon Jul 17 18:33:59 UTC 2017


We are glad to present this week's priorities and subteam report for Ironic. As usual, this is pulled directly from the Ironic whiteboard[0] and formatted.

This Week's Priorities (as of the weekly ironic meeting)
1. Docs due to the docs re-org - See http://lists.openstack.org/pipermail/openstack-dev/2017-July/119221.html
    1.1. Ironic - https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:master+topic:doc-migration
    1.2. Ironic-ui - https://review.openstack.org/#/q/status:open+project:openstack/ironic-ui+branch:master+topic:doc-migration
    1.3. ironic-python-agent - https://review.openstack.org/#/q/status:open+project:openstack/ironic-python-agent+branch:master+topic:doc-migration
    1.4. ironic-inspector - https://review.openstack.org/#/q/status:open+project:openstack/ironic-inspector+branch:master+topic:doc-migration
    1.5. Other subprojects and repos have not been started: virtualbmc, sushy, sushy-tools, moltenironic, ironic-inspector-client
2. Booting from volume:
    2.1. https://review.openstack.org/#/c/472740 - Tempest Scenario
    2.2. https://review.openstack.org/#/c/466333 - Devstack support
    2.3. https://review.openstack.org/#/c/215385 - Nova patch
3. Rolling upgrades:
    3.1. Modifications for rolling upgrades: https://review.openstack.org/#/c/476779/
    3.2.  'Add new dbsync command with first online data migration': https://review.openstack.org/#/c/408556/
4. Physnet awareness:
    4.1. API patch: https://review.openstack.org/469933
5. Nova patch for VIF attach/detach: https://review.openstack.org/#/c/419975/

Bugs (dtantsur, vdrok, TheJulia)
- Stats (diff between 10 Jul 2017 and 17 Jul 2017)
- Ironic: 257 bugs (-2) + 258 wishlist items. 28 new (+1), 209 in progress (-5), 1 critical, 32 high (-1) and 31 incomplete
- Inspector: 14 bugs (+1) + 28 wishlist items. 2 new, 12 in progress, 1 critical (+1), 4 high (+1) and 3 incomplete
- Nova bugs with Ironic tag: 17 (+1). 5 new (+1), 0 critical, 0 high

Essential Priorities

CI refactoring and missing test coverage
- not considered a priority, it's a 'do it always' thing
- Standalone CI tests (vsaienk0)
    - next patch to be reviewed, needed for 3rd party CI: https://review.openstack.org/#/c/429770/
- Missing test coverage (all)
    - portgroups and attach/detach tempest tests: https://review.openstack.org/382476
    - local boot with partition images: TODO https://bugs.launchpad.net/ironic/+bug/1531149
    - adoption: https://review.openstack.org/#/c/344975/
        - should probably be changed to use standalone tests
    - root device hints: TODO

Generic boot-from-volume (TheJulia, dtantsur)
- specs and blueprints:
    - http://specs.openstack.org/openstack/ironic-specs/specs/approved/volume-connection-information.html
        - code: https://review.openstack.org/#/q/topic:bug/1526231
    - http://specs.openstack.org/openstack/ironic-specs/specs/approved/boot-from-volume-reference-drivers.html
        - code: https://review.openstack.org/#/q/topic:bug/1559691
    - https://blueprints.launchpad.net/nova/+spec/ironic-boot-from-volume
        - code: https://review.openstack.org/#/q/topic:bp/ironic-boot-from-volume
- status as of most recent weekly meeting:
    - Python-ironicclient API support for volume connectors was landed last week.
        - Version 1.14.0 was released last week and global-requirements was updated accordingly
        - These should be part of the next release, meaning 1.15.0 when released.
    - We have observed some review activity on the nova patch: https://review.openstack.org/#/c/215385/
        - Mostly positive review feedback. Some concern from nova as to the lack of n-1 support where nova is upgraded prior to ironic, which is contrary to our upgrade guide.
    - Patch/note tracking etherpad: https://etherpad.openstack.org/p/Ironic-BFV
        Ironic Patches:
            https://review.openstack.org/#/c/214586/ - Volume Connection Information Rest API Change MERGED
            https://review.openstack.org/#/c/463930/ - CRUD notification updates for volume objects. MERGED
            https://review.openstack.org/#/c/463908/ - Enable cinder storage interface for generic hardware - MERGED
            https://review.openstack.org/#/c/463972/ - Add storage_interface to notifications  MERGED
            https://review.openstack.org/#/c/466333/ - Devstack changes or Boot from Volume - Has review feedback
            https://review.openstack.org/#/c/472740/ - Tempest test scenario for BFV
            https://review.openstack.org/#/c/479326/ - BFV deploy follow-up - Requires revision
            https://review.openstack.org/#/c/427053/ - OSC volume connector - MERGED
            https://review.openstack.org/#/c/427738 - OSC volume target - MERGED
            https://review.openstack.org/#/c/466186/ - support storage interface  - MERGED
            https://review.openstack.org/#/c/479840 - Ironic cli volume connector commands - Approved to Merge
            https://review.openstack.org/#/c/479848 - ironic cli volume target commands - Approved to Merge

Rolling upgrades and grenade-partial (rloo, jlvillal)
- spec approved; code patches: https://review.openstack.org/#/q/topic:bug/1526283
- status as of most recent weekly meeting:
    - next patches ready for reviews:
        - 'Modifications for rolling upgrades': https://review.openstack.org/#/c/476779/ 2x +2
        - 'Add new dbsync command with first online data migration': https://review.openstack.org/#/c/408556/
    - to address restarting services after unpinning, spec ready for reviews: 'SIGHUP restarts services with updated configs': https://review.openstack.org/474309
    - Testing work: done as per spec, but rloo wants to ask vasyl whether we can improve. grenade test will do upgrade so we have old API sending requests to old and/or new conductor, but rloo doesn't think there is anything to control -which- conductor handles the request, so what if old conductor handles all the requests?

Reference architecture guide (dtantsur)
- no updates

Driver composition (dtantsur)
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/approved/driver-composition-reform.html
- gerrit topic: https://review.openstack.org/#/q/status:open+topic:bug/1524745
- status as of 17 Jul 2017:
    - install guide / admin guide docs
        - done except for:
            - upgrading to hardware types: TODO
            - dev docs: TODO
            - fixing nits from previous reviews
    - new hardware types:
        - we need volunteers for SNMP
        - rpioso volunteered to do drac
    - API for hardware interface properties:
        - proposed spec: https://review.openstack.org/#/c/471174/
        - rloo checked with dtantsur, is is not needed for completion of driver composition
    - spec on the classic drivers deprecation: https://review.openstack.org/#/c/464046/
        - MERGED! no actions for Pike, let's talk on the PTG about the way we proceed in Queens.

OSC default API version change (dtantsur)
- status as of 17 Jul 2017:
    - spec merged: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/cli-default-api-version.html
    - OSC CLI patch MERGED: https://review.openstack.org/#/c/442153/
    - dtantsur to make a patch for the "ironic" tool in a similar way

High Priorities

Post-deploy VIF attach/detach (sambetts, vsaienk0)
- blueprint: https://blueprints.launchpad.net/nova/+spec/ironic-hotplug-interfaces
- gerrit topic (nova): https://review.openstack.org/#/q/topic:bp/ironic-plug-unplug-vifs-update
- Nova side https://review.openstack.org/#/c/419975/

Physical network awareness (sambetts, vsaienk0)
- Etherpad tracking networking work: https://etherpad.openstack.org/p/Ironic-networking
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/physical-network-awareness.html
- Physical network awareness implementation
    - ironic:
        - API: https://review.openstack.org/469933
        - Tempest tests: https://review.openstack.org/#/c/470915
        - TODO: Rolling upgrades support
    - python-ironicclient:
        - OSC & ironic clients: https://review.openstack.org/#/c/461893
    - Related:
        - Refactor VIFPortIDMixin, separate out neutron-specific code into a subclass: https://review.openstack.org/477597 (ready for review)
        - Create a fake network interface for tempest testing: <included in above patch currently, may be refactored out)
        - Assume neutron port exists in VIFPortIDMixin: https://review.openstack.org/#/c/476949
- networking-baremetal initial patches
    - https://review.openstack.org/448073 MERGED
    - https://review.openstack.org/448157
    - https://review.openstack.org/450714
- Routed networks support (sambetts, vsaienk0)
    - Dependant on physnet awareness
    - networking-baremetal initial patches
        - https://review.openstack.org/#/c/456235/
    - RFE: https://bugs.launchpad.net/ironic/+bug/1658964
        - No RFE right now as all planned work is in networking-baremetal, but going to keep this around in case we need to make unexpected changes to ironic

ironic-python-agent's REST API versioning (sambetts)
- RFE: https://bugs.launchpad.net/ironic-python-agent/+bug/1602265
- gerrit topic: https://review.openstack.org/#/q/topic:bug/1602265
- status as of 5 Jun 2017:
    - spec merged: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ipa_api_versioning.html
    - code needs rebasing and updating

Optional Priorities

Python 3.5 compatibility (Nisha, Ankit)
- Topic: https://review.openstack.org/#/q/topic:goal-python35+NOT+project:openstack/governance+NOT+project:openstack/releases
    - this include all projects, not only ironic
    - please tag all reviews with topic "goal-python35"
other patches for experimental gates are not merging as core reviewers are asking to add the python3 builder in running gates instead of duplicating them in project--config project. https://review.openstack.org/462487,
https://review.openstack.org/462695, https://review.openstack.org/462701 and https://review.openstack.org/462706
- Two ironic patches in review: https://review.openstack.org/473888 and  https://review.openstack.org/#/c/473913/.

node tags (zhenguo, vdrok)
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/approved/nodes-tagging.html
- topic: https://review.openstack.org/#/q/topic:bug/1526266
- status as of 19 Jun 2017:
    - next on review: https://review.openstack.org/#/c/233357/

Rescue mode (stendulker/aparnav)
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/approved/implement-rescue-mode.html
- code: https://review.openstack.org/#/q/topic:bug/1526449+status:open
- Status: 17 Jul 2017
- Need to resolve merge conflicts. Will resolve shortly.

Split away the tempest plugin (jlvillal)
- Have not had time yet to finish scripts (jlvillal) 10-July-2017
- patches waiting for review https://review.openstack.org/#/q/project:openstack/ironic-tempest-plugin
- need to port new patches in ironic/ironic-tempest-plugin (jlvillal)
- need to migrate ironic-inspector/ironic-tempest-plugin (jlvillal)

deploy steps (rloo, mat128, yolanda)
- Ocata summit design session etherpad: https://etherpad.openstack.org/p/ironic-ocata-summit-deploy-time-raid
- RFE: https://bugs.launchpad.net/ironic/+bug/1630376
- status as of most recent weekly meeting:
    - spec needs updating: https://review.openstack.org/#/c/412523/

Supported power states API (dtantsur)
- topic: https://review.openstack.org/#/q/topic:bug/1526226
    - note: the same topic as for soft power and NMI is used, as it's covered by the same spec
- status as of Jul 17th:
    - next on review: https://review.openstack.org/#/c/382194/ (needs update)
        - no updates since May, I wonder if somebody has to take it over..

Available clean steps API (rloo)
- spec had been approved in mitaka: http://specs.openstack.org/openstack/ironic-specs/specs/5.0/manual-cleaning.html#get-nodes-node-ident-cleaning-steps
- status as of most recent weekly meeting:
    - patch has been rebased and updated, but reviewer comments still need to be addressed and it needs testing: https://review.openstack.org/#/c/278566/

etags in the REST API (pas-ha)
- RFE: https://bugs.launchpad.net/ironic/+bug/1605728
- topic: https://review.openstack.org/#/q/topic:bug/1605728
- Spec has merged: http://specs.openstack.org/openstack/ironic-specs/specs/approved/evolve-etags.html
- status as of Jul 17th:
    - The first patch is for adding etags for nodes endpoint, needs rebasing: https://review.openstack.org/392213
        - everything needs a rebase
    - (galyna) Patches-POC are up. Demo: https://asciinema.org/a/103604
-Moved to later releases
These are the things we had on priorities, but they are not making it in Pike.

Deploying with Apache and WSGI in CI (vsaienk0)
- ironic part seems finished (needs double-checking)
    - do we have install-guide bits on how to do it?
- inspector is TODO and depends on https://review.openstack.org/#/q/topic:bug/1525218
    - ironic is done, inspector will be bumped to Queens

Neutron event processing (vdrok, vsaienk0)
- spec at https://review.openstack.org/343684, ready for reviews
- WIP code at https://review.openstack.org/440778


Inspector (dtantsur, milan)
- inspector's gate hosed: https://review.openstack.org/#/c/483059/
- inspector docs affected by the switch (404) patches up, client patches up (to some extend), still WIP
- reviews needed on the worker-api split https://review.openstack.org/#/c/462861/
- anupn succeeded testing the dnsmasq pxe filter in devstack \o/ so it seems it might work

Bifrost (TheJulia)
- Yolanda has proposed a patch for multiple nameservers to be supported as part of the node configuration.

ironic-ui (betherly, ppiela, TheJulia)
- Targeting Pike-3 for release, several outstanding features are in review at this time.

Until next week,

[0] https://etherpad.openstack.org/p/IronicWhiteBoard

More information about the OpenStack-dev mailing list