Open Stack

Hi,

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. rolling upgrades
    1.1. the next patch is ready for reviews: https://review.openstack.org/#/c/412397/
2. review next BFV patch:
    2.1. next: https://review.openstack.org/#/c/366197/
3. install guide updates related to the driver composition:
    3.1. configuration https://review.openstack.org/462151
    3.2. enrollment: https://review.openstack.org/463609
4. review e-tags spec: https://review.openstack.org/#/c/381991/


Bugs (dtantsur, vdrok, TheJulia)
--------------------------------
- Stats (diff between 01 May 2017 and 15 May 2017)
- Ironic: 252 bugs + 251 wishlist items. 21 new (-1), 200 in progress (-1), 0 critical, 26 high and 32 incomplete (+1)
- Inspector: 12 bugs (-2) + 28 wishlist items (-2). 1 new (-2), 14 in progress (-1), 0 critical, 2 high (+1) 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:
    - Updated driver patch to address hshiina's findings [mjturek].
    - mjturek is working on getting together devstack config updates/script changes in order to support this configuration
        - Getting back to this this week. Setting up environment and seeing how far I can get with the current patches.
    - 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/366197/ Cinder Driver
            https://review.openstack.org/#/c/406290 Wiring in attach/detach operations
            https://review.openstack.org/#/c/413324 iPXE template
            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
        - https://review.openstack.org/456166 MERGED

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"
- Nisha will be taking over this work(Nisha on leave from May 5 to May 22)
- Status as on May 5.  Raised patches in openstack-infra/project-config for adding experimental gates for the ironic governed modules
    - https://review.openstack.org/462487  - python-ironicclient
    - https://review.openstack.org/462511- IPA(has one +2)
    - https://review.openstack.org/462695- ironic-inspector
    - https://review.openstack.org/462701- ironic-lib
    - https://review.openstack.org/#/c/462706/- python-ironic-inspector-client
    - Not sure, if we want to do the same for ironic-staging-drivers module or not, hence not raised
        - one day - yes, but it's not a priority at all
- Ankit took over this work
    - Status as on May 12.
    - Upstream python3 experimental gate for ironic is failing because of failing of s-proxy process and it is giving error ImportError: No module named keystonemiddleware.auth_token
    But in the local environment with enablement of python 3 it is passing.

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:bg/1525218

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 11 May 2017:
    - client changes merged and released as part of ironicclient 1.13
    - install guide / admin guide docs
        - configuration https://review.openstack.org/462151
        - enrollment: https://review.openstack.org/463609
        - upgrading to hardware types: TODO
    - new hardware types:
        - we need volunteers for SNMP
        - rpioso volunteered to do drac
    - future of the driver properties API spec:
        - https://review.openstack.org/#/c/439907/
    - dtantsur started a spec on the classic drivers deprecation:
        - https://review.openstack.org/#/c/464046/

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)
-----------------------------------------
- 15 May 2017 update
    - Spec updated; scopes work to CLI only:  https://review.openstack.org/#/c/448661/
    - OSC CLI patch is up, passing unit and functional tests  https://review.openstack.org/#/c/442153/
        - needs updating to match the recent spec changes (mostly wording of the deprecation message)

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 needs rebasing

High Priorities
===============

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

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
- Physical network awareness implementation
    - Port data model: https://review.openstack.org/#/c/461301/
    - OSC: https://review.openstack.org/#/c/461893
    - Minor update to spec: https://review.openstack.org/#/c/464644/
    - Physical network-aware port mapping: in development
- 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

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 merged: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ipa_api_versioning.html

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

Redfish driver (lucasagomes, dtantsur)
--------------------------------------
- topic: https://review.openstack.org/#/q/topic:bug/1526477
- status as of May 15th:
    - Documentation merged: https://docs.openstack.org/developer/ironic/drivers/redfish.html
    - CI jobs run non-voting on ironic and sushy
    - Logging and error handling in Sushy needs some love: https://review.openstack.org/463680
        - and minor bug fix for session usage: https://review.openstack.org/#/c/464188/
        - we need more people reviewing sushy
- OOB inspection is in progress, but is probably outside of the scope of this particular topic

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 May 14th:
    - next on review: https://review.openstack.org/#/c/382194/ (needs update)

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 (pas-ha)
-------------------------------------------
- RFE: https://bugs.launchpad.net/ironic/+bug/1605728
- status as of most recent weekly meeting:
    - (pas-ha) I'm taking over these patches to drive them further since Galyna most probably won't have time to do this (she changed job to non-OpenStack-related).
    - (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

Subprojects
===========

Inspector (dtansur, milan)
--------------------------
- No updates

Bifrost (TheJulia)
------------------
- Ocata stable branch build may be broken.  Investigating.

ironic-ui (betherly, ppiela, TheJulia)
--------------------------------------
- No Updates

.........

Until next week,
--Rama

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