bump rubygems version in bootstrap templates

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 0.10.0
  • Fix Version/s: 0.10.4
  • Component/s: Knife
  • Labels:
    None

Description

Right now rubygems 1.3.7 is installed in the following bootstrap templates:

  • centos5-gems.erb
  • ubuntu10.04-gems.erb

This is a fairly old version of rubygems and appears to be causing some permission issues on ubuntu..namely creating a ~/.gem directory owned by root:

lubuntu@domU-12-31-39-00-C5-82:~$ ll
total 32
drwxr-xr-x 5 ubuntu ubuntu 4096 2011-04-30 03:39 ./
drwxr-xr-x 4 root   root   4096 2011-04-30 03:41 ../
-rw-r--r-- 1 ubuntu ubuntu  220 2011-02-01 21:50 .bash_logout
-rw-r--r-- 1 ubuntu ubuntu 3103 2011-02-01 21:50 .bashrc
drwx------ 2 ubuntu ubuntu 4096 2011-04-30 03:39 .cache/
drwxr-xr-x 3 root   root   4096 2011-04-30 03:39 .gem/

Bumping the rubygems version to the latest 1.7.2 fixes these issues.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Seth Chisamore added a comment -

Fix can be pulled from here:
https://github.com/schisamo/chef/tree/CHEF-2293

Show
Seth Chisamore added a comment - Fix can be pulled from here: https://github.com/schisamo/chef/tree/CHEF-2293
Hide
Permalink
Seth Chisamore added a comment -

Dan mentioned there might be some issues with 1.7.2...I'm not tied to it...just want a newer rubygems than 1.3.7 as the default.

Show
Seth Chisamore added a comment - Dan mentioned there might be some issues with 1.7.2...I'm not tied to it...just want a newer rubygems than 1.3.7 as the default.
Hide
Permalink
James Sulinski added a comment -

Ruby 1.8.7 and Rubygems 1.6.2 are required for 0.10 on el5.

It would be good to install rubygems via package, and we can update the packages to rubygems 1.7.2 to facilitate this if need be. I haven't heard of the 1.7.2 problems, but went with 1.6.2 because it was the oldest version which worked.

Show
James Sulinski added a comment - Ruby 1.8.7 and Rubygems 1.6.2 are required for 0.10 on el5. It would be good to install rubygems via package, and we can update the packages to rubygems 1.7.2 to facilitate this if need be. I haven't heard of the 1.7.2 problems, but went with 1.6.2 because it was the oldest version which worked.
Hide
Permalink
James Sulinski added a comment -

I think in the bootstrap scripts the gem update --system command is supposed to update to the latest rubygems, but that command doesn't work on 1.3.7 as far as I know. You have to install the update_rubygems gem and run the bin it creates.

Show
James Sulinski added a comment - I think in the bootstrap scripts the gem update --system command is supposed to update to the latest rubygems, but that command doesn't work on 1.3.7 as far as I know. You have to install the update_rubygems gem and run the bin it creates.
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Seth,

I believe you said you would give us a new patch, yes?

Thanks buddy!

Show
Bryan McLellan [Opscode] added a comment - Seth, I believe you said you would give us a new patch, yes? Thanks buddy!
Hide
Permalink
Seth Chisamore added a comment -

Ok updated my branch with references to 1.6.2
https://github.com/schisamo/chef/tree/CHEF-2293

Show
Seth Chisamore added a comment - Ok updated my branch with references to 1.6.2 https://github.com/schisamo/chef/tree/CHEF-2293
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Thanks sir

Show
Bryan McLellan [Opscode] added a comment - Thanks sir
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Reopening for fix version change.

Show
Bryan McLellan [Opscode] added a comment - Reopening for fix version change.
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

0.10.2 was a security release, 0.10.4 is the next maintenance release.

Show
Bryan McLellan [Opscode] added a comment - 0.10.2 was a security release, 0.10.4 is the next maintenance release.
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Bulk edit: this ticket should have been previously merged for 0.10.2, but will be released in 0.10.4

Show
Bryan McLellan [Opscode] added a comment - Bulk edit: this ticket should have been previously merged for 0.10.2, but will be released in 0.10.4

People

  • Assignee:
    Bryan McLellan [Opscode]
    Reporter:
    Seth Chisamore
Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: