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

feat(runner): allow linux starter-runner script to retrieve labels without with IMDSv2 tags option #2764

Merged
merged 13 commits into from
Mar 10, 2023

Conversation

MGSousa
Copy link
Contributor

@MGSousa MGSousa commented Dec 14, 2022

Allow to use the default start-runner script in Linux to be acknowledge with Instance Metadata Service Tags as an option to be settled out (to describe or to use an endpoint to fetch them), without the need for custom starter scripts.

  • Option only available for linux
  • Example available in mult-runner example

@npalm npalm self-requested a review December 14, 2022 15:31
modules/runners/main.tf Outdated Show resolved Hide resolved
@npalm
Copy link
Member

npalm commented Jan 5, 2023

@MGSousa can you update the PR to merge to main (and rebase) we stopped the develop branch. See also the announcements in the readme. Sorry for the incovience.

@MGSousa MGSousa changed the base branch from develop to main January 5, 2023 23:52
@github-actions
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the Stale label Feb 13, 2023
@MGSousa
Copy link
Contributor Author

MGSousa commented Feb 13, 2023

@npalm Why is this PR stale?
I already rebased and merged to the main branch on 13 January.

@github-actions github-actions bot removed the Stale label Feb 14, 2023
@npalm npalm self-requested a review March 1, 2023 13:51
@npalm npalm changed the title feat: Acknowledge starter-runner with IMDSv2 tags option feat(runner): allow linux starter-runner script to retrieve labels without with IMDSv2 tags option Mar 1, 2023
npalm and others added 2 commits March 1, 2023 15:39
- Fix start script to read ssm config tag via SDK
- Update multi runner example, one runner is now using SDK instead of meta data tags
@npalm
Copy link
Member

npalm commented Mar 1, 2023

@MGSousa Thanks for you work, I made a few minor adjustments on your branch. Please can you cross check the changes?

@MGSousa
Copy link
Contributor Author

MGSousa commented Mar 1, 2023

@npalm Looks good! Thanks

@npalm npalm merged commit 0d8a74c into philips-labs:main Mar 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants