[openstack-dev] [ironic] this week's priorities and subteam reports
Yeleswarapu, Ramamani
ramamani.yeleswarapu at intel.com
Wed Sep 6 16:17:38 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. Review specs in preparation for the PTG
2. Refactoring of the way we access clients: https://review.openstack.org/#/q/topic:bug/1699547
3. Rolling upgrades missing bit: https://review.openstack.org/#/c/497666/
3.1. disallow upgrades from ocata: https://review.openstack.org/#/c/497685/
3.2. check object versions in dbsync tool: https://review.openstack.org/#/c/497703/
4. Switch to none auth for standalone mode: https://review.openstack.org/#/c/359061/
Bugs (dtantsur, vdrok, TheJulia)
--------------------------------
- Stats (diff between 28 Aug 2017 and 4 Sep 2017)
- Ironic: 251 bugs (+1) + 258 wishlist items (+2). 21 new (-2), 191 in progress (+1), 0 critical, 31 high (+1) and 36 incomplete (+3)
- Inspector: 13 bugs (+1) + 29 wishlist items. 2 new, 11 in progress (+1), 0 critical, 3 high (+1) and 3 incomplete
- Nova bugs with Ironic tag: 15 (+1). 1 new (+1), 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
- node take over?
- resource classes integration tests: https://review.openstack.org/#/c/443628/
Essential Priorities
====================
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 4 Sep 2017:
- Meetings resume this week.
- Queens ideas/thoughts should be posted to the BFV etherpad: https://etherpad.openstack.org/p/Ironic-BFV
- 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:
- added 'ironic-dbsync online_data_migrations' call to regular grenade test (not multinode test because grenade multinode test doesn't upgrade all services) https://review.openstack.org/497949 - MERGED
- to address restarting services after unpinning, spec: 'SIGHUP restarts services with updated configs': https://review.openstack.org/474309. MERGED
- code: https://review.openstack.org/#/c/474331
- outstanding: dev documentation needs updating: https://review.openstack.org/#/c/419439/
- we forgot to set the conductor version in the database :-( So that rolling upgrades will work without having to cut another Pike release, 2 patches starting with:
- conductor saves version in db:https://review.openstack.org/#/c/497593/ MERGED (Not backporting)
- patches to switch over/disallow ocata -> queens upgrade, starts with:
- Add online data migrations for conductor version (https://review.openstack.org/#/c/497666/1)
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 28 Aug 2017:
- documentation
- upgrade guide for the remaining drivers: TODO
- ilo: https://review.openstack.org/#/c/496480/
- idrac: (rpioso) TBD
- snmp: https://review.openstack.org/#/c/498541/ MERGED
- 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/
- 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/ MERGED
- 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 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
- Raised https://review.openstack.org/495766 for testing ironic-inspector without swift functionality
- anupn to update the python3 job to build tinyipa with python3
- we need to make the ironic job voting eventually. but we need to check that nova, glance and neutron already have voting python 3 jobs, otherwise they may break us.
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: 04 Sep 2017
- 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)?
- (TheJulia) None that I'm aware of, but nova is going to expect ironic work be completed first.
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
Subprojects
===========
Inspector (dtantsur, milan)
---------------------------
- progress with RPC introduction (depends on an oslo_messaging fake driver patch to be able to functional-test it)
- patch up for disk by path in the introspection data
- patch to allow periodics to shut down inspector (after a critical failure) just received +2
Bifrost (TheJulia)
------------------
- TheJulia is working on updating shade to support, as a prelude to support more advanced features of ironic.
ironic-ui (ppiela, TheJulia)
----------------------------
- (TheJulia) Meeting resumes this week.
.........
Until next week,
--Rama
[0] https://etherpad.openstack.org/p/IronicWhiteBoard
More information about the OpenStack-dev
mailing list