potiuk commented on a change in pull request #5362: [AIRFLOW-4669] Make
airflow/dag Pylint compatible
URL: https://github.com/apache/airflow/pull/5362#discussion_r289659649
##
File path: airflow/dag/base_dag.py
##
@@ -17,6 +17,8 @@
# specific language governing
[
https://issues.apache.org/jira/browse/AIRFLOW-4564?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Asgeir Berland resolved AIRFLOW-4564.
-
Resolution: Fixed
PR is ready for review and can be merged after commit-squash.
>
[
https://issues.apache.org/jira/browse/AIRFLOW-4726?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Yu Kodama updated AIRFLOW-4726:
---
Description:
When there are duplicate operator task ids, it raises `_Broken DAG: Cycle
detected
[
https://issues.apache.org/jira/browse/AIRFLOW-4364?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bas Harenslak updated AIRFLOW-4364:
---
Description:
After a [vote on the mailing
codecov-io edited a comment on issue #5319: [AIRFLOW-4564] Azure Container
Instance bugfixes and improvements
URL: https://github.com/apache/airflow/pull/5319#issuecomment-497988863
# [Codecov](https://codecov.io/gh/apache/airflow/pull/5319?src=pr=h1)
Report
> Merging
Yu Kodama created AIRFLOW-4726:
--
Summary: Duplicate operator task ids raise inappropriate error
message
Key: AIRFLOW-4726
URL: https://issues.apache.org/jira/browse/AIRFLOW-4726
Project: Apache Airflow
jmcarp commented on issue #5259: [AIRFLOW-4089] Use shared `Resources`
instances by default.
URL: https://github.com/apache/airflow/pull/5259#issuecomment-498105179
Thanks @potiuk. I reimplemented both `Resource` and `Resources` as
`namedtuple`s both to clean up the code and to make sure
andriisoldatenko commented on a change in pull request #5362: [AIRFLOW-4669]
Make airflow/dag Pylint compatible
URL: https://github.com/apache/airflow/pull/5362#discussion_r289655393
##
File path: airflow/dag/base_dag.py
##
@@ -17,6 +17,8 @@
# specific language governing
[
https://issues.apache.org/jira/browse/AIRFLOW-4227?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16854196#comment-16854196
]
ASF GitHub Bot commented on AIRFLOW-4227:
-
jmcarp commented on pull request #5030:
jmcarp closed pull request #5030: [AIRFLOW-4227] Use python3-style type
annotations.
URL: https://github.com/apache/airflow/pull/5030
This is an automated message from the Apache Git Service.
To respond to the message,
jmcarp commented on issue #5030: [AIRFLOW-4227] Use python3-style type
annotations.
URL: https://github.com/apache/airflow/pull/5030#issuecomment-498105763
Looks like this was reimplemented in #5327. Thanks @BasPH.
This is
codecov-io edited a comment on issue #5345: AIRFLOW-4594 Pod state might be
different from task state. So don't c…
URL: https://github.com/apache/airflow/pull/5345#issuecomment-497290974
# [Codecov](https://codecov.io/gh/apache/airflow/pull/5345?src=pr=h1)
Report
> Merging
BasPH opened a new pull request #5363: [AIRFLOW-4725] Fix setup.py PEP440 &
Sphinx-PyPI-upload dependency
URL: https://github.com/apache/airflow/pull/5363
Make sure you have checked _all_ steps below.
### Jira
- [ ] My PR addresses the following [Airflow
[
https://issues.apache.org/jira/browse/AIRFLOW-4725?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16854115#comment-16854115
]
ASF GitHub Bot commented on AIRFLOW-4725:
-
BasPH commented on pull request #5363:
codecov-io edited a comment on issue #5259: [AIRFLOW-4089] Use shared
`Resources` instances by default.
URL: https://github.com/apache/airflow/pull/5259#issuecomment-497993580
# [Codecov](https://codecov.io/gh/apache/airflow/pull/5259?src=pr=h1)
Report
> Merging
Fokko commented on issue #4937: [AIRFLOW-4116] Multi-staging includes CI image
[Step 2/3]
URL: https://github.com/apache/airflow/pull/4937#issuecomment-498057486
I understand. I’ll look into it tomorrow!
This is an
[
https://issues.apache.org/jira/browse/AIRFLOW-4364?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andrii Soldatenko reassigned AIRFLOW-4364:
--
Assignee: Andrii Soldatenko
> Integrate Pylint
>
>
>
[
https://issues.apache.org/jira/browse/AIRFLOW-4364?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andrii Soldatenko reassigned AIRFLOW-4364:
--
Assignee: (was: Andrii Soldatenko)
> Integrate Pylint
>
[
https://issues.apache.org/jira/browse/AIRFLOW-4658?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andrii Soldatenko reassigned AIRFLOW-4658:
--
Assignee: Andrii Soldatenko
> Make tests/[a-zA-Z_]*.py Pylint compatible
>
[
https://issues.apache.org/jira/browse/AIRFLOW-4725?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bas Harenslak reassigned AIRFLOW-4725:
--
Assignee: Bas Harenslak
> Make setup.py run without errors/warnings
>
watthell234 commented on issue #3111: [AIRFLOW-2106] Cannot pass sandbox
argument to sales_force hook preve…
URL: https://github.com/apache/airflow/pull/3111#issuecomment-498071283
How do I use this addition in the UI? simple_salesforce implements it by
passing domain=test. What do I add
codecov-io edited a comment on issue #5259: [AIRFLOW-4089] Use shared
`Resources` instances by default.
URL: https://github.com/apache/airflow/pull/5259#issuecomment-497993580
# [Codecov](https://codecov.io/gh/apache/airflow/pull/5259?src=pr=h1)
Report
> Merging
andriisoldatenko commented on a change in pull request #5361: [AIRFLOW-4670]
Make airflow/example_dags Pylint compatible
URL: https://github.com/apache/airflow/pull/5361#discussion_r289655302
##
File path: airflow/example_dags/example_skip_dag.py
##
@@ -30,21 +32,31 @@
Bas Harenslak created AIRFLOW-4725:
--
Summary: Make setup.py run without errors/warnings
Key: AIRFLOW-4725
URL: https://issues.apache.org/jira/browse/AIRFLOW-4725
Project: Apache Airflow
codecov-io edited a comment on issue #5330: [AIRFLOW-4573] Import
airflow_local_settings after prepare_classpath
URL: https://github.com/apache/airflow/pull/5330#issuecomment-497957938
# [Codecov](https://codecov.io/gh/apache/airflow/pull/5330?src=pr=h1)
Report
> Merging
codecov-io edited a comment on issue #5330: [AIRFLOW-4573] Import
airflow_local_settings after prepare_classpath
URL: https://github.com/apache/airflow/pull/5330#issuecomment-497957938
# [Codecov](https://codecov.io/gh/apache/airflow/pull/5330?src=pr=h1)
Report
> Merging
BasPH opened a new pull request #5361: [AIRFLOW-4670] Make airflow/example_dags
Pylint compatible
URL: https://github.com/apache/airflow/pull/5361
Make sure you have checked _all_ steps below.
### Jira
- [ ] My PR addresses the following [Airflow
[
https://issues.apache.org/jira/browse/AIRFLOW-4670?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16854057#comment-16854057
]
ASF GitHub Bot commented on AIRFLOW-4670:
-
BasPH commented on pull request #5361:
[
https://issues.apache.org/jira/browse/AIRFLOW-4670?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Work on AIRFLOW-4670 started by Bas Harenslak.
--
> Make airflow/example_dags Pylint compatible
>
BasPH opened a new pull request #5362: [AIRFLOW-4669] Make airflow/dag Pylint
compatible
URL: https://github.com/apache/airflow/pull/5362
Make sure you have checked _all_ steps below.
### Jira
- [ ] My PR addresses the following [Airflow
[
https://issues.apache.org/jira/browse/AIRFLOW-4669?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16854061#comment-16854061
]
ASF GitHub Bot commented on AIRFLOW-4669:
-
BasPH commented on pull request #5362:
[
https://issues.apache.org/jira/browse/AIRFLOW-4364?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bas Harenslak updated AIRFLOW-4364:
---
Description:
After a [vote on the mailing
[
https://issues.apache.org/jira/browse/AIRFLOW-4669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Work on AIRFLOW-4669 started by Bas Harenslak.
--
> Make airflow/dag Pylint compatible
> --
milton0825 commented on a change in pull request #5360: [AIRFLOW-4101] Add
documentation note if dags directory does not exis…
URL: https://github.com/apache/airflow/pull/5360#discussion_r289628785
##
File path: docs/start.rst
##
@@ -36,6 +36,10 @@ The installation is
[
https://issues.apache.org/jira/browse/AIRFLOW-4720?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16853903#comment-16853903
]
ASF subversion and git services commented on AIRFLOW-4720:
--
Commit
[
https://issues.apache.org/jira/browse/AIRFLOW-4720?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16853902#comment-16853902
]
ASF GitHub Bot commented on AIRFLOW-4720:
-
BasPH commented on pull request #5355:
BasPH merged pull request #5355: [AIRFLOW-4720] Allow comments in
.airflowignore files.
URL: https://github.com/apache/airflow/pull/5355
This is an automated message from the Apache Git Service.
To respond to the message,
jack created AIRFLOW-4724:
-
Summary: Make params dict to be templated for operators
Key: AIRFLOW-4724
URL: https://issues.apache.org/jira/browse/AIRFLOW-4724
Project: Apache Airflow
Issue Type: Bug
CloudChaoszero commented on a change in pull request #5360: [AIRFLOW-4101] Add
documentation note if dags directory does not exis…
URL: https://github.com/apache/airflow/pull/5360#discussion_r289632803
##
File path: docs/start.rst
##
@@ -36,6 +36,10 @@ The installation is
OmerJog commented on issue #4610: [AIRFLOW-3783] Switch to HEADER option being
provided already by Redshift
URL: https://github.com/apache/airflow/pull/4610#issuecomment-498018428
@shrechak Can you please rebase ?
This is
andriisoldatenko commented on issue #5048: [AIRFLOW-3370] Add stdout output
options to Elasticsearch task log handler
URL: https://github.com/apache/airflow/pull/5048#issuecomment-498004121
@ashb fixed all ci issues.
This is
[
https://issues.apache.org/jira/browse/AIRFLOW-4723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bas Harenslak updated AIRFLOW-4723:
---
Description:
This has been deprecated since 2016 (airflow/configuration.py):
{code}
#
OmerJog commented on issue #5049: [AIRFLOW-3063] - Make DockerOperator support
all docker arguments
URL: https://github.com/apache/airflow/pull/5049#issuecomment-498014844
@MerouaneBen any progress with this PR?
This is an
BasPH commented on issue #5238: [AIRFLOW-4364] Add Pylint to CI
URL: https://github.com/apache/airflow/pull/5238#issuecomment-498006972
Hi @pgagnon could you clarify? All files are blacklisted so it should
succeed right now. Only (obviously) when you remove files from the blacklist,
will
ismailsimsek commented on issue #3138: [AIRFLOW-2221] Create DagFetcher
abstraction
URL: https://github.com/apache/airflow/pull/3138#issuecomment-498021461
looking forward to this feature. it might be good idea to have rest endpoint
to trigger the fetch operation
ex:
1. dags
potiuk commented on issue #5238: [AIRFLOW-4364] Add Pylint to CI
URL: https://github.com/apache/airflow/pull/5238#issuecomment-498038230
Yeah. Also I it should be possible to disable some rules in parts of the
files in case we find it non-compliant and there is a good reason for it. It's
[
https://issues.apache.org/jira/browse/AIRFLOW-4671?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bas Harenslak reassigned AIRFLOW-4671:
--
Assignee: Bas Harenslak
> Make airflow/executors Pylint compatible
>
[
https://issues.apache.org/jira/browse/AIRFLOW-4670?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bas Harenslak reassigned AIRFLOW-4670:
--
Assignee: Bas Harenslak
> Make airflow/example_dags Pylint compatible
>
potiuk commented on a change in pull request #5196: [AIRFLOW-4423] Improve date
handling in mysql to gcs operator.
URL: https://github.com/apache/airflow/pull/5196#discussion_r289648199
##
File path: airflow/contrib/operators/mysql_to_gcs.py
##
@@ -138,6 +156,11 @@ def
[
https://issues.apache.org/jira/browse/AIRFLOW-4721?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16854004#comment-16854004
]
ASF GitHub Bot commented on AIRFLOW-4721:
-
potiuk commented on pull request #5357:
potiuk merged pull request #5357: [AIRFLOW-4721] Remove all builtin imports
URL: https://github.com/apache/airflow/pull/5357
This is an automated message from the Apache Git Service.
To respond to the message, please log on
[
https://issues.apache.org/jira/browse/AIRFLOW-4721?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16854005#comment-16854005
]
ASF subversion and git services commented on AIRFLOW-4721:
--
Commit
[
https://issues.apache.org/jira/browse/AIRFLOW-4721?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jarek Potiuk resolved AIRFLOW-4721.
---
Resolution: Fixed
Fix Version/s: 2.0.0
> Remove all import builtins imports
>
codecov-io edited a comment on issue #5319: [AIRFLOW-4564] Azure Container
Instance bugfixes and improvements
URL: https://github.com/apache/airflow/pull/5319#issuecomment-497988863
# [Codecov](https://codecov.io/gh/apache/airflow/pull/5319?src=pr=h1)
Report
> Merging
codecov-io edited a comment on issue #5319: [AIRFLOW-4564] Azure Container
Instance bugfixes and improvements
URL: https://github.com/apache/airflow/pull/5319#issuecomment-497988863
# [Codecov](https://codecov.io/gh/apache/airflow/pull/5319?src=pr=h1)
Report
> Merging
pgagnon commented on issue #5238: [AIRFLOW-4364] Add Pylint to CI
URL: https://github.com/apache/airflow/pull/5238#issuecomment-498043930
@BasPH I didn't realize at first that a blacklist mechanism was in place.
Nevertheless I feel that we should disable `missing-docstring` for now until
potiuk commented on issue #4932: [AIRFLOW-3611] Simplified development workflow
[Depends on multi-staging]
URL: https://github.com/apache/airflow/pull/4932#issuecomment-498044271
Yep. I continuously rebase it in on top of master with the multi-staging
AIP-10. Up until recently we had a
potiuk commented on issue #4937: [AIRFLOW-4116] Multi-staging includes CI image
[Step 2/3]
URL: https://github.com/apache/airflow/pull/4937#issuecomment-498044369
Hey @ashb @Fokko -> I would love one final round of review. I would love to
merge AIP-10 completely this week before I talk
Trollgeir commented on a change in pull request #5319: [AIRFLOW-4564] Azure
Container Instance bugfixes and improvements
URL: https://github.com/apache/airflow/pull/5319#discussion_r289648763
##
File path: airflow/contrib/operators/azure_container_instances_operator.py
##
59 matches
Mail list logo