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

Merge upstream prometheus/prometheus at 31491eb #611

Merged
merged 37 commits into from
Apr 4, 2024

Conversation

bboreham and others added 30 commits March 12, 2024 15:04
This should make it easier to test.

Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
So we can mock it.

Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
Test fails due to bug in code on main.

Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
Signed-off-by: Jeanette Tan <jeanette.tan@grafana.com>
Signed-off-by: tdakkota <tanc13@yandex.ru>
Release 2.51: Cherry-pick #13803 bugfix for label_join
Restrict the capacity of first argument to `append()` to force an allocation.
This is for the slice implementation only.

Signed-off-by: Domantas Jadenkus <djadenkus@gmail.com>
Azure Discovery tests: Add test for mapping VMs to labels
Restrict the capacity of first argument to `append()` to force an allocation.
This is for the slice implementation only.

Signed-off-by: Domantas Jadenkus <djadenkus@gmail.com>
Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
* fix the bug of setting native histogram min bucket factor

Signed-off-by: Ziqi Zhao <zhaoziqi9146@gmail.com>

* Add unit test for checking that min_bucket_factor is correctly applied

Signed-off-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>

---------

Signed-off-by: Ziqi Zhao <zhaoziqi9146@gmail.com>
Signed-off-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>
Co-authored-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>
* fix the bug of setting native histogram min bucket factor

Signed-off-by: Ziqi Zhao <zhaoziqi9146@gmail.com>

* Add unit test for checking that min_bucket_factor is correctly applied

Signed-off-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>

---------

Signed-off-by: Ziqi Zhao <zhaoziqi9146@gmail.com>
Signed-off-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>
Co-authored-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>
Signed-off-by: Jeanette Tan <jeanette.tan@grafana.com>
…td dev/var

Signed-off-by: Jeanette Tan <jeanette.tan@grafana.com>
Release 2.51: Cherry-pick #13845 bugfix for DropMetricName
Release 2.51: Cherry pick #13846 fix the bug of setting native histogram min bucket factor
The definition of histograms in the test framework may create
histograms in a non-compact form. Since histogram comparison relies on
exact equality of the bucket layout, we have to compact the histograms
created by the test framework language before comparing them to
histograms returned from the PromQL engine.

Signed-off-by: beorn7 <beorn@grafana.com>
Signed-off-by: beorn7 <beorn@grafana.com>
* Cut release 2.51.1

Co-authored-by: George Krajcsovits <krajorama@users.noreply.github.com>
Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
This was an unintentional effect of splitting out Go tests into multiple
parallel blocks.

Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
promql: Fix histogram comparison in test framework
Note that we have delayed v2.52 by a week to avoid collisions with
events and travels.

Signed-off-by: beorn7 <beorn@grafana.com>
Appoint release shepherds for v2.52 and v2.53
Signed-off-by: Nicolas Takashi <nicolas.tcs@hotmail.com>
…OOChunks-struct

[refactor] moving mergedOOOChunks to ooo_head_read
CI: Publish step should require all Go tests to pass
Signed-off-by: carehabit <shenyuting@outlook.com>
Nexucis and others added 7 commits April 1, 2024 21:54
prometheus/prometheus#13446 removed the last usage of
this field, but didn't remove the field.

Signed-off-by: Julius Volz <julius.volz@gmail.com>
Signed-off-by: Nicolas Takashi <nicolas.tcs@hotmail.com>
…y fails or an unexpected series is returned (#13847)

* promql: include more details in error message when creating test query fails

Signed-off-by: Charles Korn <charles.korn@grafana.com>

* Include more details when an unexpected metric is returned

Signed-off-by: Charles Korn <charles.korn@grafana.com>

---------

Signed-off-by: Charles Korn <charles.korn@grafana.com>
The strings produced by these tests can run to thousands of characters,
which makes test logs difficult to read.

Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
[TESTS] Truncate some long test names, for readability
…pstream

# Conflicts:
#	.github/workflows/ci.yml
#	model/labels/regexp_test.go
#	promql/engine_test.go
#	tsdb/db.go
@CLAassistant
Copy link

CLAassistant commented Apr 4, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
5 out of 14 committers have signed the CLA.

✅ zenador
✅ bboreham
✅ beorn7
✅ krajorama
✅ charleskorn
❌ jDomantas
❌ tdakkota
❌ bwplotka
❌ fatsheep9146
❌ carehabit
❌ Nexucis
❌ ArthurSens
❌ nicolastakashi
❌ juliusv
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link
Contributor

@aknuds1 aknuds1 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@pracucci pracucci left a comment

Choose a reason for hiding this comment

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

I like these small but frequent syncs. They're more manageable.

@charleskorn charleskorn merged commit 99f7a07 into main Apr 4, 2024
5 of 6 checks passed
@charleskorn charleskorn deleted the charleskorn/merge-upstream branch April 4, 2024 23:54
@charleskorn
Copy link
Contributor Author

I like these small but frequent syncs. They're more manageable.

Agree. One day™️ I'd love it if these updates were automated (or if we can do away with this repo entirely and just use upstream, but I know that requires a bit of work to achieve).

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.