DevOps & Packaging Instructions — @DevOps

Scope: packaging/, Containerfile, bitbucket-pipelines.yml, ci/

CI Pipeline Architecture

Bitbucket Pipelines (bitbucket-pipelines.yml)

5 jobs, against PostgreSQL 17 and 18 on Ubuntu 24.04:

Job Purpose PG Versions Dependencies
build Compile with -Wall -Wextra, zero warnings required 17, 18 None
regress Run 84 pg_regress SQL tests (vault provider) 17, 18 build
wallet Run 84 pg_regress SQL tests (local wallet provider) 17, 18 build
tap Run TAP tests with mock Vault 17, 18 build
isolation Run isolation tests (concurrency / MVCC) 17, 18 build
memcheck Valgrind + AddressSanitizer 17, 18 build

Local CI Pipeline (ci/)

Containerized pipeline for pre-push validation. Run via make ci-* targets. See ci/README.md for full documentation.

Container Build (Containerfile)

Base image: postgres:${PG_MAJOR} (parameterized via ARG PG_MAJOR=18)
Required packages: - postgresql-server-dev-${PG_MAJOR} - libssl-dev, libcurl4-openssl-dev - pkg-config, build-essential

The container MUST: 1. Accept PG_MAJOR as a build argument (default: 18) 2. Build the extension from source 3. Install into the PostgreSQL extension directory 4. Run the full regression suite as validation

Packaging Matrix

There is a single package per (format, PG major):

Format Script Package Name Pattern Spec PG Versions
DEB packaging/build_deb.sh postgresql-${PG_MAJOR}-pg-vault-tde packaging/debian/ 17, 18
RPM packaging/build_rpm.sh postgresql${PG_MAJOR}-pg_vault_tde packaging/rpm/pg_vault_tde.spec 17, 18

There used to be additional CPU-specific package variants (-aesni, -vaes, -armce, an unpackaged sve2). They were removed: hardware-accelerated AES (AES-NI, VAES, ARM Crypto Extensions, SVE2) is provided automatically at runtime by OpenSSL’s EVP layer on this single package — the removed TDE_TARGET_ARCH compiler flags/defines were never referenced by any #ifdef/#if defined in src/, so the variant builds never produced a measurably faster .so. See wiki/Performance-and-Tuning.md and src/crypto/pg_vault_tde_hw_accel.c. Do not reintroduce a CPU-specific build without first adding actual #ifdef-gated code that uses it — otherwise it’s dead weight in the packaging matrix again.

Release Checklist

Two kinds of release, and they touch different files.

Patch release (C-only, no SQL objects changed) — e.g. 1.7.0 → 1.7.1: - [ ] Update VERSION (full three-component, e.g. 1.7.1) - [ ] Update both version fields in META.json - [ ] Update packaging/debian/changelog (version + date), Version: in packaging/rpm/pg_vault_tde.spec, VERSION= in packaging/build_rpm.sh, PKG_VERSION= in packaging/build_deb.sh — all to 1.7.1/1.7.1-1 - [ ] Leave pg_vault_tde.control (default_version) and sql/pg_vault_tde--*.sql alone: bumping extversion with no SQL delta only buys an empty upgrade script and a pointless ALTER EXTENSION UPDATE for every user - [ ] make clean before building — plain make will not rebuild already compiled .o files just because VERSION changed, so the version baked in via -DPG_VAULT_TDE_BUILD_VERSION would stay stale - [ ] Verify at runtime: SELECT pg_vault_tde_build_version() reports the new version while pg_extension.extversion still reports the old one - [ ] Tag the release commit

Minor/major release (SQL objects changed) — e.g. 1.7.x → 1.8: - [ ] Everything above, plus: - [ ] Update pg_vault_tde.control (default_version) - [ ] Add sql/pg_vault_tde--<new>.sql and the --<old>--<new>.sql upgrade script; update DATA in the Makefile and provides.file in META.json - [ ] Extend the upgrade chain in ci/scripts/run-regress.sh (phases 2 and 4)

Test Execution Environment

Local CI Pipeline (make ci-*)

All test stages run inside containers built from ci/containers/pg-test.Containerfile. The pipeline auto-detects podman or docker and uses a mock Vault for integration tests.

Command What It Tests
make ci-regress 84-test SQL regression suite (vault provider)
make ci-wallet 84-test SQL regression suite (local wallet provider)
make ci-checksums Same + page checksums (initdb -k)
make ci-tap TAP tests (extension load, backup hooks)
make ci-isolation MVCC / DEK rotation concurrency
make ci-vault Vault Transit API integration (Compose)
make ci-bench Performance benchmark (informational)
make ci-all All of the above in sequence

All commands are idempotent (can be run repeatedly without manual cleanup). Use make ci-clean to remove containers and images.

Dependency Rules

  • DevOps files MUST NOT modify any C source code
  • CI workflows MUST run the SAME test commands as local development
  • Container builds MUST use the same Makefile targets as local builds
  • Package scripts MUST NOT bundle runtime dependencies not declared in control files

Adding a New PostgreSQL Version (Checklist)

When PostgreSQL N+1 becomes GA:

  1. RPM specs: Update %global pgmajorversion or create version-specific spec
  2. DEB control: Update postgresql-XX-pg-vault-tde package name and deps
  3. DEB changelog: Add entry mentioning PG N+1 support
  4. Containerfile: Verify postgres:N+1 base image exists on Docker Hub
  5. CI matrix: Add PG N+1 to GitHub Actions matrix and Bitbucket steps
  6. ci/.env: Add N+1 to PG_SUPPORTED_VERSIONS
  7. Makefile: Update TDE_PG_MAX to N+1
  8. Test: Run PG_VERSION=N+1 make ci-all — all 84 tests must pass
  9. build_deb.sh / build_rpm.sh: Verify scripts work with new PG version
  10. Tag release: Include “Added PG N+1 support” in release notes