Bug 802235 - need change env variables to 'OPENSHIFT_GEAR' in github examples' README file and code
Summary: need change env variables to 'OPENSHIFT_GEAR' in github examples' README file...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OKD
Classification: Red Hat
Component: Containers
Version: 1.x
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Dan McPherson
QA Contact: libra bugs
URL:
Whiteboard:
: 803393 803413 803417 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-03-12 07:51 UTC by Meng Bo
Modified: 2012-04-13 18:35 UTC (History)
4 users (show)

Fixed In Version: devenv-stage_147
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-04-13 18:35:14 UTC
Target Upstream Version:

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Meng Bo 2012-03-12 07:51:47 UTC 
Description of problem:
since the environment variable changed from 'APP' to 'GEAR', the README file in github should be changed correspondingly.  

Version-Release number of selected component (if applicable):
https://github.com/openshift

How reproducible:
always

Steps to Reproduce:
1.visit https://github.com/openshift via web browser
2.check the README files for all the examples
3.
  
Actual results:
environment variables in README files are still 'OPENSHIFT_APP'


Expected results:
env variables should be changed to 'OPENSHIFT_GEAR'


Additional info:
need to be fixed files list:
openshift-twt-mongo-demo / README 
dancer-example / README 
phpbb-example / README 
cakephp-example / README 
django-example / README 
flask-example / README 
frogcms-example / README 
joomla-example / README 
reviewboard-example / README 
sinatra-example / README 
wolfcms-example / README 
wordpress-example / README 
bottle-openshift-quickstart / README 
ciphp / README 
elgg-openshift-quickstart / README 
magento-example / README 
phpmongotweet-example / README 
kitchensink-example / README 
kitchensink-html5-mobile-example / README 
tweetstream-example / environment-readme.txt
Comment 1 Dan McPherson 2012-03-12 14:06:16 UTC 
This can't be changed until the release is public.
Comment 2 Dan McPherson 2012-03-14 16:47:02 UTC 
*** Bug 803393 has been marked as a duplicate of this bug. ***
Comment 3 Dan McPherson 2012-03-14 16:47:19 UTC 
*** Bug 803401 has been marked as a duplicate of this bug. ***
Comment 4 Dan McPherson 2012-03-14 16:47:51 UTC 
*** Bug 803413 has been marked as a duplicate of this bug. ***
Comment 5 Dan McPherson 2012-03-14 16:48:16 UTC 
*** Bug 803417 has been marked as a duplicate of this bug. ***
Comment 6 Meng Bo 2012-03-15 05:27:13 UTC 
Since some examples will get failed when push without changing the env vars.
Need also change the env vars in the github example's code.

like twt-mongo-demo
Comment 7 Xiaoli Tian 2012-03-19 09:41:55 UTC 
Lower the priority since it needs to wait for release to production
Comment 8 Dan McPherson 2012-03-19 15:05:10 UTC 
Went a different route.  The only variable left that doesn't still exist that end users might use is OPENSHIFT_APP_TYPE and none of the samples use it.
Comment 9 Meng Bo 2012-03-22 01:37:04 UTC 
Still can get old env_var like 'OPENSHIFT_APP_DIR' in README files.
And found the modified date of the github examples are many days ago.
Comment 10 Dan McPherson 2012-03-22 14:31:53 UTC 
Fixed readmes and 1 reference to APP_UUID
Comment 11 Meng Bo 2012-03-23 02:47:34 UTC 
verified on latest github site, 
this issue has been fixed.
Note You need to log in before you can comment on or make changes to this bug.