FE-109 migrate to Dagster 0.13 and Appsec Blessed Python 3.9 #10
Conversation
I'll update it all to 3.12 when I upgrade the dagster version - but first things first - just get the appsec image running old dagster
… the dockerfile logic correct
… psycopg2-binary, and pyyaml for dependency clashes
…Use temurin since adopt is now temurin, essentially
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
…" - but one of the devs suggested trying "2.0.1" snok/install-poetry#164
…eInProcessGraphResult - (migrating from 0.12.14 to 0.13) https://github.com/dagster-io/dagster/blob/4fb944ec6d0d9dfd57856901f5a43ea381483187/python_modules/dagster/dagster/core/execution/execute_in_process_result.py
…o generate the requirements file for the security scanners
…epth back to checkout
| # formerly known as `service_account_key` | ||
| credentials_json: '${{ secrets.GCP_TEST_KEY }}' | ||
| - name: 'Set up Cloud SDK' | ||
| uses: 'google-github-actions/[email protected]' |
Check warning
Code scanning / CodeQL
Unpinned tag for a non-immutable Action in workflow Medium
| env: | ||
| GCP_TEST_KEY: '${{ secrets.GCP_TEST_KEY }}' | ||
| - name: 'Set up Cloud SDK' | ||
| uses: 'google-github-actions/[email protected]' |
Check warning
Code scanning / CodeQL
Unpinned tag for a non-immutable Action in workflow Medium
…ion version - v2 is being deprecated
…ion version - v2 is being deprecated
…109-new-hca-dev-env
| fetch-depth: 0 | ||
| - id: 'auth' | ||
| uses: 'google-github-actions/auth@v2' |
Check warning
Code scanning / CodeQL
Unpinned tag for a non-immutable Action in workflow Medium
| # Needed by sonar to get the git history for the branch the PR will be merged into. | ||
| with: | ||
| fetch-depth: 0 | ||
| - uses: google-github-actions/[email protected] | ||
| - id: 'auth' | ||
| uses: 'google-github-actions/auth@v2' |
Check warning
Code scanning / CodeQL
Unpinned tag for a non-immutable Action in workflow Medium
…tput_for_node, such that it was trying to find the output_value method in HcaScratchDatasetName. which does not have that method, rather than ExecuteInProcessResult, which does. The default value for output_value is result though, so this should perhaps resolve the issue. https://docs.dagster.io/api/python-api/execution dagster-io/dagster#6218
…109-new-hca-dev-env
…109-new-hca-dev-env
…109-new-hca-dev-env
Why
Relevant ticket
This PR
Checklist