Bug #3412

install.rb should not put "." first in the tmp_dirs

Added by Martin Englund about 2 years ago. Updated about 2 years ago.

Status:Closed Start date:03/23/2010
Priority:Normal Due date:
Assignee:Martin Englund % Done:

0%
Category:installation
Target version:2.6.0
Affected Puppet version:0.25.4 Branch:http://github.com/pmenglund/puppet/tree/tickets/0.25.x/3412
Keywords:
Votes: 0

Description

Since Ruby incorrectly assumes that being root is enough to be able to write a directory, install.rb will fail when installing as root from a NFS mounted directory, as it will pick the current directory (“.”) as tmp_dir.

root@ubuntu:/mnt/puppet# ls -ld .
drwxr-xr-x 24 martin staff 816 2010-03-19 15:55 .
root@ubuntu:/mnt/puppet# ruby install.rb 
install.rb:416:in `initialize': Permission denied - ./_tmp (Errno::EACCES)

Related issues

related to Facter - Bug #3411: install.rb should not put "." first in the tmp_dirs Closed 03/23/2010

History

#1
Updated by Markus Roberts about 2 years ago

  • Status changed from Unreviewed to Investigating
  • Target version set to 0.25.5

#2
Updated by Markus Roberts about 2 years ago

  • Status changed from Investigating to In Topic Branch Pending Review
  • Branch set to http://github.com/pmenglund/puppet/tree/tickets/0.25.x/3412

#3
Updated by James Turnbull about 2 years ago

Pushed in commit:d561a988c53d521d0fecf06c4f4d8a65267ac005 in branch 0.25.x

#4
Updated by Markus Roberts about 2 years ago

  • Status changed from In Topic Branch Pending Review to Merged - Pending Release

#5
Updated by James Turnbull about 2 years ago

  • Target version changed from 0.25.5 to 49

#6
Updated by James Turnbull about 2 years ago

  • Target version changed from 49 to 2.6.0

#7
Updated by Markus Roberts about 2 years ago

  • Status changed from Merged - Pending Release to Closed

In master as of commit:e6a720b67abc16e0a008021b8192f84c11060635

Also available in: Atom PDF