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

Cherry pick changes for 1.5.1 #2207

Merged
merged 17 commits into from
Dec 3, 2024
Merged

Cherry pick changes for 1.5.1 #2207

merged 17 commits into from
Dec 3, 2024

Conversation

ptodev
Copy link
Contributor

@ptodev ptodev commented Dec 3, 2024

Preparing for 1.5.1 by updating the VERSION file and cherry picking these commits:

dc9ec03
c99137c
e19c783
3bc6bfe
a319983
e61ad14
5d47b91
d82f44b
2b16ff0
45ef571
9177f33
1ba29ea
395e8cd
94dd6f2

@mattdurham unfortunately, it was difficult to cherry pick a37e481 and a914fd2 with remote queue changes.

YusifAghalar and others added 14 commits December 3, 2024 11:01
* Fixed an issue in the `otlp.exporter.prometheus` component

* Fixed an issue in the `otlp.exporter.prometheus` component
* Fix potential deadlock in import statements

* change

* typo
* Refactor ui remtoecfg components to avoid race condition

* Fix accidental cast to pointer that should have been struct

* Update changelog
* Fully prevent panic in remotecfg ui

* Address PR feedback
* Fix deadlock due to infinite retry

* changelog
* Upgrade ckit and changelog

* go mod tidy
…nally. (#2187)

* Fix issue which disables clustering unintentionally.
* prometheus.operator.*: allow setting informer_sync_timeout

* default to 1m

* docs
#2190)

* Only take into account some labels

* Reword docs
* fix: crash when updating import.http config
* Update postgres exporter

* Update changelog

* Use postgres exporter branch that implements exporter package

* Add TODO for future maintainers
@ptodev ptodev requested review from a team and clayton-cornell as code owners December 3, 2024 11:33
ptodev and others added 2 commits December 3, 2024 12:03
* Fix pyroscope.write issues with pyroscope.receive_http

The nodejs Pyroscope SDK sends profiles with a `Connection: close` header.
This header was copied to the upstream request, causing connection churn
towards Pyroscope, which can be quite bad on the CPU when using TLS. Do not
copy the `Connection` header from the incoming request to fix this issue.

Additionally, `pyroscope.write` had a single `http.Client` used for
forwarding data from `pyroscope.receive_http`, which may not work if
multiple endpoints are configured with different options. To fix this,
store a `http.Client` for each endpoint.
@ptodev ptodev mentioned this pull request Dec 3, 2024
Copy link
Collaborator

@mattdurham mattdurham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@ptodev ptodev merged commit 22e8ba6 into release/v1.5 Dec 3, 2024
18 checks passed
@ptodev ptodev deleted the ptodev/cherry-pick-1.5.1 branch December 3, 2024 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants