Skip to content

python3Packages.pgvector: temporarily disable checkPhase#426048

Merged
wolfgangwalther merged 2 commits intoNixOS:masterfrom
wolfgangwalther:python-pgvector
Jul 17, 2025
Merged

python3Packages.pgvector: temporarily disable checkPhase#426048
wolfgangwalther merged 2 commits intoNixOS:masterfrom
wolfgangwalther:python-pgvector

Conversation

@wolfgangwalther
Copy link
Contributor

@wolfgangwalther wolfgangwalther commented Jul 17, 2025

Let's disable the checkPhase to unbreak the package and its dependencies until #425384 is properly solved.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other contributing documentation in corresponding paths.

Add a 👍 reaction to pull requests you find important.

@wolfgangwalther
python3Packages.pgvector: temporarily disable checkPhase
39767f9 
Let's disable the checkPhase to unbreak the package and its dependencies
until NixOS#425384 is properly solved.
@wolfgangwalther wolfgangwalther mentioned this pull request Jul 17, 2025
Closed
13 tasks
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 6.topic: python Python is a high-level, general-purpose programming language. labels Jul 17, 2025
@nix-owners nix-owners bot requested a review from natsukium July 17, 2025 09:49
@wolfgangwalther
Copy link
Contributor Author

wolfgangwalther commented Jul 17, 2025

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 426048 --skip-package-regex .*private-gpt.*
Commit: 39767f902ebfc8ecd9722ee085cad51e0100aac7

x86_64-linux

❌ 16 packages failed to build:
  • python312Packages.langgraph
  • python312Packages.langgraph-checkpoint-postgres
  • python312Packages.langgraph-checkpoint-postgres.dist
  • python312Packages.langgraph-prebuilt
  • python312Packages.langgraph-prebuilt.dist
  • python312Packages.langgraph.dist
  • python312Packages.txtai
  • python312Packages.txtai.dist
  • python313Packages.langgraph
  • python313Packages.langgraph-checkpoint-postgres
  • python313Packages.langgraph-checkpoint-postgres.dist
  • python313Packages.langgraph-prebuilt
  • python313Packages.langgraph-prebuilt.dist
  • python313Packages.langgraph.dist
  • python313Packages.txtai
  • python313Packages.txtai.dist
✅ 10 packages built:
  • open-webui
  • open-webui.dist
  • python312Packages.llama-index-vector-stores-postgres
  • python312Packages.llama-index-vector-stores-postgres.dist
  • python312Packages.pgvector
  • python312Packages.pgvector.dist
  • python313Packages.llama-index-vector-stores-postgres
  • python313Packages.llama-index-vector-stores-postgres.dist
  • python313Packages.pgvector
  • python313Packages.pgvector.dist

aarch64-linux

❌ 16 packages failed to build:
  • python312Packages.langgraph
  • python312Packages.langgraph-checkpoint-postgres
  • python312Packages.langgraph-checkpoint-postgres.dist
  • python312Packages.langgraph-prebuilt
  • python312Packages.langgraph-prebuilt.dist
  • python312Packages.langgraph.dist
  • python312Packages.txtai
  • python312Packages.txtai.dist
  • python313Packages.langgraph
  • python313Packages.langgraph-checkpoint-postgres
  • python313Packages.langgraph-checkpoint-postgres.dist
  • python313Packages.langgraph-prebuilt
  • python313Packages.langgraph-prebuilt.dist
  • python313Packages.langgraph.dist
  • python313Packages.txtai
  • python313Packages.txtai.dist
✅ 8 packages built:
  • python312Packages.llama-index-vector-stores-postgres
  • python312Packages.llama-index-vector-stores-postgres.dist
  • python312Packages.pgvector
  • python312Packages.pgvector.dist
  • python313Packages.llama-index-vector-stores-postgres
  • python313Packages.llama-index-vector-stores-postgres.dist
  • python313Packages.pgvector
  • python313Packages.pgvector.dist

aarch64-darwin

✅ 10 packages built:
  • open-webui
  • open-webui.dist
  • python312Packages.llama-index-vector-stores-postgres
  • python312Packages.llama-index-vector-stores-postgres.dist
  • python312Packages.pgvector
  • python312Packages.pgvector.dist
  • python313Packages.llama-index-vector-stores-postgres
  • python313Packages.llama-index-vector-stores-postgres.dist
  • python313Packages.pgvector
  • python313Packages.pgvector.dist
Error logs: `x86_64-linux`
python312Packages.langgraph-checkpoint-postgres 
checking for references to /build/ in /nix/store/4wv2sxdfbin29218blmlnh86illnd3qw-python3.12-langgraph-checkpoint-postgres-2.0.21-dist...
patching script interpreter paths in /nix/store/4wv2sxdfbin29218blmlnh86illnd3qw-python3.12-langgraph-checkpoint-postgres-2.0.21-dist
Executing pythonRemoveTestsDir
Finished executing pythonRemoveTestsDir
Running phase: installCheckPhase
@nix { "action": "setPhase", "phase": "installCheckPhase" }
no Makefile or custom installCheckPhase, doing nothing
Running phase: pythonCatchConflictsPhase
@nix { "action": "setPhase", "phase": "pythonCatchConflictsPhase" }
Running phase: pythonRemoveBinBytecodePhase
@nix { "action": "setPhase", "phase": "pythonRemoveBinBytecodePhase" }
Running phase: pythonImportsCheckPhase
@nix { "action": "setPhase", "phase": "pythonImportsCheckPhase" }
Executing pythonImportsCheckPhase
Check whether the following modules can be imported: langgraph.checkpoint.postgres
Running phase: pytestCheckPhase
@nix { "action": "setPhase", "phase": "pytestCheckPhase" }
Executing pytestCheckPhase
/nix/store/04xc8i3h5qvbl17fm6dw9dfk3ls0fi5b-postgresql-test-hook/nix-support/setup-hook: line 52: type: initdb: not found
initdb not found. Did you add postgresql to the nativeCheckInputs?
python313Packages.langgraph-checkpoint-postgres 
checking for references to /build/ in /nix/store/9xpqa804sngaal57i46y4qhrjgq19lcl-python3.13-langgraph-checkpoint-postgres-2.0.21-dist...
patching script interpreter paths in /nix/store/9xpqa804sngaal57i46y4qhrjgq19lcl-python3.13-langgraph-checkpoint-postgres-2.0.21-dist
Executing pythonRemoveTestsDir
Finished executing pythonRemoveTestsDir
Running phase: installCheckPhase
@nix { "action": "setPhase", "phase": "installCheckPhase" }
no Makefile or custom installCheckPhase, doing nothing
Running phase: pythonCatchConflictsPhase
@nix { "action": "setPhase", "phase": "pythonCatchConflictsPhase" }
Running phase: pythonRemoveBinBytecodePhase
@nix { "action": "setPhase", "phase": "pythonRemoveBinBytecodePhase" }
Running phase: pythonImportsCheckPhase
@nix { "action": "setPhase", "phase": "pythonImportsCheckPhase" }
Executing pythonImportsCheckPhase
Check whether the following modules can be imported: langgraph.checkpoint.postgres
Running phase: pytestCheckPhase
@nix { "action": "setPhase", "phase": "pytestCheckPhase" }
Executing pytestCheckPhase
/nix/store/04xc8i3h5qvbl17fm6dw9dfk3ls0fi5b-postgresql-test-hook/nix-support/setup-hook: line 52: type: initdb: not found
initdb not found. Did you add postgresql to the nativeCheckInputs?
Error logs: `aarch64-linux`
python312Packages.langgraph-checkpoint-postgres 
checking for references to /build/ in /nix/store/zfhm3wm716sa5pwxw867jnlgnnk461dq-python3.12-langgraph-checkpoint-postgres-2.0.21-dist...
patching script interpreter paths in /nix/store/zfhm3wm716sa5pwxw867jnlgnnk461dq-python3.12-langgraph-checkpoint-postgres-2.0.21-dist
Executing pythonRemoveTestsDir
Finished executing pythonRemoveTestsDir
Running phase: installCheckPhase
@nix {"action":"setPhase","phase":"installCheckPhase"}
no Makefile or custom installCheckPhase, doing nothing
Running phase: pythonCatchConflictsPhase
@nix {"action":"setPhase","phase":"pythonCatchConflictsPhase"}
Running phase: pythonRemoveBinBytecodePhase
@nix {"action":"setPhase","phase":"pythonRemoveBinBytecodePhase"}
Running phase: pythonImportsCheckPhase
@nix {"action":"setPhase","phase":"pythonImportsCheckPhase"}
Executing pythonImportsCheckPhase
Check whether the following modules can be imported: langgraph.checkpoint.postgres
Running phase: pytestCheckPhase
@nix {"action":"setPhase","phase":"pytestCheckPhase"}
Executing pytestCheckPhase
/nix/store/acnrn6sjipn0g53hkp93l83rzxd49bj2-postgresql-test-hook/nix-support/setup-hook: line 52: type: initdb: not found
initdb not found. Did you add postgresql to the nativeCheckInputs?
python313Packages.langgraph-checkpoint-postgres 
checking for references to /build/ in /nix/store/5xlmjdrnq3lj636x5nkxx9mshdcq3jgn-python3.13-langgraph-checkpoint-postgres-2.0.21-dist...
patching script interpreter paths in /nix/store/5xlmjdrnq3lj636x5nkxx9mshdcq3jgn-python3.13-langgraph-checkpoint-postgres-2.0.21-dist
Executing pythonRemoveTestsDir
Finished executing pythonRemoveTestsDir
Running phase: installCheckPhase
@nix {"action":"setPhase","phase":"installCheckPhase"}
no Makefile or custom installCheckPhase, doing nothing
Running phase: pythonCatchConflictsPhase
@nix {"action":"setPhase","phase":"pythonCatchConflictsPhase"}
Running phase: pythonRemoveBinBytecodePhase
@nix {"action":"setPhase","phase":"pythonRemoveBinBytecodePhase"}
Running phase: pythonImportsCheckPhase
@nix {"action":"setPhase","phase":"pythonImportsCheckPhase"}
Executing pythonImportsCheckPhase
Check whether the following modules can be imported: langgraph.checkpoint.postgres
Running phase: pytestCheckPhase
@nix {"action":"setPhase","phase":"pytestCheckPhase"}
Executing pytestCheckPhase
/nix/store/acnrn6sjipn0g53hkp93l83rzxd49bj2-postgresql-test-hook/nix-support/setup-hook: line 52: type: initdb: not found
initdb not found. Did you add postgresql to the nativeCheckInputs?

