Exnet ff3 #48
Open
Exnet ff3 #48
Conversation
This file contains 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
Instead of defining a second job matrix for external network E2E deployments and tests, include those jobs in the E2E-Full matrix. This results in adding a post-mortem step for external E2E tests. It also de-duplicates the workflow, making maintenance easier. Signed-off-by: Daniel Farrell <[email protected]>
Include new external network deployments and tests in the periodic tests against devel meant to find flaky failures. Signed-off-by: Daniel Farrell <[email protected]>
mkolesnik
suggested changes
Apr 3, 2022
There was a problem hiding this comment.
I propose another approach, let's create e2e-experimental which hosts all experimental features (wireguard, extnet, whatever else).
It can be a clone of this one and trigger by the same ready-to-test label, but we'll have a clear differentiation of jobs that are mandatory for merge and ones that arent.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.