Bug 1057579

Summary:	HA Vm reservation check ignores host status
Product:	[Retired] oVirt	Reporter:	Moti Asayag <masayag>
Component:	ovirt-engine-core	Assignee:	Kobi <kianku>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Lukas Svaty <lsvaty>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	3.4	CC:	acathrow, dfediuck, gklein, iheim, mavital, yeylon
Target Milestone:	---	Keywords:	Triaged
Target Release:	3.4.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	sla
Fixed In Version:	ovirt-3.4.0-rc	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2014-03-31 12:25:53 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Moti Asayag 2014-01-24 12:01:38 UTC

Description of problem:
In a cluster configured for HA resource reservation, with a single host in state up which is running HA vms, and all the other hosts are in statuses Maintenance/Down, the check of HA resource reservation is ended successfully.

Version-Release number of selected component (if applicable):


How reproducible:
always

Steps to Reproduce:
1. Mark cluster for HA resource reservation
2. Have 2 hosts in the cluster:
2.1. host 1 is down
2.2. host 2 has 2 vms running on
3. wait up to 5 minutes where the following appears in the engine.log:
2014-01-24 13:51:24,315 INFO  [org.ovirt.engine.core.bll.scheduling.HaReservationHandling] (DefaultQuartzScheduler_Worker-34) HA reservation status for cluster Default is OK

Actual results:
Resource check ends successfully

Expected results:
If there are no hosts when the check is executed that can guarantee the resource reservation for the HA vms, an error should be reported both to engine.log and mostly to the event log.

Additional info:

Comment 1 Doron Fediuck 2014-02-03 11:02:18 UTC

Moti, good catch!

Comment 2 Doron Fediuck 2014-02-03 11:05:35 UTC

One more thing, 'none' policy should include the new ha reservations weight module. Currently it is missing.

Comment 3 Sandro Bonazzola 2014-03-03 14:40:03 UTC

This BZ should be fixed in oVirt 3.4.0 RC

Comment 4 Sandro Bonazzola 2014-03-31 12:25:53 UTC

this is an automated message: moving to Closed CURRENT RELEASE since oVirt 3.4.0 has been released