Change in ovirt-engine[ovirt-engine-3.5]: webadmin: Enable editing WAD when VM is running
by tnisan@redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: webadmin: Enable editing WAD when VM is running
......................................................................
webadmin: Enable editing WAD when VM is running
Fixed the regression introduced by
Ib8b9f2860fd186651311814d0087c47f4083c863.
Change-Id: Id59348e04ca25d8684716d59ed14a07561efbc7e
Bug-Url: https://bugzilla.redhat.com/1070823
Signed-off-by: Allon Mureinik <amureini(a)redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/EditDiskModel.java
1 file changed, 0 insertions(+), 1 deletion(-)
Approvals:
Tal Nisan: Looks good to me, approved
Allon Mureinik: Verified
--
To view, visit http://gerrit.ovirt.org/33243
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id59348e04ca25d8684716d59ed14a07561efbc7e
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
10 years, 6 months
Change in ovirt-engine[master]: webadmin: Enable editing WAD when VM is running
by amureini@redhat.com
Allon Mureinik has submitted this change and it was merged.
Change subject: webadmin: Enable editing WAD when VM is running
......................................................................
webadmin: Enable editing WAD when VM is running
Fixed the regression introduced by
Ib8b9f2860fd186651311814d0087c47f4083c863.
Change-Id: Id59348e04ca25d8684716d59ed14a07561efbc7e
Bug-Url: https://bugzilla.redhat.com/1070823
Signed-off-by: Allon Mureinik <amureini(a)redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/EditDiskModel.java
1 file changed, 0 insertions(+), 1 deletion(-)
Approvals:
Tal Nisan: Looks good to me, approved
Allon Mureinik: Verified
--
To view, visit http://gerrit.ovirt.org/33237
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id59348e04ca25d8684716d59ed14a07561efbc7e
Gerrit-PatchSet: 1
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: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
10 years, 6 months
Change in ovirt-engine[master]: packaging: spec: fix missing dep on JDK 1.7
by sbonazzo@redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: packaging: spec: fix missing dep on JDK 1.7
......................................................................
packaging: spec: fix missing dep on JDK 1.7
Fixes
[ERROR] javac: invalid target release: 1.7
failure while building.
Change-Id: Ia71fb7f6c21f3f7ce26a1a0cc1ab8608dfa224c3
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M ovirt-engine.spec.in
1 file changed, 7 insertions(+), 1 deletion(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/33161
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia71fb7f6c21f3f7ce26a1a0cc1ab8608dfa224c3
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(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
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years, 6 months
Change in ovirt-reports[ovirt-engine-reports-3.5]: packaging: spec: require openjdk 1.7.0
by ydary@redhat.com
Yaniv Dary has submitted this change and it was merged.
Change subject: packaging: spec: require openjdk 1.7.0
......................................................................
packaging: spec: require openjdk 1.7.0
Change-Id: I69aa839498385f99c2e4916be23472340fcca3ff
Bug-Url: https://bugzilla.redhat.com/1140746
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
(cherry picked from commit 8a45c71b38050645e37a3a00f58b8adbb9db9b81)
---
M ovirt-engine-reports.spec.in
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Yaniv Dary: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/33239
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I69aa839498385f99c2e4916be23472340fcca3ff
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-reports
Gerrit-Branch: ovirt-engine-reports-3.5
Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
10 years, 6 months
Change in ovirt-reports[master]: packaging: spec: require openjdk 1.7.0
by ydary@redhat.com
Yaniv Dary has submitted this change and it was merged.
Change subject: packaging: spec: require openjdk 1.7.0
......................................................................
packaging: spec: require openjdk 1.7.0
Change-Id: I69aa839498385f99c2e4916be23472340fcca3ff
Bug-Url: https://bugzilla.redhat.com/1140746
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
M ovirt-engine-reports.spec.in
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Yedidyah Bar David: Verified
Yaniv Dary: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/32924
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I69aa839498385f99c2e4916be23472340fcca3ff
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-reports
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
10 years, 6 months
Change in ovirt-reports[master]: packaging: setup: Use common code for remote engine pki
by ydary@redhat.com
Yaniv Dary has submitted this change and it was merged.
Change subject: packaging: setup: Use common code for remote engine pki
......................................................................
packaging: setup: Use common code for remote engine pki
Requires http://gerrit.ovirt.org/33023
Bug-Url: https://bugzilla.redhat.com/1142267
Change-Id: Ia7a549d09dc85293beba24327ea44ef1dcaf4a55
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
M packaging/setup/ovirt_engine_setup/reports/constants.py
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/pki/apache.py
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/pki/jboss.py
3 files changed, 92 insertions(+), 388 deletions(-)
Approvals:
Yaniv Dary: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/33024
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia7a549d09dc85293beba24327ea44ef1dcaf4a55
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-reports
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
10 years, 6 months
Change in ovirt-reports[ovirt-engine-reports-3.5]: packaging: setup: config engine to access reports
by ydary@redhat.com
Yaniv Dary has submitted this change and it was merged.
Change subject: packaging: setup: config engine to access reports
......................................................................
packaging: setup: config engine to access reports
Instead of updating vdc_options as was done before, write a conf file on
the engine side that will let the engine access reports.
Moved writing reports.conf to jasper deploy.
Requires http://gerrit.ovirt.org/33023 .
Bug-Url: https://bugzilla.redhat.com/1144079
Change-Id: I4b7ea55fc0c8204a88edf39c055c2bb2d53d6dde
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
(cherry picked from commit cc9b3e85982b903a8636a10e8f676c4d391cf855)
---
M packaging/setup/ovirt_engine_setup/reports/constants.py
M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-reports/db/connection.py
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/engine.py
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/db/connection.py
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py
5 files changed, 69 insertions(+), 208 deletions(-)
Approvals:
Yaniv Dary: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/33238
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4b7ea55fc0c8204a88edf39c055c2bb2d53d6dde
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-reports
Gerrit-Branch: ovirt-engine-reports-3.5
Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
10 years, 6 months
Change in ovirt-reports[master]: packaging: setup: config engine to access reports
by ydary@redhat.com
Yaniv Dary has submitted this change and it was merged.
Change subject: packaging: setup: config engine to access reports
......................................................................
packaging: setup: config engine to access reports
Instead of updating vdc_options as was done before, write a conf file on
the engine side that will let the engine access reports.
Moved writing reports.conf to jasper deploy.
Requires http://gerrit.ovirt.org/33023 .
Bug-Url: https://bugzilla.redhat.com/1144079
Change-Id: I4b7ea55fc0c8204a88edf39c055c2bb2d53d6dde
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
M packaging/setup/ovirt_engine_setup/reports/constants.py
M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-reports/db/connection.py
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/engine.py
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/db/connection.py
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py
5 files changed, 69 insertions(+), 208 deletions(-)
Approvals:
Yedidyah Bar David: Verified
Yaniv Dary: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/33148
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4b7ea55fc0c8204a88edf39c055c2bb2d53d6dde
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-reports
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
10 years, 6 months
Change in ovirt-engine[ovirt-engine-3.5]: core: Fail Add/Update Vm when CPU is unsupported
by tnisan@redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: core: Fail Add/Update Vm when CPU is unsupported
......................................................................
core: Fail Add/Update Vm when CPU is unsupported
Change-Id: Id1d22994c59b16d1d71d2315882d41454079bf32
Bug-Url: https://bugzilla.redhat.com/1096851
Signed-off-by: Roy Golan <rgolan(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CpuFlagsManagerHandler.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVmCommandTest.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateVmCommandTest.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.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
12 files changed, 102 insertions(+), 3 deletions(-)
Approvals:
Roy Golan: Verified
Omer Frenkel: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/33230
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id1d22994c59b16d1d71d2315882d41454079bf32
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years, 6 months
Change in ovirt-engine[ovirt-engine-3.5]: core: osinfo - add unsupported list of CPUs
by tnisan@redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: core: osinfo - add unsupported list of CPUs
......................................................................
core: osinfo - add unsupported list of CPUs
adding a comma separated list of unsupported CPUs to osinfo
and 2 verbs to query it:
public Map<Pair<Integer, Version>, Set<String>> getUnsupportedCpus();
public boolean isCpuSupported(int osId, Version version, String cpuId);
Change-Id: I339b87de7646b813aa634102c26f710b9d5f6f75
Bug-Url: https://bugzilla.redhat.com/1096851
Signed-off-by: Roy Golan <rgolan(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/osinfo/OsRepository.java
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/OsRepositoryImpl.java
M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/OsRepositoryImplTest.java
M packaging/conf/osinfo-defaults.properties
4 files changed, 89 insertions(+), 0 deletions(-)
Approvals:
Roy Golan: Verified
Omer Frenkel: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/33229
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I339b87de7646b813aa634102c26f710b9d5f6f75
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years, 6 months