Skip to content

Add support for grafana/otel-lgtm and LgtmStackContainer from Service Connection #41551

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

Closed

Conversation

eddumelendez
Copy link
Contributor

@eddumelendez eddumelendez commented Jul 17, 2024

Support grafana/otel-lgtm from Docker Compose and LgtmStackContainer
from Testcontainers' Grafana module as a Service Connection for Metrics
and Tracing with Opentelemetry.

The image provides Loki, Grafana, Tempo, Mimir, Prometheus and Opentelemetry Collector installed, which is nice for local development.

This PR use Testcontainers for Java 1.20.0, which contains the brand new grafana module. The new version works well with previous spring boot versions as you can see here, so, would be great to port it to 3.2.x and 3.3.x branches as well.

Thanks!

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 17, 2024
@snicoll snicoll self-assigned this Jul 26, 2024
@snicoll
Copy link
Member

snicoll commented Jul 26, 2024

This PR use Testcontainers for Java 1.20.0, which contains the brand new grafana module. The new version works well with previous spring boot versions as you can see here, so, would be great to port it to 3.2.x and 3.3.x branches as well.

Thanks for the suggestion but our upgrade policy states we won't upgrade to a new feature release in a maintenance release of Spring Boot. I've upgraded to 1.20.0 on main in a separate issue.

@snicoll snicoll added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Jul 29, 2024
@snicoll snicoll added this to the 3.4.0-M2 milestone Jul 29, 2024
snicoll pushed a commit that referenced this pull request Jul 29, 2024
snicoll added a commit that referenced this pull request Jul 29, 2024
@snicoll snicoll closed this in 3da45aa Jul 29, 2024
@snicoll
Copy link
Member

snicoll commented Jul 29, 2024

Thanks again @eddumelendez

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants