September 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: Refactor detach of local Storage Domain
by mlipchuk＠redhat.com 20 Sep '15

20 Sep '15

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Refactor detach of local Storage Domain ...................................................................... core: Refactor detach of local Storage Domain Refactor the detach local Storage Domain CDA at Remove Storage Domain Command to be more clear. Change-Id: Id456c085f08b0d7a44f9d42b63f71a75e2d534f2 Bug-Url: https://bugzilla.redhat.com/1263589 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RemoveStorageDomainCommand.java 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/46217 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id456c085f08b0d7a44f9d42b63f71a75e2d534f2 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-image-uploader[master]: ovirt-image-uploader: show informative msg if OVF data not f...
by sbonazzo＠redhat.com 18 Sep '15

18 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-image-uploader: show informative msg if OVF data not found ...................................................................... ovirt-image-uploader: show informative msg if OVF data not found This patch improves user experience without changing current behavior. Change-Id: Ib5b4fc629acc2a4e754f0ae53d7b628bd172ad6e Bug-Url: https://bugzilla.redhat.com/1258758 Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M src/__main__.py 1 file changed, 12 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Rafael Martins: Verified -- To view, visit https://gerrit.ovirt.org/46192 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib5b4fc629acc2a4e754f0ae53d7b628bd172ad6e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: master Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: calling prepareImage twice
by sbonazzo＠redhat.com 18 Sep '15

18 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: calling prepareImage twice ...................................................................... packaging: setup: calling prepareImage twice When the storageDomain has been disconnected from the bootstrap storagePool we have to call again prepareImage to get valid links for block devices under /rhev/datacenter/... Change-Id: I501796441b82f3b4ea021e30f9eb431639d006c0 Bug-Url: https://bugzilla.redhat.com/1258465 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/ha/ha_services.py M src/plugins/ovirt-hosted-engine-setup/storage/heconf.py M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 4 files changed, 19 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46343 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I501796441b82f3b4ea021e30f9eb431639d006c0 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: Typo fix in exporting vm error dialog
by tjelinek＠redhat.com 18 Sep '15

18 Sep '15

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: Typo fix in exporting vm error dialog ...................................................................... webadmin: Typo fix in exporting vm error dialog Attach an Export Domain to the Virtual machine(s) Data Center. -> Attach an Export Domain to the Virtual machine's Data Center. Change-Id: Ic4d63896fc566159416b207c799de42d2fbe473e Bug-Url: https://bugzilla.redhat.com/1149487 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Looks good to me, approved; Passed CI tests Jakub Niedermertl: Verified -- To view, visit https://gerrit.ovirt.org/46361 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic4d63896fc566159416b207c799de42d2fbe473e 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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: String fix Ok -> OK, VMs tab, Snapshots subtab
by tjelinek＠redhat.com 18 Sep '15

18 Sep '15

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: String fix Ok -> OK, VMs tab, Snapshots subtab ...................................................................... webadmin: String fix Ok -> OK, VMs tab, Snapshots subtab Making 'OK' (with capital K) used consistently accross the app. Change-Id: Ibb5e9c0422e3314130d9ee999a528f7ecfce49a2 Bug-Url: https://bugzilla.redhat.com/1149485 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Looks good to me, approved; Passed CI tests Jakub Niedermertl: Verified -- To view, visit https://gerrit.ovirt.org/46360 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibb5e9c0422e3314130d9ee999a528f7ecfce49a2 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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: Typo fix + i18n of 'Not-Configured'
by tjelinek＠redhat.com 18 Sep '15

18 Sep '15

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: Typo fix + i18n of 'Not-Configured' ...................................................................... webadmin: Typo fix + i18n of 'Not-Configured' * 'Not-Configured' -> 'Not Configured' * internationalization * VMs tab -> General subtab, Storage tab > VM Import subtab of export domain > click import Change-Id: Id0d5907e08e8f8b07a3afdc316d1ba4ce1ea2591 Bug-Url: https://bugzilla.redhat.com/1144063 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmGeneralModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmImportGeneralModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 3 files changed, 15 insertions(+), 5 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved; Passed CI tests Jakub Niedermertl: Verified -- To view, visit https://gerrit.ovirt.org/46359 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id0d5907e08e8f8b07a3afdc316d1ba4ce1ea2591 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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core, webadmin: Fix of strings regarding Cluster removal
by tjelinek＠redhat.com 18 Sep '15

18 Sep '15

Tomas Jelinek has submitted this change and it was merged. Change subject: core, webadmin: Fix of strings regarding Cluster removal ...................................................................... core, webadmin: Fix of strings regarding Cluster removal Fix of strings in Operation canceled dialog during Cluster removal. Change-Id: I43fbed0adf04b8bb079efdc638a0538fba671f84 Bug-Url: https://bugzilla.redhat.com/1149468 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 4 files changed, 8 insertions(+), 8 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46358 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I43fbed0adf04b8bb079efdc638a0538fba671f84 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: automation(a)ovirt.org

1 0

Change in jenkins[master]: ovirt-engine-wildfly: build v8 and v10
by dcaroest＠redhat.com 18 Sep '15

18 Sep '15

David Caro has submitted this change and it was merged. Change subject: ovirt-engine-wildfly: build v8 and v10 ...................................................................... ovirt-engine-wildfly: build v8 and v10 build both v8 (for 3.6) and v10 (for master) Change-Id: I99e35470a008a6c90c19518574b8cb4c2ad2773e Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-wildfly/ovirt-engine-wildfly.yaml 1 file changed, 24 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Ready for review Juan Hernandez: Looks good to me Jenkins CI: Passed CI tests David Caro: Ready for merge -- To view, visit https://gerrit.ovirt.org/46355 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I99e35470a008a6c90c19518574b8cb4c2ad2773e Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: packaging: setup: making the timeout on engine-setup configu...
by sbonazzo＠redhat.com 18 Sep '15

18 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: making the timeout on engine-setup configurable ...................................................................... packaging: setup: making the timeout on engine-setup configurable We are still suffering timeout issues on engine-setup on the CI env, making that timeout configurable. Raising the default value from 300 to 600. Change-Id: I036a85da07dce44054e0e3954ad74f472fbc2679 Bug-Url: https://bugzilla.redhat.com/1255026 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/engine/health.py 2 files changed, 23 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46272 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I036a85da07dce44054e0e3954ad74f472fbc2679 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: making the timeout on engine-setup configu...
by sbonazzo＠redhat.com 18 Sep '15

18 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: making the timeout on engine-setup configurable ...................................................................... packaging: setup: making the timeout on engine-setup configurable We are still suffering timeout issues on engine-setup on the CI env, making that timeout configurable. Raising the default value from 300 to 600. Change-Id: I036a85da07dce44054e0e3954ad74f472fbc2679 Bug-Url: https://bugzilla.redhat.com/1255026 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/engine/health.py 2 files changed, 23 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved; Passed CI tests Simone Tiraboschi: Verified Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46268 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I036a85da07dce44054e0e3954ad74f472fbc2679 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0