[GitHub] [airflow] milton0825 commented on a change in pull request #5350: [AIRFLOW-4716] Instrument dag loading time duration

milton0825 commented on a change in pull request #5350: [AIRFLOW-4716] Instrument dag loading time duration URL: https://github.com/apache/airflow/pull/5350#discussion_r289275017 ## File path: airflow/models/dagbag.py ## @@ -391,6 +391,10 @@ def collect_dags(

[GitHub] [airflow] feng-tao commented on a change in pull request #5350: [AIRFLOW-4716] Instrument dag loading time duration

feng-tao commented on a change in pull request #5350: [AIRFLOW-4716] Instrument dag loading time duration URL: https://github.com/apache/airflow/pull/5350#discussion_r289275297 ## File path: airflow/models/dagbag.py ## @@ -391,6 +391,10 @@ def collect_dags(

[GitHub] [airflow] milton0825 commented on a change in pull request #5350: [AIRFLOW-4716] Instrument dag loading time duration

milton0825 commented on a change in pull request #5350: [AIRFLOW-4716] Instrument dag loading time duration URL: https://github.com/apache/airflow/pull/5350#discussion_r289275017 ## File path: airflow/models/dagbag.py ## @@ -391,6 +391,10 @@ def collect_dags(

[GitHub] [airflow] milton0825 commented on a change in pull request #5350: [AIRFLOW-4716] Instrument dag loading time duration

milton0825 commented on a change in pull request #5350: [AIRFLOW-4716] Instrument dag loading time duration URL: https://github.com/apache/airflow/pull/5350#discussion_r289275017 ## File path: airflow/models/dagbag.py ## @@ -391,6 +391,10 @@ def collect_dags(

[GitHub] [airflow] codecov-io edited a comment on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep

codecov-io edited a comment on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep URL: https://github.com/apache/airflow/pull/5351#issuecomment-497597451 # [Codecov](https://codecov.io/gh/apache/airflow/pull/5351?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep

codecov-io commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep URL: https://github.com/apache/airflow/pull/5351#issuecomment-497597451 # [Codecov](https://codecov.io/gh/apache/airflow/pull/5351?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io edited a comment on issue #5350: [AIRFLOW-4716] Instrument dag loading time duration

codecov-io edited a comment on issue #5350: [AIRFLOW-4716] Instrument dag loading time duration URL: https://github.com/apache/airflow/pull/5350#issuecomment-497596765 # [Codecov](https://codecov.io/gh/apache/airflow/pull/5350?src=pr=h1) Report > Merging

[GitHub] [airflow] feng-tao closed pull request #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep

feng-tao closed pull request #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep URL: https://github.com/apache/airflow/pull/5351 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] feng-tao commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep

feng-tao commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep URL: https://github.com/apache/airflow/pull/5351#issuecomment-497593983 Thanks for @XD-DENG for confirming. This is an

[GitHub] [airflow] XD-DENG commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep

XD-DENG commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep URL: https://github.com/apache/airflow/pull/5351#issuecomment-497593584 @feng-tao , I don't think the error you have seen is the reason for the CI failure. Please refer to this CI:

[GitHub] [airflow] feng-tao commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep

feng-tao commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep URL: https://github.com/apache/airflow/pull/5351#issuecomment-497591940 it is hard to tell which one though(https://github.com/Azure/msrest-for-python/network/dependents?dependent_type=PACKAGE).

[GitHub] [airflow] feng-tao edited a comment on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep

feng-tao edited a comment on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep URL: https://github.com/apache/airflow/pull/5351#issuecomment-497591940 it is hard to tell which one

[GitHub] [airflow] feng-tao commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep

feng-tao commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep URL: https://github.com/apache/airflow/pull/5351#issuecomment-497591664 I assume there is another dep which pin or bring this transient dependency the msrest 0.4.29 ?

[GitHub] [airflow] XD-DENG commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep

XD-DENG commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep URL: https://github.com/apache/airflow/pull/5351#issuecomment-497590786 But `azure-mgmt-containerinstance 1.5.0` was released back on `May 23 2019`?

[GitHub] [airflow] feng-tao commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep

feng-tao commented on issue #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep URL: https://github.com/apache/airflow/pull/5351#issuecomment-497590019 cc @XD-DENG @potiuk This is an automated message from

[GitHub] [airflow] feng-tao opened a new pull request #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep

feng-tao opened a new pull request #5351: [AIRFLOW-XXX] Pin azure-mgmt-containerinstance to lower dep URL: https://github.com/apache/airflow/pull/5351 Just found that the latest release(1.5.0) of azure-mgmt-containerinstance broke the master with the following errors in CI:

[GitHub] [airflow] feng-tao commented on a change in pull request #5350: [AIRFLOW-4716] Instrument dag loading time duration

feng-tao commented on a change in pull request #5350: [AIRFLOW-4716] Instrument dag loading time duration URL: https://github.com/apache/airflow/pull/5350#discussion_r289268103 ## File path: airflow/models/dagbag.py ## @@ -391,6 +391,10 @@ def collect_dags(

[GitHub] [airflow] XD-DENG commented on a change in pull request #5350: [AIRFLOW-4716] Instrument dag loading time duration

XD-DENG commented on a change in pull request #5350: [AIRFLOW-4716] Instrument dag loading time duration URL: https://github.com/apache/airflow/pull/5350#discussion_r289266898 ## File path: airflow/models/dagbag.py ## @@ -391,6 +391,10 @@ def collect_dags(

<    1   2