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

Updated user-guide #680

Merged
merged 1 commit into from
Jul 6, 2017
Merged

Updated user-guide #680

merged 1 commit into from
Jul 6, 2017

Conversation

surajnarwade
Copy link
Contributor

Fixes issue #678. As we no longer supports setting provider by environment
variable, this PR will remove documentation for the same.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jul 5, 2017
@surajnarwade
Copy link
Contributor Author

@cdrage review please :)

You can choose targeted provider either using global option `--provider`, or by setting environment variable `PROVIDER`.
By setting environment variable `PROVIDER` you can permanently switch to OpenShift provider without need to always specify `--provider openshift` option.
If no provider is specified Kubernetes is default provider.
You can choose targeted provider using global option `--provider`, If no provider is specified Kubernetes is default provider.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grammar mistake(s).

Please replace with:

You can choose a targeted provider by using the global option --provider. If no provider is specified, Kubernetes is set by default.

Fixes issue kubernetes#678. As we no longer supports setting provider by environment
variable, this PR will remove documentation for the same.
@cdrage cdrage merged commit 1a2f33f into kubernetes:master Jul 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants