Skip to content

Increase retries for github-auth to allow self healing for up to 10 minutes#1681

Merged
ccwienk merged 1 commit into
masterfrom
increase-retries
Jul 23, 2026
Merged

Increase retries for github-auth to allow self healing for up to 10 minutes#1681
ccwienk merged 1 commit into
masterfrom
increase-retries

Conversation

@Michael5601

Copy link
Copy Markdown
Contributor

TCP connectivity between GitHub Actions runners and the OIDC Federation service is intermittently disrupted, causing read timeouts in the github-auth action. Investigation across ~50 failing runs shows the connection self-heals in as little as 20 seconds but can take over 10 minutes. Only one of 50 runs failed to recover within that window.

This PR increases the default retries from 3 to 12 as a workaround until the root cause is addressed. With the default backoff settings (backoff-base=3, backoff-cap=60), this allows up to ~580 seconds (~9.7 minutes) of cumulative wait time before giving up and thus reduces the amount of failed workflows.

Release note:


@gardener-prow gardener-prow Bot added do-not-merge/needs-kind Indicates a PR lacks a `kind/foo` label and requires one. cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jul 23, 2026
@gardener-prow gardener-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Jul 23, 2026
@gardener-prow

gardener-prow Bot commented Jul 23, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ccwienk

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gardener-prow gardener-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 23, 2026
@gardener-prow

gardener-prow Bot commented Jul 23, 2026

Copy link
Copy Markdown

LGTM label has been added.

DetailsGit tree hash: 91a06cc6e2518119dcd6c6d68ea33cdef8a57588

@ccwienk
ccwienk merged commit 69593d2 into master Jul 23, 2026
22 of 23 checks passed
@ccwienk
ccwienk deleted the increase-retries branch July 23, 2026 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. do-not-merge/needs-kind Indicates a PR lacks a `kind/foo` label and requires one. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants