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

Note that new-app --env flag doesn't apply to templates #9583

Merged
merged 1 commit into from
Jul 1, 2016

Conversation

rhcarvalho
Copy link
Contributor

Fixes #9508.

@rhcarvalho
Copy link
Contributor Author

[test] @bparees PTAL

@bparees
Copy link
Contributor

bparees commented Jun 27, 2016

lgtm, thanks @rhcarvalho!
[merge]

@bparees
Copy link
Contributor

bparees commented Jun 27, 2016

[merge]

@bparees
Copy link
Contributor

bparees commented Jun 28, 2016

[merge]

On Mon, Jun 27, 2016 at 8:50 PM, OpenShift Bot [email protected]
wrote:

continuous-integration/openshift-jenkins/merge FAILURE (
https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/5479/)


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#9583 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AEvl3k2KLzhRUMkmrLLPhj1NYHigLMmGks5qQG_WgaJpZM4I_CEQ
.

Ben Parees | OpenShift

@bparees
Copy link
Contributor

bparees commented Jun 28, 2016

[merge]

Ben Parees | OpenShift
On Jun 28, 2016 05:15, "OpenShift Bot" [email protected] wrote:

continuous-integration/openshift-jenkins/merge ABORTED (
https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/5485/)


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#9583 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AEvl3tF9Us-kqtLigArR9ftD5rXBp-fMks5qQOY6gaJpZM4I_CEQ
.

@bparees
Copy link
Contributor

bparees commented Jun 28, 2016

networking test is disabled.
[merge]

@openshift-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 29, 2016
@bparees
Copy link
Contributor

bparees commented Jun 29, 2016

@rhcarvalho needs a rebase

@rhcarvalho rhcarvalho force-pushed the issue9508-new-app-env branch from 695c0a5 to e0d7595 Compare June 30, 2016 13:32
@rhcarvalho
Copy link
Contributor Author

Strangely, rebased and there were no conflicts...

@bparees
Copy link
Contributor

bparees commented Jun 30, 2016

might have been auto-resolved, that has happened to me.

@bparees
Copy link
Contributor

bparees commented Jun 30, 2016

[merge]

@openshift-bot openshift-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 30, 2016
@bparees
Copy link
Contributor

bparees commented Jun 30, 2016

[test]

1 similar comment
@bparees
Copy link
Contributor

bparees commented Jul 1, 2016

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to e0d7595

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/5659/)

@bparees
Copy link
Contributor

bparees commented Jul 1, 2016

[merge]

@bparees
Copy link
Contributor

bparees commented Jul 1, 2016

(flake #9670 opened)

@openshift-bot
Copy link
Contributor

openshift-bot commented Jul 1, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/5707/) (Image: devenv-rhel7_4504)

@bparees
Copy link
Contributor

bparees commented Jul 1, 2016

delightful.. a new flake.
ERROR: gave up waiting for https://127.0.0.1:38443/healthz

[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to e0d7595

@openshift-bot openshift-bot merged commit 2229122 into openshift:master Jul 1, 2016
@rhcarvalho rhcarvalho deleted the issue9508-new-app-env branch July 2, 2016 10:45
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.

3 participants