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

clarify 'oc delete' help info, add 'oc delete all -l app=appName' example #8757

Merged
merged 1 commit into from
May 14, 2016

Conversation

sallyom
Copy link
Contributor

@sallyom sallyom commented May 5, 2016

No description provided.

@sallyom sallyom force-pushed the fixDelHelp branch 3 times, most recently from bd79a60 to 25a643f Compare May 6, 2016 17:10
# Delete all resources associated with a running app, includes
# buildconfig,deploymentconfig,service,imagestream,route,pod,
# where 'appName' is listed in 'Labels' of 'oc describe [resource] [resource name]' output.
$ %[1]s delete all -l app=appName
Copy link
Contributor

Choose a reason for hiding this comment

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

#8454 landed today which means we won't use prompts anymore.

@smarterclayton
Copy link
Contributor

Other than removing the $ looks good

@smarterclayton
Copy link
Contributor

Lgtm [merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented May 14, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/5913/) (Image: devenv-rhel7_4198)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 7abc4b8

@openshift-bot openshift-bot merged commit df4cc0b into openshift:master May 14, 2016
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.

4 participants