-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Set actions client timeout to 5 minutes, add logging to client #3103
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's ship it
Any chance for a new release including this patch soon? This is causing a headache for my org. |
Hey @audunsolemdal, This PR is a mitigation to help recover the client in case of network issues. The best course of action is to actually troubleshoot what is filtering packets. We can do a patch release after we confirm this PR mitigates the issue. Would you be willing to test and confirm that the issue is mitigated with the canary image? To do so, you should:
|
Ok, I will update YAML specs now and report back. Thanks for the guidance. |
I keep getting this error
even though I have updated the YAML specs in both places. Am I missing something? |
Are you using an OCI chart or a path to the chart? |
Fixes potential problems of not timing out the request on actions client.
Likely fixes #3100