[openstack-dev] [ironic] this week's priorities and subteam reports
Yeleswarapu, Ramamani
ramamani.yeleswarapu at intel.com
Fri Jan 26 23:49:42 UTC 2018
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. Sorry about the delay.
This Week's Priorities (as of the weekly ironic meeting)
========================================================
1. ironicclient version negotiation (deadline: Thu, Jan 25th)
1.1. expose negotiated latest: https://review.openstack.org/531029 MERGED
1.2. accept list of versions: https://review.openstack.org/#/c/531271/ MERGED
2. Classic drivers deprecation
2.1. upgrade: https://review.openstack.org/534373 2x+2
3. Traits
3.1. RPC objects https://review.openstack.org/#/c/532268/ MERGED
3.2. RPC API https://review.openstack.org/#/c/535296 MERGED
3.3. API https://review.openstack.org/#/c/532269/ MERGED
3.4. Client https://review.openstack.org/#/c/532622/ MERGED
3.5. API ref https://review.openstack.org/#/c/536384 MERGED
4. Rescue:
4.1. network interface update: https://review.openstack.org/#/c/509342 MERGED
4.2. rescuewait timeout: https://review.openstack.org/#/c/353156/ MERGED
4.3. Agent rescue implementation: https://review.openstack.org/#/c/400437/ APPROVED
4.4. Add API methods for [un]rescue: https://review.openstack.org/#/c/350831/ APPROVED
4.5. Client Rescue Provision States https://review.openstack.org/#/c/408341
4.6. Client rescue_interface on node https://review.openstack.org/#/c/517302
Vendor priorities
-----------------
cisco-ucs:
Patches in works for SDK update, but not posted yet, currently rebuilding third party CI infra after a disaster...
idrac:
RFE and first several patches for adding UEFI support will be posted by Tuesday, 1/9
ilo:
https://review.openstack.org/#/c/530838/ - OOB Raid spec for iLO5
irmc:
None
oneview:
Remove python-oneviewclient from oneview hardware type - https://review.openstack.org/#/c/524729/
Subproject priorities
---------------------
bifrost:
(TheJulia): Fedora support fixes - https://review.openstack.org/#/c/471750/
ironic-inspector (or its client):
(dtantsur) keystoneauth adapters https://review.openstack.org/#/c/515787/ MERGED
networking-baremetal:
neutron baremetal agent https://review.openstack.org/#/c/456235/
sushy and the redfish driver:
(dtantsur) implement redfish sessions: https://review.openstack.org/#/c/471942/ MERGED
Bugs (dtantsur, vdrok, TheJulia)
--------------------------------
- Stats (diff between 08 Jan 2018 and 15 Jan 2018)
- Ironic: 216 bugs (-3) + 260 wishlist items. 1 new (-1), 156 in progress (-2), 0 critical, 33 high (-1) and 27 incomplete (-1)
- Inspector: 14 bugs (-1) + 28 wishlist items. 0 new, 10 in progress, 0 critical, 2 high (-1) and 6 incomplete (+1)
- Nova bugs with Ironic tag: 13. 1 new, 0 critical, 0 high
- via http://dashboard-ironic.7e14.starter-us-west-2.openshiftapps.com/
- the dashboard was abruptly deleted and needs a new home :(
- HIGH bugs with patches to review:
- Clean steps are not tested in gate https://bugs.launchpad.net/ironic/+bug/1523640: Add manual clean step ironic standalone test https://review.openstack.org/#/c/429770/15
- Needs to be reproposed to the ironic tempest plugin repository.
- prepare_instance() is not called for whole disk images with 'agent' deploy interface https://bugs.launchpad.net/ironic/+bug/1713916:
- Fix ``agent`` deploy interface to call ``boot.prepare_instance`` https://review.openstack.org/#/c/499050/
- (TheJulia) Currently WF-1, as revision is required for deprecation.
- If provisioning network is changed, Ironic conductor does not behave correctly https://bugs.launchpad.net/ironic/+bug/1679260: Ironic conductor works correctly on changes of networks: https://review.openstack.org/#/c/462931/
- (rloo) needs some direction
- may be fixed as part of https://review.openstack.org/#/c/460564/
- IPA may not find partition created by conductor https://bugs.launchpad.net/ironic-lib/+bug/1739421
- Fix proposed: https://review.openstack.org/#/c/529325/
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/
- localboot with partitioned image patches:
- Ironic - add localboot partitioned image test: https://review.openstack.org/#/c/502886/
- when previous are merged TODO (vsaienko)
- Upload tinycore partitioned image to tarbals.openstack.org
- Switch ironic to use tinyipa partitioned image by default
- Missing test coverage (all)
- portgroups and attach/detach tempest tests: https://review.openstack.org/382476
- 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/
- radosgw (https://bugs.launchpad.net/ironic/+bug/1737957)
Essential Priorities
====================
Ironic client API version negotiation (TheJulia, dtantsur)
----------------------------------------------------------
- RFE https://bugs.launchpad.net/python-ironicclient/+bug/1671145
- Nova bug https://bugs.launchpad.net/nova/+bug/1739440
- gerrit topic: https://review.openstack.org/#/q/topic:bug/1671145
- status as of 22 Jan 2017:
- Nova request was accepted as a bug for now: https://bugs.launchpad.net/nova/+bug/1739440
- we will upgrade it to a blueprint if it starts looking a feature; no spec is probably needed
- TODO:
- easier access to versions in ironicclient
- see https://etherpad.openstack.org/p/ironic-api-version-negotiation
- discussion of various ways to implement it happened on the midcycle
- dtantsur wants to have an API-SIG guideline on consuming versions in SDKs
- ready for review https://review.openstack.org/532814
- patches for ironicclient by TheJulia:
- expose negotiated latest: https://review.openstack.org/531029
- accept list of versions: https://review.openstack.org/#/c/531271/
- establish foundation for using version negotiation in nova
External project authentication rework (pas-ha, TheJulia)
---------------------------------------------------------
- gerrit topic: https://review.openstack.org/#/q/topic:bug/1699547
- status as of 22 Jan 2017:
- 0 inspector patch left
- https://review.openstack.org/#/c/515786/ MERGED
- https://review.openstack.org/#/c/515787 MERGED
Classic drivers deprecation (dtantsur)
--------------------------------------
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/classic-drivers-future.html
- status as of 23 Jan 2017:
- dev documentation for hardware types: TODO
- switch documentation to hardware types:
- need help from vendors updating their pages
- migration of classic drivers to hardware types:
- updating spec based on actual code: https://review.openstack.org/#/c/536298/
- see the commit message for explanation
- support options for migrations: https://review.openstack.org/535772
- upgrade (for IPMI, SNMP and fake): https://review.openstack.org/#/c/534373/
- other drivers TODO
- migration of CI to hardware types
- switch all jobs from -_ipmitool to ipmi: https://review.openstack.org/#/c/536875/
- switch inspector CI: https://review.openstack.org/#/c/537415/
- clean up job playbooks: https://review.openstack.org/#/c/535896/
- actual deprecation: https://review.openstack.org/#/c/536928/
Traits support planning (mgoddard, johnthetubaguy, dtantsur)
------------------------------------------------------------
- http://specs.openstack.org/openstack/ironic-specs/specs/approved/node-traits.html
- Nova patches: https://review.openstack.org/#/q/topic:bp/ironic-driver-traits+(status:open+OR+status:merged)
- status as of 8 Jan 2018:
- deploy templates spec: https://review.openstack.org/504952 needs reviews
- depends on deploy-steps spec: https://review.openstack.org/#/c/412523
- patches for traits API
- DB model & DB API
- https://review.openstack.org/#/c/528238 (MERGED)
- https://review.openstack.org/#/c/530723 (MERGED)
- Add version to DB object
- https://review.openstack.org/#/c/535482 (MERGED)
- RPC objects
- https://review.openstack.org/#/c/532268 MERGED
- RPC API & conductor
- https://review.openstack.org/#/c/535296 MERGED
- API
- https://review.openstack.org/#/c/532269 MERGED
- API ref
- https://review.openstack.org/#/c/536384
- Client
- https://review.openstack.org/#/c/532622/ APPROVED
- johnthetubaguy is picking the ironic side of traits up now, mgoddard is taking a look at the nova virt driver side
- If we don't land this code (at least the API) this week, highly unlikely the nova part will land before next week's FF.
Reference architecture guide (dtantsur, sambetts)
-------------------------------------------------
- status as of 22 Jan 2017:
- dtantsur needs volunteers to help move this forward
- list of cases from https://etherpad.openstack.org/p/ironic-queens-ptg-open-discussion
- Admin-only provisioner
- small and/or rare: TODO
- large and/or frequent: TODO
- Bare metal cloud for end users
- smaller single-site: TODO
- larger single-site: TODO
- larger multi-site: TODO
High Priorities
===============
Neutron event processing (vdrok, vsaienk0, sambetts)
----------------------------------------------------
- status as of 27 Sep 2017:
- spec at https://review.openstack.org/343684, ready for reviews, replies from authors
- WIP code at https://review.openstack.org/440778
Routed network support (sambetts, vsaienk0, bfournie, hjensas)
--------------------------------------------------------------
- status as of 19 Jan 2018:
- Need reviews ... https://review.openstack.org/#/q/topic:bug/1658964+(status:open+OR+status:merged)
- With neutron fixed; patch below; the dsvm job seems stable.
- Fix for neutron issue https://review.openstack.org/#/c/534449/ merged.
- hjensas taken over as main contributor from sambetts
- There is challanges with integration to Placement due to the way the integration was done in neutron.
Neutron will create a resource provider for network segments in Placement, then it creates an os-aggregate
in Nova for the segment, adds nova compute hosts to this aggregate. Ironic nodes cannot be added to host-aggregates.
I (hjensas) had a short discussion with neutron devs (mlavalle) on the issue:
http://eavesdrop.openstack.org/irclogs/%23openstack-neutron/%23openstack-neutron.2018-01-12.log.html#t2018-01-12T17:05:38
There are patches in Nova to add support for ironic nodes in host-aggregates:
- https://review.openstack.org/#/c/526753/ allow compute nodes to be associated with host agg
- https://review.openstack.org/#/c/529135/ (Spec)
- Patches:
- https://review.openstack.org/456235 Add baremetal neutron agent
- https://review.openstack.org/#/c/533707/ start_flag = True, only first time, or conf change
- https://review.openstack.org/524709 Make the agent distributed using hashring and notifications (WIP)
- https://review.openstack.org/521838 Switch from MechanismDriver to SimpleAgentMechanismDriverBase
- https://review.openstack.org/#/c/532349/7 Add support to bind type vlan networks
- CI Patches:
- https://review.openstack.org/#/c/531275/ Devstack - use neutron segments (routed provider networks)
- https://review.openstack.org/#/c/531637/ Wait for ironic-neutron-agent to report state
- https://review.openstack.org/#/c/530117/ Devstack - Add ironic-neutron-agent
- https://review.openstack.org/#/c/530409/ Add dsvm job
Rescue mode (rloo, stendulker, aparnav)
---------------------------------------
- Status as on 15 Jan 2018
- 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
- ironic side:
- All patches are up-to-date, being actively reviewed and updated
- Tempest tests based on standalone ironic is WIP.
- nova side:
- https://blueprints.launchpad.net/nova/+spec/ironic-rescue-mode: approved for Queens; waiting for ironic part to be done first. Queens feature freeze is week of Jan 22.
- To get the nova patch merged, we need:
- release new python-ironicclient
- update ironicclient version in upper-constraints (this patch will be posted automatically)
- update ironicclient version in global-requirement (this patch needs to be posted manually)
- code patch: https://review.openstack.org/#/c/416487/
- If we don't land this code (at least the API) this week, highly unlikely the nova part will land before next week's FF.
- CI is needed for nova part to land
Clean up deploy interfaces (vdrok)
----------------------------------
- status as of 9 Jan 2017:
- patch https://review.openstack.org/524433 ready for reviews
Zuul v3 jobs in-tree (sambetts, derekh, jlvillal, rloo)
-------------------------------------------------------
- etherpad tracking zuul v3 -> intree: https://etherpad.openstack.org/p/ironic-zuulv3-intree-tracking
- cleaning up/centralizing job descriptions (eg 'irrelevant-files'): DONE
- Next TODO is to convert jobs on master, to proper ansible. NOT a high priority though.
- (pas-ha) DNM experimental patch with "devstack-tempest" as base job https://review.openstack.org/#/c/520167/
Graphical console interface (pas-ha, vdrok, rpioso)
---------------------------------------------------
- status as of 8 Jan 2017:
- spec on review: https://review.openstack.org/#/c/306074/
- there is nova part here, which has to be approved too
- dtantsur is worried by absence of progress here
- (TheJulia) I think for rocky, it might be worth making it a prime focus, or making it a background goal.
BIOS config framework (dtantsur, yolanda, rpioso)
-------------------------------------------------
- status as of 8 Jan 2017:
- spec under active review: https://review.openstack.org/#/c/496481/
Ansible deploy interface (pas-ha)
---------------------------------
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ansible-deploy-driver.html
- status as of 22 Jan 2017:
- code merged
- TODO
- CI job
- https://review.openstack.org/529640 MERGED
- https://review.openstack.org/#/c/529383/ MERGED
- done?
- docs: https://review.openstack.org/#/c/525501/
OpenStack 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"
- TODO submit the python3 job for IPA
- for ironic and ironic-inspector job enabled by disabling swift as swift is still lacking py3.5 support.
- anupn to update the python3 job to build tinyipa with python3
- (anupn): Talked with swift folks and there is a bug upstream opened https://review.openstack.org/#/c/401397 for py3 support in swift. But this is not on their priority
- Right now patch pass all gate jobs except agent_- drivers.
- 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.
- nova seems to have python 3 jobs voting, here are our patches:
- ironic https://review.openstack.org/#/c/531398/
- ironic-inspector https://review.openstack.org/#/c/531400/ MERGED
Deploying with Apache and WSGI in CI (pas-ha, vsaienk0)
-------------------------------------------------------
- ironic is mostly finished
- (pas-ha) needs to be rewritten for uWSGI, patches on review:
- https://review.openstack.org/#/c/507011/ +A
- https://review.openstack.org/#/c/507067 Needs revision
- inspector is TODO and depends on https://review.openstack.org/#/q/topic:bug/1525218
- may be delayed to after Queens, as the HA work seems to take a different direction
Split away the tempest plugin (jlvillal)
----------------------------------------
- https://etherpad.openstack.org/p/ironic-tempest-plugin-migration
- Current (8-Jan-2018) (jlvillal): All projects now using tempest plugin code from openstack/ironic-tempest-plugin
- Need to remove plugin code from master branch of openstack/ironic and openstack/ironic-inspector
- Plugin code will NOT be removed from the stable branches of openstack/ironic and openstack/ironic-inspector
- (jlvillal) 3rd Party CI has had over 3 weeks to prepare for removal. We should now move forward
- README, setup.cfg and docs cleanup: https://review.openstack.org/#/c/529538/ MERGED
- ironic-tempest-plugin 1.0.0 released
Subprojects
===========
Inspector (dtantsur)
--------------------
- trying to flip dsvm-discovery to use the new dnsmasq pxe filter and failing because of bash :Dhttps://review.openstack.org/#/c/525685/6/devstack/plugin.sh@202
- follow-ups being merged/reviewed; working on state consistency enhancements https://review.openstack.org/#/c/510928/ too (HA demo follow-up)
Bifrost (TheJulia)
------------------
- Also seems a recent authenticaiton change in keystoneauth1 has broken processing of the clouds.yaml files, i.e. `openstack` command does not work.
- TheJulia will try to look at this this week.
Drivers:
--------
DRAC (rpioso, dtantsur)
~~~~~~~~~~~~~~~~~~~~~~~
- Dell Ironic CI is being rebuilt, its back and running now (10/17/2017)
OneView (ricardoas, nicodemos, gmonteiro)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
- Re-submitting reverted patches for migration from python-oneviewclient to python-hpOneView + python-ilorest-library
- Check weekly priorities for most import patch to review
Cisco UCS (sambetts)
~~~~~~~~~~~~~~~~~~~~
- Currently rebuilding third party CI from the ground up after it bit the dust
- Patches for updating the UCS python SDKs are in the works and should be posted soon
.........
Until next week,
--Rama
[0] https://etherpad.openstack.org/p/IronicWhiteBoard
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20180126/7a0caeca/attachment.html>
More information about the OpenStack-dev
mailing list