lopesdiego12 opened a new pull request #8990:
URL: https://github.com/apache/airflow/pull/8990
---
Make sure to mark the boxes below before creating PR: [x]
- [ ] Description above provides context of the change
- [ ] Unit tests coverage for changes (not needed for
boring-cyborg[bot] commented on pull request #8990:
URL: https://github.com/apache/airflow/pull/8990#issuecomment-633180075
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
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to tag nightly-master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
*** WARNING: tag nightly-master was modified! ***
from f107338 (commit)
to 427257c (commit)
from
KevinYang21 commented on issue #8989:
URL: https://github.com/apache/airflow/issues/8989#issuecomment-633169381
Plan to work on this myself, maybe later next week
This is an automated message from the Apache Git Service.
To
boring-cyborg[bot] commented on issue #8989:
URL: https://github.com/apache/airflow/issues/8989#issuecomment-633168481
Thanks for opening your first issue here! Be sure to follow the issue
template!
This is an automated
KevinYang21 opened a new issue #8989:
URL: https://github.com/apache/airflow/issues/8989
**Description**
In `airflow backfill`, `-I` is available so that the task instance of the
first DAG run will ignore depends_on_past setting. It is a bit confusing from
its name that it is a concept
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 971cae3 Fix migration message (#8988)
add 427257c Remove defunct code from setup.py (#8982)
No new
potiuk merged pull request #8982:
URL: https://github.com/apache/airflow/pull/8982
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
[
https://issues.apache.org/jira/browse/AIRFLOW-578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17114992#comment-17114992
]
ASF GitHub Bot commented on AIRFLOW-578:
stale[bot] commented on pull request #6870:
URL:
stale[bot] commented on pull request #6870:
URL: https://github.com/apache/airflow/pull/6870#issuecomment-633153750
This issue has been automatically marked as stale because it has not had
recent activity. It will be closed if no further activity occurs. Thank you for
your contributions.
samuelkhtu commented on a change in pull request #8942:
URL: https://github.com/apache/airflow/pull/8942#discussion_r429583242
##
File path: airflow/operators/sql_branch_operator.py
##
@@ -0,0 +1,175 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
[
https://issues.apache.org/jira/browse/AIRFLOW-6944?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17114949#comment-17114949
]
ASF GitHub Bot commented on AIRFLOW-6944:
-
stale[bot] commented on pull request #7585:
URL:
[
https://issues.apache.org/jira/browse/AIRFLOW-6585?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17114950#comment-17114950
]
ASF GitHub Bot commented on AIRFLOW-6585:
-
stale[bot] commented on pull request #7153:
URL:
[
https://issues.apache.org/jira/browse/AIRFLOW-5708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17114951#comment-17114951
]
ASF GitHub Bot commented on AIRFLOW-5708:
-
stale[bot] closed pull request #6379:
URL:
stale[bot] commented on pull request #7153:
URL: https://github.com/apache/airflow/pull/7153#issuecomment-633143064
This issue has been automatically marked as stale because it has not had
recent activity. It will be closed if no further activity occurs. Thank you for
your contributions.
stale[bot] closed pull request #6379:
URL: https://github.com/apache/airflow/pull/6379
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
stale[bot] commented on pull request #7585:
URL: https://github.com/apache/airflow/pull/7585#issuecomment-633143069
This issue has been automatically marked as stale because it has not had
recent activity. It will be closed if no further activity occurs. Thank you for
your contributions.
xinbinhuang commented on a change in pull request #:
URL: https://github.com/apache/airflow/pull/#discussion_r429579522
##
File path: tests/providers/amazon/aws/operators/test_ecs_system.py
##
@@ -0,0 +1,99 @@
+#
+# Licensed to the Apache Software Foundation (ASF)
kaxil commented on pull request #8982:
URL: https://github.com/apache/airflow/pull/8982#issuecomment-633136267
Unfortunately, to fix the CI I have to push unrelated changes to this PR
This is an automated message from the
marclamberti edited a comment on pull request #8777:
URL: https://github.com/apache/airflow/pull/8777#issuecomment-633090457
@thesuperzapper There is an issue in the Helm chart.
`
redis:
enabled: false
`
produces the following error:
`
potiuk commented on pull request #8976:
URL: https://github.com/apache/airflow/pull/8976#issuecomment-633130178
I thin this one is good to go - and it helps a lot with dealing with
cache/pushing images manualy to cache as well if needed.
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/master by this push:
new 971cae3 Fix migration message (#8988)
boring-cyborg[bot] commented on pull request #8988:
URL: https://github.com/apache/airflow/pull/8988#issuecomment-633121965
Awesome work, congrats on your first merged pull request!
This is an automated message from the
kaxil merged pull request #8988:
URL: https://github.com/apache/airflow/pull/8988
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
eladkal commented on a change in pull request #8942:
URL: https://github.com/apache/airflow/pull/8942#discussion_r429570989
##
File path: airflow/operators/sql_branch_operator.py
##
@@ -0,0 +1,175 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
mustafagok commented on a change in pull request #:
URL: https://github.com/apache/airflow/pull/#discussion_r429561330
##
File path: tests/providers/amazon/aws/operators/test_ecs_system.py
##
@@ -0,0 +1,140 @@
+#
+# Licensed to the Apache Software Foundation (ASF)
marclamberti edited a comment on pull request #8777:
URL: https://github.com/apache/airflow/pull/8777#issuecomment-633090457
@thesuperzapper I think there is an issue in the Helm chart.
`
redis:
enabled: false
`
produces the following error:
`
marclamberti edited a comment on pull request #8777:
URL: https://github.com/apache/airflow/pull/8777#issuecomment-633090457
@thesuperzapper I think there is an issue in the Helm chart.
`
redis:
enabled: false
`
produces the following error:
`
marclamberti edited a comment on pull request #8777:
URL: https://github.com/apache/airflow/pull/8777#issuecomment-633090457
@thesuperzapper I think there is an issue in the Helm chart.
`
redis:
enabled: false
`
produces the following error:
`
mik-laj commented on a change in pull request #8911:
URL: https://github.com/apache/airflow/pull/8911#discussion_r429560311
##
File path: airflow/sentry.py
##
@@ -81,13 +81,17 @@ def __init__(self):
sentry_celery = CeleryIntegration()
marclamberti commented on pull request #8777:
URL: https://github.com/apache/airflow/pull/8777#issuecomment-633090457
@thesuperzapper I think there is an issue in the Helm chart.
`
redis:
enabled: false
`
produces the following error:
`
mik-laj commented on a change in pull request #8911:
URL: https://github.com/apache/airflow/pull/8911#discussion_r429560311
##
File path: airflow/sentry.py
##
@@ -81,13 +81,17 @@ def __init__(self):
sentry_celery = CeleryIntegration()
turbaszek commented on a change in pull request #8962:
URL: https://github.com/apache/airflow/pull/8962#discussion_r429556544
##
File path: airflow/operators/python.py
##
@@ -145,6 +147,141 @@ def execute_callable(self):
return self.python_callable(*self.op_args,
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 f946f96 Old json boto compat removed from dynamodb_to_s3 operator
(#8987)
add 1d36b03 Fix references
mik-laj merged pull request #8984:
URL: https://github.com/apache/airflow/pull/8984
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
mustafagok commented on a change in pull request #:
URL: https://github.com/apache/airflow/pull/#discussion_r429554430
##
File path: tests/test_utils/amazon_system_helpers.py
##
@@ -51,6 +54,27 @@ def provide_aws_s3_bucket(name):
@pytest.mark.system("amazon")
class
mustafagok commented on a change in pull request #:
URL: https://github.com/apache/airflow/pull/#discussion_r429554235
##
File path: tests/test_utils/amazon_system_helpers.py
##
@@ -60,6 +84,25 @@ def execute_with_ctx(cls, cmd: List[str]):
with
mustafagok commented on a change in pull request #:
URL: https://github.com/apache/airflow/pull/#discussion_r429553900
##
File path: airflow/providers/amazon/aws/example_dags/example_ecs_fargate.py
##
@@ -64,8 +65,8 @@
},
network_configuration={
ad-m commented on pull request #8985:
URL: https://github.com/apache/airflow/pull/8985#issuecomment-633062684
@potiuk , @mik-laj , thanks for you help!
This is an automated message from the Apache Git Service.
To respond to
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 f3456b1 Fix formatting code block in TESTING.rst (#8985)
add f946f96 Old json boto compat removed from
potiuk merged pull request #8987:
URL: https://github.com/apache/airflow/pull/8987
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
dependabot[bot] commented on pull request #8646:
URL: https://github.com/apache/airflow/pull/8646#issuecomment-633051660
OK, I won't notify you again about this release, but will get in touch when
a new version is available. If you'd rather skip all updates until the next
major or minor
dependabot[bot] closed pull request #8968:
URL: https://github.com/apache/airflow/pull/8968
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
kaxil closed pull request #8646:
URL: https://github.com/apache/airflow/pull/8646
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
boring-cyborg[bot] commented on pull request #8988:
URL: https://github.com/apache/airflow/pull/8988#issuecomment-633044802
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
clojurians-org opened a new pull request #8988:
URL: https://github.com/apache/airflow/pull/8988
run 'python -m airflow db init' will print extra information from migration
file.
```
INFO [alembic.runtime.migration] Running upgrade 6e96a59344a4 ->
d38e04c12aa2, add
potiuk opened a new pull request #8987:
URL: https://github.com/apache/airflow/pull/8987
---
Make sure to mark the boxes below before creating PR: [x]
- [x] Description above provides context of the change
- [x] Unit tests coverage for changes (not needed for documentation
potiuk commented on issue #8977:
URL: https://github.com/apache/airflow/issues/8977#issuecomment-633023929
I wanted to check what you just fixed in #8979 and #8978 so I think we can
close this one.
This is an automated
potiuk closed issue #8977:
URL: https://github.com/apache/airflow/issues/8977
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
ad-m commented on pull request #8968:
URL: https://github.com/apache/airflow/pull/8968#issuecomment-633022939
Could you fix source & target branch?
This is an automated message from the Apache Git Service.
To respond to the
joppevos opened a new pull request #8986:
URL: https://github.com/apache/airflow/pull/8986
Addresses one of the files in issue [#8278
](https://github.com/apache/airflow/issues/8278)
---
Make sure to mark the boxes below before creating PR: [x]
- [x] Description above
potiuk commented on pull request #8974:
URL: https://github.com/apache/airflow/pull/8974#issuecomment-633018907
I pushed the changes now. I also reviewed the docstrings to make sure they
are consistent between vault_client, secret and hook.
potiuk commented on a change in pull request #8974:
URL: https://github.com/apache/airflow/pull/8974#discussion_r429532400
##
File path: airflow/providers/hashicorp/hooks/vault.py
##
@@ -0,0 +1,281 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
potiuk commented on a change in pull request #8974:
URL: https://github.com/apache/airflow/pull/8974#discussion_r429531975
##
File path: airflow/providers/hashicorp/hooks/vault.py
##
@@ -0,0 +1,281 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
potiuk commented on a change in pull request #8974:
URL: https://github.com/apache/airflow/pull/8974#discussion_r429531833
##
File path: airflow/providers/hashicorp/hooks/vault.py
##
@@ -0,0 +1,281 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
mik-laj merged pull request #8985:
URL: https://github.com/apache/airflow/pull/8985
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
potiuk commented on a change in pull request #8974:
URL: https://github.com/apache/airflow/pull/8974#discussion_r429531729
##
File path: airflow/providers/hashicorp/secrets/vault.py
##
@@ -72,98 +68,90 @@ class VaultBackend(BaseSecretsBackend, LoggingMixin):
:type
potiuk commented on a change in pull request #8974:
URL: https://github.com/apache/airflow/pull/8974#discussion_r429531712
##
File path: airflow/providers/hashicorp/secrets/vault.py
##
@@ -72,98 +68,90 @@ class VaultBackend(BaseSecretsBackend, LoggingMixin):
:type
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 bdb8369 Add secrets to test_deprecated_packages (#8979)
add f3456b1 Fix formatting code block in
ad-m commented on pull request #8985:
URL: https://github.com/apache/airflow/pull/8985#issuecomment-633012339
> ![Screenshot 2020-05-23 at 11 10
04](https://user-images.githubusercontent.com/12058428/82726678-fdef8c80-9ce5-11ea-9ebd-28bd4208c603.png)
> It does not look good.
You
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 cf5cf45 Support YAML input for CloudBuildCreateOperator (#8808)
add bdb8369 Add secrets to
mik-laj merged pull request #8979:
URL: https://github.com/apache/airflow/pull/8979
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
mik-laj commented on pull request #8985:
URL: https://github.com/apache/airflow/pull/8985#issuecomment-633011697
![Screenshot 2020-05-23 at 11 10
04](https://user-images.githubusercontent.com/12058428/82726678-fdef8c80-9ce5-11ea-9ebd-28bd4208c603.png)
It does not look good.
mik-laj commented on a change in pull request #8984:
URL: https://github.com/apache/airflow/pull/8984#discussion_r429529543
##
File path: tests/test_impersonation.py
##
@@ -50,8 +50,8 @@
def mock_custom_module_path(path: str):
"""
-This decorator adds a path to
mik-laj commented on a change in pull request #8985:
URL: https://github.com/apache/airflow/pull/8985#discussion_r429529427
##
File path: TESTING.rst
##
@@ -966,15 +966,15 @@ Running BATS Tests via Docker
To run all tests:
-```
-docker run -it --workdir /airflow -v
potiuk commented on pull request #8984:
URL: https://github.com/apache/airflow/pull/8984#issuecomment-633009862
All good but one line too long in the impersonation
This is an automated message from the Apache Git Service.
To
potiuk commented on pull request #8985:
URL: https://github.com/apache/airflow/pull/8985#issuecomment-633009405
Stylelint complains - it's good but the style problems need to be fixed.
This is an automated message from the
j-y-matsubara edited a comment on pull request #8867:
URL: https://github.com/apache/airflow/pull/8867#issuecomment-632606590
@yuqian90
Thank you for your advise.
I have corrected the document that you pointed out.
And...
The continuous-integration/travis-ci/pr failure
j-y-matsubara edited a comment on pull request #8867:
URL: https://github.com/apache/airflow/pull/8867#issuecomment-632606590
@yuqian90
Thank you for your advise.
I have corrected the document that you pointed out.
And...
The continuous-integration/travis-ci/pr failure
ad-m commented on pull request #8985:
URL: https://github.com/apache/airflow/pull/8985#issuecomment-633001272
@potiuk , could you review that and provide any comments? I have been
informed that you have the widest knowledge of testing in the entire Apache
Airflow project.
ad-m opened a new pull request #8985:
URL: https://github.com/apache/airflow/pull/8985
---
Make sure to mark the boxes below before creating PR: [x]
- [X] Description above provides context of the change
- [X] Unit tests coverage for changes (not needed for documentation
mik-laj commented on issue #8980:
URL: https://github.com/apache/airflow/issues/8980#issuecomment-632998881
Can you open PR? You don't need to create a isue if you want to share the
code. The issue is only needed if it is a long-term task or you want someone
else to do the task
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 db70da2 Flush pending Sentry exceptions before exiting (#7232)
add cf5cf45 Support YAML input for
mik-laj merged pull request #8808:
URL: https://github.com/apache/airflow/pull/8808
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
[
https://issues.apache.org/jira/browse/AIRFLOW-6569?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17114563#comment-17114563
]
ASF subversion and git services commented on AIRFLOW-6569:
--
Commit
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 4d67704 Remove duplicate line from CONTRIBUTING.rst (#8981)
add db70da2 Flush pending Sentry
[
https://issues.apache.org/jira/browse/AIRFLOW-6569?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17114561#comment-17114561
]
ASF GitHub Bot commented on AIRFLOW-6569:
-
boring-cyborg[bot] commented on pull request #7232:
[
https://issues.apache.org/jira/browse/AIRFLOW-6569?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17114562#comment-17114562
]
ASF GitHub Bot commented on AIRFLOW-6569:
-
mik-laj merged pull request #7232:
URL:
boring-cyborg[bot] commented on pull request #7232:
URL: https://github.com/apache/airflow/pull/7232#issuecomment-632995201
Awesome work, congrats on your first merged pull request!
This is an automated message from the
mik-laj merged pull request #7232:
URL: https://github.com/apache/airflow/pull/7232
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
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 e742ef7 Fix typo in test_project_structure (#8978)
add 4d67704 Remove duplicate line from
mik-laj merged pull request #8981:
URL: https://github.com/apache/airflow/pull/8981
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
mik-laj commented on pull request #8982:
URL: https://github.com/apache/airflow/pull/8982#issuecomment-632994246
@kaxil Can you fix requirements on Github action?
This is an automated message from the Apache Git Service.
To
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 f107338 Add support for spark python and submit tasks in Databricks
operator(#8846)
add e742ef7 Fix
mik-laj merged pull request #8978:
URL: https://github.com/apache/airflow/pull/8978
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
mik-laj opened a new pull request #8984:
URL: https://github.com/apache/airflow/pull/8984
---
Make sure to mark the boxes below before creating PR: [x]
- [X] Description above provides context of the change
- [X] Unit tests coverage for changes (not needed for documentation
86 matches
Mail list logo