Fuel for OpenStack

[nailgun] networks are lost when interface is deleted

  1. Series 5.1.x
  2. Bug #1425901
Bug #1425901 reported by Aleksey Kasatkin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Aleksey Kasatkin
Fuel for OpenStack 6.1
5.1.x
Fix Committed
High
Aleksey Kasatkin
Fuel for OpenStack 5.1.1-updates
6.0.x
Fix Committed
High
Aleksey Kasatkin
Fuel for OpenStack 6.0-updates

Bug Description

Issue arrives this way:

1. Some networks are mapped to the node's NIC.
2. NIC is deleted from nailgun DB for some reason (e.g. it was removed from node).
3. All networks that were mapped to that NIC become unavailable for that node.

Currently, the only way for deployed nodes to return networks to the node is to edit nailgun DB.
For newly added nodes you can remove and re-add node to environment.

Aleksey Kasatkin (alekseyk-ru)
Changed in fuel:
assignee: nobody → Fuel Python Team (fuel-python)
Aleksey Kasatkin (alekseyk-ru)
tags: added: customer-found
Ryan Moe (rmoe)
Changed in fuel:
status: New → Confirmed
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
importance: Medium → High
Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :

We should warn user about changes in node hardware before deployment and reset network checker status. We should not update our database in case of hardware changes after deployment.

tags: added: size-m
Revision history for this message
Nikolay Markov (nmarkov) wrote :

What is desired behavior in this case? I think this may be even a separate feature.

Revision history for this message
Aleksey Kasatkin (alekseyk-ru) wrote :

1. After deployment - don't update nailgun DB - no HW changes accepted.
2. Before deployment - warn user that networks from removed NICs are moved or deleted. User should be able to reallocate such networks (via UI preferably). AFAIS, there are several options here:
a. move all networks to other NIC(s)
b. move them but keep available for allocation
c. delete them but allow to add again

See related bug https://bugs.launchpad.net/fuel/+bug/1425907

Dmitry Pyzhov (dpyzhov)
tags: added: feature
Dmitry Pyzhov (dpyzhov)
tags: added: feature-hardware-changes
tags: removed: nailgun
Dmitry Pyzhov (dpyzhov)
tags: added: feature-hardware-change
removed: feature-hardware-changes
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Aleksey Kasatkin (alekseyk-ru)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/169012
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=e687e76d95925ca7f0e5bac19d451ff77dc89b20
Submitter: Jenkins
Branch: master

commit e687e76d95925ca7f0e5bac19d451ff77dc89b20
Author: Aleksey Kasatkin <email address hidden>
Date: Mon Mar 30 20:10:17 2015 +0300

    Allow node interfaces update if it's in 'discover' or 'error' state

    Node inferfaces update is allowed when node is in one of the following states:
    'discover', 'error'. It's locked for other node states.
    It's not locked on node creation, i.e. when node.nic_interfaces is empty yet.

    Partial-Bug: #1425901
    Closes-Bug: #1428776

    Change-Id: Icc60c432296c28e89d71b1fd0dd864870553a467

Aleksey Kasatkin (alekseyk-ru)
Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/5.1)

Fix proposed to branch: stable/5.1
Review: https://review.openstack.org/170078

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/6.0)

Fix proposed to branch: stable/6.0
Review: https://review.openstack.org/170183

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/6.0)

Reviewed: https://review.openstack.org/170183
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=b37c553e063aefdcc8549b0388c343cc63ff52ab
Submitter: Jenkins
Branch: stable/6.0

commit b37c553e063aefdcc8549b0388c343cc63ff52ab
Author: Aleksey Kasatkin <email address hidden>
Date: Mon Mar 30 20:10:17 2015 +0300

    Allow node interfaces update if it's in 'discover' or 'error' state

    Node inferfaces update is allowed when node is in one of the following states:
    'discover', 'error'. It's locked for other node states.
    It's not locked on node creation, i.e. when node.nic_interfaces is empty yet.

    Closes-Bug: #1425901
    Closes-Bug: #1428776
    Change-Id: Icc60c432296c28e89d71b1fd0dd864870553a467

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/5.1)

Reviewed: https://review.openstack.org/170078
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=661a4f9502d2e6101ed2f5eb995bb675e6c3b2c5
Submitter: Jenkins
Branch: stable/5.1

commit 661a4f9502d2e6101ed2f5eb995bb675e6c3b2c5
Author: Aleksey Kasatkin <email address hidden>
Date: Thu Apr 2 14:05:40 2015 +0300

    Allow node interfaces update if it's in 'discover' or 'error' state

    Node inferfaces update is allowed when node is in one of the following states:
    'discover', 'error'. It's locked for other node states.
    It's not locked on node creation, i.e. when node.nic_interfaces is empty yet.

    Closes-Bug: #1425901
    Closes-Bug: #1428776
    Change-Id: Icc60c432296c28e89d71b1fd0dd864870553a467

Nikolay Markov (nmarkov)
Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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