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

Yeleswarapu, Ramamani ramamani.yeleswarapu at intel.com
Mon Aug 21 19:28:09 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. Finishing the ironic pike release
    1.1. needs the reno prelude: https://review.openstack.org/#/c/495316/
    1.2. also fix for iRMC BFV capability name? +1 rloo master: https://review.openstack.org/495736
2. Refactoring of the way we access clients: https://review.openstack.org/#/q/topic:bug/1699547
3. Review specs in preparation for the PTG

Pike Release Plan

- As of Thur Aug 17, in absence of PTL, as discussed with TheJulia, rloo, sambetts, vdrok :)
- see discussion before/after http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2017-08-17.log.html#t2017-08-17T13:18:35

1. Revert reno-prelude patch: https://review.openstack.org/494542 - MERGED
2. Cut a stable/pike release, version 9.0.0 - proposed to openstack/releases https://review.openstack.org/#/c/494530/2 - ttx has confirmed he will process the request - DONE
3.1. For CI to work on master, these need to land: 
	- Increase host_subset_size for ironic https://review.openstack.org/#/c/493990/ - MERGED on all branches
	- Adds 9.0 to release_mappings
		- master: https://review.openstack.org/494620 - MERGED
		- stable/pike; https://review.openstack.org/#/c/494662/ - MERGED
	- Get rid of sourcing stackrc in grenade settings https://review.openstack.org/#/c/480905/ MERGED
3.2. For CI to work on stable/pike, this needs to land:
	- Increase host_subset_size for ironic https://review.openstack.org/#/c/493991/ - MERGED
	- release mapping for 9.0 https://review.openstack.org/#/c/494662/ - MERGED
