Bug 761173 - SELinux related: Move grinder usage of /tmp/grinder to /var/run/grinder
Summary: SELinux related: Move grinder usage of /tmp/grinder to /var/run/grinder
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: user-experience
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: ---
Assignee: John Matthews
QA Contact: Preethi Thomas
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-12-07 20:02 UTC by John Matthews
Modified: 2013-09-09 16:36 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-02-24 20:16:06 UTC


Attachments (Terms of Use)

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.


Note You need to log in before you can comment on or make changes to this bug.