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
Fix segfault when using --prometheus-token-file #538
Fix segfault when using --prometheus-token-file #538
Conversation
7ffceeb
to
e740fee
Compare
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.
LGTM 👍
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dgrisonnet, JoaoBraveCoding, olivierlemasle The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Changes since v0.10.0 Logging flags that are klog specific (`--log-dir`, `--log-file`, `--logtostderr`, `--alsologtostderr`, `--one-output`, `--stderrthreshold`, `--log-file-max-size`, `--skip-log-headers`, `--add-dir-header`, `--skip-headers`, `--log-backtrace-at`) were deprecated in v0.10.0 and are now **removed**. Improvements * Bump go to 1.20 and k8s deps to 0.27.2 (kubernetes-sigs#586, @dgrisonnet) * Add support for OpenAPI v3 (kubernetes-sigs#586, @dgrisonnet) * Update registry location to registry.k8s.io in the manifests (kubernetes-sigs#529, @dgrisonnet) * Update manifests in deploy/manifests to be in sync with the one in kube-prometheus (kubernetes-sigs#531, @JoaoBraveCoding) * Use golangci-lint to lint the codebase (kubernetes-sigs#540, @olivierlemasle) * Set MinVersion: tls.VersionTLS12 in prometheus client's TLSClientConfig (kubernetes-sigs#544, @olivierlemasle) * Refactor adding logging flags (kubernetes-sigs#546, @olivierlemasle) Bug fixes * Fix segfault when using --prometheus-token-file (kubernetes-sigs#538, @olivierlemasle) Documentation * Fix broken links in README.md (kubernetes-sigs#518, @sillyfrog) * Fix yaml in sample config & docs (kubernetes-sigs#559, @asherf)
When
--prometheus-token-file
is used, prometheus-adapter crashes with a segmentation violation errorThis is caused by
httpClient.Transport
beingnil
and being dereferenced in theBearerAuthRoundTripper
.Fixes #478