Nir Soffer has posted comments on this change. Change subject: sp: race in domains upgrade prevents further pool upgrades ......................................................................
Patch Set 5: (1 comment) https://gerrit.ovirt.org/#/c/57315/5//COMMIT_MSG Commit Message: Line 18: domains which causes to a bug - an update will be considered running and Line 19: no further pool upgrades could be initiated. Line 20: Line 21: This patch copies the list to a temporary list before attempting to register Line 22: the callback/start the upgrade threads and by that avoiding that race. We don't do this now, so you can remove this part. Line 23: Line 24: this solution is simple as its targeted to the 3.6 branch, a more complete Line 25: solution which will also allow concurrent updates is introduced in Line 26: I8e14a3aa33bfab4751ab5d1e3becbeda892da4c3 -- To view, visit https://gerrit.ovirt.org/57315 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie384c63315214786dc62f2b4998002320e314d30 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Liron Aravot <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
