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. Make a release! 1.1. See the countdown below for blocker patches 2. Remaining hardware types: 2.1. Dell: 2.1.1. bug in the deploy interface: https://review.openstack.org/#/c/493421/ 2.1.2. the type itself: https://review.openstack.org/#/c/491263/ 2.2. UCS(m): 2.2.1. https://review.openstack.org/#/c/486100/ 3. If time allows - vendor patches listed below (see "Review requests for non-blockers") Release countdown ================= This list contains blockers for the upcoming Pike release. Please note if something is in the process of being landed, and strikeout once merged. - bifrost: - ready for release - ironic - update links in docs and code https://review.openstack.org/#/c/483797/ APPROVED - this seems to be the last change for doc-migration topic - upgrade to hardware types: moar drivers: TODO - rolling upgrade docs: https://review.openstack.org/#/c/416565/ MERGED - followup https://review.openstack.org/493597 - resource_class docs update: https://review.openstack.org/491773 MERGED - Using resource classes in nova still does not work: https://etherpad.openstack.org/p/nova-ironic-resource-class-migration - nova: migration of active instances: https://review.openstack.org/#/c/487954/ MERGED - nova: fix inventory: https://review.openstack.org/#/c/492964/ needs a revision - blocks switching to resource classes: https://review.openstack.org/476968 - BFV Functional tests: https://review.openstack.org/#/c/473717/ 1x +2 - ironic-inspector - release notes clean up: https://review.openstack.org/#/c/492489/ 1x +2 - ironic-python-agent - release notes clean up: https://review.openstack.org/#/c/492803/ 1x +2 - ironic-python-agent-builder, ironic-tempest-plugin - no release this time - ironic-ui, ironic-lib, python-ironicclient, python-ironic-inspector-client, sushy - already released! - networking-baremetal: - documentation: https://review.openstack.org/#/c/450714/ 1x +2 - sushy-tools: - ready for release - virtualbmc: - ready for release Review requests for non-blockers ================================ Please make sure your patch is acceptable and the soft feature freeze. - Introduce hpOneView and ilorest to OneView - https://review.openstack.org/#/c/468129/ - there is actually the whole patch chain: https://review.openstack.org/#/q/topic:bug/1706725 - I don't think we should merge the 1st patch without merging the remaining.. +1 rloo - Add the new capabilities to the iLO InspectInterface: https://review.openstack.org/422902 - Support SUM based firmware update as clean step for iLO drivers: https://review.openstack.org/#/c/422572/ - iRMC: Support volume boot for iRMC virtual media boot interface: https://review.openstack.org/#/c/467911/ - iLO: Refresh firmware versions on firmware update: https://review.openstack.org/#/c/346683/ - Update new proliantutils version to 2.4.0: https://review.openstack.org/#/c/478741/ needs updating - ilo hardware type doc https://review.openstack.org/#/c/476895/ 1x +2 - snmp hardware type doc https://review.openstack.org/#/c/493468/ - iDRAC: - Fix to DRAC classic driver double manage/provide bug: https://review.openstack.org/#/c/493421/ - Hardware type WIP: https://review.openstack.org/#/c/491263/ Bugs (dtantsur, vdrok, TheJulia) -------------------------------- - Stats (diff between 07 Aug 2017 and 14 Aug 2017) - Ironic: 246 bugs (-4) + 258 wishlist items (+1). 24 new (-5), 188 in progress (-9), 0 critical, 31 high (-1) and 31 incomplete (+1) - Inspector: 12 bugs (-1) + 28 wishlist items. 2 new, 10 in progress (-1), 0 critical, 2 high (-1) and 3 incomplete - Nova bugs with Ironic tag: 17 (-2). 0 new, 0 critical, 1 high 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/ - 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 Essential Priorities ==================== Nova resource class based scheduling changes (dtantsur) ------------------------------------------------------- - I have to add this, as there are things to finish by end of Pike to avoid problems in Queens - TODO as of 14 Aug 2017: - devstack: - always set resource class: https://review.openstack.org/491777 MERGED - support scheduling based on rsc: https://review.openstack.org/476968 - nova: - fix reporting inventory: https://review.openstack.org/492964 needs revision - prevent updating resource_class for active nodes: https://review.openstack.org/#/c/492216/ MERGED - upgrade documentation and reno: https://review.openstack.org/491773 MERGED - Optionally: - integration tests: https://review.openstack.org/#/c/443628/ Generic boot-from-volume (TheJulia, dtantsur) --------------------------------------------- - BFV Meetings on hold until September. - 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 14 Aug 2017: - Patch/note tracking etherpad: https://etherpad.openstack.org/p/Ironic-BFV Ironic Patches: https://review.openstack.org/#/c/479326/ - BFV deploy follow-up - MERGED https://review.openstack.org/#/c/466568/ - devstack testing guide - MERGED https://review.openstack.org/#/c/473717/ - functional tests 1x +2 Project-Config: https://review.openstack.org/#/c/490710/ - moving the experimental job to the check queue MERGED - Not required for Pike: - Nova: https://review.openstack.org/#/c/468353 - Ironic: Get IP address for volume connector 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: - last big code patch MERGED: 'Add new dbsync command with first online data migration': https://review.openstack.org/#/c/408556/ - 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? - should add 'ironic-dbsync online_data_migrations' call to regular grenade test (not multinode test because grenade multinode test doesn't upgrade all services) - to address restarting services after unpinning, spec ready for reviews: 'SIGHUP restarts services with updated configs': https://review.openstack.org/474309 - next patch ready for reviews: Rolling upgrade procedure documentation: https://review.openstack.org/#/c/416565/ - MERGED, follow up needed Reference architecture guide (dtantsur) --------------------------------------- - status as of 14 Aug 2017: - Common bits: https://review.openstack.org/487410 needs a revision - I guess this moves to Queens 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 14 Aug 2017: - important for Pike - upgrade docs on moving to hardware types: https://review.openstack.org/486696 MERGED - dev docs on writing hardware types: TODO - optional for Pike // important for Queens: - new hardware types: - UCS: https://review.openstack.org/#/c/486100/ - SNMP: https://review.openstack.org/#/c/485041/ MERGED - DRAC: (rpioso) WIP: https://review.openstack.org/#/c/491263/ - dtantsur thinks that https://bugs.launchpad.net/ironic/+bug/1676387 should be fixed first. - Fix has been proposed: https://review.openstack.org/#/c/493421/. - OneView (nicodemos/ricardoas): https://review.openstack.org/#/c/485798/ MERGED - 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: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/classic-drivers-future.html to be continued in Queens High Priorities =============== 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 - Related: - Refactor VIFPortIDMixin 2: https://review.openstack.org/#/c/486125/ - Refactor VIFPortIDMixin 3: https://review.openstack.org/#/c/486126/ - Refactor VIFPortIDMixin 4: https://review.openstack.org/#/c/486127/ - Create a fake network interface for tempest testing: https://review.openstack.org/#/c/486128/ - Use fake network interface in VIF attach tempest tests: https://review.openstack.org/#/c/486129/ - Assume neutron port exists in VIFPortIDMixin: https://review.openstack.org/#/c/476949 - networking-baremetal initial patches - https://review.openstack.org/448157 MERGED - https://review.openstack.org/450714 1x +2 - Routed networks support (sambetts, vsaienk0) - Dependent on physnet awareness - networking-baremetal initial patches - https://review.openstack.org/#/c/456235/ needs updating - 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 ironic-python-agent's REST API versioning (sambetts) ------------------------------------------------------ - RFE: https://bugs.launchpad.net/ironic-python-agent/+bug/1602265 - spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ipa_api_versioning.html - gerrit topic: https://review.openstack.org/#/q/topic:bug/1602265 - status as of 14 Aug 2017: - ironic part: https://review.openstack.org/#/c/364861/ - IPA part: https://review.openstack.org/#/c/364834/ needs a revision - blocked by the ironic part per recent discussions - discussed in ironic meeting; most likely will not land in Pike Refactoring of code accessing other services (pas-ha) ----------------------------------------------------- - gerrit topic: https://review.openstack.org/#/q/topic:bug/1699547 - status as of 1 Aug 2017: ready for review - discussed in ironic meeting; -2'd until Queens Optional 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 - Two ironic patches in review: - https://review.openstack.org/473888 - MERGED - https://review.openstack.org/#/c/473913/ - MERGED - Raised the patch to move the experimental job to non-voting https://review.openstack.org/489278 MERGED - anupn to update the python3 job to build tinyipa with python3 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 Aug 14th: - API patch: https://review.openstack.org/#/c/382194/ Moved to later releases ======================= These are the things we had on priorities, but they are not making it in Pike. 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:bug/1525218 - ironic is done, inspector will be bumped to Queens Split away the tempest plugin (jlvillal) ---------------------------------------- - Proposed patch to create all the patches: https://review.openstack.org/489762 - jlvillal talked to infra and they suggested we do a batch upload as there are about 70 patches to merge in. - 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) 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: 24 Jul 2017 - Patches in merge conflict. Need to rebase. The nova patch for Rescue is abandoned and rescue tempest patch(https://review.openstack.org/#/c/452308/) which is dependent on the nova patch is in merge conflict. - any plans to revive the nova patch soon(ish)? Neutron event processing (vdrok, vsaienk0) ------------------------------------------ - spec at https://review.openstack.org/343684, ready for reviews - WIP code at https://review.openstack.org/440778 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/ needs rebase 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/ - this needs update to spec and possibly discussion at PTG. jroll and lucasagomes (with cdent + sdague) proposed a different API. IRC discussions: - http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2016-03-16.log.html#t2016-03-16T11:51:47 - http://eavesdrop.openstack.org/irclogs/%23openstack-dev/%23openstack-dev.2016-03-16.log.html#t2016-03-16T13:55:43 - move to Queens 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/ 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 Jul 17th: - 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) --------------------------- - cleaning up release notes - PXE filter driver patches, WSGI split, etc. postponed till after the release Bifrost (TheJulia) ------------------ - 4.0.0 release proposed. ironic-ui (ppiela, TheJulia) ---------------------------- - 3.0.0 released last week. ......... 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