Change in ovirt-engine[master]: packaging: Updated names of etl and notifier services
by juan.hernandez@redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: packaging: Updated names of etl and notifier services
......................................................................
packaging: Updated names of etl and notifier services
Change-Id: Icea5953c7d233b068056b75955f440b6dd6df329
Signed-off-by: Alex Lourie <alourie(a)redhat.com>
---
M packaging/fedora/setup/basedefs.py
M packaging/fedora/setup/engine-upgrade.py
2 files changed, 14 insertions(+), 14 deletions(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/8170
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icea5953c7d233b068056b75955f440b6dd6df329
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alex Lourie <alourie(a)redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com>
12 years, 3 months
Change in ovirt-engine[master]: webadmin: New block storage - Unresponsive OK (#839521)
by derez@redhat.com
Daniel Erez has submitted this change and it was merged.
Change subject: webadmin: New block storage - Unresponsive OK (#839521)
......................................................................
webadmin: New block storage - Unresponsive OK (#839521)
Removed a call for GetStorageDomainsByConnection when
creating a new block storage domain (the query is relevant
only when creating a new file storage domain).
The redundant query prevented a creation of a new block
storage domain right after creating a file storage domain.
Change-Id: I5264bada28dbcd0c7305d2032e30ec9a8b41cc15
Bug-Url: https://bugzilla.redhat.com/839521
Signed-off-by: Daniel Erez <derez(a)redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
1 file changed, 1 insertion(+), 19 deletions(-)
Approvals:
Tal Nisan: Looks good to me, approved
Daniel Erez: Verified
--
To view, visit http://gerrit.ovirt.org/8254
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5264bada28dbcd0c7305d2032e30ec9a8b41cc15
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
12 years, 3 months
Change in ovirt-engine[master]: webadmin: disks tab - images as default (#860949)
by derez@redhat.com
Daniel Erez has submitted this change and it was merged.
Change subject: webadmin: disks tab - images as default (#860949)
......................................................................
webadmin: disks tab - images as default (#860949)
Disks main-tab and VMs-Disk sub-tab:
'Images' view is displayed by default (instead of 'All').
Change-Id: I27582c4b02ae353f7b75d2e8e319507f8cee1a72
Bug-Url: https://bugzilla.redhat.com/860949
Signed-off-by: Daniel Erez <derez(a)redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/disks/DisksViewRadioGroup.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/CommonModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/disks/DiskListModel.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/uicommon/DiskModule.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabDiskView.java
5 files changed, 10 insertions(+), 13 deletions(-)
Approvals:
Tal Nisan: Looks good to me, approved
Daniel Erez: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/8253
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I27582c4b02ae353f7b75d2e8e319507f8cee1a72
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
12 years, 3 months
Change in ovirt-engine[master]: core: Fixed exception in RemoveSnapshot
by derez@redhat.com
Daniel Erez has submitted this change and it was merged.
Change subject: core: Fixed exception in RemoveSnapshot
......................................................................
core: Fixed exception in RemoveSnapshot
Fixed exception when polling a task to remove a snapshot.
Change-Id: I32f58364cbc0cd7a9beda6785fa1227fb838cc5a
Signed-off-by: Allon Mureinik <amureini(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotCommand.java
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Daniel Erez: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/8251
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I32f58364cbc0cd7a9beda6785fa1227fb838cc5a
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
12 years, 3 months
Change in ovirt-engine[master]: engine:Validate maximum number of hosts in DC.(#771699)
by derez@redhat.com
Daniel Erez has submitted this change and it was merged.
Change subject: engine:Validate maximum number of hosts in DC.(#771699)
......................................................................
engine:Validate maximum number of hosts in DC.(#771699)
http://bugzilla.redhat.com/show_bug.cgi?id=771699
when a host is added, there should be a check whether the maximum number of
hosts attached to DC has been exceeded.
If max number of hosts is exceeded, user should be presented with an
appropriate message.
Change-Id: Ie1ef2ce160bc9ee4855b310c9d170ef7c14a0a17
Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsSpmIdCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsGroupCommand.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllErrors.java
M backend/manager/modules/dal/src/main/resources/bundles/VdsmErrors.properties
5 files changed, 47 insertions(+), 26 deletions(-)
Approvals:
Daniel Erez: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/6514
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie1ef2ce160bc9ee4855b310c9d170ef7c14a0a17
Gerrit-PatchSet: 16
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: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>
12 years, 3 months
Change in ovirt-engine[master]: tools: Use standard colors in the shell output (no bz)
by juan.hernandez@redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: tools: Use standard colors in the shell output (no bz)
......................................................................
tools: Use standard colors in the shell output (no bz)
There are tools that can work only with the standard colors,
like the AnsiColor plugin for Jenkins.
Change-Id: Iafb18728efb6b1724fb3725cbf1615621187b3a1
Signed-off-by: Rami Vaknin <rvaknin(a)redhat.com>
---
M packaging/fedora/setup/basedefs.py
1 file changed, 3 insertions(+), 3 deletions(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/7660
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iafb18728efb6b1724fb3725cbf1615621187b3a1
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Rami Vaknin <rvaknin.redhat.com(a)gmail.com>
Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Rami Vaknin <rvaknin.redhat.com(a)gmail.com>
Gerrit-Reviewer: Yaniv Kaul <ykaul(a)redhat.com>
12 years, 3 months
Change in ovirt-engine[master]: core: ActivateDeactivateSingleAsyncOperationFactory cleanup
by lhornyak@redhat.com
Laszlo Hornyak has submitted this change and it was merged.
Change subject: core: ActivateDeactivateSingleAsyncOperationFactory cleanup
......................................................................
core: ActivateDeactivateSingleAsyncOperationFactory cleanup
added import java.util.ArrayList
Change-Id: Ie724fe04effb46cbd7b7d09714c6934adcc03660
Signed-off-by: Sharad Mishra <snmishra(a)linux.vnet.ibm.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ActivateDeactivateSingleAsyncOperationFactory.java
1 file changed, 8 insertions(+), 6 deletions(-)
Approvals:
Laszlo Hornyak: Verified; Looks good to me, approved
Allon Mureinik: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.ovirt.org/8220
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie724fe04effb46cbd7b7d09714c6934adcc03660
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sharad Mishra <snmishra(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>
12 years, 3 months
Change in ovirt-engine[master]: core: fix error caused by javac 1.6.0_24 bug in VdsManager
by rgolan@redhat.com
Roy Golan has submitted this change and it was merged.
Change subject: core: fix error caused by javac 1.6.0_24 bug in VdsManager
......................................................................
core: fix error caused by javac 1.6.0_24 bug in VdsManager
seems like 1.6._024 compiler is failing to parse "(Config.<Object>"
in some circumstances. Flipping the order does the trick.
Change-Id: I2855a27b27163385530156e306fd8beca2c7cda3
Signed-off-by: Roy Golan <rgolan(a)redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Roy Golan: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/8235
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2855a27b27163385530156e306fd8beca2c7cda3
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
12 years, 3 months
Change in ovirt-image-uploader[master]: tools: Don't print help on error (BZ858968)
by kroberts@redhat.com
Keith Robertson has submitted this change and it was merged.
Change subject: tools: Don't print help on error (BZ858968)
......................................................................
tools: Don't print help on error (BZ858968)
Change-Id: I8698c1c32010d889ebae4ccb99df10c590e39ff5
Signed-off-by: Keith Robertson <kroberts(a)redhat.com>
---
M src/engine-image-uploader.py
1 file changed, 0 insertions(+), 1 deletion(-)
Approvals:
Keith Robertson: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/8090
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8698c1c32010d889ebae4ccb99df10c590e39ff5
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-image-uploader
Gerrit-Branch: master
Gerrit-Owner: Keith Robertson <kroberts(a)redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com>
Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com>
Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com>
12 years, 3 months
Change in ovirt-iso-uploader[master]: tools: Fix default port (BZ857480)
by kroberts@redhat.com
Keith Robertson has submitted this change and it was merged.
Change subject: tools: Fix default port (BZ857480)
......................................................................
tools: Fix default port (BZ857480)
Change-Id: Idab93219597d8199ffb79a0c7c664c6369c0416a
Signed-off-by: Keith Robertson <kroberts(a)redhat.com>
---
M src/engine-iso-uploader.8
M src/engine-iso-uploader.py
M src/isouploader.conf
3 files changed, 7 insertions(+), 7 deletions(-)
Approvals:
Moran Goldboim: Looks good to me, but someone else must approve
Kiril Nesenko: Looks good to me, but someone else must approve
Keith Robertson: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/8007
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idab93219597d8199ffb79a0c7c664c6369c0416a
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-iso-uploader
Gerrit-Branch: master
Gerrit-Owner: Keith Robertson <kroberts(a)redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com>
Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com>
Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com>
12 years, 3 months