Skip to content

Conversation

@tak30
Copy link
Contributor

@tak30 tak30 commented Oct 23, 2025

No description provided.

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

@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
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
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.

6 participants