Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

release-tests.yaml: remove (always failing) matrix notification #20742

Merged
merged 1 commit into from
Jun 12, 2024
Merged

release-tests.yaml: remove (always failing) matrix notification #20742

merged 1 commit into from
Jun 12, 2024

Conversation

mguetschow
Copy link
Contributor

Contribution description

For some reason, enabling matrix notifications for release test results does not seem to work: Last example is here.

As discussed with @miri64, this removes the matrix notifications until we find a working setup again.

Testing procedure

Compare changes to reverted commits (below). Merge and wait for next scheduled release test.

Issues/PRs references

This reverts commit aafb9ab and 7e6a464 from #20424.

@mguetschow mguetschow requested a review from miri64 June 11, 2024 11:38
@github-actions github-actions bot added the Area: CI Area: Continuous Integration of RIOT components label Jun 11, 2024
miri64
miri64 approved these changes Jun 12, 2024
View reviewed changes
Copy link
Member

@miri64 miri64 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK, I checked that it just reverts the mentioned commits.

@miri64 miri64 added CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation labels Jun 12, 2024
@miri64 miri64 enabled auto-merge June 12, 2024 13:04
@riot-ci
Copy link

riot-ci commented Jun 12, 2024

Murdock results

✔️ PASSED

178741b release-tests.yaml: remove (always failing) matrix notification

Success Failures Total Runtime
1 0 1 01m:11s

Artifacts

@miri64 miri64 added this pull request to the merge queue Jun 12, 2024
@github-merge-queue github-merge-queue bot merged commit 4aefcd2 into RIOT-OS:master Jun 12, 2024
29 checks passed
@mguetschow mguetschow deleted the gh-actions/remove-matrix branch June 12, 2024 13:12
@mguetschow mguetschow added this to the Release 2024.07 milestone Jul 3, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@miri64 miri64 miri64 approved these changes

Assignees
No one assigned
Labels
Area: CI Area: Continuous Integration of RIOT components CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mguetschow @riot-ci @miri64