Re: [PR] Bugfix/prevent concurrency with cached venv [airflow]

2023-10-30 Thread via GitHub
potiuk commented on PR #35258: URL: https://github.com/apache/airflow/pull/35258#issuecomment-1785179218 > hat other mechanish we have in place to debug the failed run? Yeah - the log is the way to pass it to the place it will be visible (airflow log). And it's not much different

Re: [PR] Bugfix/prevent concurrency with cached venv [airflow]

2023-10-30 Thread via GitHub
jens-scheffler-bosch commented on PR #35258: URL: https://github.com/apache/airflow/pull/35258#issuecomment-1785166976 > @jens-scheffler-bosch Just curious, what other mechanish we have in place to debug the failed run? Same here. I assume we have a general contratiction / conflict

Re: [PR] Bugfix/prevent concurrency with cached venv [airflow]

2023-10-30 Thread via GitHub
potiuk merged PR #35258: URL: https://github.com/apache/airflow/pull/35258 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] Bugfix/prevent concurrency with cached venv [airflow]

2023-10-29 Thread via GitHub
potiuk commented on PR #35258: URL: https://github.com/apache/airflow/pull/35258#issuecomment-1784278368 BTW. @jens-scheffler-bosch - first test suites are getting green in https://github.com/apache/airflow/pull/35160, so I confirm it fixes the original problem too. -- This is an

Re: [PR] Bugfix/prevent concurrency with cached venv [airflow]

2023-10-29 Thread via GitHub
potiuk commented on PR #35258: URL: https://github.com/apache/airflow/pull/35258#issuecomment-1784263022 > * Revers the efforts of @potiuk in [Handle left-over termination logs from previous runs with same cacheĀ  #35252](https://github.com/apache/airflow/pull/35252) - Sorry as termination

[PR] Bugfix/prevent concurrency with cached venv [airflow]

2023-10-29 Thread via GitHub
jens-scheffler-bosch opened a new pull request, #35258: URL: https://github.com/apache/airflow/pull/35258 This PR reverts PR #35252 as the fix made in this PR just fixed one side effect. Unfortunately when initially the PR #33355 was created I did not see that all temporary scripts