-
Notifications
You must be signed in to change notification settings - Fork 77
chore(deps): bump knative.dev/pkg for OTEL metrics support #1935
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -19,6 +19,8 @@ metadata: | |
| namespace: cosign-system | ||
|
|
||
| data: | ||
| metrics-protocol: prometheus | ||
|
|
||
| _example: | | ||
| ################################ | ||
| # # | ||
|
|
@@ -35,17 +37,17 @@ data: | |
| # this example block and unindented to be in the data block | ||
| # to actually change the configuration. | ||
|
|
||
| # metrics.backend-destination field specifies the system metrics destination. | ||
|
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We could argue this is a breaking change. But easy for me to say ;) |
||
| # It supports either prometheus (the default) or stackdriver. | ||
| # Note: Using stackdriver will incur additional charges | ||
| metrics.backend-destination: prometheus | ||
| # metrics-protocol specifies the metrics export protocol. | ||
| # Supported values: prometheus, grpc, http/protobuf, none | ||
| # Default is "none" (metrics disabled). | ||
| # "prometheus" serves a pull-based /metrics endpoint on port 9090. | ||
| # "grpc" and "http/protobuf" push metrics via OTLP (requires metrics-endpoint). | ||
| metrics-protocol: prometheus | ||
|
|
||
| # metrics.request-metrics-backend-destination specifies the request metrics | ||
| # destination. If non-empty, it enables queue proxy to send request metrics. | ||
| # Currently supported values: prometheus, stackdriver. | ||
| metrics.request-metrics-backend-destination: prometheus | ||
| # metrics-endpoint specifies the endpoint for push-based protocols (grpc, http/protobuf). | ||
| # Required for grpc and http/protobuf. Optional for prometheus (sets host:port). | ||
| # metrics-endpoint: "otel-collector.observability:4317" | ||
|
|
||
| # metrics.stackdriver-project-id field specifies the stackdriver project ID. This | ||
| # field is optional. When running on GCE, application default credentials will be | ||
| # used if this field is not provided. | ||
| metrics.stackdriver-project-id: "<your stackdriver project id>" | ||
| # metrics-export-interval specifies how often metrics are exported for push-based protocols. | ||
| # Can be overridden by the OTEL_METRIC_EXPORT_INTERVAL environment variable. | ||
| # metrics-export-interval: 30s | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could argue we should drop 1.32 as well, since the k8s communinty only support 3 versions and not 4 now a days.
But since it's currently 4 I decide against it.
The main reason todo this in the same PR was that it felt natrual since we are starting to use k8s client 1.35, shouldn't be an issue but still.
Happy to revert the changes done in e2e.