-
-
Notifications
You must be signed in to change notification settings - Fork 354
Add support for exposing ECS task metadata as Spring properties #1485
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
base: main
Are you sure you want to change the base?
Conversation
|
@maciejwalkowiak Looks like the (automatically issued) PR label is wrong, this isn't a dependency upgrade. |
c60016d to
6194fd4
Compare
maciejwalkowiak
left a comment
There was a problem hiding this 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) |
There was a problem hiding this comment.
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"?> | |||
There was a problem hiding this comment.
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?
📢 Type of change
📜 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
🔮 Next steps
TBD