Skip to content

Restore packer pkg#2031

Merged
samrose merged 3 commits intodevelopfrom
restore-packer-pkg
Jan 30, 2026
Merged

Restore packer pkg#2031
samrose merged 3 commits intodevelopfrom
restore-packer-pkg

Conversation

@samrose
Copy link
Collaborator

@samrose samrose commented Jan 30, 2026

We should not have removed packer package, so that we can control the version, and to avoid non-free license errors.

Summary by CodeRabbit

  • Chores
    • Updated PostgreSQL 17 to 17.6.1.078
    • Updated PostgreSQL 15 to 15.14.1.078
    • Updated PostgreSQL with OrioLeDB to 17.6.0.035-orioledb
  • New Features
    • Added a packaged Packer tool to the build environment
    • Switched AMI and base-image workflows to use the new Packer packaging/source for builds

✏️ 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 20:57
Copy link
Contributor

@jchancojr jchancojr left a comment

Choose a reason for hiding this comment

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

LGTM

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 30, 2026

📝 Walkthrough

Walkthrough

Adds a local Nix package for HashiCorp Packer (v1.14.1) at nix/packages/packer.nix, exports it via nix/packages/default.nix, and injects it into the build-ami and build-test-ami invocations. CI workflows (.github/workflows/ami-release-nix.yml, .github/workflows/base-image-nightly.yml) are changed to source packer via github:supabase/postgres/${GIT_SHA}#packer instead of nixpkgs#packer. Bumps three Postgres-related version strings in ansible/vars.yml (postgresorioledb-17, postgres17, postgres15).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested labels

ci, nix, infrastructure

Suggested reviewers

  • hunleyd
  • LGUG2Z
  • Crispy1975
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is brief but clearly explains the rationale (control version, avoid license errors). However, the repository template requires selecting a sub-template, which appears incomplete. Select and complete the appropriate description template (Default or Extension Upgrade) with all required sections rather than providing only a brief statement.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Restore packer pkg' directly relates to the main change: reintroducing the packer package to control its version and avoid license errors.
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 restore-packer-pkg

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 enabled auto-merge January 30, 2026 21:04
@samrose samrose disabled auto-merge January 30, 2026 21:23
@samrose samrose added this pull request to the merge queue Jan 30, 2026
Merged via the queue into develop with commit d411fcd Jan 30, 2026
25 checks passed
@samrose samrose deleted the restore-packer-pkg branch January 30, 2026 22:52
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.

3 participants