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

Do not allocate memory when there's no constraints #1296

Merged
merged 1 commit into from
Jun 27, 2023

Conversation

Okhoshi
Copy link
Contributor

@Okhoshi Okhoshi commented Jun 16, 2023

Implementing a fast path when there's no constrained label should limit a lot the number of allocations.

Closes #1276

Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

Nice, thanks for this!

I think it makes sense, but we need to clean up those benchmarks. They are mostly duplicate and will get obsolete very soon. So either we use b.Run or remove them for now.

BTW Were those useful? Did you gather some info? Or just helped you to finalized the implementation? (:

prometheus/benchmark_triple_test.go Outdated Show resolved Hide resolved
prometheus/desc.go Outdated Show resolved Hide resolved
prometheus/labels.go Outdated Show resolved Hide resolved
@Okhoshi
Copy link
Contributor Author

Okhoshi commented Jun 19, 2023

BTW Were those useful? Did you gather some info? Or just helped you to finalized the implementation? (:

They essentially helped me to see the progress and if there were some remaining optimisations that could be done.

@bwplotka
Copy link
Member

Great 💪🏽

I feel we can either drop those benchmark for now if you don't have time to clean them or create one or two with table tests so others can run it on their PRs to verify if there is any regression 🤗

Signed-off-by: Quentin Devos <4972091+Okhoshi@users.noreply.github.com>
@Okhoshi
Copy link
Contributor Author

Okhoshi commented Jun 26, 2023

Sorry for the delay, I got busy at work and couldn't find the time before :) I rewrote the benchmarks as a table test, it should be easier to maintain like this 👍

Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

So much better, thanks so much 💪🏽

@bwplotka bwplotka merged commit 644c80d into prometheus:main Jun 27, 2023
@Okhoshi Okhoshi deleted the fix-1276 branch June 29, 2023 16:31
@marten-seemann
Copy link

Any chance we cut get a release that includes this fix?

We have some tests that our metrics collection doesn't allocate in go-libp2p, and we currently can't upgrade this dependency past v1.14.0 without causing these tests to fail.
Obviously, no matter which version we specify in go-libp2p's go.mod, users of our library might still have client_golang bumped to a version greater than v1.14.0, causing a performance regression.

renovate bot referenced this pull request in open-feature/flagd Sep 27, 2023
….0 (#939)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[github.com/prometheus/client_golang](https://togithub.com/prometheus/client_golang)
| require | minor | `v1.16.0` -> `v1.17.0` |

---

### Release Notes

<details>
<summary>prometheus/client_golang
(github.com/prometheus/client_golang)</summary>

###
[`v1.17.0`](https://togithub.com/prometheus/client_golang/releases/tag/v1.17.0)

[Compare
Source](https://togithub.com/prometheus/client_golang/compare/v1.16.0...v1.17.0)

#### What's Changed

- \[CHANGE] Minimum required go version is now 1.19 (we also test
client_golang against new 1.21 version).
[#&#8203;1325](https://togithub.com/prometheus/client_golang/issues/1325)
- \[FEATURE] Add support for Created Timestamps in Counters, Summaries
and Historams.
[#&#8203;1313](https://togithub.com/prometheus/client_golang/issues/1313)
- \[ENHANCEMENT] Enable detection of a native histogram without
observations.
[#&#8203;1314](https://togithub.com/prometheus/client_golang/issues/1314)

<details><summary>Commits</summary>

- Merge v1.16.0 to main by
[@&#8203;bwplotka](https://togithub.com/bwplotka) in
[https://github.com/prometheus/client_golang/pull/1293](https://togithub.com/prometheus/client_golang/pull/1293)
- Synchronize common files from prometheus/prometheus by
[@&#8203;prombot](https://togithub.com/prombot) in
[https://github.com/prometheus/client_golang/pull/1297](https://togithub.com/prometheus/client_golang/pull/1297)
- ci: define minimal permissions to GitHub workflows by
[@&#8203;diogoteles08](https://togithub.com/diogoteles08) in
[https://github.com/prometheus/client_golang/pull/1295](https://togithub.com/prometheus/client_golang/pull/1295)
- Do not allocate memory when there's no constraints by
[@&#8203;Okhoshi](https://togithub.com/Okhoshi) in
[https://github.com/prometheus/client_golang/pull/1296](https://togithub.com/prometheus/client_golang/pull/1296)
- Bump golang.org/x/sys from 0.8.0 to 0.9.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1306](https://togithub.com/prometheus/client_golang/pull/1306)
- Bump google.golang.org/grpc from 1.45.0 to 1.53.0 in /tutorial/whatsup
by [@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1307](https://togithub.com/prometheus/client_golang/pull/1307)
- histogram: Enable detection of a native histogram without observations
by [@&#8203;beorn7](https://togithub.com/beorn7) in
[https://github.com/prometheus/client_golang/pull/1314](https://togithub.com/prometheus/client_golang/pull/1314)
- Bump github.com/prometheus/procfs from 0.10.1 to 0.11.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1305](https://togithub.com/prometheus/client_golang/pull/1305)
- Synchronize common files from prometheus/prometheus by
[@&#8203;prombot](https://togithub.com/prombot) in
[https://github.com/prometheus/client_golang/pull/1302](https://togithub.com/prometheus/client_golang/pull/1302)
- Fix data-race in metric without `code` and `method` but with
`WithLabelFromCtx` by [@&#8203;tigrato](https://togithub.com/tigrato) in
[https://github.com/prometheus/client_golang/pull/1318](https://togithub.com/prometheus/client_golang/pull/1318)
- Add missing tick "\`" in README by
[@&#8203;ZiViZiViZ](https://togithub.com/ZiViZiViZ) in
[https://github.com/prometheus/client_golang/pull/1321](https://togithub.com/prometheus/client_golang/pull/1321)
- Bump golang.org/x/sys from 0.9.0 to 0.10.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1320](https://togithub.com/prometheus/client_golang/pull/1320)
- Bump github.com/prometheus/procfs from 0.11.0 to 0.11.1 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1319](https://togithub.com/prometheus/client_golang/pull/1319)
- docs: trivial grammar fixes to improve readability in promauto Godoc
by [@&#8203;sengi](https://togithub.com/sengi) in
[https://github.com/prometheus/client_golang/pull/1322](https://togithub.com/prometheus/client_golang/pull/1322)
- Add Go 1.21 support by
[@&#8203;kakkoyun](https://togithub.com/kakkoyun) in
[https://github.com/prometheus/client_golang/pull/1325](https://togithub.com/prometheus/client_golang/pull/1325)
- Bump client_model by
[@&#8203;ArthurSens](https://togithub.com/ArthurSens) in
[https://github.com/prometheus/client_golang/pull/1323](https://togithub.com/prometheus/client_golang/pull/1323)
- histogram docs: Fixed minor nit. by
[@&#8203;bwplotka](https://togithub.com/bwplotka) in
[https://github.com/prometheus/client_golang/pull/1324](https://togithub.com/prometheus/client_golang/pull/1324)
- Update building by [@&#8203;SuperQ](https://togithub.com/SuperQ) in
[https://github.com/prometheus/client_golang/pull/1326](https://togithub.com/prometheus/client_golang/pull/1326)
- Bump golang.org/x/sys from 0.10.0 to 0.11.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1331](https://togithub.com/prometheus/client_golang/pull/1331)
- Bump github.com/prometheus/client_golang from
1.15.1-0.20230416215738-0963f595c689 to 1.16.0 in /tutorial/whatsup by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1329](https://togithub.com/prometheus/client_golang/pull/1329)
- Bump github.com/prometheus/client_golang from 1.13.1 to 1.16.0 in
/examples/middleware by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1328](https://togithub.com/prometheus/client_golang/pull/1328)
- Bump github.com/prometheus/common from 0.42.0 to 0.44.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1284](https://togithub.com/prometheus/client_golang/pull/1284)
- Bump github.com/prometheus/common from 0.42.0 to 0.44.0 in
/tutorial/whatsup by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1330](https://togithub.com/prometheus/client_golang/pull/1330)
- Bump google.golang.org/protobuf from 1.30.0 to 1.31.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1304](https://togithub.com/prometheus/client_golang/pull/1304)
- Synchronize common files from prometheus/prometheus by
[@&#8203;prombot](https://togithub.com/prombot) in
[https://github.com/prometheus/client_golang/pull/1332](https://togithub.com/prometheus/client_golang/pull/1332)
- Synchronize common files from prometheus/prometheus by
[@&#8203;prombot](https://togithub.com/prombot) in
[https://github.com/prometheus/client_golang/pull/1338](https://togithub.com/prometheus/client_golang/pull/1338)
- Cleanup golangci-lint errcheck by
[@&#8203;SuperQ](https://togithub.com/SuperQ) in
[https://github.com/prometheus/client_golang/pull/1339](https://togithub.com/prometheus/client_golang/pull/1339)
- Add go_godebug_non_default_behavior_tlsmaxrsasize_events_total by
[@&#8203;alexandear](https://togithub.com/alexandear) in
[https://github.com/prometheus/client_golang/pull/1348](https://togithub.com/prometheus/client_golang/pull/1348)
- Extend Counters, Summaries and Histograms with creation timestamp by
[@&#8203;ArthurSens](https://togithub.com/ArthurSens) in
[https://github.com/prometheus/client_golang/pull/1313](https://togithub.com/prometheus/client_golang/pull/1313)
- Fix typos in comments, tests, and errors by
[@&#8203;alexandear](https://togithub.com/alexandear) in
[https://github.com/prometheus/client_golang/pull/1346](https://togithub.com/prometheus/client_golang/pull/1346)
- Deprecated comment should begin with "Deprecated:" by
[@&#8203;alexandear](https://togithub.com/alexandear) in
[https://github.com/prometheus/client_golang/pull/1347](https://togithub.com/prometheus/client_golang/pull/1347)
- Add changelog entry for 1.17 by
[@&#8203;ArthurSens](https://togithub.com/ArthurSens) in
[https://github.com/prometheus/client_golang/pull/1352](https://togithub.com/prometheus/client_golang/pull/1352)

</details>

#### New Contributors
* @&#8203;diogoteles08 made their first
contributi[https://github.com/prometheus/client_golang/pull/1295](https://togithub.com/prometheus/client_golang/pull/1295)l/1295
* @&#8203;tigrato made their first
contributi[https://github.com/prometheus/client_golang/pull/1318](https://togithub.com/prometheus/client_golang/pull/1318)l/1318
* @&#8203;ZiViZiViZ made their first
contributi[https://github.com/prometheus/client_golang/pull/1321](https://togithub.com/prometheus/client_golang/pull/1321)l/1321
* @&#8203;sengi made their first
contributi[https://github.com/prometheus/client_golang/pull/1322](https://togithub.com/prometheus/client_golang/pull/1322)l/1322
* @&#8203;ArthurSens made their first
contributi[https://github.com/prometheus/client_golang/pull/1323](https://togithub.com/prometheus/client_golang/pull/1323)l/1323
* @&#8203;alexandear made their first
contributi[https://github.com/prometheus/client_golang/pull/1348](https://togithub.com/prometheus/client_golang/pull/1348)l/1348

**Full Changelog**:
prometheus/client_golang@v1.16.0...v1.17.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/open-feature/flagd).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi4xMDcuMiIsInVwZGF0ZWRJblZlciI6IjM2LjEwNy4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
apricote referenced this pull request in hetznercloud/hcloud-cloud-controller-manager Sep 28, 2023
)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[github.com/prometheus/client_golang](https://togithub.com/prometheus/client_golang)
| require | minor | `v1.16.0` -> `v1.17.0` |

---

### Release Notes

<details>
<summary>prometheus/client_golang
(github.com/prometheus/client_golang)</summary>

###
[`v1.17.0`](https://togithub.com/prometheus/client_golang/releases/tag/v1.17.0)

[Compare
Source](https://togithub.com/prometheus/client_golang/compare/v1.16.0...v1.17.0)

#### What's Changed

- \[CHANGE] Minimum required go version is now 1.19 (we also test
client_golang against new 1.21 version).
[#&#8203;1325](https://togithub.com/prometheus/client_golang/issues/1325)
- \[FEATURE] Add support for Created Timestamps in Counters, Summaries
and Historams.
[#&#8203;1313](https://togithub.com/prometheus/client_golang/issues/1313)
- \[ENHANCEMENT] Enable detection of a native histogram without
observations.
[#&#8203;1314](https://togithub.com/prometheus/client_golang/issues/1314)

<details><summary>Commits</summary>

- Merge v1.16.0 to main by
[@&#8203;bwplotka](https://togithub.com/bwplotka) in
[https://github.com/prometheus/client_golang/pull/1293](https://togithub.com/prometheus/client_golang/pull/1293)
- Synchronize common files from prometheus/prometheus by
[@&#8203;prombot](https://togithub.com/prombot) in
[https://github.com/prometheus/client_golang/pull/1297](https://togithub.com/prometheus/client_golang/pull/1297)
- ci: define minimal permissions to GitHub workflows by
[@&#8203;diogoteles08](https://togithub.com/diogoteles08) in
[https://github.com/prometheus/client_golang/pull/1295](https://togithub.com/prometheus/client_golang/pull/1295)
- Do not allocate memory when there's no constraints by
[@&#8203;Okhoshi](https://togithub.com/Okhoshi) in
[https://github.com/prometheus/client_golang/pull/1296](https://togithub.com/prometheus/client_golang/pull/1296)
- Bump golang.org/x/sys from 0.8.0 to 0.9.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1306](https://togithub.com/prometheus/client_golang/pull/1306)
- Bump google.golang.org/grpc from 1.45.0 to 1.53.0 in /tutorial/whatsup
by [@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1307](https://togithub.com/prometheus/client_golang/pull/1307)
- histogram: Enable detection of a native histogram without observations
by [@&#8203;beorn7](https://togithub.com/beorn7) in
[https://github.com/prometheus/client_golang/pull/1314](https://togithub.com/prometheus/client_golang/pull/1314)
- Bump github.com/prometheus/procfs from 0.10.1 to 0.11.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1305](https://togithub.com/prometheus/client_golang/pull/1305)
- Synchronize common files from prometheus/prometheus by
[@&#8203;prombot](https://togithub.com/prombot) in
[https://github.com/prometheus/client_golang/pull/1302](https://togithub.com/prometheus/client_golang/pull/1302)
- Fix data-race in metric without `code` and `method` but with
`WithLabelFromCtx` by [@&#8203;tigrato](https://togithub.com/tigrato) in
[https://github.com/prometheus/client_golang/pull/1318](https://togithub.com/prometheus/client_golang/pull/1318)
- Add missing tick "\`" in README by
[@&#8203;ZiViZiViZ](https://togithub.com/ZiViZiViZ) in
[https://github.com/prometheus/client_golang/pull/1321](https://togithub.com/prometheus/client_golang/pull/1321)
- Bump golang.org/x/sys from 0.9.0 to 0.10.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1320](https://togithub.com/prometheus/client_golang/pull/1320)
- Bump github.com/prometheus/procfs from 0.11.0 to 0.11.1 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1319](https://togithub.com/prometheus/client_golang/pull/1319)
- docs: trivial grammar fixes to improve readability in promauto Godoc
by [@&#8203;sengi](https://togithub.com/sengi) in
[https://github.com/prometheus/client_golang/pull/1322](https://togithub.com/prometheus/client_golang/pull/1322)
- Add Go 1.21 support by
[@&#8203;kakkoyun](https://togithub.com/kakkoyun) in
[https://github.com/prometheus/client_golang/pull/1325](https://togithub.com/prometheus/client_golang/pull/1325)
- Bump client_model by
[@&#8203;ArthurSens](https://togithub.com/ArthurSens) in
[https://github.com/prometheus/client_golang/pull/1323](https://togithub.com/prometheus/client_golang/pull/1323)
- histogram docs: Fixed minor nit. by
[@&#8203;bwplotka](https://togithub.com/bwplotka) in
[https://github.com/prometheus/client_golang/pull/1324](https://togithub.com/prometheus/client_golang/pull/1324)
- Update building by [@&#8203;SuperQ](https://togithub.com/SuperQ) in
[https://github.com/prometheus/client_golang/pull/1326](https://togithub.com/prometheus/client_golang/pull/1326)
- Bump golang.org/x/sys from 0.10.0 to 0.11.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1331](https://togithub.com/prometheus/client_golang/pull/1331)
- Bump github.com/prometheus/client_golang from
1.15.1-0.20230416215738-0963f595c689 to 1.16.0 in /tutorial/whatsup by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1329](https://togithub.com/prometheus/client_golang/pull/1329)
- Bump github.com/prometheus/client_golang from 1.13.1 to 1.16.0 in
/examples/middleware by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1328](https://togithub.com/prometheus/client_golang/pull/1328)
- Bump github.com/prometheus/common from 0.42.0 to 0.44.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1284](https://togithub.com/prometheus/client_golang/pull/1284)
- Bump github.com/prometheus/common from 0.42.0 to 0.44.0 in
/tutorial/whatsup by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1330](https://togithub.com/prometheus/client_golang/pull/1330)
- Bump google.golang.org/protobuf from 1.30.0 to 1.31.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1304](https://togithub.com/prometheus/client_golang/pull/1304)
- Synchronize common files from prometheus/prometheus by
[@&#8203;prombot](https://togithub.com/prombot) in
[https://github.com/prometheus/client_golang/pull/1332](https://togithub.com/prometheus/client_golang/pull/1332)
- Synchronize common files from prometheus/prometheus by
[@&#8203;prombot](https://togithub.com/prombot) in
[https://github.com/prometheus/client_golang/pull/1338](https://togithub.com/prometheus/client_golang/pull/1338)
- Cleanup golangci-lint errcheck by
[@&#8203;SuperQ](https://togithub.com/SuperQ) in
[https://github.com/prometheus/client_golang/pull/1339](https://togithub.com/prometheus/client_golang/pull/1339)
- Add go_godebug_non_default_behavior_tlsmaxrsasize_events_total by
[@&#8203;alexandear](https://togithub.com/alexandear) in
[https://github.com/prometheus/client_golang/pull/1348](https://togithub.com/prometheus/client_golang/pull/1348)
- Extend Counters, Summaries and Histograms with creation timestamp by
[@&#8203;ArthurSens](https://togithub.com/ArthurSens) in
[https://github.com/prometheus/client_golang/pull/1313](https://togithub.com/prometheus/client_golang/pull/1313)
- Fix typos in comments, tests, and errors by
[@&#8203;alexandear](https://togithub.com/alexandear) in
[https://github.com/prometheus/client_golang/pull/1346](https://togithub.com/prometheus/client_golang/pull/1346)
- Deprecated comment should begin with "Deprecated:" by
[@&#8203;alexandear](https://togithub.com/alexandear) in
[https://github.com/prometheus/client_golang/pull/1347](https://togithub.com/prometheus/client_golang/pull/1347)
- Add changelog entry for 1.17 by
[@&#8203;ArthurSens](https://togithub.com/ArthurSens) in
[https://github.com/prometheus/client_golang/pull/1352](https://togithub.com/prometheus/client_golang/pull/1352)

</details>

#### New Contributors
* @&#8203;diogoteles08 made their first
contributi[https://github.com/prometheus/client_golang/pull/1295](https://togithub.com/prometheus/client_golang/pull/1295)l/1295
* @&#8203;tigrato made their first
contributi[https://github.com/prometheus/client_golang/pull/1318](https://togithub.com/prometheus/client_golang/pull/1318)l/1318
* @&#8203;ZiViZiViZ made their first
contributi[https://github.com/prometheus/client_golang/pull/1321](https://togithub.com/prometheus/client_golang/pull/1321)l/1321
* @&#8203;sengi made their first
contributi[https://github.com/prometheus/client_golang/pull/1322](https://togithub.com/prometheus/client_golang/pull/1322)l/1322
* @&#8203;ArthurSens made their first
contributi[https://github.com/prometheus/client_golang/pull/1323](https://togithub.com/prometheus/client_golang/pull/1323)l/1323
* @&#8203;alexandear made their first
contributi[https://github.com/prometheus/client_golang/pull/1348](https://togithub.com/prometheus/client_golang/pull/1348)l/1348

**Full Changelog**:
prometheus/client_golang@v1.16.0...v1.17.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/hetznercloud/hcloud-cloud-controller-manager).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi4xMDcuMiIsInVwZGF0ZWRJblZlciI6IjM2LjEwNy4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
nono referenced this pull request in cozy/cozy-stack Oct 2, 2023
….0 (#4144)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[github.com/prometheus/client_golang](https://togithub.com/prometheus/client_golang)
| require | minor | `v1.16.0` -> `v1.17.0` |

---

### Release Notes

<details>
<summary>prometheus/client_golang
(github.com/prometheus/client_golang)</summary>

###
[`v1.17.0`](https://togithub.com/prometheus/client_golang/releases/tag/v1.17.0)

[Compare
Source](https://togithub.com/prometheus/client_golang/compare/v1.16.0...v1.17.0)

#### What's Changed

- \[CHANGE] Minimum required go version is now 1.19 (we also test
client_golang against new 1.21 version).
[#&#8203;1325](https://togithub.com/prometheus/client_golang/issues/1325)
- \[FEATURE] Add support for Created Timestamps in Counters, Summaries
and Historams.
[#&#8203;1313](https://togithub.com/prometheus/client_golang/issues/1313)
- \[ENHANCEMENT] Enable detection of a native histogram without
observations.
[#&#8203;1314](https://togithub.com/prometheus/client_golang/issues/1314)

<details><summary>Commits</summary>

- Merge v1.16.0 to main by
[@&#8203;bwplotka](https://togithub.com/bwplotka) in
[https://github.com/prometheus/client_golang/pull/1293](https://togithub.com/prometheus/client_golang/pull/1293)
- Synchronize common files from prometheus/prometheus by
[@&#8203;prombot](https://togithub.com/prombot) in
[https://github.com/prometheus/client_golang/pull/1297](https://togithub.com/prometheus/client_golang/pull/1297)
- ci: define minimal permissions to GitHub workflows by
[@&#8203;diogoteles08](https://togithub.com/diogoteles08) in
[https://github.com/prometheus/client_golang/pull/1295](https://togithub.com/prometheus/client_golang/pull/1295)
- Do not allocate memory when there's no constraints by
[@&#8203;Okhoshi](https://togithub.com/Okhoshi) in
[https://github.com/prometheus/client_golang/pull/1296](https://togithub.com/prometheus/client_golang/pull/1296)
- Bump golang.org/x/sys from 0.8.0 to 0.9.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1306](https://togithub.com/prometheus/client_golang/pull/1306)
- Bump google.golang.org/grpc from 1.45.0 to 1.53.0 in /tutorial/whatsup
by [@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1307](https://togithub.com/prometheus/client_golang/pull/1307)
- histogram: Enable detection of a native histogram without observations
by [@&#8203;beorn7](https://togithub.com/beorn7) in
[https://github.com/prometheus/client_golang/pull/1314](https://togithub.com/prometheus/client_golang/pull/1314)
- Bump github.com/prometheus/procfs from 0.10.1 to 0.11.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1305](https://togithub.com/prometheus/client_golang/pull/1305)
- Synchronize common files from prometheus/prometheus by
[@&#8203;prombot](https://togithub.com/prombot) in
[https://github.com/prometheus/client_golang/pull/1302](https://togithub.com/prometheus/client_golang/pull/1302)
- Fix data-race in metric without `code` and `method` but with
`WithLabelFromCtx` by [@&#8203;tigrato](https://togithub.com/tigrato) in
[https://github.com/prometheus/client_golang/pull/1318](https://togithub.com/prometheus/client_golang/pull/1318)
- Add missing tick "\`" in README by
[@&#8203;ZiViZiViZ](https://togithub.com/ZiViZiViZ) in
[https://github.com/prometheus/client_golang/pull/1321](https://togithub.com/prometheus/client_golang/pull/1321)
- Bump golang.org/x/sys from 0.9.0 to 0.10.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1320](https://togithub.com/prometheus/client_golang/pull/1320)
- Bump github.com/prometheus/procfs from 0.11.0 to 0.11.1 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1319](https://togithub.com/prometheus/client_golang/pull/1319)
- docs: trivial grammar fixes to improve readability in promauto Godoc
by [@&#8203;sengi](https://togithub.com/sengi) in
[https://github.com/prometheus/client_golang/pull/1322](https://togithub.com/prometheus/client_golang/pull/1322)
- Add Go 1.21 support by
[@&#8203;kakkoyun](https://togithub.com/kakkoyun) in
[https://github.com/prometheus/client_golang/pull/1325](https://togithub.com/prometheus/client_golang/pull/1325)
- Bump client_model by
[@&#8203;ArthurSens](https://togithub.com/ArthurSens) in
[https://github.com/prometheus/client_golang/pull/1323](https://togithub.com/prometheus/client_golang/pull/1323)
- histogram docs: Fixed minor nit. by
[@&#8203;bwplotka](https://togithub.com/bwplotka) in
[https://github.com/prometheus/client_golang/pull/1324](https://togithub.com/prometheus/client_golang/pull/1324)
- Update building by [@&#8203;SuperQ](https://togithub.com/SuperQ) in
[https://github.com/prometheus/client_golang/pull/1326](https://togithub.com/prometheus/client_golang/pull/1326)
- Bump golang.org/x/sys from 0.10.0 to 0.11.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1331](https://togithub.com/prometheus/client_golang/pull/1331)
- Bump github.com/prometheus/client_golang from
1.15.1-0.20230416215738-0963f595c689 to 1.16.0 in /tutorial/whatsup by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1329](https://togithub.com/prometheus/client_golang/pull/1329)
- Bump github.com/prometheus/client_golang from 1.13.1 to 1.16.0 in
/examples/middleware by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1328](https://togithub.com/prometheus/client_golang/pull/1328)
- Bump github.com/prometheus/common from 0.42.0 to 0.44.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1284](https://togithub.com/prometheus/client_golang/pull/1284)
- Bump github.com/prometheus/common from 0.42.0 to 0.44.0 in
/tutorial/whatsup by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1330](https://togithub.com/prometheus/client_golang/pull/1330)
- Bump google.golang.org/protobuf from 1.30.0 to 1.31.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/prometheus/client_golang/pull/1304](https://togithub.com/prometheus/client_golang/pull/1304)
- Synchronize common files from prometheus/prometheus by
[@&#8203;prombot](https://togithub.com/prombot) in
[https://github.com/prometheus/client_golang/pull/1332](https://togithub.com/prometheus/client_golang/pull/1332)
- Synchronize common files from prometheus/prometheus by
[@&#8203;prombot](https://togithub.com/prombot) in
[https://github.com/prometheus/client_golang/pull/1338](https://togithub.com/prometheus/client_golang/pull/1338)
- Cleanup golangci-lint errcheck by
[@&#8203;SuperQ](https://togithub.com/SuperQ) in
[https://github.com/prometheus/client_golang/pull/1339](https://togithub.com/prometheus/client_golang/pull/1339)
- Add go_godebug_non_default_behavior_tlsmaxrsasize_events_total by
[@&#8203;alexandear](https://togithub.com/alexandear) in
[https://github.com/prometheus/client_golang/pull/1348](https://togithub.com/prometheus/client_golang/pull/1348)
- Extend Counters, Summaries and Histograms with creation timestamp by
[@&#8203;ArthurSens](https://togithub.com/ArthurSens) in
[https://github.com/prometheus/client_golang/pull/1313](https://togithub.com/prometheus/client_golang/pull/1313)
- Fix typos in comments, tests, and errors by
[@&#8203;alexandear](https://togithub.com/alexandear) in
[https://github.com/prometheus/client_golang/pull/1346](https://togithub.com/prometheus/client_golang/pull/1346)
- Deprecated comment should begin with "Deprecated:" by
[@&#8203;alexandear](https://togithub.com/alexandear) in
[https://github.com/prometheus/client_golang/pull/1347](https://togithub.com/prometheus/client_golang/pull/1347)
- Add changelog entry for 1.17 by
[@&#8203;ArthurSens](https://togithub.com/ArthurSens) in
[https://github.com/prometheus/client_golang/pull/1352](https://togithub.com/prometheus/client_golang/pull/1352)

</details>

#### New Contributors
* @&#8203;diogoteles08 made their first
contributi[https://github.com/prometheus/client_golang/pull/1295](https://togithub.com/prometheus/client_golang/pull/1295)l/1295
* @&#8203;tigrato made their first
contributi[https://github.com/prometheus/client_golang/pull/1318](https://togithub.com/prometheus/client_golang/pull/1318)l/1318
* @&#8203;ZiViZiViZ made their first
contributi[https://github.com/prometheus/client_golang/pull/1321](https://togithub.com/prometheus/client_golang/pull/1321)l/1321
* @&#8203;sengi made their first
contributi[https://github.com/prometheus/client_golang/pull/1322](https://togithub.com/prometheus/client_golang/pull/1322)l/1322
* @&#8203;ArthurSens made their first
contributi[https://github.com/prometheus/client_golang/pull/1323](https://togithub.com/prometheus/client_golang/pull/1323)l/1323
* @&#8203;alexandear made their first
contributi[https://github.com/prometheus/client_golang/pull/1348](https://togithub.com/prometheus/client_golang/pull/1348)l/1348

**Full Changelog**:
prometheus/client_golang@v1.16.0...v1.17.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 6am on Monday" in timezone
Europe/Paris, Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/cozy/cozy-stack).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4wLjMiLCJ1cGRhdGVkSW5WZXIiOiIzNy4wLjMiLCJ0YXJnZXRCcmFuY2giOiJtYXN0ZXIifQ==-->
bboreham added a commit to grafana/mimir that referenced this pull request Oct 3, 2023
Particularly to bring in a performance fix prometheus/client_golang#1296.

Most changes are fixing spelling or punctuation in spelling, but there
is also the addition of creation timestamps from prometheus/client_golang#1313.
bboreham added a commit to grafana/mimir that referenced this pull request Oct 3, 2023
Particularly to bring in a performance fix prometheus/client_golang#1296.

Most changes are fixing spelling or punctuation in spelling, but there
is also the addition of creation timestamps from prometheus/client_golang#1313.
djaglowski pushed a commit to open-telemetry/opentelemetry-collector-contrib that referenced this pull request Oct 5, 2023
Had to update a Prometheus exporter test `TestCollectMetrics` to adapt
to the metric description String() output change made the recent Prom
library version prometheus/client_golang#1296.

Fixes
#27398
jmsnll pushed a commit to jmsnll/opentelemetry-collector-contrib that referenced this pull request Nov 12, 2023
Had to update a Prometheus exporter test `TestCollectMetrics` to adapt
to the metric description String() output change made the recent Prom
library version prometheus/client_golang#1296.

Fixes
open-telemetry#27398
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.

constrainLabels allocates too many memory
3 participants