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/15 23:02:49 UTC
[GitHub] [airflow] potiuk opened a new pull request #7185: [AIRFLOW-6575]
Entropy source for CI tests is changed to unblocking
potiuk opened a new pull request #7185: [AIRFLOW-6575] Entropy source for CI tests is changed to unblocking
URL: https://github.com/apache/airflow/pull/7185
On Travis CI blocking entropy source slows startup time of a number of
containers. This change changes the entropy source to unblocking one.
---
Issue link: WILL BE INSERTED BY [boring-cyborg](https://github.com/kaxil/boring-cyborg)
Make sure to mark the boxes below before creating PR: [x]
- [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 commented on issue #7185: [AIRFLOW-6575] Entropy
source for CI tests is changed to unblocking
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #7185: [AIRFLOW-6575] Entropy source for CI tests is changed to unblocking
URL: https://github.com/apache/airflow/pull/7185#issuecomment-574898482
Let's see when Travis completes, but I believe this change will speed up the startup - I changed the entropy source to be unblocking one and it should make the startup faster.
----------------------------------------------------------------
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 #7185: [AIRFLOW-6575] Entropy
source for CI tests is changed to unblocking
Posted by GitBox <gi...@apache.org>.
potiuk merged pull request #7185: [AIRFLOW-6575] Entropy source for CI tests is changed to unblocking
URL: https://github.com/apache/airflow/pull/7185
----------------------------------------------------------------
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 #7185:
[AIRFLOW-6575] Entropy source for CI tests is changed to unblocking
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7185: [AIRFLOW-6575] Entropy source for CI tests is changed to unblocking
URL: https://github.com/apache/airflow/pull/7185#issuecomment-574910352
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7185?src=pr&el=h1) Report
> Merging [#7185](https://codecov.io/gh/apache/airflow/pull/7185?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/417feda5eba6cc63c01a9a04a7b48bc24f63a3d1?src=pr&el=desc) will **increase** coverage by `0.12%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7185?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7185 +/- ##
==========================================
+ Coverage 85.25% 85.37% +0.12%
==========================================
Files 723 723
Lines 39543 39543
==========================================
+ Hits 33711 33761 +50
+ Misses 5832 5782 -50
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7185?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/scheduler\_job.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL3NjaGVkdWxlcl9qb2IucHk=) | `89.34% <0%> (+0.43%)` | :arrow_up: |
| [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+1.65%)` | :arrow_up: |
| [airflow/hooks/postgres\_hook.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9wb3N0Z3Jlc19ob29rLnB5) | `94.36% <0%> (+16.9%)` | :arrow_up: |
| [airflow/operators/postgres\_to\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvcG9zdGdyZXNfdG9fZ2NzLnB5) | `85.29% <0%> (+32.35%)` | :arrow_up: |
| [airflow/operators/postgres\_operator.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvcG9zdGdyZXNfb3BlcmF0b3IucHk=) | `100% <0%> (+100%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7185?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/7185?src=pr&el=footer). Last update [417feda...b7568be](https://codecov.io/gh/apache/airflow/pull/7185?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 #7185: [AIRFLOW-6575]
Entropy source for CI tests is changed to unblocking
Posted by GitBox <gi...@apache.org>.
codecov-io commented on issue #7185: [AIRFLOW-6575] Entropy source for CI tests is changed to unblocking
URL: https://github.com/apache/airflow/pull/7185#issuecomment-574910352
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7185?src=pr&el=h1) Report
> Merging [#7185](https://codecov.io/gh/apache/airflow/pull/7185?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/417feda5eba6cc63c01a9a04a7b48bc24f63a3d1?src=pr&el=desc) will **decrease** coverage by `0.62%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7185?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7185 +/- ##
==========================================
- Coverage 85.25% 84.62% -0.63%
==========================================
Files 723 723
Lines 39543 39543
==========================================
- Hits 33711 33463 -248
- Misses 5832 6080 +248
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7185?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/operators/mysql\_operator.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvbXlzcWxfb3BlcmF0b3IucHk=) | `0% <0%> (-100%)` | :arrow_down: |
| [airflow/operators/mysql\_to\_hive.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvbXlzcWxfdG9faGl2ZS5weQ==) | `0% <0%> (-100%)` | :arrow_down: |
| [airflow/operators/generic\_transfer.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvZ2VuZXJpY190cmFuc2Zlci5weQ==) | `0% <0%> (-100%)` | :arrow_down: |
| [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
| [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
| [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
| [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
| [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.31% <0%> (-20.49%)` | :arrow_down: |
| [airflow/utils/sqlalchemy.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy91dGlscy9zcWxhbGNoZW15LnB5) | `86.66% <0%> (-10%)` | :arrow_down: |
| [airflow/utils/dag\_processing.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy91dGlscy9kYWdfcHJvY2Vzc2luZy5weQ==) | `85.52% <0%> (-2.67%)` | :arrow_down: |
| ... and [5 more](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7185?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/7185?src=pr&el=footer). Last update [417feda...b7568be](https://codecov.io/gh/apache/airflow/pull/7185?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 #7185:
[AIRFLOW-6575] Entropy source for CI tests is changed to unblocking
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #7185: [AIRFLOW-6575] Entropy source for CI tests is changed to unblocking
URL: https://github.com/apache/airflow/pull/7185#issuecomment-574910352
# [Codecov](https://codecov.io/gh/apache/airflow/pull/7185?src=pr&el=h1) Report
> Merging [#7185](https://codecov.io/gh/apache/airflow/pull/7185?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/417feda5eba6cc63c01a9a04a7b48bc24f63a3d1?src=pr&el=desc) will **increase** coverage by `0.12%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/airflow/pull/7185?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #7185 +/- ##
==========================================
+ Coverage 85.25% 85.37% +0.12%
==========================================
Files 723 723
Lines 39543 39543
==========================================
+ Hits 33711 33761 +50
+ Misses 5832 5782 -50
```
| [Impacted Files](https://codecov.io/gh/apache/airflow/pull/7185?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [airflow/jobs/scheduler\_job.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL3NjaGVkdWxlcl9qb2IucHk=) | `89.34% <0%> (+0.43%)` | :arrow_up: |
| [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+1.65%)` | :arrow_up: |
| [airflow/hooks/postgres\_hook.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9wb3N0Z3Jlc19ob29rLnB5) | `94.36% <0%> (+16.9%)` | :arrow_up: |
| [airflow/operators/postgres\_to\_gcs.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvcG9zdGdyZXNfdG9fZ2NzLnB5) | `85.29% <0%> (+32.35%)` | :arrow_up: |
| [airflow/operators/postgres\_operator.py](https://codecov.io/gh/apache/airflow/pull/7185/diff?src=pr&el=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvcG9zdGdyZXNfb3BlcmF0b3IucHk=) | `100% <0%> (+100%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/7185?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/7185?src=pr&el=footer). Last update [417feda...b7568be](https://codecov.io/gh/apache/airflow/pull/7185?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