Bug 802235

Summary: need change env variables to 'OPENSHIFT_GEAR' in github examples' README file and code
Product: OKD Reporter: Meng Bo <bmeng>
Component: ContainersAssignee: Dan McPherson <dmcphers>
Status: CLOSED CURRENTRELEASE QA Contact: libra bugs <libra-bugs>
Severity: medium Docs Contact:
Priority: medium    
Version: 1.xCC: dmcphers, gsigrisi, mpatel, xtian
Target Milestone: ---Keywords: Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: devenv-stage_147 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-04-13 18:35:14 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

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.