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

[6/6] introspection: remove disused argument #1974

Closed

Conversation

@hdonnay hdonnay force-pushed the hack/healthmetrics branch 4 times, most recently from 4fcfd81 to efe68fc Compare February 15, 2024 17:13
@hdonnay hdonnay force-pushed the hack/healthmetrics branch from efe68fc to 2b838df Compare April 9, 2024 21:44
@hdonnay hdonnay mentioned this pull request Jul 23, 2024
hdonnay added 6 commits July 23, 2024 14:56
Signed-off-by: Hank Donnay <[email protected]>
This should allow a Clair instance to use the "native" OpenTelemetry
transport for metrics and traces.

Some parts of Clair and Claircore are directly instrumented with
Prometheus, and so the OTLP support should be considered beta-quality.

Signed-off-by: Hank Donnay <[email protected]>
This updates the `local-dev` stack. Most of these updates were needed to
test the OpenTelemetry support.

Signed-off-by: Hank Donnay <[email protected]>
This is replaced by the metrics-powered health check.

Signed-off-by: Hank Donnay <[email protected]>
@hdonnay hdonnay force-pushed the hack/healthmetrics branch from 2b838df to 61a1619 Compare July 23, 2024 20:26
@hdonnay hdonnay changed the title health: PoC for health checks via otel metrics [6/6] introspection: remove disused argument Jul 24, 2024
@hdonnay hdonnay changed the base branch from main to hdonnay/introspection-lints July 24, 2024 20:21
@hdonnay hdonnay force-pushed the hdonnay/introspection-lints branch from 1f226c7 to c31e40e Compare August 26, 2024 22:38
@github-actions github-actions bot deleted the branch quay:hdonnay/introspection-lints August 26, 2024 22:42
@github-actions github-actions bot closed this Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant