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.

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

Increase the TM SIGNAL_RESPONSE_TIMEOUT #3619

Merged

Conversation

ldoktor
Copy link
Contributor

@ldoktor ldoktor commented Apr 24, 2024

the 100s timeout is too short for some systems (reported on slightly outdated aarch64).

Note I tested it with 200s which was still failing. With 300s it succeeded 5/5.

the 100s timeout is too short for some systems (reported on slightly
outdated aarch64).

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
@dbutenhof dbutenhof added bug Agent Signal Of and relating to signal integration work Tool Meister Of and relating to the Tool Meister sub-system labels Apr 24, 2024
Copy link
Member

@dbutenhof dbutenhof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Certainly seems reasonable and safe. The frightening part about this is that nobody's attempted to produce a new Agent release since Nick left. 😁

@ldoktor
Copy link
Contributor Author

ldoktor commented Apr 24, 2024

Certainly seems reasonable and safe. The frightening part about this is that nobody's attempted to produce a new Agent release since Nick left. 😁

Hmm, I didn't know he left. Where will the new coprs live?

@dbutenhof
Copy link
Member

Certainly seems reasonable and safe. The frightening part about this is that nobody's attempted to produce a new Agent release since Nick left. 😁

Hmm, I didn't know he left. Where will the new coprs live?

Nick retired, last August. He handed the keys to his COPR over to @webbnh, and left detailed instructions on his release process ... it's just that nobody's tried it without him. 🤹🏻

Copy link
Member

@webbnh webbnh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@webbnh webbnh merged commit 9a76f13 into distributed-system-analysis:main Apr 24, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agent bug Signal Of and relating to signal integration work Tool Meister Of and relating to the Tool Meister sub-system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants