Update docker.io/library/postgres Docker tag to v14.22#221
Update docker.io/library/postgres Docker tag to v14.22#221red-hat-konflux-kflux-prd-rh02[bot] wants to merge 1 commit into
Conversation
Signed-off-by: red-hat-konflux-kflux-prd-rh02 <190377777+red-hat-konflux-kflux-prd-rh02[bot]@users.noreply.github.com>
|
Hi @red-hat-konflux-kflux-prd-rh02[bot]. Thanks for your PR. I'm waiting for a openshift-hyperfleet member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe default PostgreSQL container image tag under Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Supply chain surface (CWE-1395, CWE-693): Confirm 🚥 Pre-merge checks | ✅ 11✅ Passed checks (11 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@charts/values.yaml`:
- Line 308: The charts/README.md file is out of sync with the updated
values.yaml file. The postgres image version has been changed to 14.22 in
values.yaml, but README.md still references the old version 14.2 in three places
(database table, database.postgresql table, and database.postgresql.image row).
Run the make helm-docs command to regenerate the charts/README.md file from
values.yaml and the helm template, then commit the updated README.md file. This
will ensure the documentation stays in sync and the make test-helm target's
verify-helm-docs check will pass.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Central YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Enterprise
Run ID: 3bdc51bc-c073-4448-be77-e32eec2279db
📒 Files selected for processing (1)
charts/values.yaml
🔗 Linked repositories identified
CodeRabbit considers these linked repositories for cross-repo context during reviews:
openshift-hyperfleet/architecture(manual)openshift-hyperfleet/hyperfleet-api(manual)openshift-hyperfleet/hyperfleet-sentinel(manual)openshift-hyperfleet/hyperfleet-adapter(manual)openshift-hyperfleet/hyperfleet-broker(manual)
| enabled: true | ||
| # -- PostgreSQL container image | ||
| image: docker.io/library/postgres:14.2 | ||
| image: docker.io/library/postgres:14.22 |
There was a problem hiding this comment.
🛠️ Refactor suggestion | 🟠 Major
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Search for stale references to postgres:14.2 in docs and templates
rg 'postgres:14\.2' charts/Repository: openshift-hyperfleet/hyperfleet-api
Length of output: 1308
🏁 Script executed:
#!/bin/bash
# Check if README is auto-generated and look for related tooling
cd charts
ls -la | grep -i readme
cat README.md | head -20Repository: openshift-hyperfleet/hyperfleet-api
Length of output: 915
🏁 Script executed:
#!/bin/bash
# Search for helm-docs or similar documentation generation in Makefile
grep -r "helm-docs\|README" Makefile* .github/ 2>/dev/null || echo "No helm-docs found in standard locations"Repository: openshift-hyperfleet/hyperfleet-api
Length of output: 647
🏁 Script executed:
#!/bin/bash
# Check the actual content of README.md around the database.postgresql.image section
sed -n '140,160p' charts/README.mdRepository: openshift-hyperfleet/hyperfleet-api
Length of output: 3393
Run make helm-docs to regenerate charts/README.md.
The chart README is auto-generated from values.yaml and README.md.gotmpl using helm-docs. The current README.md still documents postgres:14.2 in three places (database table, database.postgresql table, and database.postgresql.image row), but values.yaml now specifies 14.22. Run make helm-docs to regenerate the documentation, then commit the result. The make test-helm target includes a verify-helm-docs check that will fail until README.md is in sync with values.yaml.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@charts/values.yaml` at line 308, The charts/README.md file is out of sync
with the updated values.yaml file. The postgres image version has been changed
to 14.22 in values.yaml, but README.md still references the old version 14.2 in
three places (database table, database.postgresql table, and
database.postgresql.image row). Run the make helm-docs command to regenerate the
charts/README.md file from values.yaml and the helm template, then commit the
updated README.md file. This will ensure the documentation stays in sync and the
make test-helm target's verify-helm-docs check will pass.
This PR contains the following updates:
14.2→14.2214.23Warning
Some dependencies could not be looked up. Check the warning logs for more information.
Configuration
📅 Schedule: Branch creation - "on monday" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
To execute skipped test pipelines write comment
/ok-to-test.Documentation
Find out how to configure dependency updates in MintMaker documentation or see all available configuration options in Renovate documentation.