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

Fix recover due a loss of network connectivity #267

Merged
merged 10 commits into from
Jan 19, 2024
Merged

Fix recover due a loss of network connectivity #267

merged 10 commits into from
Jan 19, 2024

Conversation

jBonoraW
Copy link
Contributor

@jBonoraW jBonoraW commented Jan 19, 2024 •
edited
Loading

🎩 What?

  • Version alfa to test rele worker stuck in consumption due a loss of network connectivity.
  • Added logs.

🤔 Why?

  • We want to test in a production environment this fix for consumption.
  • We added print since logger not working as expected. We will improve this in the release.

🔗 Related issue

@jBonoraW jBonoraW requested a review from igarridot January 19, 2024 08:18
@jsansaloni jsansaloni merged commit e1a6c6c into master Jan 19, 2024
4 checks passed
@jsansaloni jsansaloni deleted the debug-retry-connection-bug branch January 19, 2024 08:36
@jBonoraW jBonoraW mentioned this pull request Jan 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jsansaloni jsansaloni jsansaloni approved these changes

@igarridot igarridot Awaiting requested review from igarridot

@daniel-ruiz daniel-ruiz Awaiting requested review from daniel-ruiz daniel-ruiz is a code owner

@EmilioCarrion EmilioCarrion Awaiting requested review from EmilioCarrion EmilioCarrion is a code owner

@AdamjGoddard AdamjGoddard Awaiting requested review from AdamjGoddard AdamjGoddard is a code owner

@jonasae jonasae Awaiting requested review from jonasae jonasae is a code owner

Assignees

@jBonoraW jBonoraW

@jsansaloni jsansaloni

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jBonoraW @jsansaloni