Skip to content

Add service connection from Testcontainers Zipkin #35107

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

Conversation

eddumelendez
Copy link
Contributor

No description provided.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Apr 22, 2023
@dsyer
Copy link
Member

dsyer commented Apr 23, 2023

It’s already supported in spring-boot-docker-compose, so I vote for this. Maybe you could add prometheus as well (or maybe prometheus isn't often used as a service endpoint - it scrapes apps)?

@eddumelendez
Copy link
Contributor Author

or maybe prometheus isn't often used as a service endpoint - it scrapes apps

TBH, I'm not sure but given the prometheus push configuration is already possible in spring-boot it totally makes sense to add support for service connection.

I would work on this later this week but if anyone interested on contributing before that happen that would be awesome

@philwebb philwebb added type: enhancement A general enhancement theme: containers Testcontainers, Docker Compose and Buildpack features and removed status: waiting-for-triage An issue we've not yet triaged labels Apr 25, 2023
@wilkinsona wilkinsona added this to the 3.1.x milestone May 1, 2023
@mhalbritter mhalbritter self-assigned this May 2, 2023
mhalbritter pushed a commit that referenced this pull request May 3, 2023
@mhalbritter
Copy link
Contributor

Thanks a lot!

@mhalbritter mhalbritter modified the milestones: 3.1.x, 3.1.0-RC2 May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme: containers Testcontainers, Docker Compose and Buildpack features type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants