You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/01/07 04:59:13 UTC
[GitHub] [airflow] potiuk opened a new pull request #7091: [AIRFLOW-6489]
Separate integrations [WIP]
potiuk opened a new pull request #7091: [AIRFLOW-6489] Separate integrations [WIP]
URL: https://github.com/apache/airflow/pull/7091
---
Issue link: WILL BE INSERTED BY [boring-cyborg](https://github.com/kaxil/boring-cyborg)
- [x] Description above provides context of the change
- [x] Commit message/PR title starts with `[AIRFLOW-NNNN]`. AIRFLOW-NNNN = JIRA ID<sup>*</sup>
- [x] Unit tests coverage for changes (not needed for documentation changes)
- [x] Commits follow "[How to write a good git commit message](http://chris.beams.io/posts/git-commit/)"
- [x] Relevant documentation is updated including usage instructions.
- [x] I will engage committers as explained in [Contribution Workflow Example](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#contribution-workflow-example).
<sup>*</sup> For document-only changes commit message can start with `[AIRFLOW-XXXX]`.
---
In case of fundamental code change, Airflow Improvement Proposal ([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+Proposals)) is needed.
In case of a new dependency, check compliance with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
In case of backwards incompatible changes please leave a note in [UPDATING.md](https://github.com/apache/airflow/blob/master/UPDATING.md).
Read the [Pull Request Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines) for more information.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] potiuk merged pull request #7091: [AIRFLOW-6489] Separate
integrations in tests
Posted by GitBox <gi...@apache.org>.
potiuk merged pull request #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] mik-laj commented on a change in pull request #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
mik-laj commented on a change in pull request #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#discussion_r365755950
##########
File path: scripts/ci/in_container/krb5/krb5.conf
##########
@@ -17,8 +17,8 @@
# under the License.
[logging]
-default = FILE:/var/log/krb5libs.log
-kdc = FILE:/var/log/krb5kdc.log
+default = FILE:/var/log/kerberos.log
Review comment:
Why do we want to have these logs in one file?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] potiuk commented on a change in pull request #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
potiuk commented on a change in pull request #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#discussion_r365785548
##########
File path: scripts/ci/in_container/krb5/krb5.conf
##########
@@ -17,8 +17,8 @@
# under the License.
[logging]
-default = FILE:/var/log/krb5libs.log
-kdc = FILE:/var/log/krb5kdc.log
+default = FILE:/var/log/kerberos.log
Review comment:
Mistake ! Thanks. I will fix it.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/ab80880fedb7843baa5b6e0f11d3fd99ded4b5cc?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39255 39255
==========================================
- Hits 33478 33477 -1
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [ab80880...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/ab80880fedb7843baa5b6e0f11d3fd99ded4b5cc?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39255 39255
==========================================
- Hits 33478 33477 -1
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [ab80880...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `51.62%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
===========================================
- Coverage 85.28% 33.65% -51.63%
===========================================
Files 707 706 -1
Lines 39254 39297 +43
===========================================
- Hits 33477 13226 -20251
- Misses 5777 26071 +20294
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...flow/contrib/example\_dags/example\_qubole\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2V4YW1wbGVfZGFncy9leGFtcGxlX3F1Ym9sZV9zZW5zb3IucHk=) | `0% <0%> (-100%)` | :arrow_down: |
| [...ample\_dags/example\_emr\_job\_flow\_automatic\_steps.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2V4YW1wbGVfZGFncy9leGFtcGxlX2Vtcl9qb2JfZmxvd19hdXRvbWF0aWNfc3RlcHMucHk=) | `0% <0%> (-100%)` | :arrow_down: |
| [...contrib/example\_dags/example\_papermill\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2V4YW1wbGVfZGFncy9leGFtcGxlX3BhcGVybWlsbF9vcGVyYXRvci5weQ==) | `0% <0%> (-100%)` | :arrow_down: |
| [airflow/hooks/webhdfs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy93ZWJoZGZzX2hvb2sucHk=) | `0% <0%> (-100%)` | :arrow_down: |
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `0% <0%> (-100%)` | :arrow_down: |
| [airflow/operators/pig\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvcGlnX29wZXJhdG9yLnB5) | `0% <0%> (-100%)` | :arrow_down: |
| [airflow/contrib/operators/file\_to\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9maWxlX3RvX2djcy5weQ==) | `0% <0%> (-100%)` | :arrow_down: |
| [airflow/contrib/hooks/gcp\_spanner\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9zcGFubmVyX2hvb2sucHk=) | `0% <0%> (-100%)` | :arrow_down: |
| [airflow/operators/hive\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvaGl2ZV9vcGVyYXRvci5weQ==) | `0% <0%> (-100%)` | :arrow_down: |
| [...rflow/contrib/operators/bigquery\_check\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9iaWdxdWVyeV9jaGVja19vcGVyYXRvci5weQ==) | `0% <0%> (-100%)` | :arrow_down: |
| ... and [568 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/ab80880fedb7843baa5b6e0f11d3fd99ded4b5cc?src=pr&el=desc) will **not change** coverage.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
=======================================
Coverage 85.28% 85.28%
=======================================
Files 707 707
Lines 39255 39255
=======================================
Hits 33478 33478
Misses 5777 5777
```
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [ab80880...64fef51](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] nuclearpinguin commented on a change in pull request
#7091: [AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
nuclearpinguin commented on a change in pull request #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#discussion_r365761099
##########
File path: tests/providers/apache/cassandra/hooks/test_cassandra_hook.py
##########
@@ -30,15 +31,7 @@
from airflow.utils import db
-def cassandra_is_not_up():
- try:
- Cluster(["cassandra"])
- return False
- except UnresolvableContactPoints:
- return True
-
-
-@unittest.skipIf(cassandra_is_not_up(), "Cassandra is not up.")
+@pytest.mark.integration("cassandra")
class TestCassandraHook(unittest.TestCase):
Review comment:
If the reason behind Cassandra flakiness was lack of resources then this is should work.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/ab80880fedb7843baa5b6e0f11d3fd99ded4b5cc?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39255 39255
==========================================
- Hits 33478 33477 -1
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [ab80880...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] mik-laj commented on a change in pull request #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
mik-laj commented on a change in pull request #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#discussion_r365755489
##########
File path: scripts/ci/in_container/_in_container_utils.sh
##########
@@ -195,4 +195,5 @@ EOF
function stop_output_heartbeat() {
kill "${HEARTBEAT_PID}"
+ wait "${HEARTBEAT_PID}" || true 2> /dev/null
Review comment:
What do you think to start using `jobs/fg/bg/kill %1` here? Running scripts this way is a bit hacky to me.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] potiuk commented on issue #7091: [AIRFLOW-6489] Separate
integrations
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573585341
@mik-laj @ashb @dimberman @kaxil @feluelle @Fokko @nuclearpinguin -> please take a look.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] potiuk commented on a change in pull request #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
potiuk commented on a change in pull request #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#discussion_r365786565
##########
File path: tests/providers/apache/cassandra/hooks/test_cassandra_hook.py
##########
@@ -30,15 +31,7 @@
from airflow.utils import db
-def cassandra_is_not_up():
- try:
- Cluster(["cassandra"])
- return False
- except UnresolvableContactPoints:
- return True
-
-
-@unittest.skipIf(cassandra_is_not_up(), "Cassandra is not up.")
+@pytest.mark.integration("cassandra")
class TestCassandraHook(unittest.TestCase):
Review comment:
I truly hope so! It looks like so far.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] mik-laj commented on a change in pull request #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
mik-laj commented on a change in pull request #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#discussion_r365750352
##########
File path: TESTING.rst
##########
@@ -152,46 +152,262 @@ You can also specify individual tests or a group of tests:
./breeze --test-target tests/test_core.py::TestCore
-Running Full test suite via scripts from the host
--------------------------------------------------
-To run all tests with default settings (Python 3.6, Sqlite backend, "docker" environment), enter:
+Airflow Integration Tests
+=========================
-.. code-block::
+Some of the tests in Airflow are Integration tests. Those tests require not only airflow-testing docker
+image but also extra images with integrations (such as redis/mongodb etc.).
- ./scripts/ci/local_ci_run_airflow_testing.sh
+Enabling integrations
+---------------------
-To select Python 3.6 version, Postgres backend, and a ``docker`` environment, specify:
+Running Airflow integration tests cannot be run in local virtualenv. They can only run in Breeze
+environment with enabled integrations and in Travis CI.
+
+When you are in Breeze environment, by default all integrations are disabled - this way only true unit tests
+can be executed in Breeze. You can enable the integration by passing ``--integration <INTEGRATION>``
+switch when starting Breeze. You can specify multiple integrations by repeating the ``--integration`` switch
+or by using ``--integration all`` switch which enables all integrations.
+
+Note, that every integration requires separate container with the corresponding integration image,
+so they take precious resources on your PC - mainly memory. The integrations started are not stopped
+until you stop the Breeze environment with ``--stop-environment`` switch.
+
+The following integrations are available:
+
+.. list-table:: Airflow Test Integrations
+ :widths: 15 80
+ :header-rows: 1
+
+ * - Integration
+ - Description
+ * - cassandra
+ - Integration required for Cassandra hooks
+ * - kerberos
+ - Integration that provides Kerberos authentication
+ * - mongo
+ - Integration required for MongoDB hooks
+ * - openldap
+ - Integration required for OpenLDAP hooks
+ * - rabbitmq
+ - Integration required for Celery executor tests
+ * - redis
+ - Integration required for Celery executor tests
+
+Below command starts mongi integration only:
Review comment:
```suggestion
Below command starts mongo integration only:
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.28% +<.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
+ Hits 33477 33478 +1
Misses 5777 5777
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...8767391](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] potiuk commented on a change in pull request #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
potiuk commented on a change in pull request #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#discussion_r365785103
##########
File path: scripts/ci/in_container/_in_container_utils.sh
##########
@@ -195,4 +195,5 @@ EOF
function stop_output_heartbeat() {
kill "${HEARTBEAT_PID}"
+ wait "${HEARTBEAT_PID}" || true 2> /dev/null
Review comment:
Kill %1 is the same as PID but it is really intended for interactive use (when you see the output of `jobs` command). You never know which job you are killing this way. If you have some other background jobs started before your "start_output_heartbeat" then `kill %1` will kill the other job, not the heartbeat. Storing PID of the started process and using it explicitly is the only sure way I know to kill exactly the process I intend to.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/ab80880fedb7843baa5b6e0f11d3fd99ded4b5cc?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39255 39255
==========================================
- Hits 33478 33477 -1
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [ab80880...64fef51](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] potiuk commented on issue #7091: [AIRFLOW-6489] Separate
integrations
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573439357
Hello everyone. I am still running final tests but I think this one is ready to review. I think - regardless of our Github Actions move, it will be super-helpful to have faster, more stable tests and especially - the local dev environment with Breeze is now really, really convenient to use (far lest resource usage without extra integrations, full pytest marker integration etc.).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] potiuk commented on issue #7091: [AIRFLOW-6489] Separate
integrations
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573594030
Rearranged documentation slightly.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.28% +<.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
+ Hits 33477 33478 +1
Misses 5777 5777
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...64fef51](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...64fef51](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.28% +<.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
+ Hits 33477 33478 +1
Misses 5777 5777
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...8767391](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io commented on issue #7091: [AIRFLOW-6489]
Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io commented on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/ab80880fedb7843baa5b6e0f11d3fd99ded4b5cc?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39255 39255
==========================================
- Hits 33478 33477 -1
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [ab80880...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> :exclamation: No coverage uploaded for pull request base (`master@e7bf8ec`). [Click here to learn what that means](https://docs.codecov.io/docs/error-reference#section-missing-base-commit).
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
=========================================
Coverage ? 85.33%
=========================================
Files ? 707
Lines ? 39358
Branches ? 0
=========================================
Hits ? 33588
Misses ? 5770
Partials ? 0
```
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [e7bf8ec...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] potiuk commented on a change in pull request #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
potiuk commented on a change in pull request #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#discussion_r365720797
##########
File path: TESTING.rst
##########
@@ -152,46 +152,192 @@ You can also specify individual tests or a group of tests:
./breeze --test-target tests/test_core.py::TestCore
-Running Full test suite via scripts from the host
--------------------------------------------------
-To run all tests with default settings (Python 3.6, Sqlite backend, "docker" environment), enter:
+Airflow Integration Tests
+=========================
-.. code-block::
+Some of the tests in Airflow are Integration tests. Those tests require not only airflow-testing docker
+image but also extra images with integrations (such as redis/mongodb etc.).
- ./scripts/ci/local_ci_run_airflow_testing.sh
+Enabling integrations
+---------------------
-To select Python 3.6 version, Postgres backend, and a ``docker`` environment, specify:
+Running Airflow integration tests cannot be run in local virtualenv. They can only run in Breeze
+environment with enabled integrations and in Travis CI.
+
+When you are in Breeze environment, by default all integrations are disabled - this way only true unit tests
+can be executed in Breeze. You can enable the integration by passing ``--integration <INTEGRATION>``
+switch when starting Breeze. You can specify multiple integrations by repeating the ``--integration`` switch
+or by using ``--integration all`` switch which enables all integrations.
+
+Note, that every integration requires separate container with the corresponding integration image,
+so they take precious resources on your PC - mainly memory. The integrations started are not stopped
+until you stop the Breeze environment with ``--stop-environment`` switch.
+
+The following integrations are available:
+
+.. list-table:: Airflow Test Integrations
+ :widths: 15 80
+ :header-rows: 1
+
+ * - Integration
+ - Description
+ * - cassandra
+ - Integration required for Cassandra hooks
+ * - kerberos
+ - Integration that provides Kerberos authentication
+ * - mongo
+ - Integration required for MongoDB hooks
+ * - openldap
Review comment:
Note to reviewers: I could not find any openldap integration tests. Maybe some of them are simply skipped now. I'd love if you can point me to the tests that should be marked with pytest.mark.integration("openldap")
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
- Coverage 85.28% 85.28% -0.01%
==========================================
Files 707 707
Lines 39254 39255 +1
==========================================
Hits 33477 33477
- Misses 5777 5778 +1
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `91.59% <0%> (-0.29%)` | :arrow_down: |
| [airflow/operators/docker\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZG9ja2VyX29wZXJhdG9yLnB5) | `96.87% <0%> (+0.03%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...390f679](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [airflow] codecov-io edited a comment on issue #7091:
[AIRFLOW-6489] Separate integrations in tests
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7091: [AIRFLOW-6489] Separate integrations in tests
URL: https://github.com/apache/airflow/pull/7091#issuecomment-573536301
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=h1) Report
> Merging [#7091](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/0481b9a95786a62de4776a735ae80e746583ef2b?src=pr&el=desc) will **increase** coverage by `0.05%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7091 +/- ##
==========================================
+ Coverage 85.28% 85.33% +0.05%
==========================================
Files 707 707
Lines 39254 39358 +104
==========================================
+ Hits 33477 33588 +111
+ Misses 5777 5770 -7
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/contrib/operators/gcs\_list\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfbGlzdF9vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfZ2NzLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_acl\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3NfYWNsX29wZXJhdG9yLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcs\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djc19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/sensors/gcs\_sensor.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvZ2NzX3NlbnNvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/hooks/gcp\_dataproc\_hook.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2djcF9kYXRhcHJvY19ob29rLnB5) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `94.73% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/gcs\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9nY3Nfb3BlcmF0b3IucHk=) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/contrib/operators/dataproc\_operator.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9kYXRhcHJvY19vcGVyYXRvci5weQ==) | `100% <0%> (ø)` | :arrow_up: |
| [airflow/gcp/example\_dags/example\_cloud\_sql.py](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvZXhhbXBsZV9kYWdzL2V4YW1wbGVfY2xvdWRfc3FsLnB5) | `100% <0%> (ø)` | :arrow_up: |
| ... and [8 more](https://codecov.io/gh/apache/airflow/pull/7091/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=footer). Last update [0481b9a...b68676d](https://codecov.io/gh/apache/airflow/pull/7091?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services