HDDS-15211. Automatically create a github discussion for junit failures on master#10238
Draft
errose28 wants to merge 1 commit into
Draft
HDDS-15211. Automatically create a github discussion for junit failures on master#10238errose28 wants to merge 1 commit into
master#10238errose28 wants to merge 1 commit into
Conversation
master
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
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.
What changes were proposed in this pull request?
Draft generated with AI as an example to see if this is helpful.
We often do not check the master branch for flaky tests to proactively track them under HDDS-5626 and tag them with
@Flaky. Usually they don't get tagged until they disrupt PRs, but even then tests are frequently just rerun without bothering to tag them going forward. This can cause the reliability of master to slowly degrade over time, until it gets bad enough that we inspect a lot of past runs and add lots of@Flakytags all at once.To help proactively tag these flaky tests, this PR contains a github actions job that will create a new github discussion for each test run on master that has junit failures. The current draft is sending this to the
Generalcategory, but we could create a different category for these.Format of the discussion would look like this:
Other alternatives were considered but dropped due to complexity:
@Flakyannotation to the test. The corresponding Jira would then be filed by whoever reviews and merged the change.What is the link to the Apache JIRA
HDDS-15211
How was this patch tested?
I haven't done a dry run of this on my fork yet, this PR is mostly to see if the community is interested in this. If so I can proceed with testing.