Charles Couble
02/17/2023, 12:50 PMFirst, i had an issue with private github repository that serves as a python library
I could work around using <https://github.com/dagster-io/dagster/discussions/12340#discussioncomment-4976293
|this fix. >(it requires ENABLE_FAST_DEPLOYS='true')
Therefore, i had another issue with the DBT integration,
It seems to be related to <https://github.com/dagster-io/dagster/discussions/12180
|this >: (the fix proposed here requires ENABLE_FAST_DEPLOYS='false')So basically i tried both ENABLE_FAST_DEPLOYS = True / False With both fixes applied and here are the error I get.
Shalabh Chaturvedi
02/17/2023, 5:42 PMavatar_url
error in your second screenshot (ENABLE_FAST_DEPLOYS=True). I can you know when it is out and then we can try again.Charles Couble
02/17/2023, 5:43 PMShalabh Chaturvedi
02/17/2023, 5:59 PMdeploy.yml
Replace the @ tag here:
uses: dagster-io/dagster-cloud-action/actions/build_deploy_python_executable@pex-v0.1
With:
uses: dagster-io/dagster-cloud-action/actions/build_deploy_python_executable@shalabhc/fix-tests
I'm running this through tests before we make it live as @pex-v0.1
. However if you want you can try it out early and let us know if it fixes your issue. Note you will need to set ENABLE_FAST_DEPLOYS: 'true'
for this.Charles Couble
02/17/2023, 6:09 PMShalabh Chaturvedi
02/17/2023, 6:12 PMShalabh Chaturvedi
02/20/2023, 5:53 PMCharles Couble
02/21/2023, 12:31 PMCharles Couble
02/22/2023, 12:18 PMCharles Couble
02/23/2023, 3:25 PMShalabh Chaturvedi
02/23/2023, 4:46 PMuses: dagster-io/dagster-cloud-action/actions/build_deploy_python_executable@pex-v0.1
With:
uses: dagster-io/dagster-cloud-action/actions/build_deploy_python_executable@v0.1.19-pr-comment-hotfix
Charles Couble
02/23/2023, 10:26 PMAdding a github profile picture
Aswell as registering my e-mail on git CLI with "git config" command.. Before that i was pushing only with git config user.name but no e-mail.My though, but i did'nt really dig much : There is a step on the Python Deploy process that interracts with github. It seems like the script scans all profiles in commits, and my user "unregistered" / "without git config --email" configured, returns an empty variable, which prevents the script from working. I hope this helps reproducing or fixing this issue !
Shalabh Chaturvedi
02/23/2023, 11:02 PM