Skip to content
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

feat: add grafana-lgtm module #2660

Merged
merged 11 commits into from
Aug 5, 2024

Conversation

mdelapenya
Copy link
Member

@mdelapenya mdelapenya commented Jul 19, 2024

  • feat: init grafana module
  • chore: refactor layout
  • chore: add connection methods to the container
  • chore: a comprenhensive example using Otel libraries

What does this PR do?

This PR adds a new module for the Grafana LGTM docker container, which contains a single container to collect logs, traces and metrics thanks to an Otel Collector, Tempo, Loki and Prometheus, all into the single container.

The module exposes methods to get the endpoints for all the services in the format <host>:<port>, so please configure your clients accordingly (e.g. adding the http schema if/when needed).

The module also exposes an optional configuration for the admin credentials, in case you need to update the default admin/admin credentials, that the system forces you to update for first time. Passing credentials makes you access the system without updating the password.

The testable example code was taken from https:/grafana/docker-otel-lgtm/tree/main/examples/go, which is also used in the otel go sdk: https://opentelemetry.io/docs/languages/go/getting-started/

Why is it important?

With this module, it's super helpful to verify your observability story is able to send Otel data to an otel collector.

@mdelapenya mdelapenya requested a review from a team as a code owner July 19, 2024 12:45
@mdelapenya mdelapenya added the feature New functionality or new behaviors on the existing one label Jul 19, 2024
@mdelapenya mdelapenya self-assigned this Jul 19, 2024
Copy link

netlify bot commented Jul 19, 2024

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit a651cfc
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/669a8db1a88d5a00087d4a72
😎 Deploy Preview https://deploy-preview-2660--testcontainers-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mdelapenya mdelapenya merged commit 4c5f1bd into testcontainers:main Aug 5, 2024
112 checks passed
@mdelapenya mdelapenya deleted the grafana-lgtm-module branch August 5, 2024 07:48
mdelapenya added a commit that referenced this pull request Aug 5, 2024
* main:
  feat: add grafana-lgtm module (#2660)
  Added valkey module (#2639)
  fix: container.Endpoint and wait.FortHTTP to use lowest internal port (#2641)
  chore: test cleanups (#2657)
  docs: fix compilation of examples (#2656)
  feat: add custom container registry substitutor (#2647)
  fix: couchbase containers intermittently hang on startup (#2650)
  chore(deps): bump Ryuk to 0.8.1 (#2648)
  fix: retry on label error (#2644)
  perf: optimise docker authentication config lookup (#2646)
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Aug 6, 2024
* main:
  chore: print Docker Info labels in banner (testcontainers#2681)
  fix: incorrect parsing of exposedPorts in readiness check (testcontainers#2658)
  feat: add grafana-lgtm module (testcontainers#2660)
  Added valkey module (testcontainers#2639)
  fix: container.Endpoint and wait.FortHTTP to use lowest internal port (testcontainers#2641)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionality or new behaviors on the existing one
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant