Skip to content

Disable observability dev services at test time#54952

Open
brunobat wants to merge 1 commit into
quarkusio:mainfrom
brunobat:no-deafult-grafana-lgtm-on-tests
Open

Disable observability dev services at test time#54952
brunobat wants to merge 1 commit into
quarkusio:mainfrom
brunobat:no-deafult-grafana-lgtm-on-tests

Conversation

@brunobat

@brunobat brunobat commented Jun 19, 2026

Copy link
Copy Markdown
Member

Disable automatic start of the Grafana lgtm at test time.
This has started after #54276

Impact

Low because it only affects tests and I'm not aware of users using this functionality.

Why?

Only extension specific tests are using this functionality but because the dev service starts automatically with quarkus-opentelemetry, this feature extends build time and uses more resources without need.

Other changes

Updated the docs to make them consistent with the automatic start of the Grafana LGTM dev service when quarkus-opentelemetry is present.

@geoand

geoand commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Fine with me, but I'll let @alesj have the final say

@brunobat

Copy link
Copy Markdown
Member Author

Actually, this isn't a breaking change because Martin's PR is part of 3.38 and has not been released yet.

@brunobat brunobat changed the title [Breaking] Disable observability dev services at test time Disable observability dev services at test time Jun 19, 2026
@quarkus-bot

quarkus-bot Bot commented Jun 19, 2026

Copy link
Copy Markdown

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 77bf2eb.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

Warning

There are other workflow runs running, you probably need to wait for their status before merging.

You can consult the Develocity build scans.

@quarkus-bot

quarkus-bot Bot commented Jun 19, 2026

Copy link
Copy Markdown

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 77bf2eb.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@github-actions

Copy link
Copy Markdown

🎊 PR Preview 02f04e3 has been successfully built and deployed to https://quarkus-pr-main-54952-preview.surge.sh/version/main/guides/

  • Images of blog posts older than 3 months are not available.
  • Newsletters older than 3 months are not available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants