Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-2.1]: upgrade: fixing vm.conf patching method

From Simone Tiraboschi <stirabos@redhat.com>:
Simone Tiraboschi has submitted this change and it was merged. Change subject: upgrade: fixing vm.conf patching method ...................................................................... upgrade: fixing vm.conf patching method For the first boot of the new appliance we have to patch on the fly vm.conf to inject the cloud-init seed image. Patch https://gerrit.ovirt.org/67924 slightly changed the way to read vm.conf, fixing according to it. With the previous interface it was possible to refresh from the shared storage (initial vm.conf or the OVF_STORE if available) to a temporary file and edit in place. Now we can just refresh vm.conf in its final local destination under /var/run, read from there, inject the cloud-init iso and write to a temporary file as before. Change-Id: Ib6fbbdeb5bcb84d59982c12ebdc71fb20c082b29 Bug-Url: https://bugzilla.redhat.com/1420283 Signed-off-by: Simone Tiraboschi <stirabos@redhat.com> (cherry picked from commit c02b2f9ccc56c4923f063eef9bfc235b36d30aae) --- M src/plugins/gr-he-upgradeappliance/vm/runvm.py 1 file changed, 3 insertions(+), 5 deletions(-) Approvals: Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/71991 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib6fbbdeb5bcb84d59982c12ebdc71fb20c082b29 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-2.1 Gerrit-Owner: Simone Tiraboschi <stirabos@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins@redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>
participants (1)
-
Code Review