Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1213398

Summary:	[RFE] - Do not allow adding Cinder storage domain, if hosts of the selected dc, do not report they have the Ceph libraries.
Product:	[oVirt] vdsm	Reporter:	Yaniv Lavi <ylavi>
Component:	RFEs	Assignee:	Daniel Erez <derez>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Ori Gofen <ogofen>
Severity:	high	Docs Contact:
Priority:	high
Version:	---	CC:	acanan, amureini, bazulay, bugs, derez, ebenahar, ecohen, fdeutsch, gamado, gklein, iheim, lpeer, lsurette, mgoldboi, nsoffer, ogofen, rbalakri, scohen, sherold, ssaha, yeylon, ylavi
Target Milestone:	ovirt-3.6.0-rc	Keywords:	FutureFeature
Target Release:	4.17.9	Flags:	rule-engine: ovirt-3.6.0+ ylavi: planning_ack+ rule-engine: devel_ack+ rule-engine: testing_ack+
Hardware:	x86_64
OS:	Unspecified
Whiteboard:	storage
Fixed In Version:	ovirt-engine-3.6.0_alpha3	Doc Type:	Enhancement
Doc Text:		Story Points:	---
Clone Of:	1160653	Environment:
Last Closed:	2015-11-27 07:50:46 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Storage	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1160653, 1185826

Description Yaniv Lavi 2015-04-20 13:09:54 UTC

We need to add check in VDSM to report is Ceph libraries are present on the host to block two flows:
a. prevent adding a cinder storage domain when some hosts of the dc do not have the Ceph libraries capabilities.
b. prevent adding a host to a cluster that has cinder storage domain configured.

Comment 1 Allon Mureinik 2015-04-27 12:14:57 UTC

Need to research what is a sensible check, and how we can report it back to the engine.

Comment 2 Nir Soffer 2015-04-27 13:39:34 UTC

Ceph libraries will be required for cluster version 3.6, engine will use the cluster version.

Comment 3 Allon Mureinik 2015-07-09 10:45:58 UTC

Daniel, this is handled by https://gerrit.ovirt.org/#/c/42991/ and https://gerrit.ovirt.org/#/c/42990, right?

Comment 4 Nir Soffer 2015-07-19 15:08:36 UTC

Yaniv, I think this is already implemented.

Comment 5 Yaniv Lavi 2015-07-19 15:11:51 UTC

Can we move this to ON_QA then?

Comment 6 Yaniv Lavi 2015-07-19 15:12:10 UTC

And attach patches?

Comment 7 Nir Soffer 2015-07-19 16:56:27 UTC

(In reply to Yaniv Dary from comment #6)
> And attach patches?

Daniel will handle that.

Comment 8 Daniel Erez 2015-07-20 08:35:05 UTC

The behavior is as follows,

Upon host activation:
* Validate Librbd1 package existence.
* Move host to NON-OP if not valid.

Upon Cinder SD activation:
* Validate package existence on all relevant hosts.
* Fail activation if not valid.

Comment 9 Red Hat Bugzilla Rules Engine 2015-10-18 08:34:32 UTC

Bug tickets that are moved to testing must have target release set to make sure tester knows what to test. Please set the correct target release before moving to ON_QA.

Comment 10 Ori Gofen 2015-11-17 14:15:44 UTC

afaik the only "must have" package is librbd1 and it is embedded to vdsm, qemu and libvirt spec since rhevm-3.5 can't remove it without removing it's dependencies or install a fresh vdsm without having librbd installed as well.
Adding a 3.4 hypervisor (which does not have the package) failed because of cluster incompatibility.
marking as Verified on rhevm-3.6-0.2

Comment 11 Sandro Bonazzola 2015-11-27 07:50:46 UTC

Since oVirt 3.6.0 has been released, moving from verified to closed current release.