Bug 1305826
| Summary: | [DOCS] Update Database Images Docs to reflect correct usage of environment variables | ||
|---|---|---|---|
| Product: | OpenShift Container Platform | Reporter: | Vikram Goyal <vigoyal> |
| Component: | Documentation | Assignee: | Ashley Hardin <ahardin> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Xingxing Xia <xxia> |
| Severity: | medium | Docs Contact: | Vikram Goyal <vigoyal> |
| Priority: | medium | ||
| Version: | 3.1.0 | CC: | ahardin, aos-bugs, jokerman, mmccomas, vigoyal, wzheng, xxia |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2016-02-23 14:33:49 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: | |||
| Deadline: | 2016-03-15 | ||
|
Description
Vikram Goyal
2016-02-09 10:40:42 UTC
@Vikram, Please review and let me know if this satisfies the request: https://github.com/openshift/openshift-docs/pull/1604 `oc new-app`, environment variables after "-e" should use "," rather than ":" as delimiter. See following tests against both of latest Origin and OSE environments (Origin versions: openshift/oc v1.1.3-32-g5ab601a kubernetes v1.2.0-alpha.7-703-gbc4550d; OSE versions: openshift/oc v3.1.1.903 kubernetes v1.2.0-origin): 1. Use "," $ oc new-app -e ENV1=V1,ENV2=V2 openshift/hello-openshift --name=app1 $ oc env pod/app1-1-d81ia --list # pods app1-1-d81ia, container app1 ENV1=V1 ENV2=V2 2. Use ":" $ oc new-app -e ENV1=V1:ENV2=V2 openshift/hello-openshift --name=app2 $ oc env pod/app2-1-0h9m1 --list # pods app2-1-0h9m1, container app2 ENV1=V1:ENV2=V2 All rhel7 images are released to this registry: registry.access.redhat.com, so cannot use openshift3/mysql-55-rhel7 to get the image; one more thing is mongodb-26-rhel7 is under rhscl namespace like this registry.access.redhat.com/rhscl/mongodb-26-rhel7; same for postgresql-94-rhel7 image should be registry.access.redhat.com/rhscl/postgresql-94-rhel7. Edits complete. Added ifdef statements so that both OSE and Origin users get the right syntax. As of right now, the Doc PR looks good for me, except the note "there's an extra space ..." in the PR. Will move this bug to VERIFIED when the PR is merged. Thanks Ashley and Xingxing. I am not sure why when I ran this on my install it told me that the correct form is: error: environment variables must be of the form key=value:. I have since verified it and it is not giving me that error any more. So the original intent of the bug is no longer there. But it is a good thing that we are fixing the other issue identified by Wenjing. Commits pushed to master at https://github.com/openshift/openshift-docs https://github.com/openshift/openshift-docs/commit/b1fbf6be4c38d325f205408119f6a250ddbb0071 Bug 1305826, edited commands: creating new databases for MySQL, PostgresSQL,mongodb https://github.com/openshift/openshift-docs/commit/946f50fef9c2ca460f50e5b53b93c91aa06c6407 Merge pull request #1604 from ahardin-rh/enviro-variables Bug 1305826, edited commands for creating new databases for MySQL, PostgresSQL, MongoDB Extra space removed and the content is merged. The PR is merged, so move to VERIFIED. Content is now published: https://access.redhat.com/documentation/en/openshift-enterprise/version-3.1/using-images/#configuration-and-usage https://access.redhat.com/documentation/en/openshift-enterprise/version-3.1/using-images/#configuration-and-usage-2 https://access.redhat.com/documentation/en/openshift-enterprise/version-3.1/using-images/#configuration-and-usage-3 |