[RFE] Check duplicate nodes associated with the same BMC

Bug #1549664 reported by Zhenguo Niu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Confirmed
Wishlist
Unassigned

Bug Description

As I understand, one BMC should only be associated to one ironic node, but currently we can enroll any node with the BMC information which already used by another node, then assume node-A is deployed, and the new enrolled node-B becomes 'available' after node verifying, that means node-A can be override if a new nova boot request schedules to node-B, it's unacceptable.

I plan to add some check during node verifying process, then we can enroll a duplicate node but it can't be available.

Tags: rfe
Changed in ironic:
assignee: nobody → Zhenguo Niu (niu-zglinux)
description: updated
Revision history for this message
Dmitry Tantsur (divius) wrote :

This is a good call, but it's not as easy as it may seem, cause e.g. with IPMI bridging several nodes will have the same IPMI address. So maybe we should leave this check up to driver. Meaning this bug is actually an RFE.

Changed in ironic:
importance: Undecided → Wishlist
status: New → Confirmed
tags: added: rfe
summary: - Different nodes can associate the same physical BMC
+ [RFE]Different nodes can associate the same physical BMC
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic-specs (master)

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

Changed in ironic:
status: Confirmed → In Progress
summary: - [RFE]Different nodes can associate the same physical BMC
+ [RFE] Check duplicate nodes associated with the same BMC
Revision history for this message
Zhenguo Niu (niu-zglinux) wrote :
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/298184

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ironic-specs (master)

Change abandoned by Dmitry Tantsur (<email address hidden>) on branch: master
Review: https://review.openstack.org/296357
Reason: Hi! This patch has not been updated for several months, so I'm abandoning it. Please feel free to restore if you still plan to work on it.

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/298184
Reason: Hi! This patch has not been updated for several months, and the spec patch was abandoned, so I'm abandoning it. Please feel free to restore if you still plan to work on it.

Dmitry Tantsur (divius)
Changed in ironic:
assignee: Zhenguo Niu (niu-zglinux) → nobody
status: In Progress → Confirmed
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.