Conversation
elementary-data 0.22.0 dropped Python 3.9 support, causing pip install to fail inside the action's Docker container. Update base image from python:3.9.17 to python:3.10.17. Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
devin-ai-integration bot
added a commit
to elementary-data/elementary
that referenced
this pull request
Mar 1, 2026
The run-elementary-action Docker image uses Python 3.9, but elementary-data 0.22.0 dropped Python 3.9 support. Point at the fix branch (devin/1772355616-fix-python-version) which updates the Dockerfile to Python 3.10. This should be updated to the new release tag once elementary-data/run-elementary-action#15 is merged and tagged. Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
4 tasks
2 tasks
haritamar
added a commit
to elementary-data/elementary
that referenced
this pull request
Mar 1, 2026
…tion version (#2131) * Modernize test-github-action.yml to use e2e_dbt_project - Replace deprecated dbt-data-reliability integration tests with the newer e2e_dbt_project (matching test-warehouse.yml approach) - Use generate_profiles.py + profiles.yml.j2 instead of the TEST_GITHUB_ACTION_PROFILES_YML secret - Update run-elementary-action from v1.8 to v1.12 - Add concurrency management and schema cleanup - Add generate-data workflow input - Temporarily add pull_request trigger for CI testing Co-Authored-By: Itamar Hartstein <haritamar@gmail.com> * Add masking for profiles.yml credentials in GITHUB_OUTPUT Co-Authored-By: Itamar Hartstein <haritamar@gmail.com> * Fix jq validation to work correctly under bash -e Co-Authored-By: Itamar Hartstein <haritamar@gmail.com> * Temporarily point action at Python 3.10 fix branch The run-elementary-action Docker image uses Python 3.9, but elementary-data 0.22.0 dropped Python 3.9 support. Point at the fix branch (devin/1772355616-fix-python-version) which updates the Dockerfile to Python 3.10. This should be updated to the new release tag once elementary-data/run-elementary-action#15 is merged and tagged. Co-Authored-By: Itamar Hartstein <haritamar@gmail.com> * Update run-elementary-action to v1.13 (with Python 3.10 fix) Replace temporary branch reference with the official v1.13 release tag which includes the Python 3.10 Dockerfile fix. Co-Authored-By: Itamar Hartstein <haritamar@gmail.com> * Add -t snowflake to edr monitor report command Co-Authored-By: Itamar Hartstein <haritamar@gmail.com> * Remove temporary pull_request trigger Co-Authored-By: Itamar Hartstein <haritamar@gmail.com> * Remove Notify Slack job Co-Authored-By: Itamar Hartstein <haritamar@gmail.com> --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: Itamar Hartstein <haritamar@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Updates the Docker base image from
python:3.9.17topython:3.10.17.elementary-datav0.22.0 dropped Python 3.9 support. The action'sinstall_edrfunction detects the installed dbt package version and runspip install elementary-data[<adapter>]~=0.22.0inside this container — which fails because pip correctly filters out 0.22.0 as incompatible with Python 3.9.Review & Testing Checklist for Human
python:3.10.17image exists on Docker Hub and is appropriate (vs. choosing 3.11 or 3.12)requirements.txtdependencies (packaging~=21.3,pydantic~=1.10.2) are compatible with Python 3.10v1.13) so theelementaryrepo'stest-github-action.ymlcan reference the fixed versionNotes
entrypoint.py,requirements.txt, oraction.yml.elementaryrepo has a companion PR (#2131) that modernizestest-github-action.ymland depends on this fix.