Change in ovirt-engine[master]: Revert "engine: Create default network QoS for ovirtmgmt upo...
by masayag@redhat.com
Moti Asayag has submitted this change and it was merged.
Change subject: Revert "engine: Create default network QoS for ovirtmgmt upon DC creation"
......................................................................
Revert "engine: Create default network QoS for ovirtmgmt upon DC creation"
This reverts commit 7bbfdab92ac600994319aac2bc5ee35ec11f3dc9.
Change-Id: I3aad475b1b075ffc3b182082d63304f48cbb56e6
Bug-Url: https://bugzilla.redhat.com/1364145
Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/pool/AddEmptyStoragePoolCommand.java
1 file changed, 1 insertion(+), 35 deletions(-)
Approvals:
Marcin Mirecki: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Moti Asayag: Looks good to me, approved
Yevgeny Zaspitsky: Verified
--
To view, visit https://gerrit.ovirt.org/61914
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3aad475b1b075ffc3b182082d63304f48cbb56e6
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com>
Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[master]: Revert "engine: allow QoS and non-QoS networks to be attache...
by masayag@redhat.com
Moti Asayag has submitted this change and it was merged.
Change subject: Revert "engine: allow QoS and non-QoS networks to be attached to a nic"
......................................................................
Revert "engine: allow QoS and non-QoS networks to be attached to a nic"
This reverts commit 984878ccbb5084fa936dca5df39f2bf2b8fc740a.
Change-Id: Ic20d5e51741b79005ae2c9c452fdea2faf7e322a
Bug-Url: https://bugzilla.redhat.com/1364145
Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/HostSetupNetworksValidator.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/host/HostSetupNetworksValidatorTest.java
2 files changed, 118 insertions(+), 1 deletion(-)
Approvals:
Marcin Mirecki: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Moti Asayag: Looks good to me, approved
Yevgeny Zaspitsky: Verified
--
To view, visit https://gerrit.ovirt.org/61915
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic20d5e51741b79005ae2c9c452fdea2faf7e322a
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com>
Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[master]: core: ClusterValidator: clean up boolean expression
by amureini@redhat.com
Allon Mureinik has submitted this change and it was merged.
Change subject: core: ClusterValidator: clean up boolean expression
......................................................................
core: ClusterValidator: clean up boolean expression
Remove parentheses and explicit "false" in versionSupported() to make
the boolean expression more straight forward and easier to read.
Change-Id: I1cefa45708cdee9c276db41d6bb5a9e1fcda879c
Signed-off-by: Allon Mureinik <amureini(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/ClusterValidator.java
1 file changed, 2 insertions(+), 3 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Allon Mureinik: Verified
Arik Hadas: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/62013
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1cefa45708cdee9c276db41d6bb5a9e1fcda879c
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: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[master]: core: ClusterValidator type inference
by amureini@redhat.com
Allon Mureinik has submitted this change and it was merged.
Change subject: core: ClusterValidator type inference
......................................................................
core: ClusterValidator type inference
Change-Id: Ic8e97ee064492fe339581e8793a4f391c06d7d55
Signed-off-by: Allon Mureinik <amureini(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/ClusterValidator.java
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Jenkins CI: Passed CI tests
Allon Mureinik: Verified
Arik Hadas: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/62012
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic8e97ee064492fe339581e8793a4f391c06d7d55
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: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in jenkins[master]: move ovirt-system-tests to run on fc24
by eedri@redhat.com
Eyal Edri has submitted this change and it was merged.
Change subject: move ovirt-system-tests to run on fc24
......................................................................
move ovirt-system-tests to run on fc24
We don't maintain anymore fc23 repos for master
Change-Id: Ia081be48612229f8919e66fdc043d46b90a6e941
Signed-off-by: Eyal Edri <eedri(a)redhat.com>
---
M jobs/confs/projects/ovirt-system-tests/ovirt-system-tests_standard.yaml
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Eyal Edri: Verified; Looks good to me; Ready for merge
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/62016
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia081be48612229f8919e66fdc043d46b90a6e941
Gerrit-PatchSet: 1
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[master]: webadmin, userportal: VM > Containers subtab hashname fix
by mbetak@redhat.com
Martin Betak has submitted this change and it was merged.
Change subject: webadmin, userportal: VM > Containers subtab hashname fix
......................................................................
webadmin, userportal: VM > Containers subtab hashname fix
VM > Containers subtab hash name was inconsistent causing warning:
"org.ovirt.engine.ui.common.uicommon.ContextSensitiveHelpManager
INFO: No context-sensitive help was found on the server. It will not be
available for this session."
Change-Id: I0e5f98d06c6c61da4c3cce6241597dc46faed5e3
Bug-Url: https://bugzilla.redhat.com/1364466
Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmGuestContainerListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/place/UserPortalApplicationPlaces.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/place/WebAdminApplicationPlaces.java
3 files changed, 10 insertions(+), 3 deletions(-)
Approvals:
Jakub Niedermertl: Verified
Jenkins CI: Passed CI tests
Martin Betak: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/61991
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0e5f98d06c6c61da4c3cce6241597dc46faed5e3
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com>
Gerrit-Reviewer: Jakub Niedermertl <jniederm(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: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-image-uploader[ovirt-image-uploader-3.6]: ovirt-image-uploader: added --with-kerberos engine option
by rmartins@redhat.com
Rafael Martins has submitted this change and it was merged.
Change subject: ovirt-image-uploader: added --with-kerberos engine option
......................................................................
ovirt-image-uploader: added --with-kerberos engine option
Bug-Url: https://bugzilla.redhat.com/1364471
Change-Id: Ia0e8aedf8050bc88f916b638da0154f147352200
Signed-off-by: Rafael Martins <rmartins(a)redhat.com>
---
M src/__main__.py
M src/engine-image-uploader.8
2 files changed, 31 insertions(+), 10 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Rafael Martins: Verified; Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/61980
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia0e8aedf8050bc88f916b638da0154f147352200
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-image-uploader
Gerrit-Branch: ovirt-image-uploader-3.6
Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-2.0]: text: making the exit message context aware
by stirabos@redhat.com
Simone Tiraboschi has submitted this change and it was merged.
Change subject: text: making the exit message context aware
......................................................................
text: making the exit message context aware
Using softer exit messages if the error occured before
the tool really touched the system.
Change-Id: I9ad15d672366ed0a340c3e65aa5c1047230dcd0d
Bug-Url: https://bugzilla.redhat.com/1358255
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
---
M src/ovirt_hosted_engine_setup/constants.py
M src/plugins/gr-he-common/core/misc.py
M src/plugins/gr-he-rollbackupgrade/core/misc.py
M src/plugins/gr-he-setup/core/misc.py
M src/plugins/gr-he-upgradeappliance/core/misc.py
5 files changed, 59 insertions(+), 73 deletions(-)
Approvals:
Simone Tiraboschi: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
Rafael Martins: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/61989
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9ad15d672366ed0a340c3e65aa5c1047230dcd0d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: ovirt-hosted-engine-setup-2.0
Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-hosted-engine-setup[master]: text: making the exit message context aware
by stirabos@redhat.com
Simone Tiraboschi has submitted this change and it was merged.
Change subject: text: making the exit message context aware
......................................................................
text: making the exit message context aware
Using softer exit messages if the error occured before
the tool really touched the system.
Change-Id: I9ad15d672366ed0a340c3e65aa5c1047230dcd0d
Bug-Url: https://bugzilla.redhat.com/1358255
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
---
M src/ovirt_hosted_engine_setup/constants.py
M src/plugins/gr-he-common/core/misc.py
M src/plugins/gr-he-rollbackupgrade/core/misc.py
M src/plugins/gr-he-setup/core/misc.py
M src/plugins/gr-he-upgradeappliance/core/misc.py
5 files changed, 59 insertions(+), 73 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Simone Tiraboschi: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
Rafael Martins: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/61486
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9ad15d672366ed0a340c3e65aa5c1047230dcd0d
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Rafael Martins <rmartins(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: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-image-uploader[ovirt-image-uploader-4.0]: ovirt-image-uploader: added --with-kerberos engine option
by rmartins@redhat.com
Rafael Martins has submitted this change and it was merged.
Change subject: ovirt-image-uploader: added --with-kerberos engine option
......................................................................
ovirt-image-uploader: added --with-kerberos engine option
Bug-Url: https://bugzilla.redhat.com/1340963
Change-Id: Ia0e8aedf8050bc88f916b638da0154f147352200
Signed-off-by: Rafael Martins <rmartins(a)redhat.com>
(cherry picked from commit 86772c2dbd3c3dc4851a9f7959d51ee72e12fe60)
---
M src/__main__.py
M src/ovirt-image-uploader.8
2 files changed, 30 insertions(+), 10 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Rafael Martins: Verified; Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/61909
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia0e8aedf8050bc88f916b638da0154f147352200
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-image-uploader
Gerrit-Branch: ovirt-image-uploader-4.0
Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months