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

Yeleswarapu, Ramamani ramamani.yeleswarapu at intel.com
Mon Nov 13 22:13:39 UTC 2017


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. CI migration to Zuul v3: take legacy jobs in tree: https://etherpad.openstack.org/p/ironic-zuulv3-intree-tracking
    1.1. Clean up ironic-lib: https://review.openstack.org/518622 (jlvillal) MERGED
    1.2. Clean up ironic-python-agent: https://review.openstack.org/#/c/518613/ (jlvillal)
    1.3. Finish Bifrost (TheJulia)
2. Release ironic after oneview reverts below land (they are approved, waiting to merge). This is really for dtantsur to do.
3. Midcycle planning: https://etherpad.openstack.org/p/ironic-queens-midcycle
4. Rework keystone auth for Glance: https://review.openstack.org/#/c/467728/
5. BIOS interface spec: https://review.openstack.org/#/c/496481/
6. Rescue:
    6.1. db https://review.openstack.org/#/c/509334/
    6.2. driver interface https://review.openstack.org/#/c/509335/
    6.3. RPC https://review.openstack.org/#/c/509336/
    6.4. rescuewait timeout https://review.openstack.org/#/c/353156

Vendor priorities
    Patchs in works for SDK update, but not posted yet, currently rebuilding third party CI infra after a disaster...

    https://review.openstack.org/207337 - Out-of-band Boot from UEFI iSCSI volume for HPE Proliant server
    SPEC to add a new hardware type for another FUJITSU server: PRIMEQUEST MMB:
    https://review.openstack.org/#/c/515717/ 2x +2, awaits last-minute comments

    HIGH PRIORITY: need to revert changes that moved from python-oneviewclient to hpOneView library; this is blocking the ironic release (they're are already chained from 1 to 8. Ping ricardoas, nicodemos or fellypefca if there is some issue)

    Add validations for OneView ML2 driver -  https://review.openstack.org/#/c/508946/

Subproject priorities
ironic-inspector (or its client):
    dnsmasq-based inspector PXE filter driver: https://review.openstack.org/#/c/466448/ TL;DR: replaces iptables with a dynamic configuration of dnsmasq (pretty cool thing too ;)
    folks might consider trying the test patch to experiment manually with this https://review.openstack.org/#/c/468712/54
    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 06 Nov 2017 and 13 Nov 2017)
  - Ironic: 223 bugs (-13) + 247 wishlist items (-1). 13 new, 151 in progress (-15), 0 critical, 31 high (+1) and 34 incomplete (-1)
  - Inspector: 16 bugs + 31 wishlist items. 2 new, 16 in progress, 0 critical, 4 high and 3 incomplete
  - Nova bugs with Ironic tag: 13 (+1). 1 new, 0 critical, 1 high
- 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
  - prepare_instance() is not called for whole disk images with 'agent' deploy interface https://bugs.launchpad.net/ironic/+bug/1713916:
    - Fix to return 'root_uuid' as part of command status https://review.openstack.org/#/c/500719/4
    - Fix ``agent`` deploy interface to call ``boot.prepare_instance`` https://review.openstack.org/#/c/499050/
  - 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

CI refactoring and missing test coverage
- Zuul v3 jobs in-tree migration tracking https://etherpad.openstack.org/p/ironic-zuulv3-intree-tracking: everything migrated except for bifrost stable/ocata (tests failing, someone needs to look into it)
- 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:
        - IPA - build tinycore based partitioned image with grub https://review.openstack.org/#/c/504888/
        - 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
    - 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

Ironic client API version negotiation (TheJulia, dtantsur)
- RFE https://bugs.launchpad.net/python-ironicclient/+bug/1671145
- gerrit topic: https://review.openstack.org/#/q/topic:bug/1671145
- status as of 13 Nov 2017:
    - patches on review:
        - switch the "ironic" CLI as well: https://review.openstack.org/515064 MERGED
    - TODO:
        - easier access to versions in ironicclient
        - 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 27 Sep 2017:
    - review needed

Classic drivers deprecation (dtantsur)
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/classic-drivers-future.html
- status as of 13 Nov 2017:
    - dev documentation for hardware types: TODO
    - switch documentation to hardware types:
        - install guide: https://review.openstack.org/#/c/517290/ needs updating
        - admin guide:
            - idrac docs (split away in a separate patch): https://review.openstack.org/517287 MERGED
            - TODO

Reference architecture guide (dtantsur, sambetts)
- status as of 06 Nov 2017:
    - Common bits: https://review.openstack.org/487410 MERGED
        - nits: https://review.openstack.org/516376 MERGED
    - 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
    - WIP code at https://review.openstack.org/440778

