add explanation of extended-platform-tests#183
Merged
openshift-merge-robot merged 1 commit intoJan 22, 2020
Conversation
openshift-ci-robot
added
size/L
Contributor
|
Nothing to add. lgtm. Giving time for some more eyes. |
Contributor
|
/lgtm so @soltysh can have a look. |
openshift-ci-robot
added
do-not-merge/hold
openshift-ci-robot
removed
the
do-not-merge/hold
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k, mfojtik, soltysh The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
| Instead of creating mechanisms to allow this that require modification of critical release templates, | ||
| we can instead use the same technique used to create release payloads. | ||
|
|
||
| Every team that wants to contribute to the universal set of tests can do so by creating an image that has an entrypoint |
There was a problem hiding this comment.
We would need a way to also have the test run output the location for bugs and contacts for build-cop. If these are blocking we need escalation points IMO.
| which conforms to the `openshift-tests` CLI definition. | ||
| 1. run-test with dry-run | ||
| 2. run with dry-run and a common set of defined buckets. Help lists which buckets are there. | ||
| Based on that information it is possible to layer all the binaries into a single image with a new entry point that |
Contributor
|
This is not in implementable yet, still reviewing. This is only in proposed |
|
Thanks! /lgtm |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Explain how a testing repo can be a good intermediate step between where we are and where we'd like to be that allows QE to land tests without risking our ability to quickly update levels of kube.
@sttts @mfojtik @soltysh @derekwaynecarr @smarterclayton @jianzhangbjz