Skip to content

Conversation

@jpalomaki
Copy link

@jpalomaki jpalomaki commented Sep 29, 2025

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Implements support for fetching AWS ECS task metadata (e.g. TaskARN) from the standard ECS metadata endpoint v4, and exposing that metadata as properties in the Spring environment, for use e.g. in logs or OTEL metadata.

💡 Motivation and Context

Fixes #1477

💚 How did you test it?

TBD

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • I updated reference documentation to reflect the change
  • All tests passing
  • No breaking changes

🔮 Next steps

TBD

@jpalomaki
Copy link
Author

@maciejwalkowiak Looks like the (automatically issued) PR label is wrong, this isn't a dependency upgrade.

@jpalomaki jpalomaki changed the title First version of ECS task metadata lookup Add support for exposing ECS task metadata as Spring properties Sep 29, 2025
Copy link
Contributor

@maciejwalkowiak maciejwalkowiak left a comment

Choose a reason for hiding this comment

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

Thanks @jpalomaki and apologies for late reply. It got out of sync with main and i left few comments.

Before we can merge it, we definitely need tests.

* @since 3.5.0
*/
@AutoConfiguration
@ConditionalOnProperty(name = "spring.cloud.aws.ecs-task-metadata.enabled", havingValue = "true", matchIfMissing = false)
Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps it should be disabled by default? It will add some time to startup so it should be an opt-in feature.

@@ -0,0 +1,23 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a need for starter considering it does not bring any extra dependencies?

@maciejwalkowiak maciejwalkowiak added type: feature Integration with a new AWS service or bigger change in existing integration status: waiting-for-feedback Waiting for feedback from issuer and removed type: dependency-upgrade Dependency version bump labels Oct 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: waiting-for-feedback Waiting for feedback from issuer type: feature Integration with a new AWS service or bigger change in existing integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for exposing ECS (Fargate) task metadata

2 participants