Skip to content

korrect: 0.2.1 -> 0.3.3#460269

Merged
Aleksanaa merged 1 commit intoNixOS:masterfrom
dwt:update-korrect
Nov 24, 2025
Merged

korrect: 0.2.1 -> 0.3.3#460269
Aleksanaa merged 1 commit intoNixOS:masterfrom
dwt:update-korrect

Conversation

@dwt
Copy link
Contributor

@dwt dwt commented Nov 10, 2025

This release adds the ability to also get the correct version of the helm binary as well. Now both kubectl as well as helm use the version that is best suited for the cluster at hand, auto switching to the latest compatible release if available.

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.

@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. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Nov 10, 2025
@dwt dwt force-pushed the update-korrect branch 2 times, most recently from 988637a to 7c91e78 Compare November 15, 2025 10:22
@dwt dwt changed the title korrect: 0.2.1 -> 0.3.1 korrect: 0.2.1 -> 0.3.3 Nov 15, 2025
@dwt
Copy link
Contributor Author

dwt commented Nov 15, 2025

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 460269
Commit: 7c91e78bd671565d3342499f6cf2807fe956059d


x86_64-linux

❌ 1 package failed to build:
  • korrect

aarch64-linux

✅ 1 package built:
  • korrect

x86_64-darwin

✅ 1 package built:
  • korrect

aarch64-darwin

✅ 1 package built:
  • korrect

Not sure what's going on with the x86_64-linux build, manually building that with nix-build --attr korrect --eval-system x86_64-linux works fine.

@dwt
Copy link
Contributor Author

dwt commented Nov 15, 2025

The linux build fails with:

       Reason: builder failed with exit code 1.
       Output paths:
         /nix/store/n6gidz7rcxb7drwvhg4k2igfjhwd56d0-korrect-0.3.3-vendor
       Last 4 log lines:
       > Traceback (most recent call last):
       >   File "/nix/store/f2aiivbcplcfk3c30dmjgl0iw96yvwaz-fetch-cargo-vendor-util/bin/fetch-cargo-vendor-util", line 16, in <module>
       >     import requests
       > ModuleNotFoundError: No module named 'requests'

Which seems to me to mean that it's somehow a nix tooling problem.

This release adds the ability to also get the correct version of helm
@dwt
Copy link
Contributor Author

dwt commented Nov 15, 2025

After a rebase I can now reproduce this outside of nixpkgs-reveiw, I'll investigate and file bugs. I'm still pretty confident that this is an upstream problem and not a fault of this PR.

@dwt
Copy link
Contributor Author

dwt commented Nov 15, 2025

see #461884 for debugging of the build failure

Copy link
Contributor

@eclairevoyant eclairevoyant left a comment

Choose a reason for hiding this comment

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

Approved automatically following the successful run of nixpkgs-review.

@eclairevoyant
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 460269
Commit: a40339417f4b61af54d166cfa142b97d9b44d8aa


x86_64-linux

✅ 1 package built:
  • korrect

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Nov 15, 2025
@dwt
Copy link
Contributor Author

dwt commented Nov 15, 2025

@eclairevoyant Ok, so it's pretty likely that this is an issue with my rosetta based linux builder. Thanks!

@dwt dwt marked this pull request as ready for review November 15, 2025 12:30
@dwt
Copy link
Contributor Author

dwt commented Nov 24, 2025

yes, that linux builder was the reason, see fix at #463106

@Aleksanaa Aleksanaa added this pull request to the merge queue Nov 24, 2025
Merged via the queue into NixOS:master with commit 7b23d0d Nov 24, 2025
30 of 31 checks passed
@dwt dwt deleted the update-korrect branch November 24, 2025 14:33
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-already-reviewed/2617/2660

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

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants