⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@micoleaoo
Copy link
Collaborator

@micoleaoo micoleaoo commented Oct 20, 2025

note:
ci.yaml run fails because tests are designed for docker type container.
This PR has been tested against usegalaxy version utilizing new Pulsar TESP runner.
The container will be reverted back to docker and pulsar.url back to "http://pulsar_rest:8913" after final review of this PR.

  1. Added two steps to ci.yaml for robustness
  2. Added aio_pika module for sending / recieving messages asynchronously over RabbitMQ
  3. Fixed singularity volumes
  4. Increased timeout for job completion

other:

  • typo fixes
  • formatting

@BorisYourich please add additional comments if necessary

….py, added and increased timeout in settings.toml, updated error handling in pulsar_operations.py to catch asyncio.TimeoutError and treat it as a PulsarLayerConnectionError
@micoleaoo micoleaoo requested a review from martenson January 21, 2026 09:36
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