Skip to content

Added support for imdsv2#70

Merged
tak30 merged 4 commits intomainfrom
RTI-17056-imdsv2
Oct 23, 2025
Merged

Added support for imdsv2#70
tak30 merged 4 commits intomainfrom
RTI-17056-imdsv2

Conversation

@tak30
Copy link
Copy Markdown
Contributor

@tak30 tak30 commented Oct 23, 2025

No description provided.

@tak30 tak30 self-assigned this Oct 23, 2025
Copy link
Copy Markdown

@pehuen-rodriguez pehuen-rodriguez left a comment

Choose a reason for hiding this comment

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

LGTM. I think there was also the option of passing metadata to the instance setting http_tokens = optional. Not sure if we had the same issue in other systems we migrated to amz2023 🤔

Copy link
Copy Markdown
Contributor

@raz-adroll raz-adroll left a comment

Choose a reason for hiding this comment

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

LGTM we just need to add some options to the ASG and it will use imdsv1 as other scripts relay in that.

Copy link
Copy Markdown
Contributor

@raz-adroll raz-adroll left a comment

Choose a reason for hiding this comment

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

It was supposed to be an approve and not a comment 🤦

@tak30 tak30 merged commit 73fa38b into main Oct 23, 2025
5 checks passed
@tak30 tak30 deleted the RTI-17056-imdsv2 branch October 23, 2025 14:58
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.

5 participants