fix: set output_plugin_libraries for gated PG minor versions - #452
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (6)
📝 WalkthroughWalkthroughThe change makes default PostgreSQL GUCs version-aware. Patroni configuration passes the PostgreSQL version to ChangesVersion-aware PostgreSQL defaults
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
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. Comment |
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | -1 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
jason-lynch
left a comment
There was a problem hiding this comment.
Perfect! Thank you for doing this.
* fix: set output_plugin_libraries for gated PG minor versions
Summary
This PR updates the version manifest to support Spock
5.0.11and configuresoutput_plugin_librariesto includespock_outputfor PostgreSQL versions16.15+, 17.11+, and 18.5+, where it is now required for replication slot creation.Changes
postgres.DefaultGUCs()to accept a*ds.PgEdgeVersionand automatically includeoutput_plugin_libraries= "pgoutput, test_decoding, spock_output" forPostgreSQL 16.15+, 17.11+, 18.5+,and newer major versions.PgEdgeVersionfield toPatroniConfigGenerator, initialized from the instance spec in NewPatroniConfigGenerator and propagated toDefaultGUCs()through parameters().postgresql_conf.output_plugin_librariesvalues continue to take precedence since spec parameters are merged last.Spock 5.0.11images for the16.14, 17.10, and 18.4entries.Testing
go test ./server/internal/postgres/... ./server/internal/orchestrator/common/... ./server/internal/orchestrator/swarm/...gucs_test.gocovering both sides of each version gate (16.14/16.15, 17.10/17.11, 18.4/18.5, a future major, and a nil version).18.5version pointing at the real18.4-spock5.0.11image, since no genuinely patched PG image exists yet:patroni.yaml's generated parameters includeoutput_plugin_libraries: pgoutput, test_decoding, spock_outputfor a database declared aspostgres_version: 18.5.postgresql_conf.output_plugin_librariesoverride in the database spec is honored instead of the default.Checklist
PLAT-721