Skip to content

Commit

Permalink
Use correct command to upgrade pip in dbt nux
Browse files Browse the repository at this point in the history
Summary:
The previous command did not actually upgrade pip and left us on 20.0.2.

Test Plan:
Go through the NUX with this new YAML, it now correctly backtracks and is able to install the correct set of dependencies.

I'm unclear if I need to do a release of the action for this to be applied or not?
  • Loading branch information
gibsondan committed Aug 28, 2024
1 parent f13c949 commit d41e0b9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion github/serverless/dbt/branch_deployments.yml
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ jobs:
if: steps.prerun.outputs.result == 'pex-deploy'
# --upgrade-strategy eager picks up newer packages that are required for things to work
run: |
pip install pip --upgrade
python -m pip install pip --upgrade
cd project-repo/${{ env.DAGSTER_PROJECT_NAME }}
pip install . --upgrade --upgrade-strategy eager
dagster-dbt project prepare-and-package --file ${{ env.DAGSTER_PROJECT_NAME }}/project.py
Expand Down
2 changes: 1 addition & 1 deletion github/serverless/dbt/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ jobs:
if: steps.prerun.outputs.result == 'pex-deploy'
# --upgrade-strategy eager picks up newer packages that are required for things to work
run: |
pip install pip --upgrade
python -m pip install pip --upgrade
cd project-repo/${{ env.DAGSTER_PROJECT_NAME }}
pip install . --upgrade --upgrade-strategy eager
dagster-dbt project prepare-and-package --file ${{ env.DAGSTER_PROJECT_NAME }}/project.py
Expand Down

0 comments on commit d41e0b9

Please sign in to comment.