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

Yeleswarapu, Ramamani ramamani.yeleswarapu at intel.com
Mon Jul 24 18:14:38 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. molteniron - https://review.openstack.org/#/q/status:open+project:openstack/molteniron+branch:master+topic:doc-migration
    1.4. seem done: ironic-lib, ironic-ui, ironic-python-agent, sushy, sushy-tools, python-ironic-inspector-client; bifrost delayed
    1.5. TODO configuration guide for both ironic and ironic-inspector
2. Booting from volume:
    2.1.  https://review.openstack.org/#/c/484032/ -- Create boot.ipxe upon start-up - In Review/Requires revision
    2.2. https://review.openstack.org/#/c/215385 - Nova patch
    2.3. https://review.openstack.org/#/c/472740 - Tempest Scenario
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. Tempest API test: https://review.openstack.org/#/c/470915/ 1x +2
    4.2. Rolling upgrades:
5. ironicclient & ironic-inspector-client patches for release this week:
    5.1. client: https://review.openstack.org/486677 1x +2

Bugs (dtantsur, vdrok, TheJulia)
- Stats (diff between 17 Jul 2017 and 24 Jul 2017)
- Ironic: 259 bugs (+2) + 260 wishlist items (+2). 31 new (+3), 211 in progress (+2), 0 critical (-1), 32 high and 31 incomplete
- Inspector: 14 bugs + 28 wishlist items. 1 new (-1), 12 in progress, 0 critical (-1), 4 high and 3 incomplete
- Nova bugs with Ironic tag: 17. 4 new, 0 critical, 0 high

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 most recent weekly meeting:
    - Last week we identified an issue where a follow-up broke a BFV machine when progressing through to cleaning.
        - This issue was identified, and some discussion has occurred regarding how to properly fix. We are going to remove the follow-up code for a better solution to fix the same problem.
        -  https://review.openstack.org/#/c/484032/
    - Last week we also achieved the first boot of a machine via a cinder volume in OpenStack CI
        - http://logs.openstack.org/12/485812/6/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial/375227c/
        - Achieved via DNM patches - Project-config update has been proposed and presently has one +2.
    - 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 - In Review/Requires revision
            https://review.openstack.org/#/c/472740/ - Tempest test scenario for BFV
            https://review.openstack.org/#/c/479326/ - BFV deploy follow-up - Requires revision - should be landed after the tempest test can be executed.
            https://review.openstack.org/#/c/215385 - Ironic: Support boot from cinder volume - In Review
            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.

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:
    - next patches ready for reviews:
        - 'Modifications for rolling upgrades': https://review.openstack.org/#/c/476779/ MERGED
        - 'Add new dbsync command with first online data migration': https://review.openstack.org/#/c/408556/
    - 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)
- no updates

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 24 Jul 2017:
    - important for Pike
        - upgrade docs on moving to hardware types: WIP 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
        - 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 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 24 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/
    - 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:
        - API: https://review.openstack.org/469933 MERGED
        - Tempest tests: https://review.openstack.org/#/c/470915
        - Rolling upgrades support 1: https://review.openstack.org/#/c/477694
        - Rolling upgrades support 2: https://review.openstack.org/#/c/485773
    - python-ironicclient:
        - OSC & ironic clients: https://review.openstack.org/#/c/461893 MERGED
    - Related:
        - Refactor VIFPortIDMixin, separate out neutron-specific code into a subclass: https://review.openstack.org/477597 (needs update)
        - Create a fake network interface for tempest testing: <included in above patch currently, may be refactored out)
        - Assume neutron port exists in VIFPortIDMixin: https://review.openstack.org/#/c/476949
- networking-baremetal initial patches
    - https://review.openstack.org/448073 MERGED
    - https://review.openstack.org/448157
    - https://review.openstack.org/450714
- Routed networks support (sambetts, vsaienk0)
    - Dependent on physnet awareness
    - networking-baremetal initial patches
        - https://review.openstack.org/#/c/456235/
    - 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 24 Jul 2017:
    - code needs review

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 (needs update)
    - https://review.openstack.org/#/c/473913/ 1x +2

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


Inspector (dtantsur, milan)
- inspector docs affected by the switch (404) patches up, client patches up (to some extend), still WIP, better as of today; some more +2 in the patchset, contributor content merging
- reviews needed on the worker-api split https://review.openstack.org/#/c/462861/
- anupn succeeded testing the dnsmasq pxe filter in devstack \o/ so it seems it might work, updated with Ilya's suggestions
- python-ironic-inspector-client docs update merged

ironic-ui (betherly, ppiela, TheJulia)
- Expecting to release this week.


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