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

Yeleswarapu, Ramamani ramamani.yeleswarapu at intel.com
Mon Mar 26 18:07:57 UTC 2018


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
- Deploy Steps
    - https://review.openstack.org/#/c/549493/ 1x+2 and 1x-1
- Remaining Rescue patches
    - https://review.openstack.org/#/c/546919/ - Fix a bug for unrescuiing with whole disk image
        - better fix: https://review.openstack.org/#/c/499050/  - Fix ``agent`` deploy interface to call ``boot.prepare_instance`` Updated 19-Mar-2018.
    - https://review.openstack.org/#/c/538119/ - Rescue mode standalone tests
    - https://review.openstack.org/#/c/528699/ - Tempest tests with nova (This can land after nova work is done. But, it should be ready to get the nova patch reviewed.)
- Management interface boot_mode change
    - https://review.openstack.org/#/c/526773/
- Bios interface support
    - https://review.openstack.org/#/c/511162/
    - https://review.openstack.org/#/c/528609/

Vendor priorities
    Patches in works for SDK update, but not posted yet, currently rebuilding third party CI infra after a disaster...
    RFE and first several patches for adding UEFI support will be posted by Tuesday, 1/9
    https://review.openstack.org/#/c/530838/ - OOB Raid spec for iLO5
    None - a few works are work in progress
    None at this time - No subteam at present.

Subproject priorities

ironic-inspector (or its client):



sushy and the redfish driver:

Bugs (dtantsur, vdrok, TheJulia)
- (TheJulia) Ironic has moved to Storyboard. Dtantsur has indicated he will update the tool that generates these stats.
- Stats (diff between  12 Mar 2018 and 19 Mar 2018)
- Ironic: 225 bugs (+14) + 250 wishlist items (+2). 15 new (+10), 152 in progress, 1 critical, 36 high (+3) and 26 incomplete (+2)
- Inspector: 15 bugs (+1) + 26 wishlist items. 1 new (+1), 14 in progress, 0 critical, 3 high and 4 incomplete
- Nova bugs with Ironic tag: 14 (-1). 1 new, 0 critical, 0 high
- critical:
    - sushy: https://bugs.launchpad.net/sushy/+bug/1754514 (basic auth broken when SessionService is not present)
