build: delete duplicate jobs for grace/litmus, update nightly workflow #22220
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
|
Looks like you're getting build failures from #22221. Rebasing on the latest master should fix. |
danxmoran
force-pushed
the
dm-consolidate-litmus-and-grace
branch
2 times, most recently
from
8f7a8e8 to
8f44969
Compare
codyshepherd
approved these changes
Aug 17, 2021
danxmoran
force-pushed
the
dm-consolidate-litmus-and-grace
branch
from
8f44969 to
7e57a87
Compare
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.
Ever since #22186, litmus tests have been passing against merges to
masterbut failing in our nightly builds. I investigated and found that the way litmus is invoked in the nightly build caused it to pull & test thequay.io/influxdb/influxdb2:nightlyimage. This image is:Rather than pour more energy into litmus, I've updated the nightly workflow to use the same litmus-test process as the workflow that runs per-commit.
I made a similar change for our grace tests because I've noticed they can be flaky in nightly runs, I think because of timing issues when the grace suite tries to launch
influxdin Docker vs. us launchinginfluxdas a pre-step before the tests.