[GitHub] [airflow] chinwobble commented on issue #18999: Update databricks provider to use TriggerOperator

2022-05-28 Thread GitBox
chinwobble commented on issue #18999: URL: https://github.com/apache/airflow/issues/18999#issuecomment-1140382369 @eskarimov nice work getting this one merged! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] uranusjr commented on issue #23965: S3 keys provided by aws/log/s3_task_handler.py to s3 hook contain the prefix `s3://`, causing minio bad request.

2022-05-28 Thread GitBox
uranusjr commented on issue #23965: URL: https://github.com/apache/airflow/issues/23965#issuecomment-1140376432 It would be a good idea to check with Minio maintainers and potentially Amazon about this. Would you be interested in doing this? -- This is an automated message from the

[GitHub] [airflow] zachliu commented on issue #23926: Airflow 2.3.1 - gunicorn keeps removing and adding Permission menu access on Permissions to role Admin

2022-05-28 Thread GitBox
zachliu commented on issue #23926: URL: https://github.com/apache/airflow/issues/23926#issuecomment-1140371585 hey i already found a layman solution here: https://github.com/apache/airflow/issues/23512#issuecomment-1140160062 :grin: -- This is an automated message from the Apache Git

[GitHub] [airflow] zachliu commented on issue #23512: Random "duplicate key value violates unique constraint" errors when initializing the postgres database

2022-05-28 Thread GitBox
zachliu commented on issue #23512: URL: https://github.com/apache/airflow/issues/23512#issuecomment-1140371309 aha, i think it's exactly the same as https://github.com/apache/airflow/issues/23926: all gunicorn workers are executing the same code causing clashes on the database maybe

[GitHub] [airflow] github-actions[bot] commented on pull request #23997: Removing magic numbers from exceptions

2022-05-28 Thread GitBox
github-actions[bot] commented on PR #23997: URL: https://github.com/apache/airflow/pull/23997#issuecomment-1140369693 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the

[GitHub] [airflow] uranusjr commented on pull request #23972: Adding connections for sql operator unit tests so they pass

2022-05-28 Thread GitBox
uranusjr commented on PR #23972: URL: https://github.com/apache/airflow/pull/23972#issuecomment-1140369634 A pytest fixture can be implemented and used like this: ```python @pytest.fixture(scope="session") def ensure_connection(request): def _ensure_session(conn_id: str,

[GitHub] [airflow] bernardocouto opened a new pull request, #23997: Removing magic numbers from exceptions

2022-05-28 Thread GitBox
bernardocouto opened a new pull request, #23997: URL: https://github.com/apache/airflow/pull/23997 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for

[GitHub] [airflow] bernardocouto closed pull request #23996: Removing magic numbers from exceptions

2022-05-28 Thread GitBox
bernardocouto closed pull request #23996: Removing magic numbers from exceptions URL: https://github.com/apache/airflow/pull/23996 -- 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

[GitHub] [airflow] bernardocouto commented on pull request #23996: Removing magic numbers from exceptions

2022-05-28 Thread GitBox
bernardocouto commented on PR #23996: URL: https://github.com/apache/airflow/pull/23996#issuecomment-1140363780 > Please follow the contribution guide, run pre-commit checks locally, and fix the errors. I'm sorry. Adjusted. -- This is an automated message from the Apache Git

[GitHub] [airflow] uranusjr commented on issue #23512: Random "duplicate key value violates unique constraint" errors when initializing the postgres database

2022-05-28 Thread GitBox
uranusjr commented on issue #23512: URL: https://github.com/apache/airflow/issues/23512#issuecomment-1140363029 This seems somewhat related to #23926. cc @jhtimmins -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [airflow] uranusjr commented on a diff in pull request #23510: Fix permission issue for dag that has dot in name

2022-05-28 Thread GitBox
uranusjr commented on code in PR #23510: URL: https://github.com/apache/airflow/pull/23510#discussion_r884199036 ## airflow/security/permissions.py: ## @@ -66,14 +66,16 @@ DAG_ACTIONS = {ACTION_CAN_READ, ACTION_CAN_EDIT, ACTION_CAN_DELETE} -def

[GitHub] [airflow] uranusjr commented on a diff in pull request #23964: Add built-in Extrenal Link for ExternalTaskMarker operator

2022-05-28 Thread GitBox
uranusjr commented on code in PR #23964: URL: https://github.com/apache/airflow/pull/23964#discussion_r884198642 ## airflow/sensors/external_task.py: ## @@ -318,3 +325,19 @@ def get_serialized_fields(cls): if not cls.__serialized_fields:

[GitHub] [airflow] github-actions[bot] commented on pull request #23980: Add better diagnostics capabilities for pre-commits run via CI image

2022-05-28 Thread GitBox
github-actions[bot] commented on PR #23980: URL: https://github.com/apache/airflow/pull/23980#issuecomment-1140357182 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the

[GitHub] [airflow] uranusjr commented on pull request #23996: Removing magic numbers from exceptions

2022-05-28 Thread GitBox
uranusjr commented on PR #23996: URL: https://github.com/apache/airflow/pull/23996#issuecomment-1140356071 Please follow the contribution guide, run pre-commit checks locally, and fix the errors. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] uranusjr closed issue #15147: Dag created with a dag decorator in a packaged dag file (zip) has incorrect filepath property

2022-05-28 Thread GitBox
uranusjr closed issue #15147: Dag created with a dag decorator in a packaged dag file (zip) has incorrect filepath property URL: https://github.com/apache/airflow/issues/15147 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [airflow] uranusjr commented on issue #15147: Dag created with a dag decorator in a packaged dag file (zip) has incorrect filepath property

2022-05-28 Thread GitBox
uranusjr commented on issue #15147: URL: https://github.com/apache/airflow/issues/15147#issuecomment-1140355765 Seems fixed to me. ![image](https://user-images.githubusercontent.com/605277/170848535-2916ec47-03ed-47a4-8b88-a68dece187fc.png) -- This is an automated message from

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #23996: Removing magic numbers from exceptions

2022-05-28 Thread GitBox
boring-cyborg[bot] commented on PR #23996: URL: https://github.com/apache/airflow/pull/23996#issuecomment-1140355359 Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution

[GitHub] [airflow] bernardocouto opened a new pull request, #23996: Removing magic numbers from exceptions

2022-05-28 Thread GitBox
bernardocouto opened a new pull request, #23996: URL: https://github.com/apache/airflow/pull/23996 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for

[GitHub] [airflow] github-actions[bot] commented on pull request #23994: Remove fixing cncf.kubernetes provider when generating constraints

2022-05-28 Thread GitBox
github-actions[bot] commented on PR #23994: URL: https://github.com/apache/airflow/pull/23994#issuecomment-1140355161 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the

[GitHub] [airflow] uranusjr commented on pull request #23995: Remove certifi limitations from eager upgrade limits

2022-05-28 Thread GitBox
uranusjr commented on PR #23995: URL: https://github.com/apache/airflow/pull/23995#issuecomment-1140355016 Seems like all Helm Chart tests are timing out. -- 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

[GitHub] [airflow] github-actions[bot] closed pull request #22329: Add naming convention to the DB

2022-05-28 Thread GitBox
github-actions[bot] closed pull request #22329: Add naming convention to the DB URL: https://github.com/apache/airflow/pull/22329 -- 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

[GitHub] [airflow] github-actions[bot] closed pull request #22676: Fix helm name override

2022-05-28 Thread GitBox
github-actions[bot] closed pull request #22676: Fix helm name override URL: https://github.com/apache/airflow/pull/22676 -- 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

[GitHub] [airflow] github-actions[bot] commented on pull request #22996: Show a consistent duration format

2022-05-28 Thread GitBox
github-actions[bot] commented on PR #22996: URL: https://github.com/apache/airflow/pull/22996#issuecomment-1140347485 This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for

[GitHub] [airflow] potiuk opened a new pull request, #23995: Remove certifi limitations from eager upgrade limits

2022-05-28 Thread GitBox
potiuk opened a new pull request, #23995: URL: https://github.com/apache/airflow/pull/23995 The certifi limitation was introduced to keep snowflake happy while performing eager upgrade because it added limits on certifi. However seems like it is not limitation any more in latest

[GitHub] [airflow] potiuk commented on issue #10182: Move relevant information about First time contributor workshop as Airflow Blog

2022-05-28 Thread GitBox
potiuk commented on issue #10182: URL: https://github.com/apache/airflow/issues/10182#issuecomment-1140335981 I think the documentation there is outdated and is actually never used. I don't think we need to add any blog for it. -- This is an automated message from the Apache Git Service.

[GitHub] [airflow] potiuk closed issue #10182: Move relevant information about First time contributor workshop as Airflow Blog

2022-05-28 Thread GitBox
potiuk closed issue #10182: Move relevant information about First time contributor workshop as Airflow Blog URL: https://github.com/apache/airflow/issues/10182 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] potiuk commented on issue #18932: Conflicts with airflow constraints for airflow 2.2.0 python 3.7

2022-05-28 Thread GitBox
potiuk commented on issue #18932: URL: https://github.com/apache/airflow/issues/18932#issuecomment-1140335803 This has been documented in https://github.com/apache/airflow/pull/23697 in docs. closing. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] potiuk closed issue #18932: Conflicts with airflow constraints for airflow 2.2.0 python 3.7

2022-05-28 Thread GitBox
potiuk closed issue #18932: Conflicts with airflow constraints for airflow 2.2.0 python 3.7 URL: https://github.com/apache/airflow/issues/18932 -- 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

[GitHub] [airflow] potiuk commented on pull request #23981: Remove `xcom_push` from `DockerOperator`

2022-05-28 Thread GitBox
potiuk commented on PR #23981: URL: https://github.com/apache/airflow/pull/23981#issuecomment-1140335308 > We should have a discussion on the propsal in the mailing list Absolutely -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [airflow] potiuk commented on pull request #23994: Remove fixing cncf.kubernetes provider when generating constraints

2022-05-28 Thread GitBox
potiuk commented on PR #23994: URL: https://github.com/apache/airflow/pull/23994#issuecomment-1140335149 This was the reason why cncf.kubernetes 3.1.2 was generated in the "pypi" constraints - simply because I have forgotten we pinned it when we yanked the providers (so that the constraint

[GitHub] [airflow] potiuk opened a new pull request, #23994: Remove fixing cncf.kubernetes provider when generating constraints

2022-05-28 Thread GitBox
potiuk opened a new pull request, #23994: URL: https://github.com/apache/airflow/pull/23994 When we yanked cncf.kubernetes provider, we pinned 3.1.2 temporarily for provider generation. This removes the pinning as we are already at 4.0.2 version --- **^ Add meaningful

[airflow] branch main updated: Remove pinning for xmltodict (#23992)

2022-05-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new eec85d397e Remove pinning for xmltodict (#23992)

[GitHub] [airflow] potiuk closed issue #23576: The xmltodict 0.13.0 breaks some emr tests

2022-05-28 Thread GitBox
potiuk closed issue #23576: The xmltodict 0.13.0 breaks some emr tests URL: https://github.com/apache/airflow/issues/23576 -- 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

[GitHub] [airflow] potiuk merged pull request #23992: Remove pinning for xmltodict

2022-05-28 Thread GitBox
potiuk merged PR #23992: URL: https://github.com/apache/airflow/pull/23992 -- 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:

[GitHub] [airflow] eladkal closed issue #10835: System Test Automation (AIP-4)

2022-05-28 Thread GitBox
eladkal closed issue #10835: System Test Automation (AIP-4) URL: https://github.com/apache/airflow/issues/10835 -- 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,

[GitHub] [airflow] eladkal commented on issue #10835: System Test Automation (AIP-4)

2022-05-28 Thread GitBox
eladkal commented on issue #10835: URL: https://github.com/apache/airflow/issues/10835#issuecomment-1140330868 Closing as we have AIP47 -- 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

[GitHub] [airflow] eladkal commented on issue #15147: Dag created with a dag decorator in a packaged dag file (zip) has incorrect filepath property

2022-05-28 Thread GitBox
eladkal commented on issue #15147: URL: https://github.com/apache/airflow/issues/15147#issuecomment-1140330698 Does it still happens in latest Airflow version? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] eladkal closed issue #12647: Dynamic Workflows by spinning multiple task Instance during Dag Run

2022-05-28 Thread GitBox
eladkal closed issue #12647: Dynamic Workflows by spinning multiple task Instance during Dag Run URL: https://github.com/apache/airflow/issues/12647 -- 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

[GitHub] [airflow] eladkal commented on issue #12647: Dynamic Workflows by spinning multiple task Instance during Dag Run

2022-05-28 Thread GitBox
eladkal commented on issue #12647: URL: https://github.com/apache/airflow/issues/12647#issuecomment-1140330365 Closing as already implemented by AIP 42 -- 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

[GitHub] [airflow] eladkal closed issue #9555: AIP-5 Remote DagFetcher

2022-05-28 Thread GitBox
eladkal closed issue #9555: AIP-5 Remote DagFetcher URL: https://github.com/apache/airflow/issues/9555 -- 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:

[GitHub] [airflow] eladkal commented on issue #9555: AIP-5 Remote DagFetcher

2022-05-28 Thread GitBox
eladkal commented on issue #9555: URL: https://github.com/apache/airflow/issues/9555#issuecomment-1140330240 Closing as this is a discussion for the mailing list not for Github issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [airflow] eladkal commented on pull request #23981: Remove `xcom_push` from `DockerOperator`

2022-05-28 Thread GitBox
eladkal commented on PR #23981: URL: https://github.com/apache/airflow/pull/23981#issuecomment-1140329718 We should have a discussion on the propsal in the mailing list -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [airflow] github-actions[bot] commented on pull request #23992: Remove pinning for xmltodict

2022-05-28 Thread GitBox
github-actions[bot] commented on PR #23992: URL: https://github.com/apache/airflow/pull/23992#issuecomment-1140328655 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the

[GitHub] [airflow] potiuk opened a new pull request, #23992: Remove pinning for xmltodict

2022-05-28 Thread GitBox
potiuk opened a new pull request, #23992: URL: https://github.com/apache/airflow/pull/23992 We have now moto 3.1.9+ in constraints so we should remove the limit. Fixes: #23576 --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] eladkal merged pull request #23983: Replace `use_task_execution_date` with `use_task_logical_date`

2022-05-28 Thread GitBox
eladkal merged PR #23983: URL: https://github.com/apache/airflow/pull/23983 -- 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:

[airflow] branch main updated: Replace `use_task_execution_date` with `use_task_logical_date` (#23983)

2022-05-28 Thread eladkal
This is an automated email from the ASF dual-hosted git repository. eladkal pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 614b2329c1 Replace `use_task_execution_date` with

[airflow] branch constraints-2-3 updated: Updating constraints. Build id:2400830677

2022-05-28 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch constraints-2-3 in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/constraints-2-3 by this push: new e38ce67c9f Updating

[GitHub] [airflow] uranusjr commented on issue #23973: PythonVirtualenvOperator backward compatibility issue - requirements params

2022-05-28 Thread GitBox
uranusjr commented on issue #23973: URL: https://github.com/apache/airflow/issues/23973#issuecomment-1140295215 Unfortunately `requirements` being passed directly to `pip` is an implementation detail, and backward compatibility is not guaranteed. Feel free to submit a new feature to allow

[GitHub] [airflow] uranusjr closed issue #23973: PythonVirtualenvOperator backward compatibility issue - requirements params

2022-05-28 Thread GitBox
uranusjr closed issue #23973: PythonVirtualenvOperator backward compatibility issue - requirements params URL: https://github.com/apache/airflow/issues/23973 -- 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

[GitHub] [airflow] github-actions[bot] commented on pull request #23983: Replace `use_task_execution_date` with `use_task_logical_date`

2022-05-28 Thread GitBox
github-actions[bot] commented on PR #23983: URL: https://github.com/apache/airflow/pull/23983#issuecomment-1140295029 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the

[GitHub] [airflow] potiuk commented on issue #16176: Quickstart Helm Chart fails post-install

2022-05-28 Thread GitBox
potiuk commented on issue #16176: URL: https://github.com/apache/airflow/issues/16176#issuecomment-1140294318 No idea. You have to debug it. -- 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

[GitHub] [airflow] eladkal commented on issue #23954: Add missing parameter documentation in `DatabricksSubmitRunOperator` and `DatabricksSqlOperator`

2022-05-28 Thread GitBox
eladkal commented on issue #23954: URL: https://github.com/apache/airflow/issues/23954#issuecomment-1140292507 assigned -- 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

[GitHub] [airflow] yuyafukuchi commented on issue #23954: Add missing parameter documentation in `DatabricksSubmitRunOperator` and `DatabricksSqlOperator`

2022-05-28 Thread GitBox
yuyafukuchi commented on issue #23954: URL: https://github.com/apache/airflow/issues/23954#issuecomment-1140291734 Hello! Can I pick this one? -- 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

[GitHub] [airflow] javad87 commented on issue #16176: Quickstart Helm Chart fails post-install

2022-05-28 Thread GitBox
javad87 commented on issue #16176: URL: https://github.com/apache/airflow/issues/16176#issuecomment-1140291076 > So you have a problem with connecting to postgres then how can I resolve connection issue? -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow] eladkal commented on a diff in pull request #23983: Replace `use_task_execution_date` with `use_task_logical_date`

2022-05-28 Thread GitBox
eladkal commented on code in PR #23983: URL: https://github.com/apache/airflow/pull/23983#discussion_r884151696 ## airflow/operators/datetime.py: ## @@ -64,10 +66,16 @@ def __init__( self.target_upper = target_upper self.follow_task_ids_if_true =

[GitHub] [airflow] eladkal commented on a diff in pull request #23983: Replace `use_task_execution_date` with `use_task_logical_date`

2022-05-28 Thread GitBox
eladkal commented on code in PR #23983: URL: https://github.com/apache/airflow/pull/23983#discussion_r884151696 ## airflow/operators/datetime.py: ## @@ -64,10 +66,16 @@ def __init__( self.target_upper = target_upper self.follow_task_ids_if_true =

[GitHub] [airflow] uranusjr commented on a diff in pull request #23983: Replace `use_task_execution_date` with `use_task_logical_date`

2022-05-28 Thread GitBox
uranusjr commented on code in PR #23983: URL: https://github.com/apache/airflow/pull/23983#discussion_r884150686 ## airflow/operators/datetime.py: ## @@ -64,10 +66,16 @@ def __init__( self.target_upper = target_upper self.follow_task_ids_if_true =

[GitHub] [airflow] uranusjr commented on a diff in pull request #23983: Replace `use_task_execution_date` with `use_task_logical_date`

2022-05-28 Thread GitBox
uranusjr commented on code in PR #23983: URL: https://github.com/apache/airflow/pull/23983#discussion_r884150657 ## airflow/operators/datetime.py: ## @@ -64,10 +66,16 @@ def __init__( self.target_upper = target_upper self.follow_task_ids_if_true =

[GitHub] [airflow] uranusjr commented on a diff in pull request #23983: Replace `use_task_execution_date` with `use_task_logical_date`

2022-05-28 Thread GitBox
uranusjr commented on code in PR #23983: URL: https://github.com/apache/airflow/pull/23983#discussion_r884150587 ## airflow/operators/datetime.py: ## @@ -64,10 +66,16 @@ def __init__( self.target_upper = target_upper self.follow_task_ids_if_true =

[GitHub] [airflow] potiuk commented on issue #16176: Quickstart Helm Chart fails post-install

2022-05-28 Thread GitBox
potiuk commented on issue #16176: URL: https://github.com/apache/airflow/issues/16176#issuecomment-1140287196 So you have a problem with connecting to postgres then -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [airflow] javad87 commented on issue #16176: Quickstart Helm Chart fails post-install

2022-05-28 Thread GitBox
javad87 commented on issue #16176: URL: https://github.com/apache/airflow/issues/16176#issuecomment-1140286800 > i Context: microk8s <0> tail <6> 1h ClearToggle… __. Cluster: microk8s-cluster

[GitHub] [airflow] chenglongyan opened a new pull request, #23991: Migrate HTTP example DAGs to new design

2022-05-28 Thread GitBox
chenglongyan opened a new pull request, #23991: URL: https://github.com/apache/airflow/pull/23991 closes: #22448 , #22431 --- **^ Add meaningful description above** Read the **[Pull Request

[airflow] branch main updated: Replace generation of docker volumes to be done from python (#23985)

2022-05-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 882535a8a2 Replace generation of docker volumes to

[GitHub] [airflow] potiuk merged pull request #23985: Replace generation of docker volumes to be done from python

2022-05-28 Thread GitBox
potiuk merged PR #23985: URL: https://github.com/apache/airflow/pull/23985 -- 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:

[GitHub] [airflow] ephraimbuddy closed pull request #23966: Sync `v2-3-stable` with `v2-3-test` to release `2.3.2`

2022-05-28 Thread GitBox
ephraimbuddy closed pull request #23966: Sync `v2-3-stable` with `v2-3-test` to release `2.3.2` URL: https://github.com/apache/airflow/pull/23966 -- 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

[airflow] branch main updated (230a737749 -> 148013b2a8)

2022-05-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from 230a737749 Force colors in yarn test output in CI (#23986) add 148013b2a8 Fix breeze failures when there is no

[GitHub] [airflow] potiuk merged pull request #23988: Fix breeze failures when there is no buildx installed on Mac

2022-05-28 Thread GitBox
potiuk merged PR #23988: URL: https://github.com/apache/airflow/pull/23988 -- 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:

[airflow] branch constraints-main updated: Updating constraints. Build id:2401017363

2022-05-28 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch constraints-main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/constraints-main by this push: new 3c4ad3202b Updating

[GitHub] [airflow] github-actions[bot] commented on pull request #23988: Fix breeze failures when there is no buildx installed on Mac

2022-05-28 Thread GitBox
github-actions[bot] commented on PR #23988: URL: https://github.com/apache/airflow/pull/23988#issuecomment-1140270706 The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide that full

[GitHub] [airflow] potiuk commented on pull request #23980: Add better diagnostics capabilities for pre-commits run via CI image

2022-05-28 Thread GitBox
potiuk commented on PR #23980: URL: https://github.com/apache/airflow/pull/23980#issuecomment-1140268493 Green now :) -- 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

[GitHub] [airflow] potiuk commented on pull request #23988: Fix breeze failures when there is no buildx installed on Mac

2022-05-28 Thread GitBox
potiuk commented on PR #23988: URL: https://github.com/apache/airflow/pull/23988#issuecomment-1140267636 something wrong with previous branch :( -- 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

[GitHub] [airflow] potiuk opened a new pull request, #23988: Fix breeze failures when there is no buildx installed on Mac

2022-05-28 Thread GitBox
potiuk opened a new pull request, #23988: URL: https://github.com/apache/airflow/pull/23988 If you have no buildx plugin installed on Mac (for example when you use colima instead of Docker Desktop) the breeze check was failing - but buildx in fact is not needed to run typical breeze

[GitHub] [airflow] github-actions[bot] commented on pull request #23985: Replace generation of docker volumes to be done from python

2022-05-28 Thread GitBox
github-actions[bot] commented on PR #23985: URL: https://github.com/apache/airflow/pull/23985#issuecomment-1140266752 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the

[GitHub] [airflow] rbiegacz commented on issue #22727: google provider - GKEStartPodOperator without gcloud to get conf

2022-05-28 Thread GitBox
rbiegacz commented on issue #22727: URL: https://github.com/apache/airflow/issues/22727#issuecomment-1140255347 @lukas-at-harren - are you planning to work on this item to fix it? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [airflow] potiuk closed pull request #23987: Fix breeze failures when there is no buildx installed on Mac

2022-05-28 Thread GitBox
potiuk closed pull request #23987: Fix breeze failures when there is no buildx installed on Mac URL: https://github.com/apache/airflow/pull/23987 -- 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

[GitHub] [airflow] potiuk commented on pull request #23987: Fix breeze failures when there is no buildx installed on Mac

2022-05-28 Thread GitBox
potiuk commented on PR #23987: URL: https://github.com/apache/airflow/pull/23987#issuecomment-1140251852 Testing with colima -- 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

[GitHub] [airflow] potiuk commented on issue #16176: Quickstart Helm Chart fails post-install

2022-05-28 Thread GitBox
potiuk commented on issue #16176: URL: https://github.com/apache/airflow/issues/16176#issuecomment-1140251156 K9s will allow you to monitor more logs in your deployment and likely find the right problem - just explore your installation with it. -- This is an automated message from the

[GitHub] [airflow] potiuk commented on issue #16176: Quickstart Helm Chart fails post-install

2022-05-28 Thread GitBox
potiuk commented on issue #16176: URL: https://github.com/apache/airflow/issues/16176#issuecomment-1140250959 can you please use `k9s` tool and connect/monitor the migration logs ? I found out that it is much better in getting to the right logs. -- This is an automated message from the

[airflow] branch main updated: Force colors in yarn test output in CI (#23986)

2022-05-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 230a737749 Force colors in yarn test output in CI

[GitHub] [airflow] potiuk merged pull request #23986: Force colors in yarn test output in CI

2022-05-28 Thread GitBox
potiuk merged PR #23986: URL: https://github.com/apache/airflow/pull/23986 -- 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:

[GitHub] [airflow] potiuk commented on pull request #23986: Force colors in yarn test output in CI

2022-05-28 Thread GitBox
potiuk commented on PR #23986: URL: https://github.com/apache/airflow/pull/23986#issuecomment-1140250636 Much nicer: https://user-images.githubusercontent.com/595491/170825308-37c1c279-c4cb-4891-96e3-f6f9ada57edd.png;> -- This is an automated message from the Apache Git

[GitHub] [airflow] github-actions[bot] commented on pull request #23987: Fix breeze failures when there is no buildx installed on Mac

2022-05-28 Thread GitBox
github-actions[bot] commented on PR #23987: URL: https://github.com/apache/airflow/pull/23987#issuecomment-1140250570 The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide that full

[GitHub] [airflow] eladkal commented on pull request #23987: Fix breeze failures when there is no buildx installed on Mac

2022-05-28 Thread GitBox
eladkal commented on PR #23987: URL: https://github.com/apache/airflow/pull/23987#issuecomment-1140250473 Cool! -- 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

[GitHub] [airflow] github-actions[bot] commented on pull request #23986: Force colors in yarn test output in CI

2022-05-28 Thread GitBox
github-actions[bot] commented on PR #23986: URL: https://github.com/apache/airflow/pull/23986#issuecomment-1140250448 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the

[GitHub] [airflow] potiuk opened a new pull request, #23987: Fix breeze failures when there is no buildx installed on Mac

2022-05-28 Thread GitBox
potiuk opened a new pull request, #23987: URL: https://github.com/apache/airflow/pull/23987 If you have no buildx plugin installed on Mac (for example when you use colima instead of Docker Desktop) the breeze check was failing - but buildx in fact is not needed to run typical breeze

[GitHub] [airflow] potiuk commented on pull request #23987: Fix breeze failures when there is no buildx installed on Mac

2022-05-28 Thread GitBox
potiuk commented on PR #23987: URL: https://github.com/apache/airflow/pull/23987#issuecomment-1140250404 Another problem found during our workshop. -- 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

[GitHub] [airflow] potiuk opened a new pull request, #23986: Force colors in yarn test output in CI

2022-05-28 Thread GitBox
potiuk opened a new pull request, #23986: URL: https://github.com/apache/airflow/pull/23986 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more

[GitHub] [airflow] potiuk commented on pull request #23986: Force colors in yarn test output in CI

2022-05-28 Thread GitBox
potiuk commented on PR #23986: URL: https://github.com/apache/airflow/pull/23986#issuecomment-1140248027 We noticed that with @eladkal during First-time contributor's workshop :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [airflow] potiuk commented on pull request #23985: Replace generation of docker volumes to be done from python

2022-05-28 Thread GitBox
potiuk commented on PR #23985: URL: https://github.com/apache/airflow/pull/23985#issuecomment-1140246678 The next step to get rid of Bash. -- 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

[GitHub] [airflow] potiuk opened a new pull request, #23985: Replace generation of docker volumes to be done from python

2022-05-28 Thread GitBox
potiuk opened a new pull request, #23985: URL: https://github.com/apache/airflow/pull/23985 The pre-commit to generate docker volumes in docker compose file is now written in Python and it also uses the newer "volume:" syntax to define the volumes mounted in the docker-compose.

[GitHub] [airflow] javad87 commented on issue #16176: Quickstart Helm Chart fails post-install

2022-05-28 Thread GitBox
javad87 commented on issue #16176: URL: https://github.com/apache/airflow/issues/16176#issuecomment-1140242570 > What the logs of your migration pods show ? https://www.digitalocean.com/community/questions/how-to-check-the-logs-of-running-and-crashed-pods-in-kubernetes I run this

[airflow] branch main updated: use explicit --mount with types of mounts rather than --volume flags (#23982)

2022-05-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 49367642ef use explicit --mount with types of

[GitHub] [airflow] potiuk merged pull request #23982: use explicit --mount with types of mounts rather than --volume flags

2022-05-28 Thread GitBox
potiuk merged PR #23982: URL: https://github.com/apache/airflow/pull/23982 -- 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:

[GitHub] [airflow] github-actions[bot] commented on pull request #23984: Fix missing shorthand for docker buildx rm -f

2022-05-28 Thread GitBox
github-actions[bot] commented on PR #23984: URL: https://github.com/apache/airflow/pull/23984#issuecomment-1140238683 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the

[GitHub] [airflow] potiuk merged pull request #23984: Fix missing shorthand for docker buildx rm -f

2022-05-28 Thread GitBox
potiuk merged PR #23984: URL: https://github.com/apache/airflow/pull/23984 -- 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:

[GitHub] [airflow] potiuk commented on issue #16176: Quickstart Helm Chart fails post-install

2022-05-28 Thread GitBox
potiuk commented on issue #16176: URL: https://github.com/apache/airflow/issues/16176#issuecomment-1140238661 What the logs of your migration pods show ? https://www.digitalocean.com/community/questions/how-to-check-the-logs-of-running-and-crashed-pods-in-kubernetes -- This is an

[airflow] branch v2-3-test updated (f1e19f2ba1 -> ea8a570a5e)

2022-05-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch v2-3-test in repository https://gitbox.apache.org/repos/asf/airflow.git from f1e19f2ba1 Update version to 2.3.2 and add release notes add ea8a570a5e Fix missing shorthand for docker

[airflow] branch main updated: Fix missing shorthand for docker buildx rm -f (#23984)

2022-05-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new fc17fbfe96 Fix missing shorthand for docker buildx

[GitHub] [airflow] potiuk opened a new pull request, #23984: Fix missing shorthand for docker buildx rm -f

2022-05-28 Thread GitBox
potiuk opened a new pull request, #23984: URL: https://github.com/apache/airflow/pull/23984 Latest version of buildx removed -f as shorthand for --force flag. --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] github-actions[bot] commented on pull request #23982: use explicit --mount with types of mounts rather than --volume flags

2022-05-28 Thread GitBox
github-actions[bot] commented on PR #23982: URL: https://github.com/apache/airflow/pull/23982#issuecomment-1140231326 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the

  1   2   >