Skip to content

Conversation

@danielhollas
Copy link
Collaborator

This reverts changes from #7076 which somehow broke the rabbitmq nightly job that runs via pytest -m "requires_rmq" I tried to briefly investigate the root cause but the failures are non-deterministic and depend on testing order. Before I have time to investigate more, let's just revert the changes to fix CI.

btw: this is also important for the release workflow, which also runs the subset of the test suite via pytest -m requires_rmq

@codecov
Copy link

codecov bot commented Nov 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.62%. Comparing base (49af7f0) to head (ef76fcd).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7105      +/-   ##
==========================================
+ Coverage   79.61%   79.62%   +0.02%     
==========================================
  Files         566      566              
  Lines       43546    43546              
==========================================
+ Hits        34663    34669       +6     
+ Misses       8883     8877       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@GeigerJ2 GeigerJ2 left a comment

Choose a reason for hiding this comment

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

LGTM

@danielhollas danielhollas merged commit 9b69202 into aiidateam:main Nov 17, 2025
21 checks passed
@danielhollas danielhollas deleted the revert-requires-rmq branch November 17, 2025 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants