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

python312Packages.rtslib: 2.2.0 -> 2.2.2 #389871

Merged
merged 6 commits into from
Mar 21, 2025

Conversation

r-ryantm
Copy link
Contributor

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.

meta.description for python312Packages.rtslib is: Python object API for managing the Linux LIO kernel target

meta.homepage for python312Packages.rtslib is: https://github.com/open-iscsi/rtslib-fb

Updates performed
  • Ran passthru.UpdateScript
To inspect upstream changes
Impact

Checks done


  • built on NixOS
  • The tests defined in passthru.tests, if any, passed
  • found 2.2.2 with grep in /nix/store/lfkspdvcl8z3n6h8pf9hjhnn3ibgk2ps-python3.12-rtslib-2.2.2
  • found 2.2.2 in filename of file in /nix/store/lfkspdvcl8z3n6h8pf9hjhnn3ibgk2ps-python3.12-rtslib-2.2.2

Rebuild report (if merged into master) (click to expand)
4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

python312Packages.rtslib
python313Packages.rtslib
targetcli
Instructions to test this update (click to expand)

Either download from the cache:

nix-store -r /nix/store/lfkspdvcl8z3n6h8pf9hjhnn3ibgk2ps-python3.12-rtslib-2.2.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.

Or, build yourself:

nix-build -A python312Packages.rtslib https://github.com/r-ryantm/nixpkgs/archive/52ab453dd53272b6623459bfb3e742d49c7f211d.tar.gz

Or:

nix build github:r-ryantm/nixpkgs/52ab453dd53272b6623459bfb3e742d49c7f211d#python312Packages.rtslib

After you've downloaded or built it, look at the files and if there are any, run the binaries:

ls -la /nix/store/lfkspdvcl8z3n6h8pf9hjhnn3ibgk2ps-python3.12-rtslib-2.2.2
ls -la /nix/store/lfkspdvcl8z3n6h8pf9hjhnn3ibgk2ps-python3.12-rtslib-2.2.2/bin


Pre-merge build results

We have automatically built all packages that will get rebuilt due to
this change.

This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review


x86_64-linux

✅ 6 packages built:
  • python312Packages.rtslib
  • python312Packages.rtslib.dist
  • python313Packages.rtslib
  • python313Packages.rtslib.dist
  • targetcli
  • targetcli.dist

Maintainer pings

Tip

As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.


Add a 👍 reaction to pull requests you find important.

@github-actions github-actions bot added 6.topic: python 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 labels Mar 14, 2025
@Defelo
Copy link
Member

Defelo commented Mar 14, 2025

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 389871

Logs: https://github.com/Defelo/nixpkgs-review-gha/actions/runs/13863041888

Download packages from cache:
  • x86_64-linux
    nix-store -r --add-root nixpkgs-pr-389871-x86_64-linux \
      --option binary-caches 'https://cache.nixos.org/ https://attic.defelo.de/nixpkgs' \
      --option trusted-public-keys '
      cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
      nixpkgs:xeaAWa3crK09hMmFiygBeRmLq3hUjUShgaAwYVUEtw0=
      ' \
      /nix/store/lfkspdvcl8z3n6h8pf9hjhnn3ibgk2ps-python3.12-rtslib-2.2.2 \
      /nix/store/2pzyn4srqfccnnngddy79bbgfmkzf9wi-python3.12-rtslib-2.2.2-dist \
      /nix/store/b0488m51xvvhl87yzy495pnqvxl24v48-python3.13-rtslib-2.2.2 \
      /nix/store/3nn259s6igv6zgzg08fgnnyc6hhy5r6q-python3.13-rtslib-2.2.2-dist \
      /nix/store/wfs3izrwhh23wdq8v7d72y42wp7zw66v-targetcli-2.1.58 \
      /nix/store/b7j4a06gaffp5gmygh32qh497j0r6k7d-targetcli-2.1.58-dist
  • aarch64-linux
    nix-store -r --add-root nixpkgs-pr-389871-aarch64-linux \
      --option binary-caches 'https://cache.nixos.org/ https://attic.defelo.de/nixpkgs' \
      --option trusted-public-keys '
      cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
      nixpkgs:xeaAWa3crK09hMmFiygBeRmLq3hUjUShgaAwYVUEtw0=
      ' \
      /nix/store/gacxc9jh4r8kf8h00mbhmsnr8j103rdd-python3.12-rtslib-2.2.2 \
      /nix/store/dal4kph93lasbyb7hpmgnx0lzxd83sck-python3.12-rtslib-2.2.2-dist \
      /nix/store/zgza5vcgl45xf8h4ymmhgy9piybj6wys-python3.13-rtslib-2.2.2 \
      /nix/store/51gdgdr6rmp1byqv2n3r5hsjy7ls5dxp-python3.13-rtslib-2.2.2-dist \
      /nix/store/s15mjmi3dxzfk7mjgnps0y78psbakxpb-targetcli-2.1.58 \
      /nix/store/iwhkppjv15zvc9yj1fxa841by76pd52l-targetcli-2.1.58-dist

