Bug 1258651 – importing new nodes from instackenv will overwrite existing nodes capabilities

Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.

Bug 1258651 - importing new nodes from instackenv will overwrite existing nodes capabilities

Summary:	importing new nodes from instackenv will overwrite existing nodes capabilities

Status:	CLOSED ERRATA

Aliases:	None

Product:	Red Hat OpenStack
Classification:	Red Hat
Component:	os-cloud-config (Show other bugs)
Sub Component:
Version:	7.0 (Kilo)
Hardware:	Unspecified Unspecified

Priority	high Severity unspecified
Target Milestone:	y2
Target Release:	7.0 (Kilo)
Assigned To:	Jay Dobies
QA Contact:	Alexander Chuzhoy
Docs Contact:

URL:
Whiteboard:
Keywords:	Reopened, Triaged

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2015-08-31 18:45 EDT by bigswitch
Modified:	2015-12-21 11:55 EST (History)
CC List:	7 users (show)

See Also:
Fixed In Version:	os-cloud-config-0.2.8-9.el7ost
Doc Type:	Bug Fix
Doc Text:	When registering new nodes and using the existing JSON file to store their info, the director would overwrite the capabilities of existing nodes. This fix skips the existing nodes. Now users can add new nodes to the original JSON file without erasing capabilities for existing nodes.
Story Points:	---
Clone Of:
Environment:
Last Closed:	2015-12-21 11:55:07 EST
Type:	Bug
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

External Trackers
Tracker	ID	Priority	Status	Summary	Last Updated
Red Hat Product Errata	RHBA-2015:2651	normal	SHIPPED_LIVE	Red Hat Enterprise Linux OSP 7 director Bug Fix Advisory	2015-12-21 16:50:26 EST

Groups: None (edit)

Description bigswitch 2015-08-31 18:45:54 EDT

Description of problem:
when import new nodes from a new instackenv.json file, it will overwrite the capabilities field in existing nodes. 

Version-Release number of selected component (if applicable):
7.0

How reproducible:
100%

Steps to Reproduce:
1. create a new instackenv.json file with new nodes information
2. import using openstack baremetal import --json instackenv.json
3. notice existing nodes capabilities field are overwritten. roles such as controller and compute is removed

Actual results:

Expected results:
should not disturb existing node information

Additional info:

Comment 3 bigswitch 2015-09-14 14:17:33 EDT

hi,
this is a duplicate of bug 1241199.

Comment 4 bigswitch 2015-09-14 14:17:55 EDT


*** This bug has been marked as a duplicate of bug 1241199 ***

Comment 5 Brad P. Crochet 2015-09-16 20:33:39 EDT

This is not a duplicate. This is two completely different code paths.

Comment 6 Brad P. Crochet 2015-09-17 15:11:42 EDT

A simple workaround is to use a new file for new systems. Do not add new systems to the old json file.

I will do some investigation as to whether a code fix is appropriate here. But it seems that documentation is the best fix.

Comment 7 Jaromir Coufal 2015-10-20 08:18:42 EDT

So, code change might be needed - if the file contains a node which already exists, tell user that the file contains already existing nodes which were already registered, so no action will be performed on these.

Comment 8 Brad P. Crochet 2015-10-29 11:23:13 EDT

This appears to be fixed in os-cloud-config 0.2.10.

Comment 9 bigswitch 2015-11-02 13:51:33 EST

hi
I am not seeing this issue with the GA code 
FYI

Comment 11 Alexander Chuzhoy 2015-11-30 16:33:52 EST

Verified:

Environment:
os-cloud-config-0.2.8-9.el7ost.noarch

The reported issue doesn't reproduce.

Comment 13 errata-xmlrpc 2015-12-21 11:55:07 EST

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2015:2651

Note You need to log in before you can comment on or make changes to this bug.