[GitHub] [airflow] uranusjr commented on a change in pull request #17265: Use `dag_maker` fixture in test_scheduler_job.py

2021-07-27 Thread GitBox
uranusjr commented on a change in pull request #17265: URL: https://github.com/apache/airflow/pull/17265#discussion_r677991151 ## File path: tests/jobs/test_scheduler_job.py ## @@ -1242,12 +1165,12 @@ def test_execute_task_instances_unlimited(self): date =

[GitHub] [airflow] uranusjr commented on a change in pull request #17265: Use `dag_maker` fixture in test_scheduler_job.py

2021-07-27 Thread GitBox
uranusjr commented on a change in pull request #17265: URL: https://github.com/apache/airflow/pull/17265#discussion_r677981562 ## File path: tests/conftest.py ## @@ -444,11 +445,17 @@ def __exit__(self, type, value, traceback): dag.__exit__(type, value, traceback)

[GitHub] [airflow] uranusjr commented on issue #17129: All tasks fail on logging to a Windows drive (Kubernetes-Multipass-Microk8s on Windows host)

2021-07-27 Thread GitBox
uranusjr commented on issue #17129: URL: https://github.com/apache/airflow/issues/17129#issuecomment-888016545 > While I can make a quick fix, I have no development setup on a Windows machines available to validate. It’s OK, we don’t currently have Windows in CI so we wouldn’t know

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

2021-07-27 Thread GitBox
uranusjr commented on a change in pull request #17269: URL: https://github.com/apache/airflow/pull/17269#discussion_r677977379 ## 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] uranusjr commented on a change in pull request #17269: Handle connection parameters added to Extra and custom fields

2021-07-27 Thread GitBox
uranusjr commented on a change in pull request #17269: URL: https://github.com/apache/airflow/pull/17269#discussion_r677977131 ## 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] uranusjr commented on a change in pull request #15330: Add a Docker Taskflow decorator

2021-07-27 Thread GitBox
uranusjr commented on a change in pull request #15330: URL: https://github.com/apache/airflow/pull/15330#discussion_r677957000 ## File path: airflow/providers/docker/decorators/docker.py ## @@ -0,0 +1,159 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] [airflow] uranusjr commented on a change in pull request #15330: Add a Docker Taskflow decorator

2021-07-27 Thread GitBox
uranusjr commented on a change in pull request #15330: URL: https://github.com/apache/airflow/pull/15330#discussion_r677956474 ## File path: airflow/decorators/__init__.py ## @@ -139,5 +139,169 @@ def virtualenv( **kwargs, ) +@staticmethod +def

[GitHub] [airflow] uranusjr opened a new pull request #17280: Opmitise LoggingMixin.log

2021-07-27 Thread GitBox
uranusjr opened a new pull request #17280: URL: https://github.com/apache/airflow/pull/17280 The log instance is now instantiated per class, instead of per instance, saving duplicated `getLogger()` calls. Those `getLogger()` calls would return the same logger instance anyway, so the

[GitHub] [airflow] sodafountain edited a comment on issue #16573: State of this instance has been externally set to up_for_retry. Terminating instance.

2021-07-27 Thread GitBox
sodafountain edited a comment on issue #16573: URL: https://github.com/apache/airflow/issues/16573#issuecomment-887986537 Thanks for your detailed explanation @peay This makes lot of sense. I think setting `AIRFLOW__SCHEDULER__ORPHANED_TASKS_CHECK_INTERVAL` to an appropriate value

[GitHub] [airflow] sodafountain commented on issue #16573: State of this instance has been externally set to up_for_retry. Terminating instance.

2021-07-27 Thread GitBox
sodafountain commented on issue #16573: URL: https://github.com/apache/airflow/issues/16573#issuecomment-887986537 Thanks for your detailed explanation @peay I think this makes lot of sense. I think setting the value `AIRFLOW__SCHEDULER__ORPHANED_TASKS_CHECK_INTERVAL` to an

[GitHub] [airflow] ashaw27 edited a comment on issue #10026: Duplicate tasks invoked for a single task_id when manually invoked task details modal.

2021-07-27 Thread GitBox
ashaw27 edited a comment on issue #10026: URL: https://github.com/apache/airflow/issues/10026#issuecomment-799613642 I'm having this issue in 2.0.1. Is there a workaround? My timesensor failed at a seemingly random time, twice, resulting in 4 failures. I only had 3 retries set, so the

[GitHub] [airflow] huozhanfeng edited a comment on issue #10026: Duplicate tasks invoked for a single task_id when manually invoked task details modal.

2021-07-27 Thread GitBox
huozhanfeng edited a comment on issue #10026: URL: https://github.com/apache/airflow/issues/10026#issuecomment-861340733 It's still an issue with version 2.1.0. Is there anyone who has a workaround to solve it in the airflow 2.x version? ``` [2021-06-15 16:48:12,916]

[GitHub] [airflow] andrelop edited a comment on issue #10026: Duplicate tasks invoked for a single task_id when manually invoked task details modal.

2021-07-27 Thread GitBox
andrelop edited a comment on issue #10026: URL: https://github.com/apache/airflow/issues/10026#issuecomment-887776535 Same issue here, running Airflow 2.1.1 : ``` [2021-07-27 19:21:52,578] {local_task_job.py:195} WARNING - Recorded pid 656 does not match the current pid 76

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

2021-07-27 Thread GitBox
uranusjr commented on a change in pull request #17273: URL: https://github.com/apache/airflow/pull/17273#discussion_r677946939 ## File path: airflow/providers/samba/hooks/samba.py ## @@ -31,52 +62,202 @@ class SambaHook(BaseHook): conn_type = 'samba' hook_name =

[GitHub] [airflow] zkan commented on issue #14200: Update Best practises doc

2021-07-27 Thread GitBox
zkan commented on issue #14200: URL: https://github.com/apache/airflow/issues/14200#issuecomment-887979835 @kaxil Do you mind if I help on this issue, please? -- 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 a change in pull request #17273: Switch to 'smbprotocol' library

2021-07-27 Thread GitBox
uranusjr commented on a change in pull request #17273: URL: https://github.com/apache/airflow/pull/17273#discussion_r677941361 ## File path: airflow/providers/samba/hooks/samba.py ## @@ -16,12 +16,43 @@ # specific language governing permissions and limitations # under the

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

2021-07-27 Thread GitBox
uranusjr commented on a change in pull request #17273: URL: https://github.com/apache/airflow/pull/17273#discussion_r677940668 ## File path: airflow/providers/samba/hooks/samba.py ## @@ -16,12 +16,43 @@ # specific language governing permissions and limitations # under the

[GitHub] [airflow] ariesunique edited a comment on issue #11919: Add how-to Guide for Slack operators

2021-07-27 Thread GitBox
ariesunique edited a comment on issue #11919: URL: https://github.com/apache/airflow/issues/11919#issuecomment-887973502 @zkan Yes - still looking into this. -- 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] ariesunique commented on issue #11919: Add how-to Guide for Slack operators

2021-07-27 Thread GitBox
ariesunique commented on issue #11919: URL: https://github.com/apache/airflow/issues/11919#issuecomment-887973502 Yes - still looking into this. On Tue, Jul 27, 2021 at 11:01 PM Kan Ouivirach ***@***.***> wrote: > @ariesunique Are you

[GitHub] [airflow] zkan commented on issue #11919: Add how-to Guide for Slack operators

2021-07-27 Thread GitBox
zkan commented on issue #11919: URL: https://github.com/apache/airflow/issues/11919#issuecomment-887971698 @ariesunique Are you working on this? -- 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] uranusjr commented on pull request #17208: Updates to FlaskAppBuilder 3.3.2+

2021-07-27 Thread GitBox
uranusjr commented on pull request #17208: URL: https://github.com/apache/airflow/pull/17208#issuecomment-887969689 I was hoping this would get us to SQLAlchemy 1.4, but alas it is not to be  -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [airflow] chuancyzhang commented on a change in pull request #17253: add encoding config in FtpHook

2021-07-27 Thread GitBox
chuancyzhang commented on a change in pull request #17253: URL: https://github.com/apache/airflow/pull/17253#discussion_r677935796 ## File path: airflow/providers/ftp/hooks/ftp.py ## @@ -60,8 +60,10 @@ def get_conn(self) -> ftplib.FTP: if self.conn is None:

[GitHub] [airflow] uranusjr closed pull request #17208: Updates to FlaskAppBuilder 3.3.2+

2021-07-27 Thread GitBox
uranusjr closed 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:

[GitHub] [airflow] uranusjr closed pull request #16301: Fix task retries when they receive sigkill and have retries. Properly handle sigterm too

2021-07-27 Thread GitBox
uranusjr closed pull request #16301: URL: https://github.com/apache/airflow/pull/16301 -- 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] uranusjr commented on pull request #17277: Show serialization exceptions in DAG parsing log

2021-07-27 Thread GitBox
uranusjr commented on pull request #17277: URL: https://github.com/apache/airflow/pull/17277#issuecomment-887968827 Is this stuck? -- 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] uranusjr closed pull request #17277: Show serialization exceptions in DAG parsing log

2021-07-27 Thread GitBox
uranusjr closed pull request #17277: URL: https://github.com/apache/airflow/pull/17277 -- 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] dimberman edited a comment on pull request #17270: Make decorators pluggable

2021-07-27 Thread GitBox
dimberman edited a comment on pull request #17270: URL: https://github.com/apache/airflow/pull/17270#issuecomment-887937376 @ashb unfortunately the problem with leaving them where they are is that because we are loading this stuff in the `__init__` file, it leads to an infinite loop. --

[GitHub] [airflow] uranusjr commented on a change in pull request #17253: add encoding config in FtpHook

2021-07-27 Thread GitBox
uranusjr commented on a change in pull request #17253: URL: https://github.com/apache/airflow/pull/17253#discussion_r677930962 ## File path: airflow/providers/ftp/hooks/ftp.py ## @@ -60,8 +60,10 @@ def get_conn(self) -> ftplib.FTP: if self.conn is None:

[GitHub] [airflow] josh-fell commented on issue #17022: TaskInstance.previous_execution_date_success fails with: 'TaskInstance' object has no attribute 'task'

2021-07-27 Thread GitBox
josh-fell commented on issue #17022: URL: https://github.com/apache/airflow/issues/17022#issuecomment-887959994 FWIW this is also reproducible in Airflow 2. **Airflow version** 2.1.0 Using the DAG below, the `get_previous_execution_date()` call from `TaskInstance` object

[GitHub] [airflow] boring-cyborg[bot] commented on issue #17279: Logout airflow on Web UI does not work using OAuth2

2021-07-27 Thread GitBox
boring-cyborg[bot] commented on issue #17279: URL: https://github.com/apache/airflow/issues/17279#issuecomment-887954968 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

[GitHub] [airflow] CaptainJ93 opened a new issue #17279: Logout airflow on Web UI does not work using OAuth2

2021-07-27 Thread GitBox
CaptainJ93 opened a new issue #17279: URL: https://github.com/apache/airflow/issues/17279 **Apache Airflow version**: 2.0.0 **Kubernetes version (if you are using kubernetes)** (use `kubectl version`): Client Version: Major:"1", Minor:"19", GitVersion:"v1.19.0",Server Version:

[GitHub] [airflow] kloty edited a comment on issue #12035: Support AssumeRoleWithWebIdentity for AWS provider

2021-07-27 Thread GitBox
kloty edited a comment on issue #12035: URL: https://github.com/apache/airflow/issues/12035#issuecomment-887944378 > @vladiceanu @hammad13060 Isn't it enough to configure an empty AWS connection to use [the boto3 credential search

[GitHub] [airflow] kloty edited a comment on issue #12035: Support AssumeRoleWithWebIdentity for AWS provider

2021-07-27 Thread GitBox
kloty edited a comment on issue #12035: URL: https://github.com/apache/airflow/issues/12035#issuecomment-887944378 > @vladiceanu @hammad13060 Isn't it enough to configure an empty AWS connection to use [the boto3 credential search

[GitHub] [airflow] kloty commented on issue #12035: Support AssumeRoleWithWebIdentity for AWS provider

2021-07-27 Thread GitBox
kloty commented on issue #12035: URL: https://github.com/apache/airflow/issues/12035#issuecomment-887944378 > @vladiceanu @hammad13060 Isn't it enough to configure an empty AWS connection to use [the boto3 credential search

[GitHub] [airflow] chuancyzhang commented on a change in pull request #17253: add encoding config in FtpHook

2021-07-27 Thread GitBox
chuancyzhang commented on a change in pull request #17253: URL: https://github.com/apache/airflow/pull/17253#discussion_r677906416 ## File path: airflow/providers/ftp/hooks/ftp.py ## @@ -60,8 +60,10 @@ def get_conn(self) -> ftplib.FTP: if self.conn is None:

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

2021-07-27 Thread GitBox
dimberman commented on a change in pull request #17270: URL: https://github.com/apache/airflow/pull/17270#discussion_r677904629 ## File path: airflow/decorators/__init__.py ## @@ -15,15 +15,17 @@ # specific language governing permissions and limitations # under the License.

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

2021-07-27 Thread GitBox
dimberman commented on pull request #17270: URL: https://github.com/apache/airflow/pull/17270#issuecomment-887937376 @ashb unfortunately the problem with leaving them where they are is that because we are loading this stuff in the __init__ file, it leads to an infinite loop. -- This is

[GitHub] [airflow] santosh-d3vpl3x closed pull request #17278: Fix `airflow celery stop` to accept the pid file.

2021-07-27 Thread GitBox
santosh-d3vpl3x closed pull request #17278: URL: https://github.com/apache/airflow/pull/17278 -- 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 pull request #17278: Fix `airflow celery stop` to accept the pid file.

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

[GitHub] [airflow] santosh-d3vpl3x opened a new pull request #17278: Fix `airflow celery stop` to accept the pid file.

2021-07-27 Thread GitBox
santosh-d3vpl3x opened a new pull request #17278: URL: https://github.com/apache/airflow/pull/17278 **Background** Celery cli provides a nice interface to launch workers and stop them. This cli setup works very well when there is one worker running on node without a custom PID

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

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

[GitHub] [airflow] github-actions[bot] commented on pull request #15398: Uses the resource and action naming scheme.

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

[GitHub] [airflow] github-actions[bot] commented on pull request #17277: Show serialization exceptions in DAG parsing log

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

[GitHub] [airflow] kaxil edited a comment on issue #17243: Helm v1.0.0 missing, no entry in index.yaml for 1.1.0

2021-07-27 Thread GitBox
kaxil edited a comment on issue #17243: URL: https://github.com/apache/airflow/issues/17243#issuecomment-887895199 We can't and shouldn't point to archive for the latest versio as I explained in https://github.com/apache/airflow/issues/17243#issuecomment-887686173 The thing to do

[GitHub] [airflow] kaxil edited a comment on issue #17243: Helm v1.0.0 missing, no entry in index.yaml for 1.1.0

2021-07-27 Thread GitBox
kaxil edited a comment on issue #17243: URL: https://github.com/apache/airflow/issues/17243#issuecomment-887895199 We can't and shouldn't point to archive for the latest version. The thing to do would be that I won't delete the old artifact for a day or too from downloads.apache.org

[GitHub] [airflow] kaxil commented on issue #17243: Helm v1.0.0 missing, no entry in index.yaml for 1.1.0

2021-07-27 Thread GitBox
kaxil commented on issue #17243: URL: https://github.com/apache/airflow/issues/17243#issuecomment-887895199 We can't and shouldn't point to archive for the latest version. The thing to do would be that I won't delete the old artifact for a day or too from downloads.apache.org.

[GitHub] [airflow] jedcunningham commented on issue #17260: FORBIDDEN access to worker's logs from Web UI

2021-07-27 Thread GitBox
jedcunningham commented on issue #17260: URL: https://github.com/apache/airflow/issues/17260#issuecomment-887879312 @denisvolokh, this is from #16754 and implies you don't have the same `[webserver] secret_key` on your webserver and worker. How are you running Airflow? Can you make sure

[GitHub] [airflow] jedcunningham opened a new pull request #17277: Show serialization exceptions in DAG parsing log

2021-07-27 Thread GitBox
jedcunningham opened a new pull request #17277: URL: https://github.com/apache/airflow/pull/17277 Make sure that any exceptions that happen when writing serialized DAGs to the db get written to the DAG parsing log, instead of only being added to `import_errors` for consumption via the

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

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

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

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

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

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

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

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

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

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

[GitHub] [airflow] codenamestif opened a new issue #17276: Make platform version as independent parameter of ECSOperator

2021-07-27 Thread GitBox
codenamestif opened a new issue #17276: URL: https://github.com/apache/airflow/issues/17276 Currently `ECSOperator` propagates `platform_version` parameter either in case `launch_type` is `FARGATE` or there is `capacity_provider_strategy` parameter provided. The case with

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

2021-07-27 Thread GitBox
pmalafosse commented on issue #17268: URL: https://github.com/apache/airflow/issues/17268#issuecomment-887811638 This one is fine too: - Update Boto3 API calls in ECSOperator (#16050) -- This is an automated message from the Apache Git Service. To respond to the message, please log

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

2021-07-27 Thread GitBox
pmalafosse commented on issue #17268: URL: https://github.com/apache/airflow/issues/17268#issuecomment-887810873 amazon: 2.1.0rc2: - Allow attaching to previously launched task in ECSOperator (#16685) - ECSOperator / pass context to self.xcom_pull as it was missing (when using

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

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

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

2021-07-27 Thread GitBox
josh-fell commented on issue #17268: URL: https://github.com/apache/airflow/issues/17268#issuecomment-887795821 Contributions for Google, Amazon, and Salesforce providers look good! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

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

2021-07-27 Thread GitBox
levyitay commented on issue #17268: URL: https://github.com/apache/airflow/issues/17268#issuecomment-887794588 qubole: 2.0.0rc2 tested and working -- 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 commented on issue #17243: Helm v1.0.0 missing, no entry in index.yaml for 1.1.0

2021-07-27 Thread GitBox
potiuk commented on issue #17243: URL: https://github.com/apache/airflow/issues/17243#issuecomment-887790137 Yep. The documentation (and thus index.yaml) is merged after release is done. One thing we can do is we can point to archive for all versions (including the latest one). --

[GitHub] [airflow] malthe opened a new pull request #17273: Switch to 'smbprotocol' library

2021-07-27 Thread GitBox
malthe opened a new pull request #17273: URL: https://github.com/apache/airflow/pull/17273 This change switches the Samba hook implementation to the [smbprotocol](https://pypi.org/project/smbprotocol/) library. As discussed in #14054 the current implementation uses an unmaintained

[GitHub] [airflow] raphaelauv edited a comment on issue #17268: Status of testing Providers that were prepared on July 27, 2021 (rc2)

2021-07-27 Thread GitBox
raphaelauv edited a comment on issue #17268: URL: https://github.com/apache/airflow/issues/17268#issuecomment-887784405 for docker 2.1.0rc2 it's all good docker in docker work and also the XCOM ``` t3 = DockerOperator( api_version='auto',

[GitHub] [airflow] raphaelauv edited a comment on issue #17268: Status of testing Providers that were prepared on July 27, 2021 (rc2)

2021-07-27 Thread GitBox
raphaelauv edited a comment on issue #17268: URL: https://github.com/apache/airflow/issues/17268#issuecomment-887784405 for docker 2.1.0rc2 for docker in docker everything work and also the XCOM ``` t3 = DockerOperator( api_version='auto',

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

2021-07-27 Thread GitBox
raphaelauv commented on issue #17268: URL: https://github.com/apache/airflow/issues/17268#issuecomment-887784405 for docker 2.1.0rc2 for docker in docker everything work and also the XCOM ``` t3 = DockerOperator( api_version='auto',

[GitHub] [airflow] andrelop commented on issue #10026: Duplicate tasks invoked for a single task_id when manually invoked task details modal.

2021-07-27 Thread GitBox
andrelop commented on issue #10026: URL: https://github.com/apache/airflow/issues/10026#issuecomment-887776535 Same issue here, running Airflow 2.1.1 : `[2021-07-27 19:21:52,578] {local_task_job.py:195} WARNING - Recorded pid 656 does not match the current pid 76 [2021-07-27

[GitHub] [airflow] naomiaro commented on issue #17243: Helm v1.0.0 missing, no entry in index.yaml for 1.1.0

2021-07-27 Thread GitBox
naomiaro commented on issue #17243: URL: https://github.com/apache/airflow/issues/17243#issuecomment-887772702 There was about a 60-90min delay with the 1.1.0 release process that made the chart unusable yesterday which was more of the issue. 1.0.0 had been removed from

[airflow] tag providers-salesforce/3.1.0rc2 created (now 4dc622d)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-salesforce/3.1.0rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 4dc622d (commit) No new revisions were added by this update.

svn commit: r49029 - /dev/airflow/providers/

2021-07-27 Thread potiuk
Author: potiuk Date: Tue Jul 27 19:20:04 2021 New Revision: 49029 Log: Add artifacts for Airflow Providers 2021-07-27 - Salesforce Added: dev/airflow/providers/apache-airflow-providers-salesforce-3.1.0.tar.gz (with props)

[airflow] branch main updated: Prepares release for Salesforce provider (#17272)

2021-07-27 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 4dc622d Prepares release for Salesforce provider

[GitHub] [airflow] potiuk merged pull request #17272: Prepares release for Salesforce provider

2021-07-27 Thread GitBox
potiuk merged pull request #17272: URL: https://github.com/apache/airflow/pull/17272 -- 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 #17272: Prepares release for Salesforce provider

2021-07-27 Thread GitBox
potiuk commented on pull request #17272: URL: https://github.com/apache/airflow/pull/17272#issuecomment-887754657 cc: @josh-fell -- 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 opened a new pull request #17272: Prepares release for Salesforce provider

2021-07-27 Thread GitBox
potiuk opened a new pull request #17272: URL: https://github.com/apache/airflow/pull/17272 By mistake, Salesforce provider has been marked as doc-only where it had new connection type added. This PR updates Salesforce docs preparing it for separate release. --- **^

[GitHub] [airflow] orossini opened a new pull request #17271: update return value for sql, template and flex template operators

2021-07-27 Thread GitBox
orossini opened a new pull request #17271: URL: https://github.com/apache/airflow/pull/17271 closes: #17180 fixes issue #17180 currently the DataflowStartSqlJobOperator, DataflowTemplatedJobStartOperator, and DataflowStartFlexTemplateOperator return a job object. It doesn't

[GitHub] [airflow] ephraimbuddy opened a new pull request #16301: Fix task retries when they receive sigkill and have retries. Properly handle sigterm too

2021-07-27 Thread GitBox
ephraimbuddy opened a new pull request #16301: URL: https://github.com/apache/airflow/pull/16301 Closes: https://github.com/apache/airflow/issues/16285 Currently, tasks are not retried when they receive sigkill even if the task has retries. Sigkill is uncatchable on task runner.

[GitHub] [airflow] ephraimbuddy closed pull request #16301: Fix task retries when they receive sigkill and have retries. Properly handle sigterm too

2021-07-27 Thread GitBox
ephraimbuddy closed pull request #16301: URL: https://github.com/apache/airflow/pull/16301 -- 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] dimberman opened a new pull request #17270: Make decorators pluggable

2021-07-27 Thread GitBox
dimberman opened a new pull request #17270: URL: https://github.com/apache/airflow/pull/17270 This PR will allow users to add custom "@task." decorators by adding to their setup.cfg files. This will make decorators seem native to airflow while living in provider packages.

[GitHub] [airflow] ephraimbuddy closed pull request #17229: Fix running tasks with `default_impersonation` config

2021-07-27 Thread GitBox
ephraimbuddy closed pull request #17229: URL: https://github.com/apache/airflow/pull/17229 -- 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] orossini closed pull request #17182: dataflowjobstatus sensor dag example fixed for issue #17180

2021-07-27 Thread GitBox
orossini closed pull request #17182: URL: https://github.com/apache/airflow/pull/17182 -- 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] josh-fell opened a new pull request #17269: Handle connection parameters added to Extra and custom fields

2021-07-27 Thread GitBox
josh-fell opened a new pull request #17269: URL: https://github.com/apache/airflow/pull/17269 Closes: #17235 Adding logic to handle connection parameters added to the classic `Extra` fields as well as part of custom fields which use `Extra` under the hood. Currently if parameters

[GitHub] [airflow] kaxil edited a comment on issue #17243: Helm v1.0.0 missing, no entry in index.yaml for 1.1.0

2021-07-27 Thread GitBox
kaxil edited a comment on issue #17243: URL: https://github.com/apache/airflow/issues/17243#issuecomment-887686173 @matthieu-foucault Check https://infra.apache.org/mirrors#location :) Current public releases appear on downloads.apache.org/. Place current, official releases that

[GitHub] [airflow] kaxil commented on issue #17243: Helm v1.0.0 missing, no entry in index.yaml for 1.1.0

2021-07-27 Thread GitBox
kaxil commented on issue #17243: URL: https://github.com/apache/airflow/issues/17243#issuecomment-887686173 @matthieu-foucault Check https://infra.apache.org/mirrors#location :) Current public releases appear on downloads.apache.org/. Place current, official releases that the PMC

[GitHub] [airflow] orossini closed pull request #17182: dataflowjobstatus sensor dag example fixed for issue #17180

2021-07-27 Thread GitBox
orossini closed pull request #17182: URL: https://github.com/apache/airflow/pull/17182 -- 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] lgov commented on issue #17129: All tasks fail on logging to a Windows drive (Kubernetes-Multipass-Microk8s on Windows host)

2021-07-27 Thread GitBox
lgov commented on issue #17129: URL: https://github.com/apache/airflow/issues/17129#issuecomment-887669851 Thanks for the feedback @uranusjr. Changing the ':' in the time to '_' would work, or just remove them. While I can make a quick fix, I have no development setup on a Windows

[GitHub] [airflow] potiuk opened a new issue #17268: Status of testing Providers that were prepared on July 27, 2021 (rc2)

2021-07-27 Thread GitBox
potiuk opened a new issue #17268: URL: https://github.com/apache/airflow/issues/17268 This is the second attempt (rc2) of releasing July Providers :). I carried over the "checkmarks" from the last testing so there is no need to re-test those. I have a kind request for all the

[GitHub] [airflow-site] potiuk commented on pull request #453: Add docs for July 2021 providers.

2021-07-27 Thread GitBox
potiuk commented on pull request #453: URL: https://github.com/apache/airflow-site/pull/453#issuecomment-887653549 With doc-only changes for all other providers we are at 1.5 M now @kaxil :) -- This is an automated message from the Apache Git Service. To respond to the message,

[airflow] tag providers-tableau/2.1.0rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-tableau/2.1.0rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-ssh/2.1.0rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-ssh/2.1.0rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-snowflake/2.1.0rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-snowflake/2.1.0rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-sftp/2.1.0rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-sftp/2.1.0rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-postgres/2.1.0rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-postgres/2.1.0rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-mysql/2.1.0rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-mysql/2.1.0rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-microsoft-azure/3.1.0rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-microsoft-azure/3.1.0rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-jenkins/2.0.1rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-jenkins/2.0.1rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-google/5.0.0rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-google/5.0.0rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-elasticsearch/2.0.2rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-elasticsearch/2.0.2rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-docker/2.1.0rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-docker/2.1.0rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-cncf-kubernetes/2.0.1rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-cncf-kubernetes/2.0.1rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-apache-sqoop/2.0.1rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-apache-sqoop/2.0.1rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-apache-hive/2.0.1rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-apache-hive/2.0.1rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

[airflow] tag providers-apache-druid/2.0.1rc2 created (now 87f408b)

2021-07-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to tag providers-apache-druid/2.0.1rc2 in repository https://gitbox.apache.org/repos/asf/airflow.git. at 87f408b (commit) No new revisions were added by this update.

  1   2   3   >