Change in ovirt-engine[master]: restapi: RSDL - Bad Params Exposed for add/update Host-NIC
by oliel@redhat.com
Ori Liel has submitted this change and it was merged.
Change subject: restapi: RSDL - Bad Params Exposed for add/update Host-NIC
......................................................................
restapi: RSDL - Bad Params Exposed for add/update Host-NIC
Change-Id: Ie74a733c15aa5faae1a7e52185fa204ed3491d69
Signed-off-by: Ori Liel <oliel(a)redhat.com>
---
M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata_v-3.1.yaml
1 file changed, 1 insertion(+), 7 deletions(-)
Approvals:
Ori Liel: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/6111
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie74a733c15aa5faae1a7e52185fa204ed3491d69
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ori Liel <oliel(a)redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>
Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>
12 years, 4 months
Change in ovirt-engine[master]: core: add quota to vms_with_tags
by lhornyak@redhat.com
Laszlo Hornyak has submitted this change and it was merged.
Change subject: core: add quota to vms_with_tags
......................................................................
core: add quota to vms_with_tags
Adds quota_name and quota_id to the vms_with_tags view, this column will
be needed at quota search.
Change-Id: I7e617fc5d76387ad4c39a631cbdeb9568e13fa5f
Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com>
---
M backend/manager/dbscripts/create_views.sql
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Laszlo Hornyak: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/6887
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7e617fc5d76387ad4c39a631cbdeb9568e13fa5f
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>
12 years, 4 months
Change in ovirt-engine[master]: core: VmTemplateConditionFieldAutoCompleter cleanup
by lhornyak@redhat.com
Laszlo Hornyak has submitted this change and it was merged.
Change subject: core: VmTemplateConditionFieldAutoCompleter cleanup
......................................................................
core: VmTemplateConditionFieldAutoCompleter cleanup
- removed StringHelper usage (deprecated)
Change-Id: I1771a5046b36fd84f8f2cc74d997d0aa6c7090cc
Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com>
---
M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VmTemplateConditionFieldAutoCompleter.java
1 file changed, 9 insertions(+), 15 deletions(-)
Approvals:
Laszlo Hornyak: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/6886
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1771a5046b36fd84f8f2cc74d997d0aa6c7090cc
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>
12 years, 4 months
Change in ovirt-engine[master]: core: minor SearchQuery cleanup
by lhornyak@redhat.com
Laszlo Hornyak has submitted this change and it was merged.
Change subject: core: minor SearchQuery cleanup
......................................................................
core: minor SearchQuery cleanup
- removed StringHelper usage (deprecated)
- temporary variable for frequently used calculation
Change-Id: Iee43e31a42571d26860d8d25fb942ecec64ce477
Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SearchQuery.java
1 file changed, 6 insertions(+), 5 deletions(-)
Approvals:
Laszlo Hornyak: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/6885
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iee43e31a42571d26860d8d25fb942ecec64ce477
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>
12 years, 4 months
Change in ovirt-engine[master]: webadmin: [SetupNetworks] Sync network- bug fixes
by masayag@redhat.com
Moti Asayag has submitted this change and it was merged.
Change subject: webadmin: [SetupNetworks] Sync network- bug fixes
......................................................................
webadmin: [SetupNetworks] Sync network- bug fixes
https://bugzilla.redhat.com/838333
In SetupNetworks- if the network is in sync the boot protocol
is always disabled because the sync network is unchecked.
In this case the boot protocol should depend on the sync
network checkbox.
Change-Id: Ifee9224ad8dc0bd85a075a0bcc7c721729c4de3f
Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com>
---
A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/HasEnabledForContainter.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostInterfaceModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostManagementNetworkModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostSetupNetworksModel.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/popup/host/SetupNetworksManagementPopupPresenterWidget.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostInterfacePopupView.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/SetupNetworksInterfacePopupView.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/SetupNetworksManagementPopupView.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/editor/EnumRadioEditor.java
9 files changed, 44 insertions(+), 14 deletions(-)
Approvals:
Moti Asayag: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/6933
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifee9224ad8dc0bd85a075a0bcc7c721729c4de3f
Gerrit-PatchSet: 4
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: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
12 years, 4 months
Change in ovirt-engine[master]: engine: Add filling "managed" for network details (#838300)
by mkolesni@redhat.com
Mike Kolesnik has submitted this change and it was merged.
Change subject: engine: Add filling "managed" for network details (#838300)
......................................................................
engine: Add filling "managed" for network details (#838300)
https://bugzilla.redhat.com/838300
Added filling this field that indicates if a network is managed by the
engine or not.
If a network is unmanaged, the "in sync" field will be false (since it's
really not in sync) but it can be disregarded since unmanaged state is
more important.
Change-Id: I9d9bf007ba13c32dfca15c4ec1e331749331a12d
Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com>
---
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/NetworkUtils.java
M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/NetworkUtilsTest.java
2 files changed, 44 insertions(+), 4 deletions(-)
Approvals:
Mike Kolesnik: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/6908
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9d9bf007ba13c32dfca15c4ec1e331749331a12d
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer(a)redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
12 years, 4 months
Change in ovirt-engine[master]: core: Add "managed" to network details (#838300)
by mkolesni@redhat.com
Mike Kolesnik has submitted this change and it was merged.
Change subject: core: Add "managed" to network details (#838300)
......................................................................
core: Add "managed" to network details (#838300)
https://bugzilla.redhat.com/838300
This field indicates if a network is managed by the engine, or is a
custom network that is defined on the host only.
Change-Id: I2f0b387799af97050d8d0b3ec612665328b6d9f9
Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsNetworkInterface.java
M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/HostNicMapperTest.java
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/NetworkUtils.java
3 files changed, 15 insertions(+), 5 deletions(-)
Approvals:
Mike Kolesnik: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/6907
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2f0b387799af97050d8d0b3ec612665328b6d9f9
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer(a)redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>
12 years, 4 months
Change in ovirt-engine[master]: userportal: Localize "about" dialog text.
by ecohen@redhat.com
Einav Cohen has submitted this change and it was merged.
Change subject: userportal: Localize "about" dialog text.
......................................................................
userportal: Localize "about" dialog text.
The first line within the user portal's "about" dialog was hard coded.
This patch localizes it (i.e. the text is now in the
ApplicationConstants interface).
Change-Id: I030fe39c1d8b0d370b835b7a4a26a615ddffcdf0
Signed-off-by: Einav Cohen <ecohen(a)redhat.com>
---
M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/ApplicationConstants.java
M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/section/main/view/AboutPopupView.ui.xml
2 files changed, 6 insertions(+), 1 deletion(-)
Approvals:
Einav Cohen: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/6937
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I030fe39c1d8b0d370b835b7a4a26a615ddffcdf0
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Einav Cohen <ecohen(a)redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
12 years, 4 months
Change in ovirt-engine[master]: core: Do Not Allow Same Mac Address (#845837)
by msalem@redhat.com
Muli Salem has submitted this change and it was merged.
Change subject: core: Do Not Allow Same Mac Address (#845837)
......................................................................
core: Do Not Allow Same Mac Address (#845837)
https://bugzilla.redhat.com/845837
This patch removes the freeing of Mac addresses from
the MacPool in the canDoAction of UpdateVmInterface
and AddVmInterface. The reason is that this enabled
adding two VmInterfaces with the same Mac address.
Signed-off-by: Muli Salem <msalem(a)redhat.com>
Change-Id: I3cfcf0c15bc494f1debd4bdd3e15bd396b179323
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmInterfaceCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmInterfaceCommand.java
2 files changed, 3 insertions(+), 7 deletions(-)
Approvals:
Muli Salem: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/6911
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3cfcf0c15bc494f1debd4bdd3e15bd396b179323
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Muli Salem <msalem(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Muli Salem <msalem(a)redhat.com>
12 years, 4 months
Change in ovirt-engine[master]: ui: wipe after delete should be disabled for POSIXFS storage...
by oourfali@redhat.com
Oved Ourfali has submitted this change and it was merged.
Change subject: ui: wipe after delete should be disabled for POSIXFS storage (#845463)
......................................................................
ui: wipe after delete should be disabled for POSIXFS storage (#845463)
https://bugzilla.redhat.com/845463
This patch disable the wipe after delete option in the UI for disks that
rely on posixfs storage domains.
Change-Id: I3ce4c13b18f8886d24a66e51bd5458b3be8e9906
Signed-off-by: Oved Ourfali <oourfali(a)redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/DiskModel.java
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Gilad Chaplik: Looks good to me, approved
Oved Ourfali: Verified
--
To view, visit http://gerrit.ovirt.org/6934
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3ce4c13b18f8886d24a66e51bd5458b3be8e9906
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
12 years, 4 months