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. make a change to the grenade job to only upgrade conductor, ready for reviews: https://review.openstack.org/456166
    1.2. 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. review e-tags spec: https://review.openstack.org/#/c/381991/
4. next driver comp client patch: https://review.openstack.org/#/c/419274/


Bugs (dtantsur, vdrok, TheJulia)
--------------------------------
- Stats (diff between 17 Apr 2017 and 24 Apr 2017)
- Ironic: 250 bugs (-5) + 252 wishlist items (-1). 17 new (-5), 201 in progress (-4), 0 critical, 26 high (-2) and 33 incomplete (+1)
- Inspector: 14 bugs (-1) + 33 wishlist items (-1). 2 new, 20 in progress (-1), 0 critical, 1 high and 3 incomplete (-1)
- Nova bugs with Ironic tag: 11 (+1). 2 new (+1), 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 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/366197/
            https://review.openstack.org/#/c/406290
            https://review.openstack.org/#/c/413324
            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)
------------------------------------
- no updates

Python 3.5 compatibility (JayF, hurricanerix)
---------------------------------------------
- no updates

Deploying with Apache and WSGI in CI (vsaienk0)
-----------------------------------------------
- seems like we can deploy with WSGI, but it still uses a fixed port, instead of sub-path
- next one is https://review.openstack.org/#/c/444337/ MERGED
    - are we done with ironic?
- 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:
    - TODO as of 24 Apr 2017
        - install guide / admin guide docs
        - client changes:
            - driver commands update: https://review.openstack.org/419274
            - node-update update: https://review.openstack.org/#/c/431542/ MERGED
        - new hardware types:
            - ilo: https://review.openstack.org/#/c/439404/
    - contentious topics:
        - what to do about driver properties API and dynamic drivers?
            - rloo and dtantsur started brainstorming: https://etherpad.openstack.org/p/ironic-driver-properties-reform
                - proposed this as topic for https://etherpad.openstack.org/p/ironic-virtual-meetup

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.
    - no update this week

OSC default API version change (mariojv, dtantsur)
--------------------------------------------------
- 4/10 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/

Rescue mode (JayF / mariojv)
----------------------------
- 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 (JayF, mariojv)
--------------------------------------
- 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 (JayF, TheJulia)
---------------------------------------------
- Awaiting spec to be posted.

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

Optional Priorities
===================

Split away the tempest plugin (soliosg, 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 (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)
-----------------------------------
- topic: https://review.openstack.org/#/q/topic:bug/1526477
- status as of most recent weekly meeting:
    - the driver has merged \o/
    - Documentation on review: https://review.openstack.org/#/c/442537/
    - Patch to DevStack (WIP but works) to setup redfish in DevStack: https://review.openstack.org/#/c/445529/
    - we need to add sushy to global-requirements

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

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

Inspector (dtansur, milan)
--------------------------
- In discussion of API-conductor split: https://review.openstack.org/#/c/435517/
- PXE/DHCP filtering driver: implementation in progress: https://review.openstack.org/#/c/439603/ ; needs rebase & fix some unittest (maybe split in more patches) ; the spec will have to be amended based on the implementation

Bifrost (TheJulia)
------------------
- Neutron support is under development.  WIP Patches are available at https://review.openstack.org/#/q/status:open+project:openstack/bifrost+branch:master+topic:452509

ironic-ui (betherly, ppiela, TheJulia)
--------------------------------------
- The javascript unit test job is presently not executing the tests. Betherly is working on it.
.........

Until next week,
--Rama

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