Routed network support (sambetts, vsaienk0, bfournie)
- status as of 27 Sep 2017:
    - WIP code at https://review.openstack.org/#/c/456235/

Rescue mode (rloo, stendulker, aparnav)
- Status as on 13 Nov 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.
        - Tempest tests based on standalone ironic is WIP.
        - -2'd until we do an ironic release (soon)
    - nova side:
        - https://blueprints.launchpad.net/nova/+spec/ironic-rescue-mode: approved for Queens; waiting for ironic part to be done first
        - code patch: https://review.openstack.org/#/c/416487/

Clean up deploy interfaces (vdrok)
- status as of 27 Sep 2017:
    - not started

Zuul v3 jobs in-tree (sambetts, derekh, jlvillal)
- status as of 13-Nov 2017:
    - etherpad tracking zuul v3 -> intree: https://etherpad.openstack.org/p/ironic-zuulv3-intree-tracking
    - everything migrated except for bifrost stable/ocata (tests failing, someone needs to look into it)
    - Next TODO is to convert jobs on master, to proper ansible. NOT a high priority though.

Graphical console interface (pas-ha, vdrok, rpioso)
- status as of 27 Sep 2017:
    - spec posted: https://review.openstack.org/#/c/306074/
        - needs to be cleaned up and revived
    - there is nova part here, which has to be approved too

BIOS config framework (dtantsur, yolanda, rpioso)
- status as of 13 Nov 2017:
    - spec under review: https://review.openstack.org/#/c/496481/
        - seems close, more review needed

Ansible deploy interface (pas-ha, yuiryz)
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ansible-deploy-driver.html
- status as of 30 Oct 2017:
    - patch proposed: https://review.openstack.org/510522

Traits support planning (johnthetubaguy, TheJulia, dtantsur)
- status as of 13 Nov 2017:
    - spec https://review.openstack.org/#/c/504531/ 2x +2 (modulo minor comments)
    - discussion: http://lists.openstack.org/pipermail/openstack-dev/2017-October/123675.html
        - call summary: http://lists.openstack.org/pipermail/openstack-dev/2017-November/124200.html
            - recording is https://bluejeans.com/s/K3wZZ (needs flash, but can be downloaded)

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"
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 - planning to raise the same under ironic-inspector under zuulv3.
- Raised https://review.openstack.org/#/c/515036/ in ironic-inspector
- 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.

Deploying with Apache and WSGI in CI (vsaienk0)
- ironic is mostly finished
    - (pas-ha) needs to be rewritten for uWSGI, patches on review https://review.openstack.org/#/c/507011/3
    - 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

Split away the tempest plugin (jlvillal)
- Proposed patch to create all the patches: https://review.openstack.org/489762  Merged!! :)
- Current (13-Nov-2017) (jlvillal): We will wait for jlvillal to return from his vacation (late November/early December) to proceed
    - This means that the following projects need to be working for stable branches:
        - openstack/ironic: Done
        - openstack/ironic-python-agent
        - openstack/ironic-lib
        - openstack/ironic-inspector
- jlvillal talked to infra and they suggested we do a batch upload as there are about 70 patches to merge in.
- need to port new patches in ironic/ironic-tempest-plugin (jlvillal)
- need to migrate ironic-inspector/ironic-tempest-plugin (jlvillal)


Inspector (dtantsur, milan)
- https://bugs.launchpad.net/ironic/+bug/1725211 was reported; the introspection abort thru ironic (spec https://review.openstack.org/#/c/482867/) should be "based" on this (goal from virtual inspector PTG)
- let's focus on the dnsmasq-based PXE filter driver implementation as this week's inspector review priority
- HA demo seems closer: https://review.openstack.org/#/c/510928/ solved my obvious introspection state discrepancies \0/

Bifrost (TheJulia)
- Quiet - TheJulia needs to pick backup a few items here in the short term. Volunteers to work on CI job migration would be appreciated.

ironic-ui (ppiela, TheJulia)
- Selection of interfaces upon enroll is up for review and requiers another review:
    - https://review.openstack.org/#/c/473957


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

OneView (ricardoas/fellypefca)
- Need to revert changes that moved from python-oneviewclient to hpOneView library, see IRC discussion: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2017-11-09.log.html#t2017-11-09T16:39:34
- See above, 'This Week's Priorities' -> 'Vendor priorities' -> 'oneview' for list of patches

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,

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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20171113/34ef209b/attachment.html>

More information about the OpenStack-dev mailing list