Bug 1120317

Summary:	[RFE]Allow passing show_image_direct to glance::api
Product:	Red Hat OpenStack	Reporter:	Chris Dearborn <christopher_dearborn>
Component:	openstack-puppet-modules	Assignee:	Chris Dearborn <christopher_dearborn>
Status:	CLOSED CURRENTRELEASE	QA Contact:	nlevinki <nlevinki>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	5.0 (RHEL 7)	CC:	arkady_kanevsky, cdevine, christopher_dearborn, cwolfe, ichavero, jdonohue, jschluet, mmagr, nbarcet, sclewis, sreichar, yeylon
Target Milestone:	z5	Keywords:	FutureFeature, OtherQA, ZStream
Target Release:	5.0 (RHEL 7)
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	openstack-packstack-2014.1.1-0.46.dev1280.el7ost	Doc Type:	Enhancement
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-08-27 17:41:07 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:
Bug Depends On:
Bug Blocks:	1120321, 1120322, 1163445

Description Chris Dearborn 2014-07-16 17:47:30 UTC

Description of problem:
When Glance is configured to use rbd as a backend, Ceph requires show_image_direct_url to be set to True to allow copy-on-write clone of an image.
See: http://ceph.com/docs/next/rbd/rbd-openstack/

This bug is to modify glance::api in stackforge/puppet-openstack so that it allows passing in show_image_direct_url.

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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Chris Dearborn 2014-07-16 17:54:39 UTC

Taken from http://ceph.com/docs/next/rbd/rbd-openstack/:
Note that this exposes the back end location via Glance’s API, so the endpoint with this option enabled should not be publicly accessible

Comment 3 Crag Wolfe 2014-08-05 20:32:01 UTC

Included in patch: https://github.com/redhat-openstack/astapor/pull/327/files .

Comment 4 Ivan Chavero 2014-08-12 23:08:25 UTC

i see that the fix is done in astapor should the component be changed to it?

Comment 5 Chris Dearborn 2014-08-13 14:20:35 UTC

Hey Ivan,

no, I believe the component is correct.  This bug was to track the work in the stackforge/puppet-cinder repo to allow passing in the show_image_direct_url flag, which is why the component is set the way it is.  I just checked, and that work has actually already been done, so I think this bug could be bumped ahead in the process - not sure what the appropriate state is to move it to.

This BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1120321 was for the work in astapor to set show_image_direct_url to true and pass it to the stackforge/puppet-cinder class when the backend type is rbd, and as you point out, this work has also been completed, so this bug too could be bumped ahead in the process.

Comment 6 Martin Magr 2014-09-12 12:07:04 UTC

Setting correct status