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

Fix race condition with fetching short lived token #260

Merged
merged 1 commit into from
Jun 17, 2024
Merged

Conversation

alextu
Copy link
Member

@alextu alextu commented Jun 17, 2024

This somehow worked before (and in our integ test) because the setup action gave enough time to let the request short-lived token return in time 🤷.

This caused a race condition not allowing the short-lived token to be returned in time before the setup finished.
@alextu alextu requested review from bigdaz and cdsap June 17, 2024 17:03
Copy link

Job Summary for Gradle

Demo adding Build Scan® comment to PR :: failing-build-with-comment-on-failure
Gradle Root Project Requested Tasks Gradle Version Build Outcome Build Scan®
kotlin-dsl no-a-real-task 8.8 Build Scan published

Copy link

Job Summary for Gradle

Demo adding Build Scan® comment to PR :: successful-build-with-always-comment
Gradle Root Project Requested Tasks Gradle Version Build Outcome Build Scan®
kotlin-dsl build 8.8 Build Scan published

Copy link

Job Summary for Gradle

Demo adding Build Scan® comment to PR :: failing-build-with-comment-on-failure
Gradle Root Project Requested Tasks Gradle Version Build Outcome Build Scan®
kotlin-dsl no-a-real-task 8.8 Build Scan published

Copy link

Job Summary for Gradle

Demo adding Build Scan® comment to PR :: successful-build-with-always-comment
Gradle Root Project Requested Tasks Gradle Version Build Outcome Build Scan®
kotlin-dsl build 8.8 Build Scan published

@cdsap cdsap merged commit ae74429 into main Jun 17, 2024
128 checks passed
@cdsap cdsap deleted the atual/fix-await branch June 17, 2024 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants