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

Yeleswarapu, Ramamani ramamani.yeleswarapu at intel.com
Mon Jan 8 19:09:20 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)
========================================================
1. ironic-lib patches to finish before the freeze
    1.1. fix waiting for partition: https://review.openstack.org/#/c/529325/
2. Traits
    2.1. https://review.openstack.org/#/c/528238/
3. Rescue:
    3.1. RPC https://review.openstack.org/#/c/509336/
    3.2. network interface update: https://review.openstack.org/#/c/509342
4. Routed Networks - Review for input only
    4.1. Add baremetal neutron agent https://review.openstack.org/#/c/456235/
5. Finishing the CI for the ansible deploy work
    5.1. https://review.openstack.org/529640
    5.2. https://review.openstack.org/#/c/529383/
6. BIOS interface spec:
    6.1. https://review.openstack.org/#/c/496481/

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:
    Introduce hpOneView and ilorest to OneView -  https://review.openstack.org/#/c/523943/

Subproject priorities
---------------------
bifrost:
    Broken on recent fedora releases - TheJulia is working on it, patch should be up this week.
ironic-inspector (or its client):
    (dtantsur) config options refactoring: https://review.openstack.org/#/c/515786/
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/

Bugs (dtantsur, vdrok, TheJulia)
--------------------------------
- Stats (diff between 18 Dec 2017 and 08 Jan 2018)
- Ironic: 219 bugs (+1) + 260 wishlist items (-1). 2 new, 158 in progress, 0 critical, 34 high (+2) and 28 incomplete (-2)
- Inspector: 15 bugs (-2) + 28 wishlist items (-1). 0 new, 10 in progress (-5), 0 critical, 3 high (-1) and 5 incomplete
- Nova bugs with Ironic tag: 13 (+1). 1 new (-1), 0 critical, 0 high
- via http://dashboard-ironic.7e14.starter-us-west-2.openshiftapps.com/
- 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/
- Inspector: Spurious race conditions detected white-/black-listing MAC addresses in dnsmasq PXE filter
    - https://bugs.launchpad.net/ironic-inspector/+bug/1741035
    - Milan's legacy - needs triaging

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 08 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
                    - still TODO
                - 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 08 Jan 2017:
    - Ironic Done
    - 2 inspector patches left
        - https://review.openstack.org/#/c/515786/
        - https://review.openstack.org/#/c/515787

Classic drivers deprecation (dtantsur)
--------------------------------------
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/classic-drivers-future.html
- status as of 08 Jan 2017:
    - dev documentation for hardware types: TODO
    - switch documentation to hardware types:
        - status https://etherpad.openstack.org/p/ironic-switch-to-hardware-types
        - admin guide update (minus vendor bits): https://review.openstack.org/#/c/528337/
            - need review + help from vendors updating their pages
    - migration of classic drivers to hardware types, in discussion...
        - http://lists.openstack.org/pipermail/openstack-dev/2017-November/124509.html
        - spec update: https://review.openstack.org/#/c/528308/

Traits support planning (mgoddard, johnthetubaguy, TheJulia, dtantsur)
----------------------------------------------------------------------
- http://specs.openstack.org/openstack/ironic-specs/specs/approved/node-traits.html
- 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
        - https://review.openstack.org/#/c/528238/
    - johnthetubaguy is picking the ironic side of traits up now, mgoddard is taking a look at the nova virt driver side

Reference architecture guide (dtantsur, sambetts)
-------------------------------------------------
- status as of 08 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 18 Dec 2017:
    - hjensas taken over as main contributor from sambetts
    - Patches:
        - https://review.openstack.org/456235 Add baremetal neutron agent
        - https://review.openstack.org/524709 Make the agent distributed using hashring and notifications
        - https://review.openstack.org/521838 Switch from MechanismDriver to SimpleAgentMechanismDriverBase

Rescue mode (rloo, stendulker, aparnav)
---------------------------------------
- Status as on 18 Dec 2017
    - 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/

Clean up deploy interfaces (vdrok)
----------------------------------
- status as of 8 Jan 2017:
    - patch https://review.openstack.org/524433

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'): all done except inspector/client because stable/ocata CI is failing :-(
    - (dtantsur) nice to do, but not a priority.
- 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 8 Jan 2017:
    - code merged
    - TODO
        - CI job
            - https://review.openstack.org/529640
            - https://review.openstack.org/#/c/529383/
        - 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/

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/

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.
- Fedora support is currently broken, thejulia has a patch in progress

Drivers:
--------

DRAC (rpioso, dtantsur)
~~~~~~~~~~~~~~~~~~~~~~~
- Dell Ironic CI is being rebuilt, its back and running now (10/17/2017)


OneView (ricardoas/fellypefca)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
- 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/20180108/f48b87fc/attachment.html>


More information about the OpenStack-dev mailing list