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 ----------------- - 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. - CI and docs work for classic drivers deprecation (see status below) - Required Backports/Nice to haves below - CRITICAL bugs (must be fixed and backported to queens before the release) - ironic port list fix: https://review.openstack.org/545069 MERGED - backport: https://review.openstack.org/#/c/545892/MERGED - 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 - Prepare for the PTG - https://etherpad.openstack.org/p/ironic-rocky-ptg Required Queens Backports ------------------------- - Fix for incorrect query during timeout check: https://review.openstack.org/545041 MERGED - queens: https://review.openstack.org/#/c/545806/ MERGED - Problem with traits and cleaning: https://bugs.launchpad.net/ironic/+bug/1750027 - https://review.openstack.org/#/c/545830/ MERGED - backport to stable/queens: https://review.openstack.org/#/c/546830/ MERGED - Problem with unrescue and netboot: https://review.openstack.org/#/c/544278/ MERGED - https://review.openstack.org/#/c/546026/ MERGED - rescue and UEFI: https://review.openstack.org/#/c/545186/ MERGED - backport: https://review.openstack.org/#/c/546955/ MERGED - configdrive overflow: https://review.openstack.org/#/c/334967/ MERGED - backport: https://review.openstack.org/#/c/546551/ MERGED - 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: None 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 12 Feb 2018 and 19 Feb 2018) - Ironic: 208 bugs (-1) + 243 wishlist items (-4). 3 new (+1), 155 in progress (-2), 1 critical, 33 high (+4) and 23 incomplete (+3) - Inspector: 13 bugs (-4) + 26 wishlist items (+1). 0 new, 12 in progress (-2), 0 critical (-2), 2 high (-1) and 4 incomplete - Nova bugs with Ironic tag: 16 (+2). 2 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 - 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 - establish foundation for using version negotiation in nova - nothing more for Queens. Stay tuned... - need to make sure that we discuss/agree with nova about how to do this Classic drivers deprecation (dtantsur) -------------------------------------- - spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/classic-drivers-future.html - status as of 19 Feb 2017: - switch documentation to hardware types: - need help from vendors updating their pages! - ilo: https://review.openstack.org/#/c/542593/ - irmc: https://review.openstack.org/#/c/541171/ MERGED - idrac looks fine for now - api-ref examples: TODO - ironic-inspector: https://review.openstack.org/#/c/545285/ - 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) ------------------------------------------------------------ - 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: 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: 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/ - rescue ramdisk doesn't boot on UEFI: https://review.openstack.org/#/c/545186/ - 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/ Ansible deploy interface (pas-ha) --------------------------------- - spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ansible-deploy-driver.html - status as of 5 Feb 2017: - code merged, CI coverage via the standalone job - docs: https://review.openstack.org/#/c/525501/ MERGED 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. - 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
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