@wolfgangwalther
Copy link
Contributor Author

wolfgangwalther commented Jul 17, 2025

This already unbreaks a few things, but python3Packages.langgraph-checkpoint-postgres needs the same.

@nix-owners nix-owners bot requested review from drupol and sarahec July 17, 2025 12:53
JohnRTitor
JohnRTitor approved these changes Jul 17, 2025
View reviewed changes
Copy link
Member

@JohnRTitor JohnRTitor left a comment

Choose a reason for hiding this comment

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

this is not a work around we should normally use, but fine.

@wolfgangwalther
Copy link
Contributor Author

wolfgangwalther commented Jul 17, 2025

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 426048 --skip-package-regex .*private-gpt.*
Commit: 6dee2b6377c673da32fdc08289f59c304a77043f

x86_64-linux

❌ 4 packages failed to build:
  • python312Packages.txtai
  • python312Packages.txtai.dist
  • python313Packages.txtai
  • python313Packages.txtai.dist
✅ 22 packages built:
  • open-webui
  • open-webui.dist
  • python312Packages.langgraph
  • python312Packages.langgraph-checkpoint-postgres
  • python312Packages.langgraph-checkpoint-postgres.dist
  • python312Packages.langgraph-prebuilt
  • python312Packages.langgraph-prebuilt.dist
  • python312Packages.langgraph.dist
  • python312Packages.llama-index-vector-stores-postgres
  • python312Packages.llama-index-vector-stores-postgres.dist
  • python312Packages.pgvector
  • python312Packages.pgvector.dist
  • python313Packages.langgraph
  • python313Packages.langgraph-checkpoint-postgres
  • python313Packages.langgraph-checkpoint-postgres.dist
  • python313Packages.langgraph-prebuilt
  • python313Packages.langgraph-prebuilt.dist
  • python313Packages.langgraph.dist
  • python313Packages.llama-index-vector-stores-postgres
  • python313Packages.llama-index-vector-stores-postgres.dist
  • python313Packages.pgvector
  • python313Packages.pgvector.dist

