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

Yeleswarapu, Ramamani ramamani.yeleswarapu at intel.com
Mon Jul 31 18:26:58 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-inspector - https://review.openstack.org/#/q/status:open+project:openstack/ironic-inspector+branch:master+topic:doc-migration
    1.3. bifrost: https://review.openstack.org/#/q/status:open+project:openstack/bifrost+branch:master+topic:doc-migration
2. Booting from volume:
    2.1. Functional tests: https://review.openstack.org/#/c/473717/
    2.2. api-ref for volumes API: https://review.openstack.org/#/c/478798/
3. Rolling upgrades:
    3.1.  'Add new dbsync command with first online data migration': https://review.openstack.org/#/c/408556/
4. Physnet awareness:
    4.1. Refactoring: https://review.openstack.org/#/q/topic:refactor-vif-attach-mixin
5. IPA API versioning:
    5.1. IPA part: https://review.openstack.org/#/c/364834/

Bugs (dtantsur, vdrok, TheJulia)
- Stats (diff between 24 Jul 2017 and 31 Jul 2017)
- Ironic: 257 bugs (-2) + 259 wishlist items (-1). 30 new (-1), 207 in progress (-4), 0 critical, 30 high (-2) and 31 incomplete
- Inspector: 13 bugs (-1) + 28 wishlist items. 3 new (+2), 10 in progress (-2), 0 critical, 2 high (-2) and 3 incomplete
- Nova bugs with Ironic tag: 18 (+1). 0 new, 0 critical, 1 high (+1)
    - added https://bugs.launchpad.net/nova/+bug/1706772

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 31 Jul 2017:
    - Patch/note tracking etherpad: https://etherpad.openstack.org/p/Ironic-BFV
        Ironic Patches:
            https://review.openstack.org/#/c/466333/ - Devstack changes or Boot from Volume - MERGED
            https://review.openstack.org/#/c/484032/ -- Create boot.ipxe upon start-up - MERGED
            https://review.openstack.org/#/c/472740/ - Tempest test scenario for BFV - MERGED
                as soon as this merges, we need to promote the CI job to non-voting check
            https://review.openstack.org/#/c/486853/ - minor nits - APPROVED
            https://review.openstack.org/#/c/479326/ - BFV deploy follow-up - Requires revision
            https://review.openstack.org/#/c/215385 - Ironic: Support boot from cinder volume - MERGED
            https://review.openstack.org/#/c/468353 - Ironic: Get IP address for volume connector - --NOT REQUIRED FOR PIKE-- - Is a follow-up to the first nova patch to allow
            https://review.openstack.org/#/c/480332 - Add experimental gate job for BFV MERGED

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:
    - last big code patch, ready for reviews:
        - 'Add new dbsync command with first online data migration': https://review.openstack.org/#/c/408556/
            - this is the last critical code patch for this feature
    - 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)
- status as of 31 Jul 2017:
    - Work has started!
    - WIP of the structure and common bits: https://review.openstack.org/487410

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 31 Jul 2017:
    - important for Pike
        - upgrade docs on moving to hardware types: https://review.openstack.org/486696
        - dev docs on writing hardware types: TODO
    - optional for Pike // important for Queens:
        - nits from previous install-guide reviews: https://review.openstack.org/486651 MERGED
        - new hardware types:
            - UCS: https://review.openstack.org/#/c/486100/
            - SNMP: https://review.openstack.org/#/c/485041/
            - DRAC: (rpioso) https://bugs.launchpad.net/bugs/1705741 was approved. Coding is underway.
            - OneView (nicodemos/ricardoas): https://review.openstack.org/#/c/485798/
        - 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: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/classic-drivers-future.html to be continued in Queens

OSC default API version change (dtantsur)
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/cli-default-api-version.html
- status as of 31 Jul 2017:
    - OSC CLI patch MERGED: https://review.openstack.org/#/c/442153/
    - ironic CLI patch: https://review.openstack.org/#/c/484866/ MERGED
        - follow-up with nits: https://review.openstack.org/#/c/486677/ MERGED
    - done for Pike. the actual switch expected mid-Queens.

High Priorities

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:
        - Tempest tests: https://review.openstack.org/#/c/470915 MERGED
        - Tempest tests nit: https://review.openstack.org/487550 APPROVED
        - Rolling upgrades support 1: https://review.openstack.org/#/c/477694 MERGED
        - Rolling upgrades support 2: https://review.openstack.org/#/c/485773 APPROVED
    - Related:
        - Refactor VIFPortIDMixin 1: https://review.openstack.org/#/c/486124/ MERGED
        - Refactor VIFPortIDMixin 2: https://review.openstack.org/#/c/486125/
        - Refactor VIFPortIDMixin 3: https://review.openstack.org/#/c/486126/
        - Refactor VIFPortIDMixin 4: https://review.openstack.org/#/c/486127/
        - Create a fake network interface for tempest testing: https://review.openstack.org/#/c/486128/
        - Use fake network interface in VIF attach tempest tests: https://review.openstack.org/#/c/486129/
        - Assume neutron port exists in VIFPortIDMixin: https://review.openstack.org/#/c/476949
        - Refactor get_physnets_by_portgroup_id: https://review.openstack.org/485772 MERGED
- networking-baremetal initial patches
    - https://review.openstack.org/448073 MERGED
    - https://review.openstack.org/448157 needs updating
    - https://review.openstack.org/450714 1x +2
- Routed networks support (sambetts, vsaienk0)
    - Dependant on physnet awareness
    - networking-baremetal initial patches
        - https://review.openstack.org/#/c/456235/ needs updating
    - 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
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ipa_api_versioning.html
- gerrit topic: https://review.openstack.org/#/q/topic:bug/1602265
- status as of 31 Jul 2017:
    - code needs review: https://review.openstack.org/#/c/364834/

Refactoring of code accessing other services (pas-ha)
- gerrit topic: https://review.openstack.org/#/q/topic:bug/1699547
- status as of 31 Jul 2017:
    - needs requirements sync to bump min KSA version: https://review.openstack.org/#/c/488117/
    - in progress of adding features from new KSA

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 - MERGED
    - https://review.openstack.org/#/c/473913/ - MERGED
    - Raised the patch to move the experimental job to non-voting https://review.openstack.org/489278

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 24th:
    - next on review: https://review.openstack.org/#/c/382194/

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/
-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

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)

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: 24 Jul 2017
- Patches in merge conflict. Need to rebase. The nova patch for Rescue is abandoned and rescue tempest patch(https://review.openstack.org/#/c/452308/) which is dependent on the nova patch is in merge conflict.
    - any plans to revive the nova patch soon(ish)?

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

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/ needs rebase

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/

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



iRMC (hshiina)
- The CI for the driver has been stopped because of our internal resource issues. The issue has been solved now and we will make them start again in August.


Until next week,

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

More information about the OpenStack-dev mailing list