Change in ovirt-engine[master]: core: Refactor detach of local Storage Domain
by mlipchuk@redhat.com
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
9 years, 1 month
Change in ovirt-image-uploader[master]: ovirt-image-uploader: show informative msg if OVF data not f...
by sbonazzo@redhat.com
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
9 years, 1 month
Change in ovirt-hosted-engine-setup[master]: packaging: setup: calling prepareImage twice
by sbonazzo@redhat.com
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
9 years, 1 month
Change in ovirt-engine[master]: webadmin: Typo fix in exporting vm error dialog
by tjelinek@redhat.com
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
9 years, 1 month
Change in ovirt-engine[master]: webadmin: String fix Ok -> OK, VMs tab, Snapshots subtab
by tjelinek@redhat.com
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
9 years, 1 month
Change in ovirt-engine[master]: webadmin: Typo fix + i18n of 'Not-Configured'
by tjelinek@redhat.com
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
9 years, 1 month
Change in ovirt-engine[master]: core, webadmin: Fix of strings regarding Cluster removal
by tjelinek@redhat.com
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
9 years, 1 month
Change in jenkins[master]: ovirt-engine-wildfly: build v8 and v10
by dcaroest@redhat.com
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>
9 years, 1 month
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
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
9 years, 1 month
Change in ovirt-hosted-engine-setup[master]: packaging: setup: making the timeout on engine-setup configu...
by sbonazzo@redhat.com
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
9 years, 1 month