- note: the increase in bug count is probably because now the dashboard tracks virtualbmc and networking-baremetal
- the dashboard was abruptly deleted and needs a new home :(
- use it locally with `tox -erun` if you need to
- 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.


Deploy Steps (rloo, mgoddard)
- status as of 26 March 2018:
    - spec for deployment steps framework: https://review.openstack.org/#/c/549493/
        - ready for reviews, 1 +2 already (but might need another update)

BIOS config framework(zshi, yolanda, mgoddard, hshiina)
- status as of 26 March 2018:
    - Spec has merged: https://review.openstack.org/#/c/496481/
    - List of ordered patches:
        - BIOS Settings: Add DB model: https://review.openstack.org/511162
        - Add bios_interface db field https://review.openstack.org/528609
        - BIOS Settings: Add DB API: https://review.openstack.org/511402
        - BIOS Settings: Add RPC object https://review.openstack.org/511714
        - Add BIOSInterface to base driver class https://review.openstack.org/507793
        - BIOS Settings: Add BIOS caching: https://review.openstack.org/512200
        - Add Node BIOS support - REST API: https://review.openstack.org/512579

Conductor Location Awareness (jroll, dtantsur)
- no update, will write spec soonish

Reference architecture guide (dtantsur, jroll)
- status as of 26 Mar 2018:
    - Dublin PTG consensus was to start with small architectural building blocks.
        - basic architecture explanation: https://review.openstack.org/554284 MERGED
    - list of cases from the Denver 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?

Graphical console interface (mkrai, anup-d-navare, TheJulia)
- status as of 19 Mar 2018:
    - VNC Graphical console spec: https://review.openstack.org/#/c/306074/
        - needs update, address comments
    - nova blueprint: https://blueprints.launchpad.net/nova/+spec/ironic-vnc-console

Neutron event processing (vdrok)
- status as of 19 Mar 2018:
    - spec at https://review.openstack.org/343684
        - Needs update
    - WIP code at https://review.openstack.org/440778
        - code is being rewritten to look a bit nicer (major rewrite), spec update coming afterwards


Updating nova virt to use REST API (TheJulia)
Status as of 26 Mar 2018:
    Attempting to determine a new path of least resistance. Multiple cached clients is gaining resistance in nova. Changing the microversion mid-flight is problematic.

Storyboard migration (TheJulia, dtantsur)
Status as of March 26th
TheJulia to propose patches to docs where appropriate.
dtantsur to rewrite the bug dashboard

Management interface refactoring (etingof, dtantsur)
- Status as of March 26th:
    - boot mode in ManagementInterface: https://review.openstack.org/#/c/526773/ needs review

Getting clean steps (rloo, TheJulia)
- Status as of March 26th:
    - Cleanhold specification updated - https://review.openstack.org/#/c/507910/

Project vision (jroll, TheJulia)
- Status as of March 26th:
    - jroll to send email detailing the session this week, failed last week, ENOTENOUGHTIME

SIGHUP support (rloo)
- Proposed for ironic by rloo -- this is done: https://review.openstack.org/474331 MERGED\o/
    - ironic-inspector
    - networking-baremetal

Stretch Goals
NOTE: These items will be migrated into storyboard and will be removed from the weekly whiteboard once storyboard is in-place

Classic driver removal formerly Classic drivers deprecation (dtantsur)
- spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/classic-drivers-future.html
- status as of 26 Mar 2018:
    - switch documentation to hardware types:
        - api-ref examples: TODO
        - update https://wiki.openstack.org/wiki/Ironic/Drivers: TODO
            - or should we kill it with fire in favour of the docs?
    - ironic-inspector:
        - documentation: https://review.openstack.org/#/c/545285/ MERGED
            - backport: https://review.openstack.org/#/c/554586/
        - enable fake-hardware in devstack: https://review.openstack.org/#/c/550811/ MERGED
        - change the default discovery driver: https://review.openstack.org/#/c/550464/
    - migration of CI to hardware types
        - IPA: https://review.openstack.org/553431 MERGED
        - ironic-lib: https://review.openstack.org/#/c/552537/ MERGED
        - python-ironicclient: https://review.openstack.org/552543 MERGED
        - python-ironic-inspector-client: https://review.openstack.org/552546 +A MERGED
        - virtualbmc: https://review.openstack.org/#/c/555361/ MERGED
    - started an ML thread tagging potentially affected projects: http://lists.openstack.org/pipermail/openstack-dev/2018-March/128438.html

Redfish OOB inspection (etingof, deray, stendulker)

Zuul v3 playbook refactoring (sambetts, pas-ha)

Before Rocky

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/ Rebase/update required
        - 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)

Queens High Priorities

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:
    Challenges 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:
    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
            - TheJulia, March 19th 2018: This RFE seems not to contain detail on what is desired to be improved upon, and ultimately just seems like refactoring/improvement work and may not then need an rfe.
        - https://bugs.launchpad.net/networking-baremetal/+bug/1749162
            - TheJulia, March 19th 2018: This RFE makes sense, although I would classify it as a general improvement. If we wish to adhere to strict RFE approval for networking-baremetal work, then I think we should consider this approved since it is minor enhancement to improve operation.

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
            - Rescue mode standalone tests: https://review.openstack.org/#/c/538119/ (failing CI, not ready for reviews)
            - 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 - Done
            - 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) Posted https://review.openstack.org/554673
        - code patch: https://review.openstack.org/#/c/416487/ Needs revision
        - 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/

OpenStack Priorities

- TheJulia needs to just declare this done.

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
    - (TheJulia, March 19th, 2018) Perhaps because of the different direction, we should consider ourselves done?


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.


OneView (???)
- Oneview presently does not have a subteam.

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,

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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20180326/36d21552/attachment.html>

More information about the OpenStack-dev mailing list