-
Notifications
You must be signed in to change notification settings - Fork 192
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CI: Test with RabbitMQ 4.0.x #6649
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6649 +/- ##
==========================================
+ Coverage 77.51% 77.93% +0.43%
==========================================
Files 560 563 +3
Lines 41444 41662 +218
==========================================
+ Hits 32120 32464 +344
+ Misses 9324 9198 -126 ☔ View full report in Codecov by Sentry. |
@@ -24,7 +24,7 @@ ENV SYSTEM_USER="${SYSTEM_USER}" | |||
|
|||
# Install all OS dependencies for notebook server that starts but lacks all | |||
# features (e.g., download as all possible file formats) | |||
ENV DEBIAN_FRONTEND noninteractive | |||
ENV DEBIAN_FRONTEND=noninteractive |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually I like if we see tiny things (hard to say how tiny is tiny, for me it is like one line change not related to the code logic and sure it is 100% correct) like this we clean it up.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The tests all pass, so all good . Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The nightly test also failed and I guess it is because of #6640
Whoops, RMQ and nightly tests are failing due the uv lockfile PR. I'll fix that. |
Let's see if RMQ 4.0 passes the tests.
https://aiida.discourse.group/t/supported-rabbitmq-versions/392/11?u=danielhollas