Bug 974632
Summary: | Ruby default gem/location assumptions are "wrong" | ||
---|---|---|---|
Product: | OKD | Reporter: | Clayton Coleman <ccoleman> |
Component: | Containers | Assignee: | Hiro Asari <hasari> |
Status: | CLOSED UPSTREAM | QA Contact: | libra bugs <libra-bugs> |
Severity: | medium | Docs Contact: | |
Priority: | unspecified | ||
Version: | 2.x | CC: | jhonce, jkeck |
Target Milestone: | --- | Keywords: | FutureFeature |
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Enhancement | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2013-06-28 18:24:09 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: | 974983 |
Description
Clayton Coleman
2013-06-14 16:53:16 UTC
PR is wrong. We can create ~/.gem and have it be owned by the gear user for all gears. However, in all cases other than the Ruby 1.9, the current default Ruby version is 1.8.7, where the installation directory is set to /usr/lib/ruby/gems/1.8. It is not clear to me at the moment if this is a 1.8 issue, or the way the RPM package is configured. In order for this to work, then, we need to set GEM_HOME explicitly to $OPENSHIFT_HOMEDIR/.gem for all gears. See https://github.com/openshift/origin-server/pull/2865 Postponing for now. Existing gears need to be migrated. Commit pushed to master at https://github.com/openshift/origin-server https://github.com/openshift/origin-server/commit/651e1f606b68ed8c031f3816ba30b35a17346277 Bug 974632 Create ~/.gem, owned by the gear user, for all cartridges. To have user gems be installed in this directory, we now set $GEM_HOME explicitly on all cartridges. Subsequently, the Ruby 1.9 cartridge need not create ~/.gem. See https://trello.com/c/71WPHuYT for status. |