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

Sync latest image-streams and templates #6179

Merged
merged 1 commit into from
Dec 13, 2017

Conversation

alexcern
Copy link
Contributor

This mainly adds the python 3.6 image-stream following openshift/origin#17281

@openshift-ci-robot openshift-ci-robot added needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Nov 20, 2017
@rh-atomic-bot
Copy link

Can one of the admins verify this patch?
I understand the following commands:

  • bot, add author to whitelist
  • bot, test pull request
  • bot, test pull request once

@michaelgugino
Copy link
Contributor

/ok-to-test

@sdodson does this look okay to you?

@openshift-ci-robot openshift-ci-robot removed the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Nov 28, 2017
@sdodson
Copy link
Member

sdodson commented Nov 28, 2017

Yup, looks fine.
/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 28, 2017
@alexcern
Copy link
Contributor Author

alexcern commented Dec 6, 2017

/retest

@alexcern
Copy link
Contributor Author

@sdodson thanks for the review.

I'm not sure what to do next: tests are green except a note about "Required Github CI test is not green: fedora/27/atomic" but it looks like that particular test is missing altogether in this PR (while it's present in other PRs I checked). My last /retest command did not seem to help with this. Any suggestion to unblock the situation?

@michaelgugino michaelgugino reopened this Dec 12, 2017
@mtnbikenc mtnbikenc merged commit 1328147 into openshift:master Dec 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants