[GitHub] [airflow] kolfild26 commented on pull request #28998: write to DB a user who triggered a dag via API endpoint request

2023-02-02 Thread via GitHub
kolfild26 commented on PR #28998: URL: https://github.com/apache/airflow/pull/28998#issuecomment-1413759778 Above there was a conflict with the very fresh commit https://github.com/apache/airflow/commit/b94f36bf563f5c8372086cec63b74eadef638ef8 in `main`. It also changed the same test in the

[GitHub] [airflow] kolfild26 commented on pull request #28998: write to DB a user who triggered a dag via API endpoint request

2023-01-31 Thread via GitHub
kolfild26 commented on PR #28998: URL: https://github.com/apache/airflow/pull/28998#issuecomment-140613 Hi, @ephraimbuddy I added test [_check_last_log](https://github.com/apache/airflow/pull/28998/commits/e2b670c4382b9e00adece76d5574268d9eb113f3) similarly it's done in [test_conne

[GitHub] [airflow] kolfild26 commented on pull request #28998: write to DB a user who triggered a dag via API endpoint request

2023-01-26 Thread via GitHub
kolfild26 commented on PR #28998: URL: https://github.com/apache/airflow/pull/28998#issuecomment-1405236443 > We also need a test Ok, will update once it's done. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[GitHub] [airflow] kolfild26 commented on pull request #28998: write to DB a user who triggered a dag via API endpoint request

2023-01-25 Thread via GitHub
kolfild26 commented on PR #28998: URL: https://github.com/apache/airflow/pull/28998#issuecomment-1404158145 @ephraimbuddy Thanks for these inputs. I was actually looking at [cli_action_loggers.py](https://github.com/apache/airflow/blob/main/airflow/utils/cli_action_loggers.py). It diffe