Change in ovirt-engine[master]: scheduling: Add PreferredHosts policy unit
by rgolan@redhat.com
Roy Golan has submitted this change and it was merged.
Change subject: scheduling: Add PreferredHosts policy unit
......................................................................
scheduling: Add PreferredHosts policy unit
This policy unit replaces the old behaviour of destHostIdList
to keep the preferred host for VM functionality, but doing it
in the "proper" way that can be disabled in the cluster policy.
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1243811
Change-Id: I2f55caf42b787458d031296cdd8f837f79293a3b
Signed-off-by: Martin Sivák <msivak(a)redhat.com>
---
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/scheduling/InternalClusterPolicies.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/InternalPolicyUnits.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java
A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PreferredHostsWeightPolicyUnit.java
A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/policyunits/PreferredHostsWeightPolicyUnitTest.java
6 files changed, 132 insertions(+), 2 deletions(-)
Approvals:
Martin Sivák: Verified
Jenkins CI: Passed CI tests
Roy Golan: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/45816
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2f55caf42b787458d031296cdd8f837f79293a3b
Gerrit-PatchSet: 36
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 6 months
Change in ovirt-engine[ovirt-engine-3.5-gluster]: gluster: fix geo rep session key parsing issue
by sabose@redhat.com
Sahina Bose has submitted this change and it was merged.
Change subject: gluster: fix geo rep session key parsing issue
......................................................................
gluster: fix geo rep session key parsing issue
There is a change in the geo rep ression key from
glusterfs 3.7.12 onwards. Slave volume UUID is added as lost
component to session key. Updating the session key
parsing to handle both old and new versions.
Change-Id: I8717817065c17b3d9d50b3ff6f524d7a530f2039
Bug-Url: https://bugzilla.redhat.com/1343650
Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumeGeoRepStatusForXmlRpc.java
1 file changed, 3 insertions(+), 2 deletions(-)
Approvals:
Sahina Bose: Looks good to me, approved; Passed CI tests
Ramesh N: Verified
--
To view, visit https://gerrit.ovirt.org/58848
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8717817065c17b3d9d50b3ff6f524d7a530f2039
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5-gluster
Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com>
Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 6 months
Change in ovirt-engine[master]: gluster: fix geo rep session key parsing issue
by sabose@redhat.com
Sahina Bose has submitted this change and it was merged.
Change subject: gluster: fix geo rep session key parsing issue
......................................................................
gluster: fix geo rep session key parsing issue
There is a change in the geo rep ression key from
glusterfs 3.7.12 onwards. Slave volume UUID is added as lost
component to session key. Updating the session key
parsing to handle both old and new versions.
Change-Id: I8717817065c17b3d9d50b3ff6f524d7a530f2039
Bug-Url: https://bugzilla.redhat.com/1343650
Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumeGeoRepStatusForXmlRpc.java
1 file changed, 3 insertions(+), 2 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Sahina Bose: Looks good to me, approved
Ramesh N: Verified
--
To view, visit https://gerrit.ovirt.org/58793
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8717817065c17b3d9d50b3ff6f524d7a530f2039
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com>
Gerrit-Reviewer: Aravinda VK <avishwan(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 6 months
Change in ovirt-engine[master]: webadmin: hide unnecessary label from Edit VM popup
by tnisan@redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: webadmin: hide unnecessary label from Edit VM popup
......................................................................
webadmin: hide unnecessary label from Edit VM popup
This patch removes the unnecessary label "Disks Allocation:" from the
"Resource Allocation" tab in the "Edit Virtual Machine" popup.
It sets the visibility of the label to be false so that only when there
are disks to display, it is set to true.
Change-Id: I422ed365b83d1560c9393910a00570b7e42f7d65
Bug-Url: https://bugzilla.redhat.com/1251928
Signed-off-by: Idan Shaby <ishaby(a)redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/DisksAllocationView.ui.xml
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Tal Nisan: Looks good to me, approved
Jenkins CI: Passed CI tests
Idan Shaby: Verified
--
To view, visit https://gerrit.ovirt.org/58801
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I422ed365b83d1560c9393910a00570b7e42f7d65
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 6 months
Change in ovirt-release[ovirt-4.0]: node: Allow a separate suffix for the placeholder package
by fabiand@redhat.com
Fabian Deutsch has submitted this change and it was merged.
Change subject: node: Allow a separate suffix for the placeholder package
......................................................................
node: Allow a separate suffix for the placeholder package
This is needed to allow snapshot NGN builds in the stable branches
Change-Id: I559b25168fcee4b3c11931f064394da44a3e173d
Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org>
---
M ovirt-release40.spec.in
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Fabian Deutsch: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58839
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I559b25168fcee4b3c11931f064394da44a3e173d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-release
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 6 months
Change in ovirt-release[ovirt-4.0]: node: Add timestamp to placeholder
by fabiand@redhat.com
Fabian Deutsch has submitted this change and it was merged.
Change subject: node: Add timestamp to placeholder
......................................................................
node: Add timestamp to placeholder
This is needed to allow daily builds of node in the snapshot branch
Change-Id: If7c9cca98d9828b4a8366d15cf8631d16204a977
Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org>
---
M automation/build-artifacts.sh
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Fabian Deutsch: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58840
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If7c9cca98d9828b4a8366d15cf8631d16204a977
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-release
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 6 months
Change in ovirt-release[master]: node: Add hint for an additional bug
by fabiand@redhat.com
Fabian Deutsch has submitted this change and it was merged.
Change subject: node: Add hint for an additional bug
......................................................................
node: Add hint for an additional bug
Change-Id: If26de5b185bcf79d6389749abb3a0f57be5af4d1
Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org>
---
M ovirt-release-master.spec.in
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Fabian Deutsch: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58837
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If26de5b185bcf79d6389749abb3a0f57be5af4d1
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-release
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 6 months
Change in ovirt-release[master]: node: Drop he-setup NM check
by fabiand@redhat.com
Fabian Deutsch has submitted this change and it was merged.
Change subject: node: Drop he-setup NM check
......................................................................
node: Drop he-setup NM check
This is now in master with https://gerrit.ovirt.org/#/c/56146/
Change-Id: I898aae7ce19b5ca3bb51d2bf531dd7660e6ae97f
Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org>
---
M ovirt-release-master.spec.in
1 file changed, 0 insertions(+), 5 deletions(-)
Approvals:
Fabian Deutsch: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58836
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I898aae7ce19b5ca3bb51d2bf531dd7660e6ae97f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-release
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 6 months
Change in ovirt-release[master]: node: Drop hack
by fabiand@redhat.com
Fabian Deutsch has submitted this change and it was merged.
Change subject: node: Drop hack
......................................................................
node: Drop hack
This hack does not work, because th eproblem is authconfig.
authconfig is run as a post-installation task, and overriding
nsswitch.conf. Because it is not aware of myhostname, it will
always not add it to the hosts line.
Change-Id: Ic85d77baadabff7357c76ee88b100746499a3c54
Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org>
---
M ovirt-release-master.spec.in
1 file changed, 0 insertions(+), 7 deletions(-)
Approvals:
Fabian Deutsch: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58835
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic85d77baadabff7357c76ee88b100746499a3c54
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-release
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 6 months
Change in ovirt-release[master]: node: Allow a separate suffix for the placeholder package
by fabiand@redhat.com
Fabian Deutsch has submitted this change and it was merged.
Change subject: node: Allow a separate suffix for the placeholder package
......................................................................
node: Allow a separate suffix for the placeholder package
This is needed to allow snapshot NGN builds in the stable branches
Change-Id: I559b25168fcee4b3c11931f064394da44a3e173d
Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org>
---
M ovirt-release-master.spec.in
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Fabian Deutsch: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58838
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I559b25168fcee4b3c11931f064394da44a3e173d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-release
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 6 months