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

Yeleswarapu, Ramamani ramamani.yeleswarapu at intel.com
Mon May 1 19:57:43 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. Pike priorities update: https://review.openstack.org/#/c/460086/
2. rolling upgrades
    2.1. make a change to the grenade job to only upgrade conductor, ready for reviews: https://review.openstack.org/456166
    2.2. the next patch is ready for reviews: https://review.openstack.org/#/c/412397/
3. review next BFV patch:
    3.1. next: https://review.openstack.org/#/c/366197/
4. review e-tags spec: https://review.openstack.org/#/c/381991/

Bugs (dtantsur, vdrok, TheJulia)
- Stats (diff between 24 Apr 2017 and 01 May 2017)
- Ironic: 252 bugs (+2) + 251 wishlist items (-1). 22 new (+5), 201 in progress, 0 critical, 26 high and 31 incomplete (-2)
- Inspector: 14 bugs + 30 wishlist items (-3). 3 new (+1), 15 in progress (-5), 0 critical, 1 high and 3 incomplete
- Nova bugs with Ironic tag: 11. 2 new, 0 critical, 0 high

Essential Priorities

CI refactoring and missing test coverage
- Standalone CI tests (vsaienk0)
    - next patch to be reviewed: 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

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:
    - mjturek is looking into identifying how to address/support a behavior of cinder in regards to required information where cinder expects data, but dirvers do not necessarilly use it.  The cinder driver code originally expected it to be optional, but hshiina found an additional check inside cinder's code base that requires both pieces of information.
    - mjturek is working on getting together devstack config updates/script changes in order to support this configuration
        - It's looking more like all setup can/should happen during tempest.
    - hshiina is looking in Nova side changes and is attempting to obtain clarity on some of the issues that tenant network separation introduced into the deployment workflow.
    - Patch/note tracking etherpad: https://etherpad.openstack.org/p/Ironic-BFV
        Ironic Patches:
            https://review.openstack.org/#/c/454243/ - WIP logic changes for deployment process.  Tenant network separation introduced some additional complexity, quick conceptual feedback requested.
            https://review.openstack.org/#/c/214586/ - Volume Connection Information Rest API Change
        Additional patches exist, for python-ironicclient and one for nova.  Links in the patch/note tracking etherpad.

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:
    - patches ready for reviews. Next one: 'Add version column' (https://review.openstack.org/#/c/412397/)
    - Testing work:
        - 27-Mar-2017: Grenade multi-node is non-voting
        - need to change grenade to only upgrade the conductor, ready for reviews: https://review.openstack.org/456166

Reference architecture guide (jroll, dtantsur)
- no updates

Python 3.5 compatibility (Nisha)
- no updates
- Nisha will be taking over this work(Nisha on leave from May 5 to May 22)

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

Driver composition (dtantsur, jroll)
- 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 most recent weekly meeting:
    - client changes merged
    - TODO as of 01 May 2017
        - install guide / admin guide docs
            - dtantsur volunteered, will try to get to it this week
        - new hardware types:
            - ilo: https://review.openstack.org/#/c/439404/
            - rpioso volunteered to do drac
        - what to do about driver properties API and dynamic drivers?
            - we discussed it on the virtual meetup, rloo to write a spec

Feature parity between two CLIs (rloo, dtantsur)
- OSC driver-properties spec is work in progress: https://review.openstack.org/#/c/439907/
    - we don't have API to show driver properties for dynamic drivers (we show hardware type + default interfaces): https://bugs.launchpad.net/ironic/+bug/1671549. This should not be a blocker for the missing OSC commands but since this will also need OSC support, it might have an impact on the OSC commands we eventually decide on. Discussed in last week's meetup, starting around L137: https://etherpad.openstack.org/p/ironic-virtual-meetup
    - no update this week

OSC default API version change (dtantsur)
- 5/1 update
    - Spec updated; scopes work to CLI only:  https://review.openstack.org/#/c/448661/
        - some comments on the spec, dtantsur to follow-up this week
    - OSC CLI patch is up, passing unit and functional tests  https://review.openstack.org/#/c/442153/

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
- 4/10 update
    - Current goal: get tempest test for rescue passing against CoreOS w/DHCP
    - API/conductor and rescuewait timeout patches up to date
    - Agent driver still needs some work, today's update broke something

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
- code updated, waiting for reviews

High Priorities

specific fault support (?????)
- !!! this topic is candidate to removing from the list per https://review.openstack.org/460086 !!!
- gerrit topic: https://review.openstack.org/#/q/topic:bug/1596107
- status 4/10:
    - Spec needs an update, but please continue reviewing
    - https://review.openstack.org/#/c/334113/

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)
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/physical-network-awareness.html
- networking-baremetal initial patches
    - https://review.openstack.org/448073
    - 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
        - needs approval?
        - Not sure if we need an RFE I think all work is in the networking-baremetal repo.

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

Documentation reorganization (TheJulia)
- !!! this topic is candidate to removing from the list per https://review.openstack.org/460086 !!!

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 most recent weekly meeting:
    - spec updated, waiting for reviews. Has two +2, rloo will +A today if no negative feedback: https://review.openstack.org/#/c/341086/

Optional Priorities

Split away the tempest plugin (jlvillal)
- patches waiting for review https://review.openstack.org/#/q/project:openstack/ironic-tempest-plugin
- need to port new patches in ironic/ironic-tempest-plugin (soliosg)
- need to migrate ironic-inspector/ironic-tempest-plugin (soliosg)

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 under review: https://review.openstack.org/#/c/412523/

Redfish driver (lucasagomes, jroll, dtantsur)
- topic: https://review.openstack.org/#/q/topic:bug/1526477
- status as of most May 1st:
    - Documentation merged: https://docs.openstack.org/developer/ironic/drivers/redfish.html
    - Patch to DevStack (WIP but works) to setup redfish in DevStack: https://review.openstack.org/#/c/445529/
    - CI job: TODO

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 most recent weekly meeting:
    - 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
- patch needs updating, rebasing, and testing: https://review.openstack.org/#/c/278566/
    - no update

etags in the REST API (gzholtkevych)
- RFE: https://bugs.launchpad.net/ironic/+bug/1605728
- status as of most recent weekly meeting:
    - (galyna): Spec: https://review.openstack.org/#/c/381991/. Updated, the spec is a kind of ready, few questions remained
    - (galyna) Patches-POC are up. Demo: https://asciinema.org/a/103604

IPA authentication (jroll, joanna)
- !!! this topic is candidate to removing from the list per https://review.openstack.org/460086 !!!


Inspector (dtansur, milan)
- API/worker split spec merged: http://specs.openstack.org/openstack/ironic-inspector-specs/specs/splitting-service-on-API-and-worker.html
- func3 (with Python 3.5) job is now voting

Bifrost (TheJulia)
- Some fixes/improvements have been proposed around inspector use/support.  Thank you Mark Goddard!
- Review of neutron support requested - https://review.openstack.org/#/c/452514/

ironic-ui (betherly, ppiela, TheJulia)
- Needing reviews. Any one willing to pickup bugs please check with TheJulia.


CIMC and UCSM (sambetts)
- sam wants to rename the Cisco UCSM driver (https://review.openstack.org/#/c/400673/). Given the dynamic drivers, is it worth doing?

SNMP (mat128/xhku/lindycoder)
- snmp gate failure was fixed!

Until next week,

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

More information about the OpenStack-dev mailing list