Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build script needs update due to change in openshift environment variables #2

Open
rw950431 opened this issue Nov 12, 2012 · 3 comments
Assignees

Comments

@rw950431
Copy link

Openshift have deleted the trailing / off all of their enviroment variables.

Hence lines in .openshift/action_hooks/build like
${OPENSHIFT_RUN_DIR}redis.pid
need to change to
${OPENSHIFT_RUN_DIR}/redis.pid

It would be nice to bump redis version to latest also..

@quatermain
Copy link

Openshift team changed back these environments.
echo ${OPENSHIFT_RUN_DIR}/redis.pid
/var/lib/openshift/......./ruby-1.9/run//redis.pid

echo ${OPENSHIFT_RUN_DIR}redis.pid
/var/lib/openshift/......./ruby-1.9/run/redis.pid

@rw950431
Copy link
Author

You're absolutely right! Doh!

Wish they would make up their mind...
Fortunately most scripts seem to tolerate the '//'

@ghost ghost assigned razorinc Nov 12, 2012
@razorinc
Copy link
Owner

@rw950431 thanks for your feedback 👍 I hence left a bit the development of this because the openshift guys took it in their account, but absolutely i'm gonna update the new version.

And yeah the consequent // is the same as / in shell scripting :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants