quarkus icon indicating copy to clipboard operation
quarkus copied to clipboard

Observability Dev UI Card and Log Console

Open melloware opened this issue 1 year ago • 12 comments

Fix #41241

Adds DEVUI card and Console logger for the LGTM in the Dev UI.

Card

image

Log console:

image

melloware avatar Jun 18 '24 12:06 melloware

/cc @cescoffier (devui), @phillip-kruger (devui)

quarkus-bot[bot] avatar Jun 18 '24 12:06 quarkus-bot[bot]

@brunobat I just pre-emptively updated the PR to use the two new property names for grafana.endpoint and otel-collector.url

melloware avatar Jun 18 '24 12:06 melloware

Excellent. Will make my best to make it available today.

brunobat avatar Jun 18 '24 12:06 brunobat


:waning_crescent_moon: This workflow status is outdated as a new workflow run has been triggered.


Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 24ba713398f40db205893117d52cf56e746b668a.

:white_check_mark: 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.

You can consult the Develocity build scans.

quarkus-bot[bot] avatar Jun 18 '24 12:06 quarkus-bot[bot]

Just a question for now, what do we do with these DevUI cards, when we add / have more then just LGTM? One for each, or a joined card ... ?

alesj avatar Jun 18 '24 13:06 alesj

Up to you! We can create the Card as "LGTM" instead of "Observability" etc.

We can create many cards or 1 single card with many values.

melloware avatar Jun 18 '24 13:06 melloware

Would you prefer I rename this card "LGTM" ?

melloware avatar Jun 18 '24 13:06 melloware

@alesj check this out. @phillip-kruger it won't let me rename the Card Title it looks like it always uses the extension name? So if one extension were to produce many cards they would all have the same name I think?

image

melloware avatar Jun 18 '24 14:06 melloware


:waning_crescent_moon: This workflow status is outdated as a new workflow run has been triggered.


Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 1c8bf754e35fddddaab579d98c83b6c96e4a0bfe.

:white_check_mark: 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.

You can consult the Develocity build scans.

quarkus-bot[bot] avatar Jun 18 '24 15:06 quarkus-bot[bot]

Every extension get's one card. (Else we might have too many cards). You can create multiple pages (sub menus in the maxi view of a card)

phillip-kruger avatar Jun 19 '24 00:06 phillip-kruger

Thanks @phillip-kruger that makes sense.

melloware avatar Jun 19 '24 00:06 melloware


:waning_crescent_moon: This workflow status is outdated as a new workflow run has been triggered.


Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 42616ae66118e2df15bf78b5b91456c1e2cd1c72.

:white_check_mark: 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.

You can consult the Develocity build scans.

quarkus-bot[bot] avatar Jun 26 '24 23:06 quarkus-bot[bot]

@brunobat i merged your changes from your recently accepted PR but it won't let me squash them now?

melloware avatar Jul 12 '24 18:07 melloware

You should have rebased...

brunobat avatar Jul 13 '24 15:07 brunobat

I made the mistake of fixing a merge conflict in the GitHub editor and that does a merge instead of rebase

melloware avatar Jul 14 '24 00:07 melloware

@brunobat i fixed it back to a squash commit!

melloware avatar Jul 14 '24 12:07 melloware


:waning_crescent_moon: This workflow status is outdated as a new workflow run has been triggered.


Status for workflow Quarkus Documentation CI

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

Failing Jobs

:warning: Artifacts of the workflow run were not available thus the report misses some details.

Status Name Step Failures Logs Raw logs Build scan
Documentation Build Build :warning: Check → Logs Raw logs :construction:

quarkus-bot[bot] avatar Jul 14 '24 12:07 quarkus-bot[bot]


:waning_crescent_moon: This workflow status is outdated as a new workflow run has been triggered.


Status for workflow Quarkus CI

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

Failing Jobs

Status Name Step Failures Logs Raw logs Build scan
Initial JDK 17 Build Build Failures Logs Raw logs :mag:

You can consult the Develocity build scans.

Failures

:gear: Initial JDK 17 Build #

- Failing: extensions/observability-devservices/deployment 
! Skipped: docs 

:package: extensions/observability-devservices/deployment

Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.13.0:compile (default-compile) on project quarkus-observability-devservices-deployment: Compilation failure /home/runner/work/quarkus/quarkus/extensions/observability-devservices/deployment/src/main/java/io/quarkus/observability/deployment/ObservabilityDevServiceProcessor.java:[166,17] cannot find symbol symbol: variable configBuildProducer location: class io.quarkus.observability.deployment.ObservabilityDevServiceProcessor

quarkus-bot[bot] avatar Jul 14 '24 12:07 quarkus-bot[bot]

Docs updated

melloware avatar Jul 15 '24 11:07 melloware


:waning_crescent_moon: This workflow status is outdated as a new workflow run has been triggered.


Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 027b565212ba90bfe3c162ed2f66eccaebae3c90.

:white_check_mark: 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[bot] avatar Jul 15 '24 12:07 quarkus-bot[bot]


:waning_crescent_moon: This workflow status is outdated as a new workflow run has been triggered.


Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 027b565212ba90bfe3c162ed2f66eccaebae3c90.

:white_check_mark: 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.

quarkus-bot[bot] avatar Jul 15 '24 12:07 quarkus-bot[bot]

I tested the PR on a local project and I found a few problems:

  1. We were missing the new Metrics signal... I created this PR to fix the issue: https://github.com/quarkusio/quarkus/pull/41903
  2. There are resources that should be registered in order not to be tracked in the traces. I imagine these are new static resources. See: Screenshot 2024-07-15 at 17 43 05
  3. The PR I created above seems to break the card. See: Screenshot 2024-07-15 at 17 47 43

brunobat avatar Jul 15 '24 16:07 brunobat

@phillip-kruger didn't we run into something like before where static resources were breaking the resolving of the card?

melloware avatar Jul 15 '24 16:07 melloware

@phillip-kruger didn't we run into something like before where static resources were breaking the resolving of the card?

Yes, if the extension has it's own web resources that also use WebJarBuildItem. Is that the case here ?

phillip-kruger avatar Jul 16 '24 01:07 phillip-kruger


:waning_crescent_moon: This workflow status is outdated as a new workflow run has been triggered.


Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 8e2682cd9e30f6d09b96e7bf01e639a603ab6e1d.

:white_check_mark: 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[bot] avatar Jul 16 '24 16:07 quarkus-bot[bot]


:waning_crescent_moon: This workflow status is outdated as a new workflow run has been triggered.


Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 8e2682cd9e30f6d09b96e7bf01e639a603ab6e1d.

:white_check_mark: 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.

quarkus-bot[bot] avatar Jul 16 '24 16:07 quarkus-bot[bot]


:waning_crescent_moon: This workflow status is outdated as a new workflow run has been triggered.


Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 5a60c968c4d2049edc392bfa92ea3e8864e8331c.

:white_check_mark: 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[bot] avatar Jul 26 '24 01:07 quarkus-bot[bot]


:waning_crescent_moon: This workflow status is outdated as a new workflow run has been triggered.


Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 5a60c968c4d2049edc392bfa92ea3e8864e8331c.

:white_check_mark: 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.

quarkus-bot[bot] avatar Jul 26 '24 01:07 quarkus-bot[bot]

What's the status of this?

IMHO this is a really good usability improvement

geoand avatar Jul 31 '24 10:07 geoand


:waning_crescent_moon: This workflow status is outdated as a new workflow run has been triggered.


Status for workflow Quarkus CI

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

:white_check_mark: 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[bot] avatar Aug 02 '24 19:08 quarkus-bot[bot]