Change in ovirt-engine-sdk-java[master]: sdk: prepare for the next development iteration
by mpastern@redhat.com
Michael Pasternak has submitted this change and it was merged.
Change subject: sdk: prepare for the next development iteration
......................................................................
sdk: prepare for the next development iteration
Change-Id: Ifa470cac5025a9df1e85903ad2a46de5af913c5f
Signed-off-by: Michael Pasternak <mpastern(a)redhat.com>
---
M ovirt-engine-sdk-java/pom.xml
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Michael Pasternak: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/11542
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifa470cac5025a9df1e85903ad2a46de5af913c5f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-sdk-java
Gerrit-Branch: master
Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>
11 years, 10 months
Change in ovirt-engine[master]: engine: remove unused VmDynamic constructor
by lhornyak@redhat.com
Laszlo Hornyak has submitted this change and it was merged.
Change subject: engine: remove unused VmDynamic constructor
......................................................................
engine: remove unused VmDynamic constructor
Long VmDynamic constructor was not used, it was removed.
Change-Id: I70ddc980f8ed832b02e8a2974097893dff4757f3
Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmDynamic.java
1 file changed, 0 insertions(+), 38 deletions(-)
Approvals:
Laszlo Hornyak: Verified; Looks good to me, approved
Michael Kublin: Looks good to me, approved
Alissa Bonas: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.ovirt.org/11959
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I70ddc980f8ed832b02e8a2974097893dff4757f3
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>
11 years, 10 months
Change in ovirt-engine[master]: engine: remove unused VmStatic constructor
by lhornyak@redhat.com
Laszlo Hornyak has submitted this change and it was merged.
Change subject: engine: remove unused VmStatic constructor
......................................................................
engine: remove unused VmStatic constructor
Long VmStatic constructor was not used, it was removed.
Change-Id: I13e27b2bc75a2c92ad8bacba84115b1389da40e8
Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmStatic.java
1 file changed, 0 insertions(+), 53 deletions(-)
Approvals:
Laszlo Hornyak: Verified; Looks good to me, approved
Michael Kublin: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/11958
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I13e27b2bc75a2c92ad8bacba84115b1389da40e8
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>
11 years, 10 months
Change in ovirt-engine[master]: engine: remove unused VM constructor
by lhornyak@redhat.com
Laszlo Hornyak has submitted this change and it was merged.
Change subject: engine: remove unused VM constructor
......................................................................
engine: remove unused VM constructor
The long constructor of VM was not used, it was removed.
Change-Id: I5f562f0cc51a62a70b7c2ae57504e0d286f52ab2
Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java
1 file changed, 0 insertions(+), 77 deletions(-)
Approvals:
Laszlo Hornyak: Verified; Looks good to me, approved
Michael Kublin: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/11957
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5f562f0cc51a62a70b7c2ae57504e0d286f52ab2
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>
11 years, 10 months
Change in ovirt-engine[master]: webadmin: When vnic's property become disabled again it shou...
by alkaplan@redhat.com
Alona Kaplan has submitted this change and it was merged.
Change subject: webadmin: When vnic's property become disabled again it should be reseted
......................................................................
webadmin: When vnic's property become disabled again it should be reseted
Some vnic's properties update is blocked if the vm is up and the vnic is
plugged.
Opening the update vnic dialog and changing the vnic to unplugged,
unblocks these properties.
Changing the vnic back to plugged should revert the changes done on these
properties.
Change-Id: I39d70d8948f32e8e6f45f2d1eba5a3d26bb04975
Bug-Url: https://bugzilla.redhat.com/909852
Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/BaseEditVmInterfaceModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/EditVmInterfaceModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewVmInterfaceModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmInterfaceModel.java
4 files changed, 66 insertions(+), 23 deletions(-)
Approvals:
Alona Kaplan: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/11930
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I39d70d8948f32e8e6f45f2d1eba5a3d26bb04975
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
11 years, 10 months
Change in ovirt-engine[master]: packaging: changed override-firewall in firewall-manager
by juan.hernandez@redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: packaging: changed override-firewall in firewall-manager
......................................................................
packaging: changed override-firewall in firewall-manager
In engine-setup, changed the option override-firewall that the user
may expect to be boolean into firewall-manager that accept "None",
"Firewalld" or "IPtables" as values.
Updated the output messages explaining the meaning of None as
firewall manager.
Change-Id: Ib5d92079a7caa301de248e90740e77fa5577b068
Bug-Url: https://bugzilla.redhat.com/901987
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M packaging/fedora/setup/engine-setup.py
M packaging/fedora/setup/output_messages.py
2 files changed, 14 insertions(+), 8 deletions(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/11956
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib5d92079a7caa301de248e90740e77fa5577b068
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: Moran Goldboim <mgoldboi(a)redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
11 years, 10 months
Change in ovirt-engine[master]: engine: RemoveVmInterface CanDoAction Message
by masayag@redhat.com
Moti Asayag has submitted this change and it was merged.
Change subject: engine: RemoveVmInterface CanDoAction Message
......................................................................
engine: RemoveVmInterface CanDoAction Message
This patch adds a message for when user tries
to remove a VmNetworkInterface that is plugged to
a running VM.
Change-Id: I0976cedbbd25301293d70cddfc99c36b7cf95923
Bug-Url: https://bugzilla.redhat.com/906405
Signed-off-by: Muli Salem <msalem(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/RemoveVmInterfaceCommand.java
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
5 files changed, 12 insertions(+), 0 deletions(-)
Approvals:
Muli Salem: Verified
Alona Kaplan: Looks good to me, but someone else must approve
Moti Asayag: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/11955
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0976cedbbd25301293d70cddfc99c36b7cf95923
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Muli Salem <msalem(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Muli Salem <msalem(a)redhat.com>
11 years, 10 months
Change in ovirt-dwh[master]: history: added storage domain status to 3.2 views
by ydary@redhat.com
Yaniv Dary has submitted this change and it was merged.
Change subject: history: added storage domain status to 3.2 views
......................................................................
history: added storage domain status to 3.2 views
Change-Id: Ib25d5d8baf093dc6393c502eca862cec41e887a4
Signed-off-by: Yaniv Dary <ydary(a)redhat.com>
---
M data-warehouse/historydbscripts_postgres/create_views_3_0.sql
M data-warehouse/historydbscripts_postgres/create_views_3_1.sql
M data-warehouse/historydbscripts_postgres/create_views_3_2.sql
3 files changed, 42 insertions(+), 30 deletions(-)
Approvals:
Yaniv Dary: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/11954
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib25d5d8baf093dc6393c502eca862cec41e887a4
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-dwh
Gerrit-Branch: master
Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>
11 years, 10 months
Change in ovirt-engine[master]: tools: Warning about last domain deletion should be prior to...
by yzaslavs@redhat.com
Yair Zaslavsky has submitted this change and it was merged.
Change subject: tools: Warning about last domain deletion should be prior to confirmation
......................................................................
tools: Warning about last domain deletion should be prior to confirmation
The following patch moves the warning print aobut last domain erasure
prior to the confirmation question.
In case of forceDelete option is used - this warning is not presented.
Bug-Url: https://bugzilla.redhat.com/905370
Change-Id: Id91bd35ad832a4c8b8947c0e86bc0679cef3fa7b
Signed-off-by: Yair Zaslavsky<yzaslavs(a)redhat.com>
---
M backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomains.java
1 file changed, 6 insertions(+), 4 deletions(-)
Approvals:
Yair Zaslavsky: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/11800
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id91bd35ad832a4c8b8947c0e86bc0679cef3fa7b
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
11 years, 10 months
Change in ovirt-engine[master]: core: fixing a broken view
by emesika@redhat.com
Eli Mesika has submitted this change and it was merged.
Change subject: core: fixing a broken view
......................................................................
core: fixing a broken view
this patch fixes a broken view resulted from changes of the following
commit : 5ca1ec0337d921f1057321da2688ad2a014b8ff
A comma separator was missing between view fields.
Change-Id: Id022f7f82b69e2af98713b2767a4dc1edf096a8e
Signed-off-by: Eli Mesika <emesika(a)redhat.com>
---
M backend/manager/dbscripts/create_views.sql
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Eli Mesika: Verified; Looks good to me, approved
Einav Cohen: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/11945
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id022f7f82b69e2af98713b2767a4dc1edf096a8e
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
11 years, 10 months