Alon Bar-Lev has posted comments on this change. Change subject: bootstrap: send complete bootstrap from engine ......................................................................
Patch Set 17: > 1. Still want to make sure we're ok with concurrency, see my comments. I think we are good. We use static assignment for initialization only, last wins, no need for synchronization in this case. We use move within same filesystem, this is atomic operation, either old or new will be used, there is no breakage, nor need to synchronize. That's true that if two parallel threads will run at the exact same time when reload time expires, both will create a new archive... but it is lighter than synchronise always. > 2. Does bootstrap2 mean we cannot support previous ovirt nodes? No... I always test with RHEVM-H, as it is the only working image I have... > 3. Still need to sort out the new import. OK. Just tell me how. -- To view, visit http://gerrit.ovirt.org/6963 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6f4a09ca9e66f0c9f5f4f7b283a5f43986b7e603 Gerrit-PatchSet: 17 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Alex Lourie <alou...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com> Gerrit-Reviewer: Doron Fediuck <dfedi...@redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsl...@redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschr...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches