Skip to content

Conversation

@mtnbikenc
Copy link
Member

Backports #6361

@mtnbikenc mtnbikenc requested a review from sdodson December 12, 2017 13:54
@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Dec 12, 2017
@mtnbikenc mtnbikenc changed the title Playbook Consolidation - byo/config.yml [3.8] Playbook Consolidation - byo/config.yml Dec 12, 2017
@sdodson
Copy link
Member

sdodson commented Dec 12, 2017

We're working to get 3.8 deployed to the online clusters with what's currently in release-3.8 then I had intended to merge all changes from master back into release-3.8. Should we just wait for that or is this a critical change that can't wait for that?

@mtnbikenc
Copy link
Member Author

This PR needs #6440 merged.

@mtnbikenc
Copy link
Member Author

@sdodson Not all of the playbook consolidation PRs were on 3.8 and I was gettings issues from QE on their testing. Fine with me to wait, just keep in mind not all the playbooks are rearranged.

@openshift-ci-robot
Copy link

@mtnbikenc: The following tests failed, say /retest to rerun them all:

Test name Commit Details Rerun command
ci/openshift-jenkins/logging dd30310 link /test logging
ci/openshift-jenkins/tox dd30310 link /test tox
ci/openshift-jenkins/install dd30310 link /test install
ci/openshift-jenkins/containerized dd30310 link /test containerized
ci/openshift-jenkins/system-containers dd30310 link /test system-containers
ci/openshift-jenkins/extended_conformance_install_crio dd30310 link /test crio

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@mtnbikenc mtnbikenc self-assigned this Dec 12, 2017
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 12, 2017
@openshift-merge-robot
Copy link
Contributor

@mtnbikenc PR needs rebase

@mtnbikenc
Copy link
Member Author

Closing in favor of #6452

@mtnbikenc mtnbikenc closed this Dec 12, 2017
@mtnbikenc mtnbikenc deleted the release-3.8-consolidate-deploy-cluster branch December 12, 2017 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. 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.

4 participants