Skip to content

Remove janus_core as dependency, instead require external setup#270

Draft
oerc0122 wants to merge 1 commit into
stfc:mainfrom
oerc0122:remove-janus-dep
Draft

Remove janus_core as dependency, instead require external setup#270
oerc0122 wants to merge 1 commit into
stfc:mainfrom
oerc0122:remove-janus-dep

Conversation

@oerc0122
Copy link
Copy Markdown
Collaborator

No description provided.

@oerc0122 oerc0122 force-pushed the remove-janus-dep branch 5 times, most recently from 6c242a0 to 0ccadd9 Compare March 26, 2026 14:04
@ajjackson
Copy link
Copy Markdown
Contributor

I think you could clear the RabbitMQ complaints out of test output by changing the workflow to use 3.8.14 instead of "latest".

I can't find reference to RabbitMQ version constraints in the AiiDA docs any more, so it might be that the latest aiida-core is happier with an up-to-date RabbitMQ.

@ElliottKasoar
Copy link
Copy Markdown
Member

I think you could clear the RabbitMQ complaints out of test output by changing the workflow to use 3.8.14 instead of "latest".

I can't find reference to RabbitMQ version constraints in the AiiDA docs any more, so it might be that the latest aiida-core is happier with an up-to-date RabbitMQ.

We're unfortunately not currently compatible with the last few releases of aiida-core, but it also seems to still have the same compatibility issues with the default configuration, the note is just a bit hidden away: https://aiida.readthedocs.io/projects/aiida-core/en/latest/installation/troubleshooting.html#rabbitmq-incompatibility.

@ajjackson
Copy link
Copy Markdown
Contributor

The default RabbitMQ timeout of 30 mins should be fine for CI, so another option is to disable the warning with verdi config set warnings.rabbitmq_version False

@oerc0122 oerc0122 force-pushed the remove-janus-dep branch from 315a004 to 5ec2afc Compare May 28, 2026 09:33
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.

3 participants