[RFE] Allow names in {provisioning,cleaning}_network option

Bug #1614938 reported by Dmitry Tantsur
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Wishlist
Dmitry Tantsur

Bug Description

Names are useful for humans. Let's use them for provisioning and cleaning networks.

A suggestion:
1. Rename {provisioning,cleaning}_network_uuid to {provisioning,cleaning}_network (with deprecation)
2. Accept network names in these fields
3. Only validate these network names during the network interface validate() call, but not on start up. This allows the name to be set and ironic to be brought up before the network is created, which can be useful for installers.

A suggestion to discuss: have some name as a default.

Tags: rfe-approved
Dmitry Tantsur (divius)
summary: - [RFE] Allow names in {provisioning,cleaning}_network option to fix
- chicken-and-egg problem in installers
+ [RFE] Allow names in {provisioning,cleaning}_network option
description: updated
Revision history for this message
Jim Rollenhagen (jim-rollenhagen) wrote :

+1

description: updated
description: updated
tags: added: rfe-approved
removed: rfe
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

Fix proposed to branch: master
Review: https://review.openstack.org/357983

Changed in ironic:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ironic (master)

Change abandoned by Dmitry Tantsur (<email address hidden>) on branch: master
Review: https://review.openstack.org/357983
Reason: Abandoning this for now, seems like it's too tricky to implement in a safe way.

Dmitry Tantsur (divius)
Changed in ironic:
status: In Progress → Triaged
assignee: Dmitry Tantsur (divius) → nobody
Changed in ironic:
assignee: nobody → Dmitry Tantsur (divius)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/357983
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=b6a2919b6198c8fb24d6f7e597f79a08f6ffe049
Submitter: Jenkins
Branch: master

commit b6a2919b6198c8fb24d6f7e597f79a08f6ffe049
Author: Dmitry Tantsur <email address hidden>
Date: Fri Aug 19 18:01:31 2016 +0200

    Support names for {cleaning,provisioning}_network

    Names are easier to use for humans, so let's allow them.
    The actual name -> UUID convertion happens during validation and its
    result is cached for each interface instance.

    This change will allow recreating a network without changing Ironic
    configuration or setting a network name in advance (e.g. by an installer).

    Note that when names are used, an administrator is responsible for ensuring
    that no other networks have the same name (names are not unique in Neutron).

    Change-Id: I9905bdc3f778310ba191b2bed6eb11bad11a2846
    Closes-Bug: #1614938

Changed in ironic:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ironic 7.0.0

This issue was fixed in the openstack/ironic 7.0.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.