Barak Azulay has posted comments on this change.

Change subject: bootstrap: node id: consider bonding MAC addresses as well
......................................................................


Patch Set 3:

1 - only caps.py uses the utils.getHostUUID and I'm almost certain the engine 
does not act on a change in the host UUID reported in caps (need to check)

2 - even when we are talking about upgrade in case the upgraded machine has 
bond (that actually had this issue in the past ... change the UUID) will have 
that affect and this is a fix for that problem.

3 - when reinstalled, again  it will effect only bonded machines that had this 
issue in the past, and would have suffered this issue again with or without the 
new patch

4 - we should also change the implementation of the utils.getHostUUID.

--
To view, visit http://gerrit.ovirt.org/7596
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iae0713f95c25e3e05b24bf8254499e4cd8e64d10
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsl...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to