Add CCSDSPy Configuration to Image#28
Conversation
There was a problem hiding this comment.
Pull request overview
This PR introduces a “baked config” mechanism for upstream Python packages (notably ccsdspy) by shipping version-controlled config files in the Lambda image, mirroring them into /tmp at container init, and wiring packages to those writable runtime config paths via Docker ENV.
Changes:
- Add documentation describing the repo → image →
/tmpconfig flow and how to extend it for additional packages. - Add a baked
ccsdspyYAML config undersrc/config/ccsdspy/. - Seed
/tmp/configfrom/lambda_function/configinentry_script.shand setccsdspy_CONFIGDIRin the Dockerfile.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| lambda_function/src/config/README.md | Documents the baked-config convention and runtime mirroring strategy. |
| lambda_function/src/config/ccsdspy/config.yml | Adds a baked ccsdspy configuration intended to prevent import-time failures in Lambda. |
| lambda_function/entry_script.sh | Mirrors baked configs into /tmp/config before the Python runtime starts. |
| lambda_function/Dockerfile | Exposes ccsdspy_CONFIGDIR pointing at the writable /tmp mirror. |
Comments suppressed due to low confidence (1)
lambda_function/entry_script.sh:19
$@should be quoted to preserve argument boundaries and avoid glob/word-splitting issues when the entrypoint is invoked with flags or values containing spaces.
if [ -z "${AWS_LAMBDA_RUNTIME_API}" ]; then
exec /usr/local/bin/aws-lambda-rie python3 -m awslambdaric $@
else
exec python3 -m awslambdaric $@
fi
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #28 +/- ##
=======================================
Coverage ? 62.54%
=======================================
Files ? 2
Lines ? 251
Branches ? 0
=======================================
Hits ? 157
Misses ? 94
Partials ? 0 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
No description provided.