[openstack-dev] [ironic] this week's priorities and subteam reports
Yeleswarapu, Ramamani
ramamani.yeleswarapu at intel.com
Mon Mar 12 18:53:01 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.
This Week's Priorities (as of the weekly ironic meeting)
========================================================
Weekly priorities
-----------------
- Rocky Priorities - https://review.openstack.org/#/c/550174/
- Remaining Rescue patches
- https://review.openstack.org/#/c/546919/ - Fix a bug for unrescuing with whole disk image
- https://review.openstack.org/#/c/524118/ - Devstack changes to enable testing add support for rescue mode
- https://review.openstack.org/#/c/538119/ - Rescue mode standalone tests
- BIOS interface specification
- https://review.openstack.org/#/c/496481/
Old priorities list that needs to be reconciled
----------------------------------------------
- Fix the multitenant grenade - https://bugs.launchpad.net/ironic/+bug/1744139 - Testing another possibility
- Disable .pyc file creation https://review.openstack.org/544750 MERGED
- Avoids library incompatibility issue by disabling .pyc files from being written to disk in the scenario.
- backport to stable/queens: https://review.openstack.org/#/c/545089/ MERGED
- The nova issue noted under critical bugs is also needed to make multitenant grenade reliable again.
- Required Backports/Nice to haves below
- CRITICAL bugs (must be fixed and backported to queens before the release)
- Nova
- Placement has issues after upgrade if ironic is unreachable for too long - Current WIP: https://review.openstack.org/#/c/545479/
- https://bugs.launchpad.net/nova/+bug/1750450
Required Queens Backports
-------------------------
- detached VIF reappearing: https://bugs.launchpad.net/ironic/+bug/1750785
- workaround: https://review.openstack.org/546584 abandoned
- decided to revert the original patch: https://review.openstack.org/546705 MERGED
- backport to stable/queens: https://review.openstack.org/546719 APPROVED
Nice to have backports
----------------------
- Ansible docs - https://review.openstack.org/#/c/525501/ MERGED
- backport https://review.openstack.org/#/c/546079/ MERGED
- inspector: do not try passing non-MACs as switch_id: https://review.openstack.org/542214 APPROVED
- stable/queens - https://review.openstack.org/543961 MERGED
- Fix for CLEANING on conductor restart: https://review.openstack.org/349971 MERGED
- backport: https://review.openstack.org/#/c/545893/ MERGED
- Reset reservations on take over: https://review.openstack.org/546273
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:
https://review.openstack.org/#/c/543883/ - rescue support for irmc-virtual-media boot
oneview:
Subproject priorities
---------------------
bifrost:
ironic-inspector (or its client):
networking-baremetal:
networking-generic-switch:
sushy and the redfish driver:
Bugs (dtantsur, vdrok, TheJulia)
--------------------------------
- Stats (diff between 19 Feb 2018 and 12 Mar 2018)
- Ironic: 211 bugs (+3) + 248 wishlist items (+5). 5 new (+2), 152 in progress (-3), 1 critical, 33 high and 24 incomplete (+1)
- Inspector: 14 bugs (+1) + 26 wishlist items. 0 new, 14 in progress (+2), 0 critical, 3 high (+1) and 4 incomplete
- Nova bugs with Ironic tag: 15 (-1). 1 new (-1), 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 :(
- use it locally with `tox -erun` if you need to
- critical:
- sushy: https://bugs.launchpad.net/sushy/+bug/1754514 (basic auth broken when SessionService is not present)
- 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.
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 12 Feb 2017:
- TODO:
- API-SIG guideline on consuming versions in SDKs https://review.openstack.org/532814 on review
- Rocky cycle work is different and will be client oriented in nova. TheJulia will abandon the remaining patches proposed for the client
Classic drivers deprecation (dtantsur)
--------------------------------------
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/classic-drivers-future.html
- status as of 12 Mar 2017:
- switch documentation to hardware types:
- need help from vendors updating their pages!
- ilo: https://review.openstack.org/#/c/542593/ MERGED
- idrac looks fine for now
- api-ref examples: TODO
- ironic-inspector:
- documentation: https://review.openstack.org/#/c/545285/
- enable fake-hardware in devstack: https://review.openstack.org/#/c/550811/
- change the default discovery driver: https://review.openstack.org/#/c/550464/
- migration of CI to hardware types
- IPA: TODO
- ironic-lib: TODO?
- python-ironicclient: TODO?
- python-ironic-inspector-client: TODO?
- virtualbmc: TODO?
Traits support planning (mgoddard, johnthetubaguy, dtantsur) - To be removed from rocky cycle list
--------------------------------------------------------------------------------------------------
- status as of 12 Feb 2018:
- deploy templates spec: https://review.openstack.org/504952 needs reviews
- depends on deploy-steps spec: https://review.openstack.org/#/c/412523
- traits API:
- need to validate node's instance_info['traits'] at deploy time (https://bugs.launchpad.net/ironic/+bug/1722194/comments/31)
- https://review.openstack.org/#/c/543461
- will need to backport this to stable/queens
- notes on next steps: https://etherpad.openstack.org/p/ironic-node-instance-traits
Reference architecture guide (dtantsur, sambetts)
-------------------------------------------------
- status as of 19 Feb 2017:
- dtantsur is returning to this after the release
- TheJulia suggested we do it right on the PTG
- list of cases from the PTG
- Admin-only provisioner
- small and/or rare: TODO
- non-HA acceptable, noop/flat network acceptable
- large and/or frequent: TODO
- HA required, neutron network or noop (static) network
- Bare metal cloud for end users
- smaller single-site: TODO
- non-HA, ironic conductors on controllers and noop/flat network acceptable
- larger single-site: TODO
- HA, split out ironic conductors, neutron networking, virtual media > iPXE > PXE/TFTP
- split out TFTP servers if you need them?
- larger multi-site: TODO
- cells v2
- ditto as single-site otherwise?
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 12 Feb 2018:
- All code patches are merged.
- One CI patch left, rework devstack baremetal simulation. To be done in Rocky?
- This is to have actual 'flat' networks in CI.
- Placement API work to be done in Rocky due to:
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:
- CI Patches:
- https://review.openstack.org/#/c/392959/ Rework Ironic devstack baremetal network simulation
- RFEs (Rocky)
- https://bugs.launchpad.net/networking-baremetal/+bug/1749166
- https://bugs.launchpad.net/networking-baremetal/+bug/1749162
Rescue mode (rloo, stendulker)
------------------------------
- Status as on 12 Feb 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+OR+status:merged
- ironic side:
- all code patches have merged except for
- Add documentation for rescue mode: https://review.openstack.org/#/c/431622/ MERGED
- Devstack changes to enable testing add support for rescue mode: https://review.openstack.org/#/c/524118/
- We need to be careful with this, in that we can't use python-ironicclient changes that have not been released.
- Update "standalone" job for supporting rescue mode: https://review.openstack.org/#/c/537821/
- Rescue mode standalone tests: https://review.openstack.org/#/c/538119/ (failing CI, not ready for reviews)
- Bugs:
- unrescue fails with partition user image: https://review.openstack.org/#/c/544278/ MERGED
- rescue ramdisk doesn't boot on UEFI: https://review.openstack.org/#/c/545186/ MERGED
- Can't Merge until we do a client release with rescue support (in Rocky):
- Tempest tests with nova: https://review.openstack.org/#/c/528699/
- Run the tempest test on the CI: https://review.openstack.org/#/c/528704/
- succeeded in rescuing: http://logs.openstack.org/04/528704/16/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/4b74169/logs/screen-ir-cond.txt.gz#_Feb_02_09_44_12_940007
- nova side:
- https://blueprints.launchpad.net/nova/+spec/ironic-rescue-mode:
- approved for Queens but didn't get the ironic code (client) done in time
- (TheJulia) Nova has indicated that this is deferred until Rocky.
- 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/
- CI is needed for nova part to land
- tiendc is working for CI
Clean up deploy interfaces (vdrok)
----------------------------------
- status as of 5 Feb 2017:
- patch https://review.openstack.org/524433 needs update and rebase
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/
OpenStack Priorities
====================
Mox
---
- TheJulia needs to just declare this done.
SIGHUP support
--------------
- Proposed for ironic by rloo -- this is done: https://review.openstack.org/474331 MERGED \o/
- TODO:
- ironic-inspector
- networking-baremetal
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.
- (TheJulia) It seems we might not have py3 compatibility with swift until the T- cycle.
- updating setup.cfg (part of requirements for the goal):
- ironic: https://review.openstack.org/#/c/539500/ - MERGED
- ironic-inspector: https://review.openstack.org/#/c/539502/ - 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/507067
- inspector is TODO and depends on https://review.openstack.org/#/q/topic:bug/1525218
- delayed as the HA work seems to take a different direction
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 authentication 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:
--------
Cisco UCS (sambetts) Last updated 2018/02/05
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
- Cisco CIMC driver CI back up and working on every patch
- Cisco UCSM driver CI in development
- 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/20180312/ef8a8935/attachment.html>
More information about the OpenStack-dev
mailing list