4. Land patches in master and backport to stable/pike. NOTE: the backported patch must pass CI first, before landing master patch. - DONE
	- patches for iDRAC hardware type (https://review.openstack.org/#/c/491263/ MERGED on master)
		- https://review.openstack.org/494737 MERGED
5. reno-prelude patch - actually, TheJulia and rloo think this should ONLY go into stable/pike branch... ++
	- (TheJulia) Local tests reveal that the output still appears as expected. Master branch will need to receieve "sem-ver: feature" afterwards.
6. Cut first offical stable/pike release 9.1.0 from head of stable/pike
	(TheJulia) I believe 2 are required, 9.0.1 with a reno saying stating what occured in case anyone tries to package >9.0.x,<9.1.0 9.0.1 released, 9.1.0 to follow in a few days
post-mortem: discuss grenade issues wrt not having a stable/release branch until after 'every project that counts' has; we need to come up with a process to handle this since it has occurred since Ocata: http://lists.openstack.org/pipermail/openstack-dev/2017-February/111849.html

Bugs (dtantsur, vdrok, TheJulia)
- Stats (diff between 14 Aug 2017 and 21 Aug 2017)
- Ironic: 246 bugs + 259 wishlist items (+1). 23 new (-1), 187 in progress (-1), 0 critical, 32 high (+1) and 31 incomplete
- Inspector: 12 bugs + 29 wishlist items (+1). 2 new, 10 in progress, 0 critical, 2 high and 3 incomplete
- Nova bugs with Ironic tag: 17. 0 new, 0 critical, 1 high

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

Essential Priorities

Nova resource class based scheduling changes (dtantsur)
- I have to add this, as there are things to finish by end of Pike to avoid problems in Queens
- TODO as of 21 Aug 2017:
    - all done
    - devstack:
        - always set resource class: https://review.openstack.org/491777 MERGED
        - support scheduling based on rsc: https://review.openstack.org/476968 MERGED
    - nova:
        - fix reporting inventory: https://review.openstack.org/492964
    - prevent updating resource_class for active nodes: https://review.openstack.org/#/c/492216/ MERGED
    - upgrade documentation and reno: https://review.openstack.org/491773 MERGED
- Optionally:
    - integration tests: https://review.openstack.org/#/c/443628/

Generic boot-from-volume (TheJulia, dtantsur)
- BFV Meetings on hold until September.
- 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 21 Aug 2017:
    - Pike stuff is done and released, including also the iRMC implementation
    - Patch/note tracking etherpad: https://etherpad.openstack.org/p/Ironic-BFV
        Ironic Patches:
            https://review.openstack.org/#/c/479326/ - BFV deploy follow-up - MERGED
            https://review.openstack.org/#/c/466568/ - devstack testing guide - MERGED
            https://review.openstack.org/#/c/473717/ - functional tests 1x +2
            https://review.openstack.org/#/c/490710/ - moving the experimental job to the check queue MERGED
    - Not required for Pike:
        - Nova:
            https://review.openstack.org/#/c/468353 - Ironic: Get IP address for volume connector

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:
    - 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?
        - outstanding: add 'ironic-dbsync online_data_migrations' call to regular grenade test (not multinode test because grenade multinode test doesn't upgrade all services)
    - to address restarting services after unpinning, spec ready for reviews: 'SIGHUP restarts services with updated configs': https://review.openstack.org/474309
    - outstanding: dev documentation needs updating: https://review.openstack.org/#/c/419439/

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 21 Aug 2017:
    - documentation
        - upgrade docs on moving to hardware types: https://review.openstack.org/486696 MERGED
            - more drivers: TODO
        - dev docs on writing hardware types: TODO
    - new hardware types:
        - apparently all merged in Pike
    - API for hardware interface properties:
        - proposed spec: https://review.openstack.org/#/c/471174/
            - rloo checked with dtantsur, is 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

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
    - Related:
        - Refactor VIFPortIDMixin 2: https://review.openstack.org/#/c/486125/
        - Refactor VIFPortIDMixin 3: https://review.openstack.org/#/c/486126/
        - Refactor VIFPortIDMixin 4: https://review.openstack.org/#/c/486127/
        - Create a fake network interface for tempest testing: https://review.openstack.org/#/c/486128/
        - Use fake network interface in VIF attach tempest tests: https://review.openstack.org/#/c/486129/
        - Assume neutron port exists in VIFPortIDMixin: https://review.openstack.org/#/c/476949
- networking-baremetal initial patches
    - https://review.openstack.org/448157 MERGED
    - https://review.openstack.org/450714 MERGED
- Routed networks support (sambetts, vsaienk0)
    - Dependent on physnet awareness
    - networking-baremetal initial patches
        - https://review.openstack.org/#/c/456235/ needs updating
    - 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

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 - MERGED
    - https://review.openstack.org/#/c/473913/ - MERGED
    - Raised the patch to move the experimental job to non-voting https://review.openstack.org/489278 MERGED
    - Raised https://review.openstack.org/495766 for testing ironic-inspector without swift functionality
- anupn to update the python3 job to build tinyipa with python3

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 Aug 21st:
    - API patch: https://review.openstack.org/#/c/382194/

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)
- Proposed patch to create all the patches: https://review.openstack.org/489762
- jlvillal talked to infra and they suggested we do a batch upload as there are about 70 patches to merge in.
- 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)

Reference architecture guide (dtantsur)
- status as of 14 Aug 2017:
    - Common bits: https://review.openstack.org/487410 needs a revision
    - I guess this moves to Queens

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

Refactoring of code accessing other services (pas-ha)
- gerrit topic: https://review.openstack.org/#/q/topic:bug/1699547
- status as of 1 Aug 2017: ready for review
- discussed in ironic meeting; -2'd until Queens

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 14 Aug 2017:
    - ironic part: https://review.openstack.org/#/c/364861/
    - IPA part: https://review.openstack.org/#/c/364834/ needs a revision
        - blocked by the ironic part per recent discussions
    - discussed in ironic meeting; most likely will not land in Pike

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

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/
    - this needs update to spec and possibly discussion at PTG. jroll and lucasagomes (with cdent + sdague) proposed a different API. IRC discussions:
        - http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2016-03-16.log.html#t2016-03-16T11:51:47
        - http://eavesdrop.openstack.org/irclogs/%23openstack-dev/%23openstack-dev.2016-03-16.log.html#t2016-03-16T13:55:43
        - move to Queens

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 was released for Pike

Until next week,

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

More information about the OpenStack-dev mailing list