zikun opened a new issue #9033:
URL: https://github.com/apache/airflow/issues/9033
**Apache Airflow version**: 1.10.10 / master
**Kubernetes version (if you are using kubernetes)** (use `kubectl version`):
**Environment**:
Production Docker image
*
mik-laj commented on pull request #9023:
URL: https://github.com/apache/airflow/pull/9023#issuecomment-634600193
@ad-m We have temporary problems with Kubernetes tests running on Travis Ci.
This change does not apply to Kubernetes, so it is safe. @dimberman is working
on improvements for
kaxil commented on a change in pull request #8721:
URL: https://github.com/apache/airflow/pull/8721#discussion_r431046930
##
File path: openapi.yaml
##
@@ -0,0 +1,2427 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. S
mik-laj commented on pull request #268:
URL: https://github.com/apache/airflow-site/pull/268#issuecomment-634597352
@mschickensoup Do you have time to look at it?
This is an automated message from the Apache Git Service.
To
stijndehaes commented on issue #8963:
URL: https://github.com/apache/airflow/issues/8963#issuecomment-634596428
@ywan2017 I also have a PR open on airflow to work with spark 3.0
https://github.com/apache/airflow/pull/8730
Th
BasPH commented on a change in pull request #8721:
URL: https://github.com/apache/airflow/pull/8721#discussion_r431044194
##
File path: openapi.yaml
##
@@ -0,0 +1,2427 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. S
BasPH commented on a change in pull request #8721:
URL: https://github.com/apache/airflow/pull/8721#discussion_r431044194
##
File path: openapi.yaml
##
@@ -0,0 +1,2427 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. S
[
https://issues.apache.org/jira/browse/AIRFLOW-5615?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17117664#comment-17117664
]
ASF subversion and git services commented on AIRFLOW-5615:
--
Co
This is an automated email from the ASF dual-hosted git repository.
ash pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 30b12a9 Filter dags by clicking on tag (#8897)
add 8ac90b0 [AIRFLOW-5615] Reduce duplicated logic around job
he
[
https://issues.apache.org/jira/browse/AIRFLOW-5615?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17117663#comment-17117663
]
ASF GitHub Bot commented on AIRFLOW-5615:
-
ashb merged pull request #6311:
URL:
ashb merged pull request #6311:
URL: https://github.com/apache/airflow/pull/6311
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
mik-laj commented on issue #8970:
URL: https://github.com/apache/airflow/issues/8970#issuecomment-634592478
@tanjinP Fantastic! I am waiting for your contribution.
This is an automated message from the Apache Git Service.
To
mik-laj closed issue #8821:
URL: https://github.com/apache/airflow/issues/8821
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 s
mik-laj merged pull request #8897:
URL: https://github.com/apache/airflow/pull/8897
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
ad-m commented on pull request #9023:
URL: https://github.com/apache/airflow/pull/9023#issuecomment-634591770
@potiuk , could you confirm that TravisCI fail is not related to that PR?
This is an automated message from the Apa
This is an automated email from the ASF dual-hosted git repository.
kamilbregula pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 5a7a3d1 Add ADDITIONAL_AIRFLOW_EXTRAS (#9032)
add 30b12a9 Filter dags by clicking on tag (#8897)
No ne
boring-cyborg[bot] commented on pull request #8897:
URL: https://github.com/apache/airflow/pull/8897#issuecomment-634591805
Awesome work, congrats on your first merged pull request!
This is an automated message from the A
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 6fc555d Add ADDITIONAL_PYTHON_DEPS (#9031)
add 5a7a3d1 Add ADDITIONAL_AIRFLOW_EXTRAS (#9032)
No new revision
potiuk merged pull request #9032:
URL: https://github.com/apache/airflow/pull/9032
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 t
olchas removed a comment on pull request #9018:
URL: https://github.com/apache/airflow/pull/9018#issuecomment-634583902
LGTM
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
olchas commented on pull request #9018:
URL: https://github.com/apache/airflow/pull/9018#issuecomment-634583902
LGTM
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
[
https://issues.apache.org/jira/browse/AIRFLOW-6569?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17117645#comment-17117645
]
ASF GitHub Bot commented on AIRFLOW-6569:
-
ad-m commented on pull request #7232
ad-m commented on pull request #7232:
URL: https://github.com/apache/airflow/pull/7232#issuecomment-634583299
@kaxil , thank you for information and add to milestone "Airflow 1.10.11".
This is an automated message from the Ap
[
https://issues.apache.org/jira/browse/AIRFLOW-6569?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17117639#comment-17117639
]
ASF GitHub Bot commented on AIRFLOW-6569:
-
kaxil commented on pull request #723
mik-laj commented on pull request #8942:
URL: https://github.com/apache/airflow/pull/8942#issuecomment-634582759
@samuelkhtu we can do it in a separate PR. This will allow us to build a
better git history.
This is an autom
kaxil commented on pull request #7232:
URL: https://github.com/apache/airflow/pull/7232#issuecomment-634580331
Yes can be included in 1.10.11
This is an automated message from the Apache Git Service.
To respond to the messag
turbaszek commented on a change in pull request #9018:
URL: https://github.com/apache/airflow/pull/9018#discussion_r431013369
##
File path: airflow/jobs/scheduler_job.py
##
@@ -1440,94 +1436,89 @@ def _change_state_for_tasks_failed_to_execute(self,
session=None):
:p
turbaszek commented on a change in pull request #9018:
URL: https://github.com/apache/airflow/pull/9018#discussion_r431013108
##
File path: airflow/jobs/scheduler_job.py
##
@@ -1440,94 +1436,89 @@ def _change_state_for_tasks_failed_to_execute(self,
session=None):
:p
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 0b0e4f7 Preparing for RC3 relase of backports (#9026)
add 6fc555d Add ADDITIONAL_PYTHON_DEPS (#9031)
No new
olchas commented on a change in pull request #9018:
URL: https://github.com/apache/airflow/pull/9018#discussion_r430996595
##
File path: airflow/jobs/scheduler_job.py
##
@@ -1440,94 +1436,89 @@ def _change_state_for_tasks_failed_to_execute(self,
session=None):
:para
potiuk commented on pull request #9031:
URL: https://github.com/apache/airflow/pull/9031#issuecomment-634554402
There are a few places - breeze-complete and breeze are the two scripts. You
can just search for another flag and do it similarly, or I can add it later
after your PRs are merged
potiuk merged pull request #9031:
URL: https://github.com/apache/airflow/pull/9031
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 t
boring-cyborg[bot] commented on pull request #9031:
URL: https://github.com/apache/airflow/pull/9031#issuecomment-634554520
Awesome work, congrats on your first merged pull request!
This is an automated message from the A
wittfabian commented on pull request #9031:
URL: https://github.com/apache/airflow/pull/9031#issuecomment-634552807
@potiuk Must the flag be added here?
https://github.com/apache/airflow/blob/master/breeze#L770
This is an
olchas commented on a change in pull request #9018:
URL: https://github.com/apache/airflow/pull/9018#discussion_r430993095
##
File path: airflow/jobs/scheduler_job.py
##
@@ -1440,94 +1436,89 @@ def _change_state_for_tasks_failed_to_execute(self,
session=None):
:para
olchas commented on a change in pull request #9018:
URL: https://github.com/apache/airflow/pull/9018#discussion_r430993095
##
File path: airflow/jobs/scheduler_job.py
##
@@ -1440,94 +1436,89 @@ def _change_state_for_tasks_failed_to_execute(self,
session=None):
:para
kaxil commented on a change in pull request #8721:
URL: https://github.com/apache/airflow/pull/8721#discussion_r430988478
##
File path: openapi.yaml
##
@@ -0,0 +1,2411 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. S
kaxil commented on a change in pull request #8721:
URL: https://github.com/apache/airflow/pull/8721#discussion_r430985695
##
File path: openapi.yaml
##
@@ -0,0 +1,2411 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. S
potiuk commented on pull request #9032:
URL: https://github.com/apache/airflow/pull/9032#issuecomment-634543457
Nice! As mentioned before - those should also be added as flags in Breeze,
but it can be done separately unless you want to do it @wittfabian !
-
potiuk commented on a change in pull request #9032:
URL: https://github.com/apache/airflow/pull/9032#discussion_r430981688
##
File path: Dockerfile
##
@@ -166,7 +167,8 @@ ARG AIRFLOW_VERSION
ENV AIRFLOW_VERSION=${AIRFLOW_VERSION}
ARG AIRFLOW_EXTRAS
-ENV AIRFLOW_EXTRAS=${AIR
potiuk commented on a change in pull request #9032:
URL: https://github.com/apache/airflow/pull/9032#discussion_r430981688
##
File path: Dockerfile
##
@@ -166,7 +167,8 @@ ARG AIRFLOW_VERSION
ENV AIRFLOW_VERSION=${AIRFLOW_VERSION}
ARG AIRFLOW_EXTRAS
-ENV AIRFLOW_EXTRAS=${AIR
swecooo edited a comment on pull request #9019:
URL: https://github.com/apache/airflow/pull/9019#issuecomment-634493060
Thank you for the review @feluelle, the comment should now be acknowledged.
The only remaining problem is the failing build but the same error occurs on
`master`.
potiuk commented on issue #8866:
URL: https://github.com/apache/airflow/issues/8866#issuecomment-634536103
I used to use AIRFLOW everywher, but ... AIRFLOW is the whole context so we
know those are AIRFLOW extras :)
This is
mik-laj commented on issue #8649:
URL: https://github.com/apache/airflow/issues/8649#issuecomment-634531019
@mdediana We had long discussions about whether to support multiple
scheduler intervals. Many people think that this can affect the presentation
and readability of the collected dat
albertocalderari commented on issue #8903:
URL: https://github.com/apache/airflow/issues/8903#issuecomment-634528875
@turbaszek Hey, sorry work was hell this past week and did not find the time
or the energy to reply :(
I have noticed that inner job does that, and that we'd be building t
wittfabian opened a new pull request #9032:
URL: https://github.com/apache/airflow/pull/9032
**Description**
It would be great to be able to pass "ADDITIONAL_EXTRAS" build arg and add
extras to the "base" extras of Production Docker image.
**Use case / motivation**
If s
wittfabian commented on pull request #9031:
URL: https://github.com/apache/airflow/pull/9031#issuecomment-634526550
I have not yet worked with Breeze, but I would like to see the result ;)
I am already working on https://github.com/apache/airflow/issues/8866.
---
turbaszek commented on a change in pull request #8922:
URL: https://github.com/apache/airflow/pull/8922#discussion_r430958055
##
File path: tests/jobs/test_local_task_job.py
##
@@ -406,3 +409,23 @@ def success_callback(context):
self.assertTrue(data['called'])
wittfabian commented on issue #8866:
URL: https://github.com/apache/airflow/issues/8866#issuecomment-634517765
Which name is better?
"ADDITIONAL_EXTRAS" or "AIRLOW_ADDITIONAL_EXTRAS"
This is an automated message from the
potiuk commented on pull request #9031:
URL: https://github.com/apache/airflow/pull/9031#issuecomment-634497097
You can mark it as ready for review to merge it @wittfabian - or you might
try to extend Breeze with an option to add --additional-python-deps flag if you
are brave enough :)
-
swecooo commented on pull request #9019:
URL: https://github.com/apache/airflow/pull/9019#issuecomment-634493060
Thank you for the review @feluelle, the comment should now be acknowledged.
The only problem is now the failing build but the same error occurs on `master`.
If it's OK, fe
turbaszek commented on a change in pull request #8954:
URL: https://github.com/apache/airflow/pull/8954#discussion_r430910630
##
File path: airflow/providers/google/cloud/hooks/datafusion.py
##
@@ -435,20 +435,52 @@ def _get_pipeline(
"workflows",
"Dat
101 - 152 of 152 matches
Mail list logo