[openstack-dev] [ironic] this week's priorities and subteam reports
Yeleswarapu, Ramamani
ramamani.yeleswarapu at intel.com
Mon Jun 5 20:13:08 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. Rolling upgrades:
2.1. spec update: https://review.openstack.org/#/c/469940/
2.2. the next patch (add version column): https://review.openstack.org/#/c/412397/
3. OSC commands for ironic driver-related commands
3.1. review the spec: https://review.openstack.org/#/c/439907/
4. Physical network topology awareness:
4.1. next patch on review: https://review.openstack.org/#/c/461301/
Bugs (dtantsur, vdrok, TheJulia)
--------------------------------
- Stats (diff between 22 May 2017 and 5 Jun 2017)
- Ironic: 249 bugs (+6) + 252 wishlist items (+1). 24 new, 196 in progress (+5), 1 critical (+1), 30 high (+4) and 32 incomplete
- Inspector: 14 bugs (+2) + 30 wishlist items (+2). 1 new (-2), 12 in progress, 0 critical, 2 high (+1) and 3 incomplete
- Nova bugs with Ironic tag: 12. 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
TheJulia will sync-up with the other contributors to this effort and see about getting the feedback addressed in the patchset.
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 redesigned (conceptually simpler) the way IronicObject versioning is handled by services running different releases:
- update to the spec, ready for reviews: 'Rolling upgrades: different object versions' https://review.openstack.org/#/c/469940/
- next patch ready for reviews, if update is approved: '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 soon(ish)
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"
- 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
- Disabled swift in experimental gate as swift is not yet ported to python3.5.
- Devstack fails to come up in py3.5 for ironic. Raised the bug https://bugs.launchpad.net/ironic/+bug/1695135
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 5 Jun 2017:
- install guide / admin guide docs
- done except for:
- upgrading to hardware types: TODO
- dev docs: TODO
- fixing nits from previous reviews
- 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)
------------------------------------------------
- status as of most recent weekly meeting:
- OSC driver-properties spec is ready for reviews; has two +2 already: https://review.openstack.org/#/c/439907/
OSC default API version change (dtantsur)
-----------------------------------------
- status as of 05 Jun 2017:
- 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 05 Jun 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
- Status: 06/June/17
- API/conductor patch is up to date, rescuewait has minor comments which needs addressing.
- Tempest test is failing for fake driver because FakeRescue method is missing. Working on that part.
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
- ironic:
- Port data model: https://review.openstack.org/#/c/461301/
- Move create_port to conductor: https://review.openstack.org/469931
- Validate portgroup physical network consistency: https://review.openstack.org/469932
- Physical network-aware port mapping: https://review.openstack.org/#/c/470406
- API: https://review.openstack.org/469933
- Tempest tests (WIP, will be added to API review): https://review.openstack.org/#/c/470915
- python-ironicclient:
- OSC: https://review.openstack.org/#/c/461893
- 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 5 Jun 2017:
- spec merged: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ipa_api_versioning.html
- code needs rebasing and updating
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)
- jlvillal is on vacation; back the week of June 12
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/
Redfish driver (lucasagomes, dtantsur)
--------------------------------------
- topic: https://review.openstack.org/#/q/topic:bug/1526477
- status as of Jun 5th:
- the power actions do not wait for the power state to change, dtantsur will fix it
- 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 Jun 5th:
- 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
- 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/
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 most recent weekly meeting:
- 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
-Subprojects
Inspector (dtantsur, milan)
---------------------------
- Support for changing IPMI credentials in-band was removed from tree
.........
Until next week,
--Rama
[0] https://etherpad.openstack.org/p/IronicWhiteBoard
More information about the OpenStack-dev
mailing list