Skip to content

[Backport release-25.05] victoriametrics: 1.119.0 -> 1.127.0#450723

Merged
yorickvP merged 9 commits intoNixOS:release-25.05from
transcaffeine:backport-victoriametrics/1.127.0-to-release-25.05
Oct 11, 2025
Merged

[Backport release-25.05] victoriametrics: 1.119.0 -> 1.127.0#450723
yorickvP merged 9 commits intoNixOS:release-25.05from
transcaffeine:backport-victoriametrics/1.127.0-to-release-25.05

Conversation

@transcaffeine
Copy link
Member

Backport of:

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

transcaffeine and others added 9 commits October 10, 2025 18:09
(cherry picked from commit 2e547a5)
(cherry picked from commit 9a38b16)
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 4.workflow: backport This targets a stable branch labels Oct 10, 2025
Copy link
Contributor

@Shawn8901 Shawn8901 left a comment

Choose a reason for hiding this comment

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

havent tested, but diff lgtm

General question (as some of the bumps are from me), so i think we agree on backporting all bumps for VictoriaMetrics (and also maybe VictoriaLogs)?

@transcaffeine
Copy link
Member Author

Yes, i just forgot to put the backport label too often and ended up just cherrypicking the 'queue' into a single PR.

Regarding victorialogs, i could take a look at backporting any missing PRs later

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels Oct 10, 2025
Copy link
Member

@leona-ya leona-ya left a comment

Choose a reason for hiding this comment

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

I checked the release notes, and found nothing breaking in there. So this is fine more me.

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 2 This PR was reviewed and approved by two persons. and removed 12.approvals: 1 This PR was reviewed and approved by one person. labels Oct 10, 2025
@yorickvP
Copy link
Contributor

@ofborg build victoriametrics, victoriametrics.passthru.tests

@yorickvP yorickvP added this pull request to the merge queue Oct 11, 2025
Merged via the queue into NixOS:release-25.05 with commit d1f24b5 Oct 11, 2025
34 of 41 checks passed
@Shawn8901
Copy link
Contributor

Shawn8901 commented Oct 11, 2025

Regarding victorialogs, i could take a look at backporting any missing PRs later

forget about that, i was looking at it and noticed that the init for victorialogs standalone package was after branch-off for 25.05, thus a breaking change, and VL was removed from the repo in 1.122.0 (#427162) (AFAIK after we init'ed the standalone package in unstable).
Which is forgot about, i thought that the split was before 25.05...

So we basically removed the victorialogs from stable here.
So the only way to revive it is either by reverting back to 1.121.0 (which is out of date) or introduce the victorialogs package in overall (with the breaking change for the modules default package).
The oldest LTS version is 1.110.x, which is even older than what we had before.

Should i create backports for the victorialogs package code init to bring it back?

@transcaffeine
Copy link
Member Author

I think backporting the VL package init would be the cleanest way.

@Shawn8901
Copy link
Contributor

see #451137

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4.workflow: backport This targets a stable branch 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 12.approvals: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants