Skip to content

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