settingsLogin | Registersettings

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

0 votes

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. rolling upgrades
1.1. make a change to the grenade job to only upgrade conductor, ready for reviews: https://review.openstack.org/456166
1.2. the next patch is ready for reviews: https://review.openstack.org/#/c/412397/
2. review next BFV patch:
2.1. next: https://review.openstack.org/#/c/366197/
3. review e-tags spec: https://review.openstack.org/#/c/381991/
4. next driver comp client patch: https://review.openstack.org/#/c/419274/

Bugs (dtantsur, vdrok, TheJulia)


- Stats (diff between 17 Apr 2017 and 24 Apr 2017)
- Ironic: 250 bugs (-5) + 252 wishlist items (-1). 17 new (-5), 201 in progress (-4), 0 critical, 26 high (-2) and 33 incomplete (+1)
- Inspector: 14 bugs (-1) + 33 wishlist items (-1). 2 new, 20 in progress (-1), 0 critical, 1 high and 3 incomplete (-1)
- Nova bugs with Ironic tag: 11 (+1). 2 new (+1), 0 critical, 0 high

Essential Priorities
====================

CI refactoring and missing test coverage

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
- It's looking more like all setup can/should happen during tempest.
- 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/366197/
https://review.openstack.org/#/c/406290
https://review.openstack.org/#/c/413324
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:
- patches ready for reviews. Next one: 'Add version column' (https://review.openstack.org/#/c/412397/)
- Testing work:
- 27-Mar-2017: Grenade multi-node is non-voting
- need to change grenade to only upgrade the conductor, ready for reviews: https://review.openstack.org/456166

Reference architecture guide (jroll)


- no updates

Python 3.5 compatibility (JayF, hurricanerix)


- no updates

Deploying with Apache and WSGI in CI (vsaienk0)


- seems like we can deploy with WSGI, but it still uses a fixed port, instead of sub-path
- next one is https://review.openstack.org/#/c/444337/ MERGED
- are we done with ironic?
- inspector is TODO and depends on https://review.openstack.org/435517

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 most recent weekly meeting:
- TODO as of 24 Apr 2017
- install guide / admin guide docs
- client changes:
- driver commands update: https://review.openstack.org/419274
- node-update update: https://review.openstack.org/#/c/431542/ MERGED
- new hardware types:
- ilo: https://review.openstack.org/#/c/439404/
- contentious topics:
- what to do about driver properties API and dynamic drivers?
- rloo and dtantsur started brainstorming: https://etherpad.openstack.org/p/ironic-driver-properties-reform
- proposed this as topic for https://etherpad.openstack.org/p/ironic-virtual-meetup

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.
- no update this week

OSC default API version change (mariojv, dtantsur)


- 4/10 update
- Spec updated; scopes work to CLI only: https://review.openstack.org/#/c/448661/
- OSC CLI patch is up, passing unit and functional tests https://review.openstack.org/#/c/442153/

Rescue mode (JayF / mariojv)


- 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

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
- code updated, waiting for reviews

High Priorities
===============

specific fault support (JayF, mariojv)

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
- 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

Documentation reorganization (JayF, TheJulia)


- Awaiting spec to be posted.

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 updated, waiting for reviews: https://review.openstack.org/#/c/341086/

Optional Priorities
===================

Split away the tempest plugin (soliosg, jlvillal)

deploy steps (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, jroll)


- topic: https://review.openstack.org/#/q/topic:bug/1526477
- status as of most recent weekly meeting:
- the driver has merged \o/
- Documentation on review: https://review.openstack.org/#/c/442537/
- Patch to DevStack (WIP but works) to setup redfish in DevStack: https://review.openstack.org/#/c/445529/
- we need to add sushy to global-requirements

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 most recent weekly meeting:
- 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
- patch needs updating, rebasing, and testing: https://review.openstack.org/#/c/278566/
- no update

etags in the REST API (gzholtkevych)


- RFE: https://bugs.launchpad.net/ironic/+bug/1605728
- status as of most recent weekly meeting:
- (galyna): Spec: https://review.openstack.org/#/c/381991/. Updated, the spec is a kind of ready, few questions remained
- (galyna) Patches-POC are up. Demo: https://asciinema.org/a/103604

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

Inspector (dtansur, milan)

Bifrost (TheJulia)


- Neutron support is under development. WIP Patches are available at https://review.openstack.org/#/q/status:open+project:openstack/bifrost+branch:master+topic:452509

ironic-ui (betherly, ppiela, TheJulia)


- The javascript unit test job is presently not executing the tests. Betherly is working on it.
.........

Until next week,
--Rama

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


OpenStack Development Mailing List (not for usage questions)
Unsubscribe: OpenStack-dev-request@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
asked Apr 24, 2017 in openstack-dev by Yeleswarapu,_Ramaman (1,240 points)   1

1 Response

0 votes

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. Pike priorities update: https://review.openstack.org/#/c/460086/
2. rolling upgrades
2.1. make a change to the grenade job to only upgrade conductor, ready for reviews: https://review.openstack.org/456166
2.2. the next patch is ready for reviews: https://review.openstack.org/#/c/412397/
3. review next BFV patch:
3.1. next: https://review.openstack.org/#/c/366197/
4. review e-tags spec: https://review.openstack.org/#/c/381991/

Bugs (dtantsur, vdrok, TheJulia)


- Stats (diff between 24 Apr 2017 and 01 May 2017)
- Ironic: 252 bugs (+2) + 251 wishlist items (-1). 22 new (+5), 201 in progress, 0 critical, 26 high and 31 incomplete (-2)
- Inspector: 14 bugs + 30 wishlist items (-3). 3 new (+1), 15 in progress (-5), 0 critical, 1 high and 3 incomplete
- Nova bugs with Ironic tag: 11. 2 new, 0 critical, 0 high

Essential Priorities

> CI refactoring and missing test coverage

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 looking into identifying how to address/support a behavior of cinder in regards to required information where cinder expects data, but dirvers do not necessarilly use it. The cinder driver code originally expected it to be optional, but hshiina found an additional check inside cinder's code base that requires both pieces of information.
- mjturek is working on getting together devstack config updates/script changes in order to support this configuration
- It's looking more like all setup can/should happen during tempest.
- 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/366197/
https://review.openstack.org/#/c/406290
https://review.openstack.org/#/c/413324
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:
- patches ready for reviews. Next one: 'Add version column' (https://review.openstack.org/#/c/412397/)
- Testing work:
- 27-Mar-2017: Grenade multi-node is non-voting
- need to change grenade to only upgrade the conductor, ready for reviews: https://review.openstack.org/456166

Reference architecture guide (jroll, dtantsur)


- no updates

Python 3.5 compatibility (Nisha)


- no updates
- Nisha will be taking over this work(Nisha on leave from May 5 to May 22)

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/435517

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 most recent weekly meeting:
- client changes merged
- TODO as of 01 May 2017
- install guide / admin guide docs
- dtantsur volunteered, will try to get to it this week
- new hardware types:
- ilo: https://review.openstack.org/#/c/439404/
- rpioso volunteered to do drac
- what to do about driver properties API and dynamic drivers?
- we discussed it on the virtual meetup, rloo to write a spec

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)


- 5/1 update
- Spec updated; scopes work to CLI only: https://review.openstack.org/#/c/448661/
- some comments on the spec, dtantsur to follow-up this week
- OSC CLI patch is up, passing unit and functional tests https://review.openstack.org/#/c/442153/

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

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
- code updated, waiting for reviews

High Priorities
===============

specific fault support (?????)

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
- 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

Documentation reorganization (TheJulia)


- !!! this topic is candidate to removing from the list per https://review.openstack.org/460086 !!!

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 updated, waiting for reviews. Has two +2, rloo will +A today if no negative feedback: https://review.openstack.org/#/c/341086/

Optional Priorities

Split away the 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, jroll, dtantsur)


- topic: https://review.openstack.org/#/q/topic:bug/1526477
- status as of most May 1st:
- Documentation merged: https://docs.openstack.org/developer/ironic/drivers/redfish.html
- Patch to DevStack (WIP but works) to setup redfish in DevStack: https://review.openstack.org/#/c/445529/
- CI job: TODO

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 most recent weekly meeting:
- 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
- patch needs updating, rebasing, and testing: https://review.openstack.org/#/c/278566/
- no update

etags in the REST API (gzholtkevych)


- RFE: https://bugs.launchpad.net/ironic/+bug/1605728
- status as of most recent weekly meeting:
- (galyna): Spec: https://review.openstack.org/#/c/381991/. Updated, the spec is a kind of ready, few questions remained
- (galyna) Patches-POC are up. Demo: https://asciinema.org/a/103604

IPA authentication (jroll, joanna)


- !!! this topic is candidate to removing from the list per https://review.openstack.org/460086 !!!

Subprojects

Inspector (dtansur, milan)

Bifrost (TheJulia)


- Some fixes/improvements have been proposed around inspector use/support. Thank you Mark Goddard!
- Review of neutron support requested - https://review.openstack.org/#/c/452514/

ironic-ui (betherly, ppiela, TheJulia)


- Needing reviews. Any one willing to pickup bugs please check with TheJulia.

Drivers:


CIMC and UCSM (sambetts)
~~~~~~~~~~~~~~~~~~~~~~~~
- sam wants to rename the Cisco UCSM driver (https://review.openstack.org/#/c/400673/). Given the dynamic drivers, is it worth doing?

SNMP (mat128/xhku/lindycoder)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
- snmp gate failure was fixed!
.........

Until next week,
--Rama

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


OpenStack Development Mailing List (not for usage questions)
Unsubscribe: OpenStack-dev-request@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
responded May 1, 2017 by Yeleswarapu,_Ramaman (1,240 points)   1
...