aarch64-linux

❌ 4 packages failed to build:
  • python312Packages.txtai
  • python312Packages.txtai.dist
  • python313Packages.txtai
  • python313Packages.txtai.dist
✅ 20 packages built:
  • python312Packages.langgraph
  • python312Packages.langgraph-checkpoint-postgres
  • python312Packages.langgraph-checkpoint-postgres.dist
  • python312Packages.langgraph-prebuilt
  • python312Packages.langgraph-prebuilt.dist
  • python312Packages.langgraph.dist
  • python312Packages.llama-index-vector-stores-postgres
  • python312Packages.llama-index-vector-stores-postgres.dist
  • python312Packages.pgvector
  • python312Packages.pgvector.dist
  • python313Packages.langgraph
  • python313Packages.langgraph-checkpoint-postgres
  • python313Packages.langgraph-checkpoint-postgres.dist
  • python313Packages.langgraph-prebuilt
  • python313Packages.langgraph-prebuilt.dist
  • python313Packages.langgraph.dist
  • python313Packages.llama-index-vector-stores-postgres
  • python313Packages.llama-index-vector-stores-postgres.dist
  • python313Packages.pgvector
  • python313Packages.pgvector.dist

aarch64-darwin

✅ 10 packages built:
  • open-webui
  • open-webui.dist
  • python312Packages.llama-index-vector-stores-postgres
  • python312Packages.llama-index-vector-stores-postgres.dist
  • python312Packages.pgvector
  • python312Packages.pgvector.dist
  • python313Packages.llama-index-vector-stores-postgres
  • python313Packages.llama-index-vector-stores-postgres.dist
  • python313Packages.pgvector
  • python313Packages.pgvector.dist

@wolfgangwalther
Copy link
Contributor Author

wolfgangwalther commented Jul 17, 2025

This is literally a workaround we use almost all the time - disabling tests when they are broken somehow :)

@wolfgangwalther wolfgangwalther merged commit caa49d2 into NixOS:master Jul 17, 2025
24 of 27 checks passed
@wolfgangwalther wolfgangwalther deleted the python-pgvector branch July 17, 2025 13:05
@lostmsu
Copy link
Contributor

lostmsu commented Jul 17, 2025

Can I set this in my configuration.nix directly somehow?

@aviallon
Copy link
Contributor

aviallon commented Jul 19, 2025

Can I set this in my configuration.nix directly somehow?

The easiest thing to do, quite frankly, is to simply soft-fork the nixos-unstable branch, and cherry-pick the commit of interest into it.
Then, when you want to update, simply rebase on nixos-unstable again.
It honestly is much more time effective than doing overlays and overrides.

@wolfgangwalther wolfgangwalther mentioned this pull request Jul 19, 2025
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JohnRTitor JohnRTitor JohnRTitor approved these changes

@B4dM4n B4dM4n Awaiting requested review from B4dM4n

@natsukium natsukium Awaiting requested review from natsukium

@sarahec sarahec Awaiting requested review from sarahec

@drupol drupol Awaiting requested review from drupol

Assignees

No one assigned

Labels

6.topic: python Python is a high-level, general-purpose programming language. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@wolfgangwalther @lostmsu @aviallon @JohnRTitor