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. Docs due to the docs re-org - See http://lists.openstack.org/pipermail/openstack-dev/2017-July/119221.html 1.1. Ironic - https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:master+topic:doc-migration 1.2. Ironic-ui - https://review.openstack.org/#/q/status:open+project:openstack/ironic-ui+branch:master+topic:doc-migration 1.3. ironic-python-agent - https://review.openstack.org/#/q/status:open+project:openstack/ironic-python-agent+branch:master+topic:doc-migration 1.4. ironic-inspector - https://review.openstack.org/#/q/status:open+project:openstack/ironic-inspector+branch:master+topic:doc-migration 1.5. Other subprojects and repos have not been started: virtualbmc, ironic-lib, sushy, sushy-tools, moltenironic, bifrost, ironic-inspector-client 2. Booting from volume: 2.1. https://review.openstack.org/#/c/427053/ - OSC volume connector 2.2. https://review.openstack.org/#/c/427738 - OSC volume target 2.3. https://review.openstack.org/#/c/466186/ - support storage interface 3. Rolling upgrades: 3.1. Modifications for rolling upgrades: https://review.openstack.org/#/c/476779/ 3.2. 'Add new dbsync command with first online data migration': https://review.openstack.org/#/c/408556/ 4. Nova patch for VIF attach/detach: https://review.openstack.org/#/c/419975/ 5. Deprecation warning about default CLI version: https://review.openstack.org/#/c/442153/ 6. Classic driver deprecation spec: https://review.openstack.org/#/c/464046/ Bugs (dtantsur, vdrok, TheJulia) -------------------------------- - Stats (diff between 26 Jun 2017 and 10 Jul 2017) - Ironic: 259 bugs (+6) + 258 wishlist items (+3). 27 new (+3), 214 in progress (+7), 1 critical (-1), 33 high (+2) and 31 incomplete - Inspector: 13 bugs (+1) + 28 wishlist items (-2). 1 new, 12 in progress (-1), 0 critical, 3 high and 3 incomplete - Nova bugs with Ironic tag: 16 (+3). 4 new (+1), 0 critical, 0 high Essential Priorities ==================== CI refactoring and missing test coverage ---------------------------------------- - Standalone CI tests (vsaienk0) - next patch to be reviewed, needed for 3rd party CI: https://review.openstack.org/#/c/429770/ - 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 Generic boot-from-volume (TheJulia, dtantsur) --------------------------------------------- - specs and blueprints: - http://specs.openstack.org/openstack/ironic-specs/specs/approved/volume-connection-information.html - code: https://review.openstack.org/#/q/topic:bug/1526231 - http://specs.openstack.org/openstack/ironic-specs/specs/approved/boot-from-volume-reference-drivers.html - code: https://review.openstack.org/#/q/topic:bug/1559691 - https://blueprints.launchpad.net/nova/+spec/ironic-boot-from-volume - code: https://review.openstack.org/#/q/topic:bp/ironic-boot-from-volume - status as of most recent weekly meeting: - Python-ironicclient API support for volume connectors was landed last week. - Version 1.14.0 has been requested to be released - https://review.openstack.org/#/c/482143/ - More python-ironicclient patches exist and need to be reviewed. - We have observed some review activity on the nova patch: https://review.openstack.org/#/c/215385/ - The above python-ironicclient version needs to be released before the nova patch can be landed. - Patch/note tracking etherpad: https://etherpad.openstack.org/p/Ironic-BFV Ironic Patches: https://review.openstack.org/#/c/214586/ - Volume Connection Information Rest API Change. MERGED https://review.openstack.org/#/c/463930/ - CRUD notification updates for volume objects. MERGED https://review.openstack.org/#/c/463908/ - Enable cinder storage interface for generic hardware - Approved, Waiting to Merge https://review.openstack.org/#/c/463972/ - Add storage_interface to notifications - 4x+2 - Approved, Waiting to Merge https://review.openstack.org/#/c/466333/ - Devstack changes or Boot from Volume https://review.openstack.org/#/c/472740/ - Tempest test scenario for BFV python-ironicclient: https://review.openstack.org/#/c/427053/ - OSC volume connector https://review.openstack.org/#/c/427738 - OSC volume target https://review.openstack.org/#/c/466186/ - support storage interface - Requires 463908 to land https://review.openstack.org/#/c/479840 - Ironic cli volume connector commands https://review.openstack.org/#/c/479848 - ironic cli volume target commands Rolling upgrades and grenade-partial (rloo, jlvillal) ----------------------------------------------------- - spec approved; code patches: https://review.openstack.org/#/q/topic:bug/1526283 - status as of most recent weekly meeting: - next patches ready for reviews: - 'Modifications for rolling upgrades': https://review.openstack.org/#/c/476779/ - 'Add new dbsync command with first online data migration': https://review.openstack.org/#/c/408556/ - to address restarting services after unpinning, spec ready for reviews: 'SIGHUP restarts services with updated configs': https://review.openstack.org/474309 - Testing work: done as per spec, but rloo wants to ask vasyl whether we can improve. grenade test will do upgrade so we have old API sending requests to old and/or new conductor, but rloo doesn't think there is anything to control -which- conductor handles the request, so what if old conductor handles all the requests? 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 - Two ironic patches in review: https://review.openstack.org/473888 and https://review.openstack.org/#/c/473913/. Deploying with Apache and WSGI in CI (vsaienk0) ----------------------------------------------- - ironic part seems finished (needs double-checking) - do we have install-guide bits on how to do it? - inspector is TODO and depends on https://review.openstack.org/#/q/topic:bg/1525218 Driver composition (dtantsur) ----------------------------- - spec: http://specs.openstack.org/openstack/ironic-specs/specs/approved/driver-composition-reform.html - gerrit topic: https://review.openstack.org/#/q/status:open+topic:bug/1524745 - status as of 10 Jul 2017: - no progress this week - install guide / admin guide docs - done except for: - upgrading to hardware types: TODO - dev docs: TODO - fixing nits from previous reviews - new hardware types: - we need volunteers for SNMP - rpioso volunteered to do drac - API for hardware interface properties: - proposed spec: https://review.openstack.org/#/c/471174/ - rloo checked with dtantsur, is not needed for completion of driver composition - spec on the classic drivers deprecation: https://review.openstack.org/#/c/464046/ - updated based on comments Feature parity between two CLIs (rloo, dtantsur) ------------------------------------------------ - Now that we have feature parity: - It is "official": client patch with a release note stating the feature parity (in the "prelude" section): reno: feature parity between ironic & OSC https://review.openstack.org/478230 has merged. - RFE to track deprecation of Ironic CLI: https://bugs.launchpad.net/python-ironicclient/+bug/1700815 - Bug to track replacement of ironic commands with OSC baremetal commands: https://bugs.launchpad.net/python-ironicclient/+bug/1703409 - From PTG discussion (https://etherpad.openstack.org/p/ironic-pike-ptg-operations): - No more 'ironic' commands in docs. - No deprecation in Pike though, defer to Queens - We will still need to have feature parity (new CLI features in both ironic & OSC CLI) until we deprecate ironic CLI - This can be removed from the subteam reporting (rloo) OSC default API version change (dtantsur) ----------------------------------------- - status as of 10 Jul 2017: - spec merged: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/cli-default-api-version.html - OSC CLI patch needs review: https://review.openstack.org/#/c/442153/ node tags (zhenguo, vdrok) -------------------------- - spec: http://specs.openstack.org/openstack/ironic-specs/specs/approved/nodes-tagging.html - topic: https://review.openstack.org/#/q/topic:bug/1526266 - status as of 19 Jun 2017: - next on review: https://review.openstack.org/#/c/233357/ High Priorities =============== Rescue mode (stendulker/aparnav) -------------------------------- - 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 - Status: 10/7/17 - Need to resolve merge conflicts. Will resolve shortly. Post-deploy VIF attach/detach (sambetts, vsaienk0) -------------------------------------------------- - blueprint: https://blueprints.launchpad.net/nova/+spec/ironic-hotplug-interfaces - gerrit topic (nova): https://review.openstack.org/#/q/topic:bp/ironic-plug-unplug-vifs-update - Nova side https://review.openstack.org/#/c/419975/ Physical network awareness (sambetts, vsaienk0) ----------------------------------------------- - Etherpad tracking networking work: https://etherpad.openstack.org/p/Ironic-networking - spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/physical-network-awareness.html - Physical network awareness implementation - ironic: - Physical network-aware port mapping: https://review.openstack.org/#/c/470406 +A'd waiting for a recheck after static.o.o failure - API: https://review.openstack.org/469933 - Tempest tests: https://review.openstack.org/#/c/470915 - python-ironicclient: - OSC & ironic clients: https://review.openstack.org/#/c/461893 - Related: - Refactor VIFPortIDMixin, separate out neutron-specific code into a subclass: https://review.openstack.org/477597 (ready for review) - Create a fake network interface for tempest testing: <included in above patch currently, may be refactored out) - Assume neutron port exists in VIFPortIDMixin: https://review.openstack.org/#/c/476949 - networking-baremetal initial patches - https://review.openstack.org/448073 - https://review.openstack.org/448157 - https://review.openstack.org/450714 - Routed networks support (sambetts, vsaienk0) - Dependant on physnet awareness - networking-baremetal initial patches - https://review.openstack.org/#/c/456235/ - RFE: https://bugs.launchpad.net/ironic/+bug/1658964 - No RFE right now as all planned work is in networking-baremetal, but going to keep this around in case we need to make unexpected changes to ironic Neutron event processing (vdrok, vsaienk0) ------------------------------------------ - spec at https://review.openstack.org/343684, ready for reviews - WIP code at https://review.openstack.org/440778 ironic-python-agent's REST API versioning (sambetts) ------------------------------------------------------ - RFE: https://bugs.launchpad.net/ironic-python-agent/+bug/1602265 - gerrit topic: https://review.openstack.org/#/q/topic:bug/1602265 - status as of 5 Jun 2017: - spec merged: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ipa_api_versioning.html - code needs rebasing and updating Optional Priorities =================== Split away the tempest plugin (jlvillal) ---------------------------------------- - Have not had time yet to finish scripts (jlvillal) 10-July-2017 - patches waiting for review https://review.openstack.org/#/q/project:openstack/ironic-tempest-plugin - need to port new patches in ironic/ironic-tempest-plugin (jlvillal) - need to migrate ironic-inspector/ironic-tempest-plugin (jlvillal) deploy steps (rloo, mat128, yolanda) ------------------------------------ - Ocata summit design session etherpad: https://etherpad.openstack.org/p/ironic-ocata-summit-deploy-time-raid - RFE: https://bugs.launchpad.net/ironic/+bug/1630376 - status as of most recent weekly meeting: - spec needs updating: https://review.openstack.org/#/c/412523/ Redfish driver (lucasagomes, dtantsur) -------------------------------------- - topic: https://review.openstack.org/#/q/topic:bug/1526477 - status as of Jun 30: - fix for power actions: https://review.openstack.org/478958 MERGED - I think this concludes this topic! - we need more people reviewing sushy - OOB inspection is in progress, but is probably outside of the scope of this particular topic Supported power states API (dtantsur) ------------------------------------- - topic: https://review.openstack.org/#/q/topic:bug/1526226 - note: the same topic as for soft power and NMI is used, as it's covered by the same spec - status as of Jun 26th: - next on review: https://review.openstack.org/#/c/382194/ (needs update) Available clean steps API (rloo) -------------------------------- - spec had been approved in mitaka: http://specs.openstack.org/openstack/ironic-specs/specs/5.0/manual-cleaning.html#get-nodes-node-ident-cleaning-steps - status as of most recent weekly meeting: - patch has been rebased and updated, but reviewer comments still need to be addressed and it needs testing: https://review.openstack.org/#/c/278566/ etags in the REST API (pas-ha) ------------------------------ - RFE: https://bugs.launchpad.net/ironic/+bug/1605728 - topic: https://review.openstack.org/#/q/topic:bug/1605728 - Spec has merged: http://specs.openstack.org/openstack/ironic-specs/specs/approved/evolve-etags.html - status as of most recent weekly meeting: - The first patch is for adding etags for nodes endpoint, needs rebasing: https://review.openstack.org/392213 - everything needs a rebase - (galyna) Patches-POC are up. Demo: https://asciinema.org/a/103604 Subprojects =========== Inspector (dtantsur, milan) --------------------------- - inspector docs affected by the switch (404) patches up, need to do the same for the client - reviews needed on the worker-api split https://review.openstack.org/#/c/462861/ - PXE filter driver introduced https://review.openstack.org/#/c/439603/ further refactoring work ongoing to switch the iptables filter to the new driver API as well as to introduce the dnsmasq driver ironic-ui (betherly, ppiela, TheJulia) -------------------------------------- - Targeting Pike-3 for release, several outstanding features are in review at this time. ......... 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