Skip to content

Conversation

@samrose
Copy link
Collaborator

@samrose samrose commented Jan 30, 2026

in previous work we dropped the packer package and use upstream. workflows needed to account for this too

Summary by CodeRabbit

  • Chores
    • Updated PostgreSQL component versions for postgres15, postgres17, and postgresorioledb-17
    • Upgraded build infrastructure tooling configuration to use nixpkgs-based packaging

✏️ Tip: You can customize this high-level summary in your review settings.

@samrose samrose requested review from a team as code owners January 30, 2026 19:00
@samrose samrose enabled auto-merge January 30, 2026 19:02
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 30, 2026

📝 Walkthrough

Walkthrough

This PR updates CI/CD workflows and version configurations. It replaces GitHub-hosted packer invocations with nixpkgs-based alternatives in two workflow files (ami-release-nix.yml and base-image-nightly.yml), and increments version numbers for three PostgreSQL packages in the ansible configuration file (postgresorioledb-17, postgres17, and postgres15).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Suggested labels

ci, infrastructure

Suggested reviewers

  • jfroche
  • staaldraad
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The description briefly explains the context ('dropped the packer package and use upstream') and states what needs to be updated ('workflows needed to account for this'), but lacks specific details about what was changed in which files. Consider adding specific details about which workflow files were modified and how the packer invocations were updated (e.g., from github:supabase/postgres to nixpkgs#packer).
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: packer pkg dropped using upstream' clearly and concisely summarizes the main change: replacing custom packer invocations with upstream nixpkgs packer across workflow files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch packer-fix

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@samrose samrose added this pull request to the merge queue Jan 30, 2026
Merged via the queue into develop with commit 8558ae8 Jan 30, 2026
22 checks passed
@samrose samrose deleted the packer-fix branch January 30, 2026 20:35
This was referenced Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants