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

Yeleswarapu, Ramamani ramamani.yeleswarapu at intel.com
Tue May 30 17:19:01 UTC 2017


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. booting from volume:
    1.1. the next patch: https://review.openstack.org/#/c/406290
2. driver composition documentation:
    2.1. explaining the defaults: https://review.openstack.org/466741
    2.2. ipmi docs update: https://review.openstack.org/466734
3. OSC commands for ironic driver-related commands
    3.1. finish and review the spec: https://review.openstack.org/#/c/439907/


Bugs (dtantsur, vdrok, TheJulia)
--------------------------------
- Stats (diff between 15 May 2017 and 22 May 2017)
- Ironic: 243 bugs (-9) + 251 wishlist items. 24 new (+3), 191 in progress (-9), 0 critical, 26 high and 32 incomplete
- Inspector: 12 bugs + 28 wishlist items. 3 new (+2), 12 in progress (-2), 0 critical, 1 high (-1) and 3 incomplete
- Nova bugs with Ironic tag: 12 (+1). 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 working on getting together devstack config updates/script changes in order to support this configuration No updates
        - hshiina uploaded some devstack patches [See etherpad]
    - 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/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:
    - Based on feedback from vdrok and jlvillal, rloo is rethinking/reworking the next patch to see if it can be simplified: 'Add version column' https://review.openstack.org/#/c/412397/
    - 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 (jroll, dtantsur)
----------------------------------------------
- no updates, dtantsur plans to start working on some text for the install-guide this week

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
- Status as on May 18.
    - Raised a patch in devstack for enabling swift and ironic for python3.5. https://review.openstack.org/#/c/464932/
    - Swift is not completely compatible with python3.5. Getting error while installing devstack. Raised a bug in swift https://bugs.launchpad.net/swift/+bug/1691090
- Status as on May29.
Raised a patch in openstack-infra for disabling swift in experimental gate as swift is still not compatible for py3.5.https://review.openstack.org/467538
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

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 29 May 2017:
    - install guide / admin guide docs
        - ipmi docs update: https://review.openstack.org/466734
        - explaining defaults calculation: https://review.openstack.org/466741
        - upgrading to hardware types: TODO
        - dev docs: 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/
            - needs addressing comments

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)
-----------------------------------------
- 29 May 2017 update
    - spec merged: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/cli-default-api-version.html
    - 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
- status as of 29 May 2017:
    - 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
- any updates here?

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 29th:
    - the power actions do not wait for the power state to change, dtantsur will fix it
    - sushy 0.2.0 released with the latest improvements
    - 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 29th:
    - 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 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
- 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).
        - The first patch is for adding etags for nodes endpoint, https://review.openstack.org/392213
        - everything needs a rebase
    - (galyna): Spec has merged: http://specs.openstack.org/openstack/ironic-specs/specs/approved/evolve-etags.html
    - (galyna) Patches-POC are up. Demo: https://asciinema.org/a/103604

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

Drivers:
--------

OneView (ricardoas/xavierr)
~~~~~~~~~~~~~~~~~~~~~~~~~~~
- stable/newton of agent_pxe_oneview does not work
- deprecating python-oneviewclient (using python-hpOneView and python-ilorest-library instead)

.........

Until next week,
--Rama

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





More information about the OpenStack-dev mailing list