Skip to content

Updated CLI content with some restructuring [tech_review]#65

Merged
nhr merged 1 commit intoopenshift:masterfrom
Bilhar-rh:os_cli
Jan 14, 2015
Merged

Updated CLI content with some restructuring [tech_review]#65
nhr merged 1 commit intoopenshift:masterfrom
Bilhar-rh:os_cli

Conversation

@Bilhar-rh
Copy link
Copy Markdown

@nhr @danmcp My first take on some rework of the CLI content that was migrated. A couple of questions though:

  1. What is osc? How do you get it/install it?
  2. In "Configuring CLI" it mentions that "configuration options are well documented." Where? Shouldn't they be documented here? If not, can you point to where they are documented?

@nhr
Copy link
Copy Markdown
Member

nhr commented Jan 13, 2015

@CowboysFan:

  1. Good question. In reality, it is a symbolic link to the openshift binary, both of which must be available in $PATH. When invoked in this context, the binary is clever enough to behave as though the user is running openshift cli. However, until there is an installer for OpenShift 3, this symlink must be created manually. We can document that, but the situation is hopefully temporary.
  2. Sorry, thinking like an engineer, not a human :-) osc --help returns all of the possible arguments to osc.

@nhr
Copy link
Copy Markdown
Member

nhr commented Jan 13, 2015

@CowboysFan let me know if you want to update your PR based on that feedback or merge it as-is.

@Bilhar-rh
Copy link
Copy Markdown
Author

@nhr Thanks for the explanation. If no. 1 is temporary then I think we'll leave that as-is. As for no. 2 it sounds like I have it covered. If you can ack the tech_review, then please merge, or I'll be happy to merge after ack.

nhr added a commit that referenced this pull request Jan 14, 2015
Updated CLI content with some restructuring [tech_review]
@nhr nhr merged commit 716b3ad into openshift:master Jan 14, 2015
@Bilhar-rh Bilhar-rh deleted the os_cli branch January 15, 2015 00:03
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

Successfully merging this pull request may close these issues.

2 participants