Conversation
PostgreSQL Extension Dependency Analysis: PR #2460
SummaryNo extensions had dependencies with MAJOR version updates. Full Analysis ResultsPostgreSQL 15 Extension DependenciesPostgreSQL 17 Extension DependenciesOrioleDB 17 Extension Dependencies |
PostgreSQL Package Dependency Analysis: PR #2460
SummaryNo packages had MAJOR version updates. Full Analysis ResultsPostgreSQL 15 Dependency ChangesExtracting PostgreSQL 15 dependencies...
Runtime Closure Size
Raw Dependency ClosurePostgreSQL 17 Dependency ChangesExtracting PostgreSQL 17 dependencies...
Runtime Closure Size
Raw Dependency Closure |
mmlb
force-pushed
the
push-znnqzquvpnxp
branch
6 times, most recently
from
September 21, 2026 13:58
f3210db to
1a74e61
Compare
I noticed that there was a bunch of dead code here so delete it. dbmate-tool doesn't need hcl files nor ARGS/result so those go away. The steps creating POSTGRES_MAJOR_VERSION and PGMAJOR were also completely unnecessary as the output from prepare step/matrix value is usable as is, I decided to rename it so it uses the common named used throughout the repo for consistency.
No point running this on its own *and* as part of nix-build so lets drop the non-workflow-call cases.
All these names have a long common prefix which means the GHA UI truncates the most interesting information. So I'm renaming these so the Summary view is more informative at a glance. I also tweaked some `needs` that didn't really need to be explicit.
These are all ephemeral instances so no need to stop anything.
parallel-eval supported by detsys-nix is much faster than nix-eval-jobs which I'll swtich to in the next commit. Assisted-By: Claude Sonnet 5 <noreply@anthropic.com>
This way we detect stalled downloads and retry faster, retry uses range-request so its not from scratch.
Splits the single nix-eval job into a per-system matrix each on its
matching blacksmith runner. This replaces nix-eval-jobs with the
parallel-eval prototype Determinate Nix.
Each nix-build-{packages,checks}-<system> job now depends directly on
nix-eval-<system>, so one system's builds no longer wait on the other
two evaluations (or on a merge step).
Assisted-By: Claude Fable 5.1 <noreply@anthropic.com>
Now that we have per-system nix-eval lets split other jobs too. This way we don't have artificial dependencies that hold back otherwise ready jobs. Report disk usage job now needs to be in nix-build since testinfra-ami-build runs once per arch. Assisted-By: Claude Fable 5.1 <noreply@anthropic.com>
mmlb
force-pushed
the
push-znnqzquvpnxp
branch
from
September 21, 2026 22:21
1a74e61 to
98747f7
Compare
Every job in the graph is now keyed by (pg major, system): 15, 17 and orioledb-17, plus a "common" group for the attrs that aren't versioned (the ext-* VM tests, wal-g, cli-smoke-test, ...). The eval script takes the pg version and only walks the top-level attrs whose name carries it, so each nix-eval-<pgver>-<system> job evaluates a quarter of the tree and the build/check jobs downstream depend only on their own group. legacyPackages has no unversioned attrs, so common only has a checks job. test.yml and testinfra-ami-build.yml take a required postgres_version input instead of reading the list from ansible/vars.yml, and are invoked once per (major, arch). Assisted-By: Claude Fable 5.1 <noreply@anthropic.com>
github_matrix.py no longer ran nix-eval-jobs, only reshaped JSONL, but `nix run .#github-matrix` still pulled in nix-eval-jobs and thereby a full from-source Nix 2.33 closure (~5-9 min per eval job in the last run, incl. a stalled S3 download). The same logic is ~60 lines of jq wrapped in a bash+jq-only package. Fixtures under nix/packages/github-matrix/tests cover the evaluation-error annotations, cacheStatus/dedupe/placeholder handling and the runner routing per system; they run in the package's checkPhase (and so in `nix flake check`). Assisted-By: Claude Fable 5.1 <noreply@anthropic.com>
mmlb
force-pushed
the
push-znnqzquvpnxp
branch
from
September 21, 2026 22:27
98747f7 to
683bb7e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What kind of change does this PR introduce?
Bug fix, feature, docs update, ...
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.