Skip to content

Warn if a docker compose service has no ports#9009

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
smarterclayton:warn_service_missing_ports
May 27, 2016
Merged

Warn if a docker compose service has no ports#9009
openshift-bot merged 1 commit intoopenshift:masterfrom
smarterclayton:warn_service_missing_ports

Conversation

@smarterclayton
Copy link
Contributor

@smarterclayton smarterclayton commented May 24, 2016

Fixes #8943

@smarterclayton smarterclayton mentioned this pull request May 24, 2016
Closed
@smarterclayton
Copy link
Contributor Author

smarterclayton commented May 26, 2016

[test] @csrwng

@csrwng
Copy link
Contributor

csrwng commented May 26, 2016

LGTM

@smarterclayton
Copy link
Contributor Author

smarterclayton commented May 26, 2016

[merge]

@smarterclayton
Copy link
Contributor Author

smarterclayton commented May 26, 2016

[merge]

On Wed, May 25, 2016 at 11:55 PM, OpenShift Bot notifications@github.com
wrote:

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

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#9009 (comment)

@smarterclayton
Copy link
Contributor Author

smarterclayton commented May 26, 2016

[merge]

On Thu, May 26, 2016 at 4:45 AM, OpenShift Bot notifications@github.com
wrote:

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

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#9009 (comment)

@openshift-bot
Copy link
Contributor

openshift-bot commented May 26, 2016 •
edited
Loading

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

@smarterclayton smarterclayton force-pushed the warn_service_missing_ports branch 2 times, most recently from f647b4d to acad55e Compare May 26, 2016 21:52
@openshift-bot
Copy link
Contributor

openshift-bot commented May 26, 2016

Evaluated for origin test up to acad55e

@openshift-bot
Copy link
Contributor

openshift-bot commented May 26, 2016

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

@openshift-bot
Copy link
Contributor

openshift-bot commented May 27, 2016

Evaluated for origin merge up to acad55e

@openshift-bot openshift-bot merged commit 81aecc8 into openshift:master May 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@smarterclayton @csrwng @openshift-bot