x86_64-linux

✅ 6 packages built:
  • python312Packages.rtslib
  • python312Packages.rtslib.dist
  • python313Packages.rtslib
  • python313Packages.rtslib.dist
  • targetcli
  • targetcli.dist

aarch64-linux

✅ 6 packages built:
  • python312Packages.rtslib
  • python312Packages.rtslib.dist
  • python313Packages.rtslib
  • python313Packages.rtslib.dist
  • targetcli
  • targetcli.dist

Copy link
Member

@Defelo Defelo 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.

@GaetanLepage
Copy link
Contributor

The package has been renamed to rtslib-fb, we should operate this rename too.
(Including adding an entry in python-aliases.nix.

@GaetanLepage GaetanLepage marked this pull request as draft March 14, 2025 19:05
@GaetanLepage GaetanLepage force-pushed the auto-update/python312Packages.rtslib branch from 52ab453 to 1290585 Compare March 14, 2025 21:10
@GaetanLepage GaetanLepage marked this pull request as ready for review March 14, 2025 21:12
@nix-owners nix-owners bot requested a review from natsukium March 14, 2025 21:13
@Defelo
Copy link
Member

Defelo commented Mar 14, 2025

@GaetanLepage I created a pr for this at #389906, you should have received a review request

@GaetanLepage GaetanLepage force-pushed the auto-update/python312Packages.rtslib branch from 1290585 to 40f1984 Compare March 14, 2025 21:22
@github-actions github-actions bot added 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: module (update) This PR changes an existing module in `nixos/` labels Mar 14, 2025
@GaetanLepage GaetanLepage force-pushed the auto-update/python312Packages.rtslib branch 3 times, most recently from a7e3a92 to bbdac1d Compare March 14, 2025 22:17
@GaetanLepage
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 389871


x86_64-linux

⏩ 2 packages blacklisted:
  • nixos-install-tools
  • tests.nixos-functions.nixos-test
✅ 6 packages built:
  • python312Packages.rtslib-fb
  • python312Packages.rtslib-fb.dist
  • python313Packages.rtslib-fb
  • python313Packages.rtslib-fb.dist
  • targetcli-fb
  • targetcli-fb.dist

aarch64-linux

⏩ 2 packages blacklisted:
  • nixos-install-tools
  • tests.nixos-functions.nixos-test
✅ 6 packages built:
  • python312Packages.rtslib-fb
  • python312Packages.rtslib-fb.dist
  • python313Packages.rtslib-fb
  • python313Packages.rtslib-fb.dist
  • targetcli-fb
  • targetcli-fb.dist

@GaetanLepage GaetanLepage requested a review from NickCao March 14, 2025 22:30
@GaetanLepage GaetanLepage force-pushed the auto-update/python312Packages.rtslib branch from bbdac1d to 87f27cb Compare March 14, 2025 23:42
@GaetanLepage GaetanLepage force-pushed the auto-update/python312Packages.rtslib branch 2 times, most recently from 66842d3 to e7807f5 Compare March 14, 2025 23:46
@NickCao
Copy link
Member

NickCao commented Mar 14, 2025

@ofborg test iscsi-root iscsi-multipath-root

@GaetanLepage GaetanLepage force-pushed the auto-update/python312Packages.rtslib branch 2 times, most recently from 7c18dec to 57fffd6 Compare March 15, 2025 00:23
@GaetanLepage
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 389871


x86_64-linux

⏩ 1 package blacklisted:
  • nixos-install-tools
✅ 13 packages built:
  • nvmet-cli
  • nvmet-cli.dist
  • python312Packages.configshell-fb
  • python312Packages.configshell-fb.dist
  • python312Packages.rtslib-fb
  • python312Packages.rtslib-fb.dist
  • python313Packages.configshell-fb
  • python313Packages.configshell-fb.dist
  • python313Packages.rtslib-fb
  • python313Packages.rtslib-fb.dist
  • spdk
  • targetcli-fb
  • targetcli-fb.dist

aarch64-linux

⏩ 1 package blacklisted:
  • nixos-install-tools
✅ 12 packages built:
  • nvmet-cli
  • nvmet-cli.dist
  • python312Packages.configshell-fb
  • python312Packages.configshell-fb.dist
  • python312Packages.rtslib-fb
  • python312Packages.rtslib-fb.dist
  • python313Packages.configshell-fb
  • python313Packages.configshell-fb.dist
  • python313Packages.rtslib-fb
  • python313Packages.rtslib-fb.dist
  • targetcli-fb
  • targetcli-fb.dist

x86_64-darwin

✅ 4 packages built:
  • python312Packages.configshell-fb
  • python312Packages.configshell-fb.dist
  • python313Packages.configshell-fb
  • python313Packages.configshell-fb.dist

aarch64-darwin

✅ 4 packages built:
  • python312Packages.configshell-fb
  • python312Packages.configshell-fb.dist
  • python313Packages.configshell-fb
  • python313Packages.configshell-fb.dist

@GaetanLepage GaetanLepage requested a review from NickCao March 15, 2025 00:31
@github-actions github-actions bot added 10.rebuild-darwin: 1-10 and removed 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin labels Mar 15, 2025
@nix-owners nix-owners bot requested a review from Hoverbear March 15, 2025 00:31
@NickCao
Copy link
Member

NickCao commented Mar 15, 2025

@ofborg test iscsi-root iscsi-multipath-root

@GaetanLepage GaetanLepage force-pushed the auto-update/python312Packages.rtslib branch from 57fffd6 to 988fef7 Compare March 15, 2025 11:17
@GaetanLepage
Copy link
Contributor

@ofborg test iscsi-root iscsi-multipath-root

@NickCao
Copy link
Member

NickCao commented Mar 15, 2025

iscsi-root passed locally, but iscsi-multipath-root failed.

@GaetanLepage
Copy link
Contributor

iscsi-root passed locally, but iscsi-multipath-root failed.

iscsi-multipath-root is already failing as darwin (as rtslib/bin/targetctl does not exist).
Now it fails with:

initiatorAuto # [   11.790184] iscsiadm[1002]: iscsiadm: cannot make connection to 2001:db8:1::3: Connection refused
(finished: waiting for unit iscsid.service, in 12.72 seconds)
initiatorAuto: waiting for unit iscsi.service
initiatorAuto # [   12.791980] iscsiadm[1002]: iscsiadm: cannot make connection to 2001:db8:1::3: Connection refused
initiatorAuto # [   13.792591] iscsiadm[1002]: iscsiadm: cannot make connection to 2001:db8:1::3: Connection refused
initiatorAuto # [   14.793392] iscsiadm[1002]: iscsiadm: cannot make connection to 2001:db8:1::3: Connection refused
initiatorAuto # [   15.793970] iscsiadm[1002]: iscsiadm: cannot make connection to 2001:db8:1::3: Connection refused
initiatorAuto # [   16.795276] iscsiadm[1002]: iscsiadm: cannot make connection to 2001:db8:1::3: Connection refused
initiatorAuto # [   16.796399] iscsiadm[1002]: iscsiadm: connection login retries (reopen_max) 5 exceeded
initiatorAuto # [   16.797912] iscsiadm[1002]: iscsiadm: Could not perform SendTargets discovery: iSCSI PDU timed out
initiatorAuto # [   16.799454] systemd[1]: iscsi.service: Control process exited, code=exited, status=11/n/a
initiatorAuto # [   16.800606] systemd[1]: iscsi.service: Failed with result 'exit-code'.
initiatorAuto # [   16.802064] systemd[1]: Failed to start Login and scanning of iSCSI devices.
initiatorAuto # [   16.803209] systemd[1]: iscsi.service: Consumed 8ms CPU time, 1.6M memory peak, 360B incoming IP traffic, 480B outgoing IP traffic.
initiatorAuto # [   16.805299] systemd[1]: Reached target Remote File Systems.
initiatorAuto # [   16.807615] systemd[1]: Starting Permit User Sessions...
initiatorAuto # [   16.827594] systemd[1]: Finished Permit User Sessions.
initiatorAuto # [   16.830333] systemd[1]: Started Getty on tty1.
initiatorAuto # [   16.831572] systemd[1]: Reached target Login Prompts.
initiatorAuto # [   16.832590] systemd[1]: Reached target Multi-User System.
initiatorAuto # [   16.833583] systemd[1]: Startup finished in 2.969s (kernel) + 13.861s (userspace) = 16.831s.

Technically it's not a regression, but it's not ideal.
I am not familiar at all with this software, so I'm not sure about what we should do.

@NickCao
Copy link
Member

NickCao commented Mar 15, 2025

cc @ajs124 and @grahamc who created the iscsi/target module

@GaetanLepage
Copy link
Contributor

No answer, should we go ahead and merge?

@NickCao
Copy link
Member

NickCao commented Mar 21, 2025

No answer, should we go ahead and merge?

I guess so.

@NickCao NickCao merged commit d2fcc2f into NixOS:master Mar 21, 2025
32 checks passed
@r-ryantm r-ryantm deleted the auto-update/python312Packages.rtslib branch March 22, 2025 00:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 6.topic: python 8.has: module (update) This PR changes an existing module in `nixos/` 10.rebuild-darwin: 1-10 10.rebuild-linux: 1-10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants