[GitHub] [airflow] XD-DENG commented on issue #16013: CLI 'kubernetes cleanup-pods' fails on invalid label key

2021-07-28 Thread GitBox
XD-DENG commented on issue #16013: URL: https://github.com/apache/airflow/issues/16013#issuecomment-32164 Thanks all for your feedbacks, and sorry for responding slowly (due to my day work schedule) @dlampa , many thanks for helping look into the issue. I will find a time to rev

[GitHub] [airflow] XD-DENG removed a comment on issue #16013: CLI 'kubernetes cleanup-pods' fails on invalid label key

2021-07-28 Thread GitBox
XD-DENG removed a comment on issue #16013: URL: https://github.com/apache/airflow/issues/16013#issuecomment-31150 Thanks all for your feedbacks, and sorry for responding slowly (die to my day works) -- This is an automated message from the Apache Git Service. To respond to th

[GitHub] [airflow] XD-DENG commented on issue #16013: CLI 'kubernetes cleanup-pods' fails on invalid label key

2021-07-28 Thread GitBox
XD-DENG commented on issue #16013: URL: https://github.com/apache/airflow/issues/16013#issuecomment-31150 Thanks all for your feedbacks, and sorry for responding slowly (die to my day works) -- This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [airflow] eladkal commented on pull request #17144: deprecate dummy trigger rule infavor of always

2021-07-28 Thread GitBox
eladkal commented on pull request #17144: URL: https://github.com/apache/airflow/pull/17144#issuecomment-24085 addressed all comments - should be good to go now. test failures seem to be unrelated. I'm leaving the merge decision for one of the PMCs as I'm not sure if we should wai

[GitHub] [airflow] uranusjr commented on pull request #17016: Move the DagRun-TaskInstance relationship to TaskInstance

2021-07-28 Thread GitBox
uranusjr commented on pull request #17016: URL: https://github.com/apache/airflow/pull/17016#issuecomment-24175 I’m not actively working on this right now. Feel free to submit a superceding PR. -- This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [airflow] uranusjr commented on a change in pull request #16352: AIP-39: ``DagRun.data_interval_start`` and ``data_interval_end``

2021-07-28 Thread GitBox
uranusjr commented on a change in pull request #16352: URL: https://github.com/apache/airflow/pull/16352#discussion_r678841333 ## File path: airflow/timetables/interval.py ## @@ -43,6 +43,13 @@ def __eq__(self, other: Any) -> bool: def validate(self) -> None: self

[GitHub] [airflow] uranusjr commented on a change in pull request #16352: AIP-39: ``DagRun.data_interval_start`` and ``data_interval_end``

2021-07-28 Thread GitBox
uranusjr commented on a change in pull request #16352: URL: https://github.com/apache/airflow/pull/16352#discussion_r678841333 ## File path: airflow/timetables/interval.py ## @@ -43,6 +43,13 @@ def __eq__(self, other: Any) -> bool: def validate(self) -> None: self

[GitHub] [airflow] uranusjr commented on a change in pull request #16352: AIP-39: ``DagRun.data_interval_start`` and ``data_interval_end``

2021-07-28 Thread GitBox
uranusjr commented on a change in pull request #16352: URL: https://github.com/apache/airflow/pull/16352#discussion_r678841333 ## File path: airflow/timetables/interval.py ## @@ -43,6 +43,13 @@ def __eq__(self, other: Any) -> bool: def validate(self) -> None: self

[GitHub] [airflow] potiuk commented on issue #17301: ResolutionImpossible when rebuilding image with Breeze

2021-07-28 Thread GitBox
potiuk commented on issue #17301: URL: https://github.com/apache/airflow/issues/17301#issuecomment-18723 BTW @uranusjr The new generated constraints are here: https://github.com/apache/airflow/commit/f31598ed62fed9d2d8575ab10b768df1f09c9701 - as you can see from the history - in most

[GitHub] [airflow] potiuk commented on issue #17301: ResolutionImpossible when rebuilding image with Breeze

2021-07-28 Thread GitBox
potiuk commented on issue #17301: URL: https://github.com/apache/airflow/issues/17301#issuecomment-17398 It is only really needed when you want to force-update some dependency and increase the minimum version (like we did with FAB) - in all other cases it will work just fine and Breeze

[GitHub] [airflow] potiuk commented on issue #17301: ResolutionImpossible when rebuilding image with Breeze

2021-07-28 Thread GitBox
potiuk commented on issue #17301: URL: https://github.com/apache/airflow/issues/17301#issuecomment-15668 Should be fixed now - I generated them yesterday, but have not pushed the change so this was very quick. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow] potiuk closed issue #17301: ResolutionImpossible when rebuilding image with Breeze

2021-07-28 Thread GitBox
potiuk closed issue #17301: URL: https://github.com/apache/airflow/issues/17301 -- 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: commits-unsubscr..

[airflow] branch constraints-main updated: Adding latest constraint files after FAB upgrade

2021-07-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk 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 f31598e Adding latest cons

[GitHub] [airflow] uranusjr commented on a change in pull request #16352: AIP-39: ``DagRun.data_interval_start`` and ``data_interval_end``

2021-07-28 Thread GitBox
uranusjr commented on a change in pull request #16352: URL: https://github.com/apache/airflow/pull/16352#discussion_r678836464 ## File path: airflow/www/views.py ## @@ -1537,11 +1537,12 @@ def trigger(self, session=None): 'airflow/trigger.html', dag_id=dag_

[GitHub] [airflow] uranusjr commented on a change in pull request #16352: AIP-39: ``DagRun.data_interval_start`` and ``data_interval_end``

2021-07-28 Thread GitBox
uranusjr commented on a change in pull request #16352: URL: https://github.com/apache/airflow/pull/16352#discussion_r678836464 ## File path: airflow/www/views.py ## @@ -1537,11 +1537,12 @@ def trigger(self, session=None): 'airflow/trigger.html', dag_id=dag_

[GitHub] [airflow] uranusjr commented on a change in pull request #16352: AIP-39: ``DagRun.data_interval_start`` and ``data_interval_end``

2021-07-28 Thread GitBox
uranusjr commented on a change in pull request #16352: URL: https://github.com/apache/airflow/pull/16352#discussion_r678836464 ## File path: airflow/www/views.py ## @@ -1537,11 +1537,12 @@ def trigger(self, session=None): 'airflow/trigger.html', dag_id=dag_

[GitHub] [airflow] potiuk edited a comment on issue #17301: ResolutionImpossible when rebuilding image with Breeze

2021-07-28 Thread GitBox
potiuk edited a comment on issue #17301: URL: https://github.com/apache/airflow/issues/17301#issuecomment-14787 Yep. I have not pushed latest constraints yesterday. My fault. It should happen automatically after successful merge to main. But since we are now having the mssql it

[GitHub] [airflow] potiuk commented on issue #17301: ResolutionImpossible when rebuilding image with Breeze

2021-07-28 Thread GitBox
potiuk commented on issue #17301: URL: https://github.com/apache/airflow/issues/17301#issuecomment-14787 Yep. I have not pushed latest constraints yesterday. My fault. It should happen automatically after successful merge to main. But since we are now having the mssql it has to

[GitHub] [airflow] uranusjr commented on a change in pull request #16352: AIP-39: ``DagRun.data_interval_start`` and ``data_interval_end``

2021-07-28 Thread GitBox
uranusjr commented on a change in pull request #16352: URL: https://github.com/apache/airflow/pull/16352#discussion_r678828040 ## File path: airflow/models/dag.py ## @@ -112,6 +113,51 @@ def get_last_dagrun(dag_id, session, include_externally_triggered=False): return quer

[GitHub] [airflow] penggongkui commented on pull request #17016: Move the DagRun-TaskInstance relationship to TaskInstance

2021-07-28 Thread GitBox
penggongkui commented on pull request #17016: URL: https://github.com/apache/airflow/pull/17016#issuecomment-04482 Any update on this? I have to change the code by myself to avoid the delete dag issue after migrate airflow. Is it possible fixed it in 2.1.3? -- This is an automated me

[GitHub] [airflow] uranusjr opened a new issue #17301: ResolutionImpossible when rebuilding image with Breeze

2021-07-28 Thread GitBox
uranusjr opened a new issue #17301: URL: https://github.com/apache/airflow/issues/17301 ```console $ ./breeze build-image ... #16 30.11 ERROR: Cannot install apache-airflow[devel-ci]==2.2.0.dev0 because these package versions have conflicting dependencies. #16 30.11 #16 30.11

[GitHub] [airflow] boring-cyborg[bot] commented on issue #17300: Tasks occasionally receive SIGTERM at the end of a successful run

2021-07-28 Thread GitBox
boring-cyborg[bot] commented on issue #17300: URL: https://github.com/apache/airflow/issues/17300#issuecomment-888763153 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [airflow] jhh3000 opened a new issue #17300: Tasks occasionally receive SIGTERM at the end of a successful run

2021-07-28 Thread GitBox
jhh3000 opened a new issue #17300: URL: https://github.com/apache/airflow/issues/17300 **Apache Airflow version**: 2.1.0 **Environment**: Docker - **Cloud provider or hardware configuration**: AWS ECS Fargate - **OS** (e.g. from /etc/os-release): - **Kernel** (e.g. `uname

[GitHub] [airflow] uranusjr commented on a change in pull request #17269: Handle connection parameters added to Extra and custom fields

2021-07-28 Thread GitBox
uranusjr commented on a change in pull request #17269: URL: https://github.com/apache/airflow/pull/17269#discussion_r678786577 ## File path: airflow/www/views.py ## @@ -3226,6 +3226,12 @@ def process_form(self, form, is_created): for key in self.extra_fields

[GitHub] [airflow] josh-fell commented on a change in pull request #17269: Handle connection parameters added to Extra and custom fields

2021-07-28 Thread GitBox
josh-fell commented on a change in pull request #17269: URL: https://github.com/apache/airflow/pull/17269#discussion_r678780369 ## File path: airflow/www/views.py ## @@ -3226,6 +3226,12 @@ def process_form(self, form, is_created): for key in self.extra_fields

[GitHub] [airflow] github-actions[bot] commented on pull request #17236: [Airflow 16364] Add conn_timeout and cmd_timeout params to SSHOperator; add conn_timeout param to SSHHook

2021-07-28 Thread GitBox
github-actions[bot] commented on pull request #17236: URL: https://github.com/apache/airflow/pull/17236#issuecomment-888747412 The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not m

[GitHub] [airflow] Gabriel39 commented on pull request #17181: POC for event notes

2021-07-28 Thread GitBox
Gabriel39 commented on pull request #17181: URL: https://github.com/apache/airflow/pull/17181#issuecomment-888728657 > Could you give some screenshots of where this shows up in the UI etc please? @ashb I think it is similar to the screenshot in this page https://github.com/apache/ai

[GitHub] [airflow] fatmumuhomer commented on pull request #17236: [Airflow 16364] Add conn_timeout and cmd_timeout params to SSHOperator; add conn_timeout param to SSHHook

2021-07-28 Thread GitBox
fatmumuhomer commented on pull request #17236: URL: https://github.com/apache/airflow/pull/17236#issuecomment-888727601 Thanks, @uranusjr! Those two changes are done. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[airflow] branch main updated (ac0c2d1 -> 53e9349)

2021-07-28 Thread uranusjr
This is an automated email from the ASF dual-hosted git repository. uranusjr pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from ac0c2d1 Doc: Fix a broken link in an ssh-related warning message (#17294) add 53e9349 Fix breeze kind-cluste

[GitHub] [airflow] uranusjr merged pull request #17293: Fix breeze kind-cluster deploy failing with ECONREFUSED

2021-07-28 Thread GitBox
uranusjr merged pull request #17293: URL: https://github.com/apache/airflow/pull/17293 -- 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: commits-uns

[GitHub] [airflow] uranusjr commented on pull request #17293: Fix breeze kind-cluster deploy failing with ECONREFUSED

2021-07-28 Thread GitBox
uranusjr commented on pull request #17293: URL: https://github.com/apache/airflow/pull/17293#issuecomment-888724418 Hmm, main is broken… this PR is fine on itself. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

[GitHub] [airflow] uranusjr commented on a change in pull request #17278: Fix `airflow celery stop` to accept the pid file.

2021-07-28 Thread GitBox
uranusjr commented on a change in pull request #17278: URL: https://github.com/apache/airflow/pull/17278#discussion_r678751397 ## File path: tests/cli/commands/test_celery_command.py ## @@ -142,6 +143,42 @@ def test_same_pid_file_is_used_in_start_and_stop( celery_comma

[GitHub] [airflow] uranusjr commented on a change in pull request #17236: [Airflow 16364] Add conn_timeout and cmd_timeout params to SSHOperator; add conn_timeout param to SSHHook

2021-07-28 Thread GitBox
uranusjr commented on a change in pull request #17236: URL: https://github.com/apache/airflow/pull/17236#discussion_r678748100 ## File path: airflow/providers/ssh/hooks/ssh.py ## @@ -148,7 +155,17 @@ def __init__( self.pkey = self._pkey_from_private_key(pri

[GitHub] [airflow] fatmumuhomer commented on pull request #17236: [Airflow 16364] Add conn_timeout and cmd_timeout params to SSHOperator; add conn_timeout param to SSHHook

2021-07-28 Thread GitBox
fatmumuhomer commented on pull request #17236: URL: https://github.com/apache/airflow/pull/17236#issuecomment-888703109 @uranusjr I made your suggested changes. Thanks for reviewing. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [airflow] github-actions[bot] closed pull request #16230: Apache Flagon UserALE JS analytics backend

2021-07-28 Thread GitBox
github-actions[bot] closed pull request #16230: URL: https://github.com/apache/airflow/pull/16230 -- 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] closed pull request #16257: Move `UPDATING.md` to `updating` directory in `docs/apache-airflow`

2021-07-28 Thread GitBox
github-actions[bot] closed pull request #16257: URL: https://github.com/apache/airflow/pull/16257 -- 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 #16003: Fixed Discord Webhook endpoint

2021-07-28 Thread GitBox
github-actions[bot] commented on pull request #16003: URL: https://github.com/apache/airflow/pull/16003#issuecomment-888698963 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 y

[GitHub] [airflow] github-actions[bot] closed pull request #16284: Make TriggerDagRunOperator compatible with taskflow API

2021-07-28 Thread GitBox
github-actions[bot] closed pull request #16284: URL: https://github.com/apache/airflow/pull/16284 -- 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] boring-cyborg[bot] commented on issue #17299: Airflow 2 not working with VPN on Ubuntu

2021-07-28 Thread GitBox
boring-cyborg[bot] commented on issue #17299: URL: https://github.com/apache/airflow/issues/17299#issuecomment-888672539 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [airflow] ppetruneac opened a new issue #17299: Airflow 2 not working with VPN on Ubuntu

2021-07-28 Thread GitBox
ppetruneac opened a new issue #17299: URL: https://github.com/apache/airflow/issues/17299 **Apache Airflow version**: `apache-airflow==2.1.2` **Kubernetes version (if you are using kubernetes)** (use `kubectl version`): NA **Environment**: - **Cloud provider or

[GitHub] [airflow] dlampa commented on issue #16013: CLI 'kubernetes cleanup-pods' fails on invalid label key

2021-07-28 Thread GitBox
dlampa commented on issue #16013: URL: https://github.com/apache/airflow/issues/16013#issuecomment-88867 Hey @jean-malo I've reached the same conclusion re. `V1LabelSelector` and had the solution ready for a few days - please take a look at #17298 . I've created a simple script using

[GitHub] [airflow] jedcunningham closed pull request #17293: Fix breeze kind-cluster deploy failing with ECONREFUSED

2021-07-28 Thread GitBox
jedcunningham closed pull request #17293: URL: https://github.com/apache/airflow/pull/17293 -- 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: commit

[GitHub] [airflow] dlampa commented on pull request #17298: Fix for #16013 - CLI 'kubernetes cleanup-pods' fails on invalid label key

2021-07-28 Thread GitBox
dlampa commented on pull request #17298: URL: https://github.com/apache/airflow/pull/17298#issuecomment-888640913 In addition to running the complete battery of tests, this was also tested in the breeze KinD cluster by disabling the deletion of worker pods (i.e. `AIRFLOW__KUBERNETES__DELET

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #17298: Fix for #16013 - CLI 'kubernetes cleanup-pods' fails on invalid label key

2021-07-28 Thread GitBox
boring-cyborg[bot] commented on pull request #17298: URL: https://github.com/apache/airflow/pull/17298#issuecomment-888636745 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 Con

[GitHub] [airflow] dlampa opened a new pull request #17298: Fix for #16013 - CLI 'kubernetes cleanup-pods' fails on invalid label key

2021-07-28 Thread GitBox
dlampa opened a new pull request #17298: URL: https://github.com/apache/airflow/pull/17298 closes: #16013 This is a workaround for the issue with `kubernetes-client` that results in an invalid API request by `V1LabelSelector` and `match_expressions`. This employs an alternative a

[GitHub] [airflow] mik-laj commented on issue #17240: bash operator overrides environment variables instead of updating them

2021-07-28 Thread GitBox
mik-laj commented on issue #17240: URL: https://github.com/apache/airflow/issues/17240#issuecomment-888629930 Related issues: https://github.com/apache/airflow/issues/13445 https://github.com/apache/airflow/issues/14620 -- This is an automated message from the Apache Git Service. To resp

[GitHub] [airflow] potiuk commented on a change in pull request #17273: Switch to 'smbprotocol' library

2021-07-28 Thread GitBox
potiuk commented on a change in pull request #17273: URL: https://github.com/apache/airflow/pull/17273#discussion_r678660274 ## File path: airflow/providers/samba/hooks/samba.py ## @@ -16,67 +16,229 @@ # specific language governing permissions and limitations # under the Lice

[GitHub] [airflow] malthe commented on a change in pull request #17273: Switch to 'smbprotocol' library

2021-07-28 Thread GitBox
malthe commented on a change in pull request #17273: URL: https://github.com/apache/airflow/pull/17273#discussion_r678644321 ## File path: airflow/providers/samba/hooks/samba.py ## @@ -16,67 +16,229 @@ # specific language governing permissions and limitations # under the Lice

[GitHub] [airflow] potiuk commented on a change in pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
potiuk commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r678651712 ## File path: airflow/decorators/__init__.py ## @@ -41,103 +46,13 @@ def __call__( """ return self.python(python_callable=python_callab

[GitHub] [airflow] potiuk commented on a change in pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
potiuk commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r678649250 ## File path: airflow/decorators/__init__.py ## @@ -41,103 +46,13 @@ def __call__( """ return self.python(python_callable=python_callab

[GitHub] [airflow] potiuk commented on a change in pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
potiuk commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r678649250 ## File path: airflow/decorators/__init__.py ## @@ -41,103 +46,13 @@ def __call__( """ return self.python(python_callable=python_callab

[GitHub] [airflow] potiuk commented on a change in pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
potiuk commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r678649250 ## File path: airflow/decorators/__init__.py ## @@ -41,103 +46,13 @@ def __call__( """ return self.python(python_callable=python_callab

[GitHub] [airflow] potiuk commented on a change in pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
potiuk commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r678644765 ## File path: airflow/decorators/__init__.py ## @@ -41,103 +46,13 @@ def __call__( """ return self.python(python_callable=python_callab

[GitHub] [airflow] potiuk commented on a change in pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
potiuk commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r678644765 ## File path: airflow/decorators/__init__.py ## @@ -41,103 +46,13 @@ def __call__( """ return self.python(python_callable=python_callab

[GitHub] [airflow] malthe commented on a change in pull request #17273: Switch to 'smbprotocol' library

2021-07-28 Thread GitBox
malthe commented on a change in pull request #17273: URL: https://github.com/apache/airflow/pull/17273#discussion_r678644321 ## File path: airflow/providers/samba/hooks/samba.py ## @@ -16,67 +16,229 @@ # specific language governing permissions and limitations # under the Lice

[GitHub] [airflow] mik-laj commented on issue #17191: Healthcheck endpoint for workers

2021-07-28 Thread GitBox
mik-laj commented on issue #17191: URL: https://github.com/apache/airflow/issues/17191#issuecomment-888611296 @jedcunningham I am not sure. It seems to me that the main process should monitor the child process and restart it when it detects problems with it. Similarly, Scheduler monitors D

[GitHub] [airflow] potiuk commented on a change in pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
potiuk commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r678639451 ## File path: airflow/decorators/__init__.py ## @@ -41,103 +46,13 @@ def __call__( """ return self.python(python_callable=python_callab

[GitHub] [airflow] potiuk commented on a change in pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
potiuk commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r678639451 ## File path: airflow/decorators/__init__.py ## @@ -41,103 +46,13 @@ def __call__( """ return self.python(python_callable=python_callab

[GitHub] [airflow] ashb commented on a change in pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
ashb commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r678637972 ## File path: airflow/decorators/__init__.py ## @@ -41,103 +46,13 @@ def __call__( """ return self.python(python_callable=python_callable

[GitHub] [airflow] potiuk commented on a change in pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
potiuk commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r678637429 ## File path: airflow/decorators/__init__.py ## @@ -41,103 +46,13 @@ def __call__( """ return self.python(python_callable=python_callab

[GitHub] [airflow] potiuk commented on a change in pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
potiuk commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r678637429 ## File path: airflow/decorators/__init__.py ## @@ -41,103 +46,13 @@ def __call__( """ return self.python(python_callable=python_callab

[GitHub] [airflow] ashb commented on a change in pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
ashb commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r678635963 ## File path: airflow/decorators/__init__.py ## @@ -41,103 +46,13 @@ def __call__( """ return self.python(python_callable=python_callable

[GitHub] [airflow] potiuk commented on a change in pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
potiuk commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r678634716 ## File path: airflow/decorators/__init__.py ## @@ -41,103 +46,13 @@ def __call__( """ return self.python(python_callable=python_callab

[airflow] tag helm-chart/1.1.0 created (now c7ed265)

2021-07-28 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to tag helm-chart/1.1.0 in repository https://gitbox.apache.org/repos/asf/airflow.git. at c7ed265 (commit) No new revisions were added by this update.

[GitHub] [airflow] ashb commented on pull request #17270: Make decorators pluggable

2021-07-28 Thread GitBox
ashb commented on pull request #17270: URL: https://github.com/apache/airflow/pull/17270#issuecomment-888602217 Probably worth linking to https://setuptools.readthedocs.io/en/latest/userguide/entry_point.html -- This is an automated message from the Apache Git Service. To respond to the

[airflow] branch main updated (8e76f3c -> ac0c2d1)

2021-07-28 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 8e76f3c Doc: Strip unnecessary arguments from MariaDB JIRA URL (#17296) add ac0c2d1 Doc: Fix a broken link

[GitHub] [airflow] kaxil merged pull request #17294: fix a broken link in an ssh-related warning message

2021-07-28 Thread GitBox
kaxil merged pull request #17294: URL: https://github.com/apache/airflow/pull/17294 -- 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: commits-unsubs

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #17294: fix a broken link in an ssh-related warning message

2021-07-28 Thread GitBox
boring-cyborg[bot] commented on pull request #17294: URL: https://github.com/apache/airflow/pull/17294#issuecomment-888600355 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [airflow] kaxil merged pull request #17296: Doc: Strip unnecessary arguments from MariaDB JIRA URL

2021-07-28 Thread GitBox
kaxil merged pull request #17296: URL: https://github.com/apache/airflow/pull/17296 -- 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: commits-unsubs

[airflow] branch main updated (6d7fa87 -> 8e76f3c)

2021-07-28 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 6d7fa87 Updates to FlaskAppBuilder 3.3.2+ (#17208) add 8e76f3c Doc: Strip unnecessary arguments from MariaDB

[GitHub] [airflow] jedcunningham commented on issue #17238: Unexpected skipped state for tasks run with KubernetesExecutor

2021-07-28 Thread GitBox
jedcunningham commented on issue #17238: URL: https://github.com/apache/airflow/issues/17238#issuecomment-888593645 Just to clarify, the `Event: {pod_name} Succeeded` line is referring to the the pod state, not the task state, so those won't always match. Can you see if it also happe

[GitHub] [airflow] ferruzzi commented on pull request #16571: Implemented Basic EKS Integration

2021-07-28 Thread GitBox
ferruzzi commented on pull request #16571: URL: https://github.com/apache/airflow/pull/16571#issuecomment-888588265 Pull/Rebased from `main` and added the commit to remove the list and describe operators. -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [airflow] ian-byrne edited a comment on issue #16025: AirflowException: Celery command failed on host: [...]

2021-07-28 Thread GitBox
ian-byrne edited a comment on issue #16025: URL: https://github.com/apache/airflow/issues/16025#issuecomment-888489320 I am having a similar issue where all celery tasks are marked as state=Failure. All of the tasks are success in the Airflow UI and appear to have worked proper with moving

[GitHub] [airflow] github-actions[bot] commented on pull request #17296: Doc: Strip unnecessary arguments from MariaDB JIRA URL

2021-07-28 Thread GitBox
github-actions[bot] commented on pull request #17296: URL: https://github.com/apache/airflow/pull/17296#issuecomment-888577707 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

[GitHub] [airflow-site] Bl3f commented on pull request #459: Add Airflow Summit 2021 Takeaways

2021-07-28 Thread GitBox
Bl3f commented on pull request #459: URL: https://github.com/apache/airflow-site/pull/459#issuecomment-888575104 Oh 🙃 that's my bad. -- 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 speci

[GitHub] [airflow-site] potiuk closed pull request #459: Add Airflow Summit 2021 Takeaways

2021-07-28 Thread GitBox
potiuk closed pull request #459: URL: https://github.com/apache/airflow-site/pull/459 -- 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: commits-unsu

[GitHub] [airflow] kaxil opened a new pull request #17296: Doc: Strip unnecessary arguments from MariaDB JIRA URL

2021-07-28 Thread GitBox
kaxil opened a new pull request #17296: URL: https://github.com/apache/airflow/pull/17296 This was included in https://github.com/apache/airflow/pull/17287 but we can strip other args --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](h

[GitHub] [airflow] BasPH commented on issue #17241: Python Operator on Apple M1, python 3.9.6, airflow 2.1.2

2021-07-28 Thread GitBox
BasPH commented on issue #17241: URL: https://github.com/apache/airflow/issues/17241#issuecomment-888574075 Before starting anything you must initialize the metastore (i.e. create all tables). This can be done by running either: - `airflow db init` (initialize everything on empty met

[GitHub] [airflow-site] potiuk commented on pull request #459: Add Airflow Summit 2021 Takeaways

2021-07-28 Thread GitBox
potiuk commented on pull request #459: URL: https://github.com/apache/airflow-site/pull/459#issuecomment-888574098 No no. I understood me wrong. I suggested to add it to https://medium.com/apache-airflow publication . I think the blog posts at Airflow we have for announcements and similar

[GitHub] [airflow] ashb commented on a change in pull request #16571: Implemented Basic EKS Integration

2021-07-28 Thread GitBox
ashb commented on a change in pull request #16571: URL: https://github.com/apache/airflow/pull/16571#discussion_r678602421 ## File path: airflow/providers/amazon/aws/operators/eks.py ## @@ -0,0 +1,797 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more c

[GitHub] [airflow-site] Bl3f opened a new pull request #459: Add Airflow Summit 2021 Takeaways

2021-07-28 Thread GitBox
Bl3f opened a new pull request #459: URL: https://github.com/apache/airflow-site/pull/459 As @potiuk suggested on my [medium post](https://medium.com/@_blef/airflow-summit-takeaways-c3e703a104c1) I propose a PR to add the takeaways I've written after watching the whole summit sessions.

[GitHub] [airflow] potiuk closed issue #16944: Include `Authlib` in Airflow Docker images

2021-07-28 Thread GitBox
potiuk closed issue #16944: URL: https://github.com/apache/airflow/issues/16944 -- 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: commits-unsubscr..

[airflow] branch main updated (396f30f -> 6d7fa87)

2021-07-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 396f30f Uses current sources when running k8s tests (#17289) add 6d7fa87 Updates to FlaskAppBuilder 3.3.2+ (#17

[GitHub] [airflow] potiuk merged pull request #17208: Updates to FlaskAppBuilder 3.3.2+

2021-07-28 Thread GitBox
potiuk merged pull request #17208: URL: https://github.com/apache/airflow/pull/17208 -- 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: commits-unsub

[GitHub] [airflow] fjmacagno closed issue #17295: Webserver doesn't show `params` in tast details

2021-07-28 Thread GitBox
fjmacagno closed issue #17295: URL: https://github.com/apache/airflow/issues/17295 -- 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: commits-unsubsc

[GitHub] [airflow] fjmacagno commented on issue #17295: Webserver doesn't show `params` in tast details

2021-07-28 Thread GitBox
fjmacagno commented on issue #17295: URL: https://github.com/apache/airflow/issues/17295#issuecomment-888568825 Aha that makes sense. Definitely a little confusing, I would expect to see that in the task instance data. Thanks! -- This is an automated message from the Apache Git Servi

[airflow] branch main updated (96bd4a1 -> 396f30f)

2021-07-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 96bd4a1 Do not use constraints when preparing venv for k8s tests on CI (#17290) add 396f30f Uses current sourc

[GitHub] [airflow] potiuk merged pull request #17289: Uses current sources when running k8s tests

2021-07-28 Thread GitBox
potiuk merged pull request #17289: URL: https://github.com/apache/airflow/pull/17289 -- 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: commits-unsub

[airflow] branch main updated (667a45c -> 96bd4a1)

2021-07-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 667a45c Quarantine TestSchedulerJob.test_retry_still_in_executor (#17292) add 96bd4a1 Do not use constraints w

[GitHub] [airflow] potiuk merged pull request #17290: Do not use constraints when preparing venv for k8s tests on CI

2021-07-28 Thread GitBox
potiuk merged pull request #17290: URL: https://github.com/apache/airflow/pull/17290 -- 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: commits-unsub

[GitHub] [airflow] warrenstephens commented on issue #17241: Python Operator on Apple M1, python 3.9.6, airflow 2.1.2

2021-07-28 Thread GitBox
warrenstephens commented on issue #17241: URL: https://github.com/apache/airflow/issues/17241#issuecomment-888568426 Yes, my Mac M1 is running the Airflow UI, but K8S is used to run the Airflow tasks and such on a remote cluster -- part of a new RENCI HeLx platform deployment for an NIH pr

[GitHub] [airflow] jedcunningham edited a comment on issue #17295: Webserver doesn't show `params` in tast details

2021-07-28 Thread GitBox
jedcunningham edited a comment on issue #17295: URL: https://github.com/apache/airflow/issues/17295#issuecomment-888556091 @fjmacagno, I think you may be confusing `params` with the DagRun `conf`. ``` DummyOperator(task_id="task1", params={"hello": "world"}) ``` ![Screen Shot

[GitHub] [airflow] jedcunningham commented on issue #17295: Webserver doesn't show `params` in tast details

2021-07-28 Thread GitBox
jedcunningham commented on issue #17295: URL: https://github.com/apache/airflow/issues/17295#issuecomment-888556091 @fjmacagno, I think you may be confusing `params` with the DagRun `conf`. ``` DummyOperator(task_id="task1", params={"hello": "world"}) ``` ![Screen Shot 2021-0

[GitHub] [airflow] fjmacagno opened a new issue #17295: Webserver doesn't show `params` in tast details

2021-07-28 Thread GitBox
fjmacagno opened a new issue #17295: URL: https://github.com/apache/airflow/issues/17295 **Apache Airflow version**: 2.1.2 **Kubernetes version (if you are using kubernetes)** (use `kubectl version`): **Environment**: - **Cloud provider or hardware configuration**: b

[GitHub] [airflow] github-actions[bot] commented on pull request #17294: fix a broken link in an ssh-related warning message

2021-07-28 Thread GitBox
github-actions[bot] commented on pull request #17294: URL: https://github.com/apache/airflow/pull/17294#issuecomment-888543590 The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not m

[GitHub] [airflow] MatrixManAtYrService opened a new pull request #17294: fix a broken link in an ssh-related warning message

2021-07-28 Thread GitBox
MatrixManAtYrService opened a new pull request #17294: URL: https://github.com/apache/airflow/pull/17294 Warning Message: # # WARNING: You should set dags.gitSync.knownHosts # #

[GitHub] [airflow] github-actions[bot] commented on pull request #17293: Fix breeze kind-cluster deploy failing with ECONREFUSED

2021-07-28 Thread GitBox
github-actions[bot] commented on pull request #17293: URL: https://github.com/apache/airflow/pull/17293#issuecomment-888541683 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 tak

[GitHub] [airflow] ephraimbuddy opened a new pull request #17293: Fix breeze kind-cluster deploy failing with ECONREFUSED

2021-07-28 Thread GitBox
ephraimbuddy opened a new pull request #17293: URL: https://github.com/apache/airflow/pull/17293 Currently, kind-cluster deploy fails occasionally due to yarn install when compiling assets. This PR fixes it by using the recommended option --network-concurrency=1 when running yarn ins

[GitHub] [airflow] iostreamdoth commented on issue #17268: Status of testing Providers that were prepared on July 27, 2021 (rc2)

2021-07-28 Thread GitBox
iostreamdoth commented on issue #17268: URL: https://github.com/apache/airflow/issues/17268#issuecomment-888531655 fix: get_cluster call project_id as self.project_id (#17075) looks good. -- This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [airflow] efulet commented on issue #15978: DAG getting stuck in "running" state indefinitely

2021-07-28 Thread GitBox
efulet commented on issue #15978: URL: https://github.com/apache/airflow/issues/15978#issuecomment-888531327 Same issue here. I dropped airflow db and it worked. For some reason, the scheduler cannot see new tasks in the pipeline -- This is an automated message from the Apache Git Servic

  1   2   3   4   >