Skip to content

support AZ ID in TMDS - #4848

Merged
sparrc merged 1 commit into
aws:devfrom
Vanggie:availability_zone_id
Jan 30, 2026
Merged

sparrc merged 1 commit into
aws:devfrom
Vanggie:availability_zone_id

Conversation

@Vanggie

@Vanggie Vanggie commented Jan 20, 2026

Copy link
Copy Markdown
Contributor

Summary

In this PR, we populate AZ id from IMDS to TMDS.

Implementation details

Agent queries IMDS for AZ ID and saves it to memory.
Later the AZ ID is passed to TMDS server and sent as response.

Testing

make test and make run-integ-tests

New tests cover the changes:
no

Description for the changelog

  • Feature - Support AZ ID for TMDS

Additional Information

Does this PR include breaking model changes? If so, Have you added transformation functions?

Does this PR include the addition of new environment variables in the README?

Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@Vanggie
Vanggie requested a review from a team as a code owner January 20, 2026 22:09
@Vanggie
Vanggie force-pushed the availability_zone_id branch 19 times, most recently from f7fefb3 to ab1635c Compare January 26, 2026 22:31
Comment thread agent/app/agent.go Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can this one also use the common func?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

For target lifecycle it used a different retry strategy (retry 3 times), is it ok to use the common func?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

yeah that's likely arbitrary, let's just normalize and use the same

Comment thread agent/app/agent.go Resolved
@Vanggie
Vanggie force-pushed the availability_zone_id branch 8 times, most recently from ea89ac6 to ea0fa6c Compare January 27, 2026 19:28
@Vanggie
Vanggie force-pushed the availability_zone_id branch 8 times, most recently from 5a79da2 to 9fe00bc Compare January 27, 2026 23:20
sparrc
sparrc previously approved these changes Jan 27, 2026
Comment thread agent/app/agent_test.go Resolved
@Vanggie
Vanggie force-pushed the availability_zone_id branch from 3e03088 to 62279f9 Compare January 28, 2026 19:43
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