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(runners): retry aws metadata token download on Linux #3408

Merged
merged 1 commit into from
Aug 8, 2023

Conversation

sdarwin
Copy link
Contributor

@sdarwin sdarwin commented Aug 3, 2023

Adding more thorough logic to fully retry the aws metadata token download, on Linux. Continuing #3292. See explanation there.

I have tested and it works, however Ubuntu usually succeeds. We will observe further how this goes or if more changes are needed.

@npalm npalm changed the title fix: Retry aws metadata token download on Linux fix(runners): retry aws metadata token download on Linux Aug 8, 2023
Copy link
Member

@npalm npalm left a comment

Choose a reason for hiding this comment

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

@sdarwin thanks!

@npalm npalm merged commit ef46827 into philips-labs:main Aug 8, 2023
npalm pushed a commit that referenced this pull request Aug 8, 2023
🤖 I have created a release *beep* *boop*
---


##
[4.1.0](v4.0.2...v4.1.0)
(2023-08-08)


### Features

* **runners:** add configurable eviction strategy to idle config
([#3375](#3375))
([896f473](896f473))


### Bug Fixes

* **lambda:** bump the aws group in /lambdas with 5 updates
([#3413](#3413))
([1acc8ba](1acc8ba))
* **runners:** retry aws metadata token download on Linux
([#3408](#3408))
([ef46827](ef46827))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: forest-releaser[bot] <80285352+forest-releaser[bot]@users.noreply.github.com>
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.

2 participants