Bug 761173

Summary:	SELinux related: Move grinder usage of /tmp/grinder to /var/run/grinder
Product:	[Retired] Pulp	Reporter:	John Matthews <jmatthew>
Component:	user-experience	Assignee:	John Matthews <jmatthew>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Preethi Thomas <pthomas>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	1.0.0	CC:	skarmark
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-02-24 20:16:06 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description John Matthews 2011-12-07 20:02:31 UTC

Description of problem:

Based on Dan Walsh's suggestion we should move usage of '/tmp/grinder' to '/var/run/grinder'.  

Remember to update SELinux policy in Pulp to account for this directory change after it's made.



QE:

 Run a sync and verify that you do not see a /tmp/grinder being created.

 Look at the pulp-server.fc which is under /usr/share/doc/pulp.../pulp-server.fc  verify that there is a line addressing /var/run/grinder

Comment 1 John Matthews 2011-12-07 21:29:59 UTC

Consider changing grinder to accept a location of where to write for temp usage, have Pulp base this value off something from /var/lib/pulp this way when running in the cloud we can continue with convention of instructing users to provide ample storage to /var/lib/pulp.

We would like to avoid requiring users to provide extra temp space on various mountpoints.

Comment 2 John Matthews 2011-12-15 20:24:44 UTC

Added a method in Pulp for
util.tmp_cache_location() which uses Pulp's constant for LOCAL_STORAGE.

Also changed grinder to allow the specification of tmp_path to YumRepoGrinder.

This means temporary yum downloading for metadata is now stored under /var/lib/pulp/cache/grinder

Commit:
http://git.fedorahosted.org/git/?p=pulp.git;a=commitdiff;h=eb11a8976a84945cdc399db8f93e0f376756b29d

Requires Grinder 0.134

Comment 3 Jeff Ortel 2012-01-04 21:48:42 UTC

build: 0.256

Comment 4 Preethi Thomas 2012-01-06 16:24:10 UTC

verified
[root@preethi ~]# rpm -q pulp

pulp-0.0.256-1.fc15.noarch


[root@preethi ~]# ls -l /var/lib/pulp/cache/grinder/
total 4
drwxr-xr-x. 3 apache apache 4096 Jan  6 11:35 27383
[root@preethi ~]# 


see that the /tmo/grinder is from before 256

[root@preethi ~]# ls -l /tmp/grinder/
total 8
drwxr-xr-x. 2 apache apache 4096 Jan  4 14:59 13017
drwxr-xr-x. 2 apache apache 4096 Jan  4 14:59 13047
[root@preethi ~]#

Comment 5 Preethi Thomas 2012-02-24 20:16:06 UTC

Pulp v1.0 is released
Closed Current Release.

Comment 6 Preethi Thomas 2012-02-24 20:17:34 UTC

Pulp v1.0 is released.