June 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: Fix of NPE while creating Template from Snapshot
by tjelinek＠redhat.com 17 Jun '16

17 Jun '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: Fix of NPE while creating Template from Snapshot ...................................................................... webadmin: Fix of NPE while creating Template from Snapshot Problem: Setting entity value of UnitVmModel property `isHighlyAvailable` in VmSnapshotListModel.cloneTemplate method invoked listener method VmModelBehaviorBase.updateMigrationAvailability in which NPE was thrown since `isAutoAssign` was null. This problem appeared e.g. if the snapshoted VM was highly available. Fix: Method updateMigrationAvailability() just returns if UnitVmModel is not yet fully initialized - required properties are `null`. Change-Id: I8bf28adbda195dd4d60344764ac800d9d41d3092 Bug-Url: https://bugzilla.redhat.com/1346095 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java 1 file changed, 7 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59281 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8bf28adbda195dd4d60344764ac800d9d41d3092 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: node: Add markers to the repos we want to keep enabled
by fabiand＠redhat.com 17 Jun '16

17 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Add markers to the repos we want to keep enabled ...................................................................... node: Add markers to the repos we want to keep enabled This is needded to let imgbased - which is used during the build - know what repos should be enabled, because by default, allr epos are getting disabled. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1340382 Change-Id: Ief40f9ff7d5a87a7e6f8ef54e14ce032852f9de1 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M configure.ac M ovirt-release40.spec.in 2 files changed, 7 insertions(+), 2 deletions(-) Approvals: Fabian Deutsch: 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/59390 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief40f9ff7d5a87a7e6f8ef54e14ce032852f9de1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: node: Make some units enabled by default
by fabiand＠redhat.com 17 Jun '16

17 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Make some units enabled by default ...................................................................... node: Make some units enabled by default Use the ln approach, because we want the defaults to be in /usr, not in /etc /etc is completely left to the user. Using systemctl preset or enable would add links to /etc. Change-Id: I73e68e909590815938eb8d6f9f8b24293ad32289 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release40.spec.in 1 file changed, 9 insertions(+), 6 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved; Passed CI tests Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/59389 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I73e68e909590815938eb8d6f9f8b24293ad32289 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: Add a folder for local storage domains for ngn
by fabiand＠redhat.com 17 Jun '16

17 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: Add a folder for local storage domains for ngn ...................................................................... Add a folder for local storage domains for ngn Create the appropriate path and set the ownership Change-Id: I55d7311433a84b7ca6bc4c4fbf21eadd3d99600a Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1339906 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- M ovirt-release40.spec.in 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Fabian Deutsch: 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/59190 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55d7311433a84b7ca6bc4c4fbf21eadd3d99600a Gerrit-PatchSet: 5 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: Add a folder for local storage domains for ngn
by fabiand＠redhat.com 17 Jun '16

17 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: Add a folder for local storage domains for ngn ...................................................................... Add a folder for local storage domains for ngn Create the appropriate path and set the ownership Change-Id: I455d966b56a057fce6fb5c06dc7a9d2a17b26a73 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1339906 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- M ovirt-release-master.spec.in 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Fabian Deutsch: 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/59189 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I455d966b56a057fce6fb5c06dc7a9d2a17b26a73 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Document movement of "in_sync" attribute
by juan.hernandez＠redhat.com 17 Jun '16

17 Jun '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Document movement of "in_sync" attribute ...................................................................... restapi: Document movement of "in_sync" attribute In version 4 of the API the "in_sync" attribute has been moved from the list of reported configurations to the network attachment. Change-Id: I97ee6fce61c988ddeee9723387f3393522af9dc9 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/README.adoc 1 file changed, 36 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/59325 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I97ee6fce61c988ddeee9723387f3393522af9dc9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Meni Yakove <myakove(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Document the rename of "Slaves" to "HostNics"
by juan.hernandez＠redhat.com 17 Jun '16

17 Jun '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Document the rename of "Slaves" to "HostNics" ...................................................................... restapi: Document the rename of "Slaves" to "HostNics" In version 4 of the API the names of the XML complex types are systematically generated from the names of the types of the model. For example, the name for the XML complex type "HostNic" is "HostNics". But in version 3 of the API this specific type was named differently, because the XML schema was written manually. This patch updates the documentation to include this type name change. Change-Id: Ia231664e171ed4390557f7bf5662c12ec548ca0f Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/README.adoc 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/59311 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia231664e171ed4390557f7bf5662c12ec548ca0f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Meni Yakove <myakove(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Fix README.adoc format
by juan.hernandez＠redhat.com 17 Jun '16

17 Jun '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix README.adoc format ...................................................................... restapi: Fix README.adoc format This patches fixes a minor format error in the README.adoc, that makes it render incorrectly. Change-Id: I0b2620bce46372b66c5447a58df437f2825becf0 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/README.adoc 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59324 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0b2620bce46372b66c5447a58df437f2825becf0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: Make some units enabled by default
by fabiand＠redhat.com 16 Jun '16

16 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Make some units enabled by default ...................................................................... node: Make some units enabled by default Use the ln approach, because we want the defaults to be in /usr, not in /etc /etc is completely left to the user. Using systemctl preset or enable would add links to /etc. Change-Id: I73e68e909590815938eb8d6f9f8b24293ad32289 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release-master.spec.in 1 file changed, 10 insertions(+), 8 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/59280 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I73e68e909590815938eb8d6f9f8b24293ad32289 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: Add markers to the repos we want to keep enabled
by fabiand＠redhat.com 16 Jun '16

16 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Add markers to the repos we want to keep enabled ...................................................................... node: Add markers to the repos we want to keep enabled This is needded to let imgbased - which is used during the build - know what repos should be enabled, because by default, allr epos are getting disabled. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1340382 Change-Id: Ief40f9ff7d5a87a7e6f8ef54e14ce032852f9de1 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release-master.spec.in 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59374 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief40f9ff7d5a87a7e6f8ef54e14ce032852f9de1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0