Fix #9572: Deployment of helm chart crashing with operation not permitted

[simonra]

The root cause appears to be that the base image has moved the location of python from /usr/bin/ to /usr/local/bin/. This prevented the workaround done by the init containers copying out the binary with the users permissions from working.

Summary by CodeRabbit

• Bug Fixes
  • Resolved container configuration issues to improve application stability and compatibility.

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

[coderabbitai]

Walkthrough

Updates Python binary paths in a Helm deployment template for the pgadmin4 container. One volumeMount path and one init container binary search path are changed to reference alternate Python binary locations.

Changes

Cohort / File(s)	Summary
Helm Deployment Configuration pkg/helm/templates/deployment.yaml	Updated Python binary mount path from /usr/bin/python3 to /usr/local/bin/python3 for pgadmin4 container volumeMount, and init container binary search pattern from /usr/bin/python3.* to /venv/bin/python3.*.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 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 title directly addresses the fix for issue #9572, accurately describing the deployment crash resolution through Python binary path correction in the helm chart.
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.}

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@pkg/helm/templates/deployment.yaml`:
- Line 208: Update the args entry that selects and copies the Python binary:
narrow the glob from python3.* to python3.[0-9][0-9] to avoid matching
non-binaries, and add explicit failure handling so the container fails loudly if
no matching binary is found (e.g., run the pipeline with pipefail/set -e or
check the ls result and echo an error + exit 1 when empty) before attempting to
cp; locate this change in the args: [...] line that currently contains "ls
/venv/bin/python3.* | sort -V -r | head -n 1 | xargs -i cp {} python3" and
replace it with the stricter glob and an explicit non-zero exit path on
no-match.

[jykae]

Tested and works great, hopefully we get this released soon.