Skip to content

remove references to personal repositories #14178

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

Merged
merged 1 commit into from
May 14, 2017

Conversation

bparees
Copy link
Contributor

@bparees bparees commented May 12, 2017

in partial fulfillment of #9899

@bparees
Copy link
Contributor Author

bparees commented May 12, 2017

[test]
[testextended][extended:core(builds)]

@@ -350,8 +350,6 @@ os::cmd::expect_success 'oc delete secret dbsecret'
os::cmd::expect_success_and_text 'oc new-app ruby-helloworld-sample -l app=helloworld -o name' 'service/frontend'
os::cmd::expect_success 'oc delete all -l app=helloworld'
os::cmd::expect_success 'oc delete secret dbsecret'
# create from template with code explicitly set is not supported
os::cmd::expect_failure 'oc new-app [email protected]:mfojtik/sinatra-app-example'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this was a totally broken test so i don't feel much guilt about just removing it outright.

Copy link
Contributor

Choose a reason for hiding this comment

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

I've considered removing os::cmd::expect_failure for this reason...

@bparees
Copy link
Contributor Author

bparees commented May 12, 2017

@openshift/devex ptal

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to f01ec12

@openshift-bot
Copy link
Contributor

Evaluated for origin testextended up to f01ec12

@smarterclayton
Copy link
Contributor

Lgtm

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/1414/) (Base Commit: 4f2b9d7)

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/testextended SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin_extended/385/) (Base Commit: 4f2b9d7) (Extended Tests: core(builds))

@bparees
Copy link
Contributor Author

bparees commented May 12, 2017

[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to f01ec12

@openshift-bot
Copy link
Contributor

openshift-bot commented May 14, 2017

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin/646/) (Base Commit: 8052ada) (Image: devenv-rhel7_6228)

@openshift-bot openshift-bot merged commit d84c74f into openshift:master May 14, 2017
@bparees bparees deleted the external_repos branch May 18, 2017 20:13
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