Skip to content

Conversation

@maqeel75
Copy link
Member

@maqeel75 maqeel75 commented Jan 5, 2026

Summary by CodeRabbit

  • Chores
    • Updated pgedge-lolor package to version 1.2.2-1.el9 across all supported PostgreSQL versions and architectures (amd64 and arm64).
    • Updated build image epoch values for versioning consistency.

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

@maqeel75 maqeel75 force-pushed the feature/BR-385/lolor-update branch from 3d1df7e to 2469ec1 Compare January 13, 2026 08:03
@coderabbitai
Copy link

coderabbitai bot commented Jan 13, 2026

📝 Walkthrough

Walkthrough

This change updates the pgedge-lolor package version from 1.2.1-1.el9 to 1.2.2-1.el9 across 24 package list configurations spanning multiple PostgreSQL versions and CPU architectures. Additionally, epoch values in the image build script are incremented from 2/3 to 3/4 to reflect these updates.

Changes

Cohort / File(s) Summary
Package Version Updates - amd64
packagelists/amd64/pg16.10-spock5.0.4-*.txt, packagelists/amd64/pg16.11-spock5.0.4-*.txt, packagelists/amd64/pg17.6-spock5.0.4-*.txt, packagelists/amd64/pg17.7-spock5.0.4-*.txt, packagelists/amd64/pg18.0-spock5.0.4-*.txt, packagelists/amd64/pg18.1-spock5.0.4-*.txt
Bumps pgedge-lolor package from version 1.2.1-1.el9 to 1.2.2-1.el9 across 12 amd64 minimal and standard configurations
Package Version Updates - arm64
packagelists/arm64/pg16.10-spock5.0.4-*.txt, packagelists/arm64/pg16.11-spock5.0.4-*.txt, packagelists/arm64/pg17.6-spock5.0.4-*.txt, packagelists/arm64/pg17.7-spock5.0.4-*.txt, packagelists/arm64/pg18.0-spock5.0.4-*.txt, packagelists/arm64/pg18.1-spock5.0.4-*.txt
Bumps pgedge-lolor package from version 1.2.1-1.el9 to 1.2.2-1.el9 across 12 arm64 minimal and standard configurations
Image Build Configuration
scripts/build_pgedge_images.py
Increments epoch values (2/3 → 3/4) in multiple image definitions across several PostgreSQL/SPOCK version blocks to align with package updates

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 hops excitedly
Version bumps cascade,
Lolor 1.2.2 takes the stage!
Across architectures, the versions aligned,
Epochs incremented, a new build designed.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately summarizes the main change: updating the lolor package version from 1.2.1 to 1.2.2 across multiple package list files and configuration.
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 docstrings

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 98dd332 and 2469ec1.

📒 Files selected for processing (25)
  • packagelists/amd64/pg16.10-spock5.0.4-minimal.txt
  • packagelists/amd64/pg16.10-spock5.0.4-standard.txt
  • packagelists/amd64/pg16.11-spock5.0.4-minimal.txt
  • packagelists/amd64/pg16.11-spock5.0.4-standard.txt
  • packagelists/amd64/pg17.6-spock5.0.4-minimal.txt
  • packagelists/amd64/pg17.6-spock5.0.4-standard.txt
  • packagelists/amd64/pg17.7-spock5.0.4-minimal.txt
  • packagelists/amd64/pg17.7-spock5.0.4-standard.txt
  • packagelists/amd64/pg18.0-spock5.0.4-minimal.txt
  • packagelists/amd64/pg18.0-spock5.0.4-standard.txt
  • packagelists/amd64/pg18.1-spock5.0.4-minimal.txt
  • packagelists/amd64/pg18.1-spock5.0.4-standard.txt
  • packagelists/arm64/pg16.10-spock5.0.4-minimal.txt
  • packagelists/arm64/pg16.10-spock5.0.4-standard.txt
  • packagelists/arm64/pg16.11-spock5.0.4-minimal.txt
  • packagelists/arm64/pg16.11-spock5.0.4-standard.txt
  • packagelists/arm64/pg17.6-spock5.0.4-minimal.txt
  • packagelists/arm64/pg17.6-spock5.0.4-standard.txt
  • packagelists/arm64/pg17.7-spock5.0.4-minimal.txt
  • packagelists/arm64/pg17.7-spock5.0.4-standard.txt
  • packagelists/arm64/pg18.0-spock5.0.4-minimal.txt
  • packagelists/arm64/pg18.0-spock5.0.4-standard.txt
  • packagelists/arm64/pg18.1-spock5.0.4-minimal.txt
  • packagelists/arm64/pg18.1-spock5.0.4-standard.txt
  • scripts/build_pgedge_images.py
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
  • GitHub Check: test (17.7-spock5.0.4-minimal-4, arm, minimal, ubuntu-24.04-arm)
  • GitHub Check: test (17.7-spock5.0.4-standard-4, arm, standard, ubuntu-24.04-arm)
  • GitHub Check: test (16.11-spock5.0.4-minimal-4, arm, minimal, ubuntu-24.04-arm)
  • GitHub Check: Codacy Static Code Analysis
🔇 Additional comments (25)
packagelists/amd64/pg18.1-spock5.0.4-standard.txt (1)

4-4: LGTM!

The pgedge-lolor_18 version update is correctly formatted and consistent with the PR's objective of updating to version 1.2.2 across all configurations.

packagelists/arm64/pg17.6-spock5.0.4-minimal.txt (1)

4-4: LGTM!

The version update is properly applied to the minimal package list configuration.

packagelists/arm64/pg17.7-spock5.0.4-standard.txt (1)

4-4: Version update is properly applied across all package lists.

The pgedge-lolor_17 package version has been consistently updated to 1.2.2 across all related packagelists (arm64 and amd64, PostgreSQL 16–18, standard and minimal variants). No remaining 1.2.1 references exist in the repository.

packagelists/arm64/pg18.0-spock5.0.4-standard.txt (1)

4-4: LGTM! Consistent version update for PostgreSQL 18.

The lolor package version update is properly applied for the PostgreSQL 18 variant, maintaining consistency with other package lists in this PR.

packagelists/amd64/pg17.6-spock5.0.4-standard.txt (1)

4-4: LGTM! Cross-architecture consistency verified.

The lolor package version update is correctly applied for the amd64 architecture, matching the pattern in the arm64 variants.

packagelists/arm64/pg17.6-spock5.0.4-standard.txt (1)

4-4: Version bump is correct and properly updated across all supported PostgreSQL versions.

All 24 package list files have been consistently updated with pgedge-lolor 1.2.2-1.el9 across PostgreSQL 16, 17, and 18 versions. The version update is legitimate, addressing important pg_upgrade fixes and security patches (CVE-2022-26520, GHSA-673j-qm5f-xpv8). Compatibility requirements are met for all supported PostgreSQL versions (16+).

packagelists/arm64/pg17.7-spock5.0.4-minimal.txt (1)

4-4: LGTM! ARM64 variant is consistent.

The lolor package version update for ARM64 architecture matches the AMD64 variant, ensuring consistency across architectures.

packagelists/amd64/pg17.7-spock5.0.4-minimal.txt (1)

4-4: LGTM! Version update is correct and consistent across all package lists.

The lolor package version bump from 1.2.1 to 1.2.2 follows the correct package naming format and aligns with the PR objectives. All 24 package lists have been updated to 1.2.2 with no remaining 1.2.1 versions, confirming consistent updates across all PostgreSQL versions, architectures, and variants.

packagelists/amd64/pg16.10-spock5.0.4-minimal.txt (1)

4-4: Version update looks good.

The lolor package update for PostgreSQL 16 is correctly formatted. The 1.2.2 release includes CVE fixes (CVE-2022-26520, GHSA-673j-qm5f-xpv8) and improvements to table OID caching. If pg_upgrade is planned, ensure lolor is updated to 1.2.2 first before running pg_upgrade.

packagelists/arm64/pg18.1-spock5.0.4-standard.txt (1)

4-4: LGTM - consistent version update.

The lolor package version bump is correctly applied in this standard variant packagelist, consistent with the minimal variant.

packagelists/amd64/pg18.0-spock5.0.4-minimal.txt (1)

4-4: LGTM - consistent cross-architecture update.

The version update is correctly applied for the amd64 architecture, maintaining consistency with the arm64 packagelists.

packagelists/arm64/pg18.0-spock5.0.4-minimal.txt (1)

4-4: Version update is consistent across all packagelists.

All 24 packagelists have been correctly and consistently updated to pgedge-lolor 1.2.2-1.el9 (covering arm64 and amd64 architectures, PostgreSQL versions 16–18, and both standard and minimal variants). No remaining 1.2.1 references exist.

Still verify:

  1. Epoch increments in scripts/build_pgedge_images.py align with this package update
  2. The lolor 1.2.2 release notes for any breaking changes or new requirements
packagelists/amd64/pg17.6-spock5.0.4-minimal.txt (1)

4-4: LGTM! Version bump is consistent.

The lolor package version update follows the same pattern as other package lists in this PR.

packagelists/amd64/pg17.7-spock5.0.4-standard.txt (1)

4-4: LGTM! Version bump applied consistently.

The lolor package version update to 1.2.2 is consistent with the other 23 package list files in this PR.

packagelists/arm64/pg16.10-spock5.0.4-standard.txt (1)

4-4: LGTM. The version bump to 1.2.2 is documented and appropriate. No security advisories found for this release.

packagelists/arm64/pg16.11-spock5.0.4-minimal.txt (1)

4-4: Version update is consistent with other package lists.

The pgedge-lolor_16 version bump to 1.2.2-1.el9 maintains consistency with the updates across all PostgreSQL versions in this PR.

packagelists/amd64/pg18.0-spock5.0.4-standard.txt (1)

4-4: Version update maintains consistency across architectures.

The pgedge-lolor_18 version bump to 1.2.2-1.el9 is consistent with the arm64 package lists and maintains the correct format for the standard configuration.

packagelists/arm64/pg18.1-spock5.0.4-minimal.txt (1)

4-4: Version update to 1.2.2 is valid and follows the package naming convention.

The package pgedge-lolor v1.2.2 is confirmed to be a legitimate pgEdge component with official documentation. The version bump is syntactically correct and has been properly tracked in the repository history.

packagelists/arm64/pg16.10-spock5.0.4-minimal.txt (1)

4-4: LGTM!

Version bump for pgedge-lolor_16 to 1.2.2-1.el9 aligns with PR objectives.

packagelists/amd64/pg18.1-spock5.0.4-minimal.txt (1)

4-4: LGTM!

Version bump for pgedge-lolor_18 to 1.2.2-1.el9 is consistent with the PR-wide lolor update.

scripts/build_pgedge_images.py (1)

166-206: LGTM!

Epoch increments from 2/3 to 3/4 across all image definitions are appropriate to reflect the updated pgedge-lolor package version. This ensures new immutable image tags are generated for the builds containing the updated dependency.

packagelists/arm64/pg16.11-spock5.0.4-standard.txt (1)

4-4: LGTM!

Version bump for pgedge-lolor_16 to 1.2.2-1.el9 in the ARM64 standard package list.

packagelists/amd64/pg16.11-spock5.0.4-standard.txt (1)

4-4: LGTM!

Version bump for pgedge-lolor_16 to 1.2.2-1.el9 in the AMD64 standard package list.

packagelists/amd64/pg16.11-spock5.0.4-minimal.txt (1)

4-4: LGTM: Version update is consistent.

The version bump is consistent with the other package list files in this PR. The format is correct and matches the expected pattern.

packagelists/amd64/pg16.10-spock5.0.4-standard.txt (1)

4-4: Pgedge-lolor version 1.2.2 is valid and compatible.

The package version 1.2.2 is documented and requires PostgreSQL 16+. The inclusion in the pg16.10 package list is appropriate. lolor is a large-object extension for logical replication support and integrates properly with this PostgreSQL 16.10 configuration.


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.

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.

2 participants