Change in ovirt-engine[master]: frontend: Expose host.devicePassthroughEnabled in Host SubTab
by tjelinek@redhat.com
Tomas Jelinek has submitted this change and it was merged.
Change subject: frontend: Expose host.devicePassthroughEnabled in Host SubTab
......................................................................
frontend: Expose host.devicePassthroughEnabled in Host SubTab
Change-Id: I76d1529194407d3b9bc717794b6437695847fae7
Signed-off-by: Martin Betak <mbetak(a)redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostGeneralModel.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/host/SubTabHostGeneralInfoView.java
3 files changed, 25 insertions(+), 1 deletion(-)
Approvals:
Tomas Jelinek: Looks good to me, approved
Jenkins CI: Passed CI tests
Martin Betak: Verified
--
To view, visit https://gerrit.ovirt.org/42685
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I76d1529194407d3b9bc717794b6437695847fae7
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbetak(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 4 months
Change in ovirt-engine[master]: core: creating dbutils SPs in engine-setup
by emesika@redhat.com
Eli Mesika has submitted this change and it was merged.
Change subject: core: creating dbutils SPs in engine-setup
......................................................................
core: creating dbutils SPs in engine-setup
Moving SP creation from dbutils to dbscripts in order to make
engine-setup responsible for installing those SPs with the right DB
user.
Also remove calls to install SPs from the utilities and removed the 3.5
split done in https://gerrit.ovirt.org/#/c/38147/ since this is for 3.6
Change-Id: Ide9f79ccfaff710b9b6bd0c41ce44d0b9bfee942
https://bugzilla.redhat.com/show_bug.cgi?id=1080542
Signed-off-by: emesika <emesika(a)redhat.com>
---
R packaging/dbscripts/fkvalidator_sp.sql
R packaging/dbscripts/taskcleaner_sp.sql
M packaging/setup/dbutils/fkvalidator.sh
M packaging/setup/dbutils/taskcleaner.sh
D packaging/setup/dbutils/taskcleaner_sp_3_5.sql
5 files changed, 8 insertions(+), 108 deletions(-)
Approvals:
Eli Mesika: Verified; Looks good to me, approved; Passed CI tests
Yedidyah Bar David: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/42655
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ide9f79ccfaff710b9b6bd0c41ce44d0b9bfee942
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 4 months
Change in ovirt-engine[master]: core: rename config value
by ahadas@redhat.com
Arik Hadas has submitted this change and it was merged.
Change subject: core: rename config value
......................................................................
core: rename config value
The name of the config value for compatibility versions for VM-stats
events was inconsistent between the code and the database, which
lead to wrong decisions on which clusters the feature is supported.
The fix is to use the name that exists in the code also in the
database (VmStatsEventsSupported).
Change-Id: I86f419b92b86501d9385ae0ea481e6503df88d03
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
1 file changed, 7 insertions(+), 1 deletion(-)
Approvals:
Omer Frenkel: Looks good to me, approved
Arik Hadas: Verified; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/42715
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I86f419b92b86501d9385ae0ea481e6503df88d03
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 4 months
Change in ovirt-engine[master]: gluster: Corrected injection of BackendInternal
by sabose@redhat.com
Sahina Bose has submitted this change and it was merged.
Change subject: gluster: Corrected injection of BackendInternal
......................................................................
gluster: Corrected injection of BackendInternal
Corrected injection of class BackednInternal
Change-Id: I275a55a831a584cb45290fc29c9145a274aaa160
Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJob.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJobTest.java
2 files changed, 15 insertions(+), 7 deletions(-)
Approvals:
Shubhendu Tripathi: Verified
Sahina Bose: Looks good to me, approved; Passed CI tests
Moti Asayag: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/42524
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I275a55a831a584cb45290fc29c9145a274aaa160
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 4 months
Change in ovirt-engine[ovirt-engine-3.5-gluster]: gluster: Dont list striped volumes types while volume creation
by sabose@redhat.com
Sahina Bose has submitted this change and it was merged.
Change subject: gluster: Dont list striped volumes types while volume creation
......................................................................
gluster: Dont list striped volumes types while volume creation
Change-Id: Ib493645662f493a271635613264c8dff1e33c339
Bug-Url: https://bugzilla.redhat.com/1233621
Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeModel.java
1 file changed, 4 insertions(+), 0 deletions(-)
Approvals:
Shubhendu Tripathi: Verified
Sahina Bose: Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/42674
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib493645662f493a271635613264c8dff1e33c339
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5-gluster
Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 4 months
Change in ovirt-engine[ovirt-engine-3.5-gluster]: engine: add snapshot mount point to filter list
by sabose@redhat.com
Sahina Bose has submitted this change and it was merged.
Change subject: engine: add snapshot mount point to filter list
......................................................................
engine: add snapshot mount point to filter list
Add '/var/run/gluster/snaps/' to the list of mount
points ignored for storage devices. lvm snapshots will
be mouted at /var/run/gluster/snaps in older versions of
Fedora and RHEL-6. With recent version of Fedora and
RHEL-7 it will be mounted at /run/gluster/snaps/.
Change-Id: If9dabf1966a82d54a3ae5cddfb0c7f5df6b09429
Bug-Url: https://bugzilla.redhat.com/1227885
Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com>
---
M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Eli Mesika: Looks good to me, approved
Sahina Bose: Passed CI tests
Ramesh N: Verified
--
To view, visit https://gerrit.ovirt.org/42677
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If9dabf1966a82d54a3ae5cddfb0c7f5df6b09429
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5-gluster
Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 4 months
Change in ovirt-engine[master]: webadmin: clicking system tree doesn't update main tab
by awels@redhat.com
Alexander Wels has submitted this change and it was merged.
Change subject: webadmin: clicking system tree doesn't update main tab
......................................................................
webadmin: clicking system tree doesn't update main tab
- When clicking through items in the system tree, if there was
only one item in the tab grid, it would auto select the item and
change the place accordingly. However it would do this before
the tab had fully loaded and it would cause the tab to not properly
update. This patch fixes the issue by making sure the auto selection
does not occur unless the view of the table that is being auto
selection is attached to the DOM.
Change-Id: I9ee1171f16ec239332c981bf28f7d5c9fa690edc
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1194293
Signed-off-by: Alexander Wels <awels(a)redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/AbstractActionTable.java
1 file changed, 21 insertions(+), 10 deletions(-)
Approvals:
Alexander Wels: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/42238
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9ee1171f16ec239332c981bf28f7d5c9fa690edc
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 4 months
Change in ovirt-engine[master]: core: Use Inject for libvirtSecretDao.
by mlipchuk@redhat.com
Maor Lipchuk has submitted this change and it was merged.
Change subject: core: Use Inject for libvirtSecretDao.
......................................................................
core: Use Inject for libvirtSecretDao.
Use Inject for libvirtSecretDao
Change-Id: Icf657d32402b9a06ee1c60ad9636168242f4bfb1
Bug-Url: https://bugzilla.redhat.com/1185826
Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ConnectHostToStoragePoolServersCommand.java
1 file changed, 7 insertions(+), 2 deletions(-)
Approvals:
Allon Mureinik: Looks good to me, approved
Maor Lipchuk: Verified; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/42652
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icf657d32402b9a06ee1c60ad9636168242f4bfb1
Gerrit-PatchSet: 12
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: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 4 months
Change in ovirt-engine-cli[cli_3.5]: cli: Check if Spice port is available
by juan.hernandez@redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: cli: Check if Spice port is available
......................................................................
cli: Check if Spice port is available
Currently when the Spice port isn't available the CLI generates a
command line that includes "None" as the value for the "-p" option. This
causes a failure of the "spicec" command. To solve that problem this
patch changes the CLI so that it won't generate the "-p" option in that
case.
Change-Id: I36556b0dd5ee0cc794dbb7867719e4f90edaf924
Bug-Url: https://bugzilla.redhat.com/1234382
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
(cherry picked from commit a01e57eefd18ae12c7eb226f8cd8950a1f33aa60)
---
M src/ovirtcli/platform/posix/spice.py
1 file changed, 6 insertions(+), 2 deletions(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/42680
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I36556b0dd5ee0cc794dbb7867719e4f90edaf924
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-cli
Gerrit-Branch: cli_3.5
Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 4 months
Change in ovirt-engine-cli[master]: cli: Check if Spice port is available
by juan.hernandez@redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: cli: Check if Spice port is available
......................................................................
cli: Check if Spice port is available
Currently when the Spice port isn't available the CLI generates a
command line that includes "None" as the value for the "-p" option. This
causes a failure of the "spicec" command. To solve that problem this
patch changes the CLI so that it won't generate the "-p" option in that
case.
Change-Id: I36556b0dd5ee0cc794dbb7867719e4f90edaf924
Bug-Url: https://bugzilla.redhat.com/1234382
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
---
M src/ovirtcli/platform/posix/spice.py
1 file changed, 6 insertions(+), 2 deletions(-)
Approvals:
Martin Tessun: Verified; Looks good to me, but someone else must approve
Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/42678
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I36556b0dd5ee0cc794dbb7867719e4f90edaf924
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-cli
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Martin Tessun <mtessun(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 4 months