Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Message ack timeout #59

Merged
merged 3 commits into from
Aug 15, 2024
Merged

Message ack timeout #59

merged 3 commits into from
Aug 15, 2024

Conversation

Foxcapades
Copy link
Member

Description

Update compute platform to use the new rabbitmq wrapper lib version which enables automatic channel reconnects and message ack timeout config.

Closes #58

Changes

  • Add new queue config option which will enable services to specify rabbitmq message ack timeouts greater than RabbitMQ's default value of 30 min
  • Fix test script

PR Checklist

  • Updated relevant source docs
  • Updated readme / docs
  • Updated dependencies

@Foxcapades Foxcapades added the enhancement New feature or request label Aug 14, 2024
@Foxcapades Foxcapades requested a review from dmgaldi August 14, 2024 18:15
@Foxcapades Foxcapades self-assigned this Aug 14, 2024
@Foxcapades Foxcapades linked an issue Aug 14, 2024 that may be closed by this pull request
Incorporate upstream changes from rabbit wrapper lib. #58
Closed
@Foxcapades Foxcapades merged commit d26fb1a into main Aug 15, 2024
1 check passed
@Foxcapades Foxcapades deleted the issue-58 branch August 15, 2024 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dmgaldi dmgaldi dmgaldi approved these changes

Assignees

@Foxcapades Foxcapades

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Incorporate upstream changes from rabbit wrapper lib.
2 participants
@Foxcapades @dmgaldi