Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Nov 28, 2020

Some parts of our CI are installing (for tests) provider packages
inside the CI container. That has proven to be rather brittle in
terms of keeping the requirements in a consistent state (and is
totally out of our control as described in #12692, however in our
tests we can continue using constraints to make sure we use
consistent set of requirements.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

@potiuk potiuk requested a review from ashb November 28, 2020 18:24
@potiuk potiuk assigned kaxil and turbaszek and unassigned kaxil and turbaszek Nov 28, 2020
@potiuk potiuk requested review from kaxil and turbaszek November 28, 2020 18:24
@potiuk
Copy link
Member Author

potiuk commented Nov 28, 2020

This will not solve the #12692, but at least should prevent our tests from failing on suddenly upgraded requirements.

@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from 66a7f75 to c7699f7 Compare November 28, 2020 19:14
@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from c7699f7 to 9af42a8 Compare November 29, 2020 06:34
@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from 9af42a8 to c3ef45b Compare November 29, 2020 14:44
@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from c3ef45b to 4d82885 Compare November 30, 2020 00:46
@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from 4d82885 to 3299eb6 Compare November 30, 2020 01:50
@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@github-actions
Copy link

The Workflow run is cancelling this PR. Building image for the PR has been cancelled

@potiuk potiuk marked this pull request as draft November 30, 2020 18:50
@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from 3299eb6 to 9d1eab5 Compare November 30, 2020 21:42
@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from 9d1eab5 to 72ad6ae Compare November 30, 2020 22:22
@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from 72ad6ae to babaa30 Compare December 4, 2020 16:45
@github-actions
Copy link

github-actions bot commented Dec 7, 2020

The Workflow run is cancelling this PR. Building images for the PR has failed. Follow the the workflow link to check the reason.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from 4019720 to 6909d91 Compare December 7, 2020 22:28
@github-actions
Copy link

github-actions bot commented Dec 7, 2020

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@github-actions
Copy link

github-actions bot commented Dec 7, 2020

The Workflow run is cancelling this PR. Building image for the PR has been cancelled

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch 3 times, most recently from 7b100fe to 2c5c454 Compare December 8, 2020 03:48
@github-actions
Copy link

github-actions bot commented Dec 8, 2020

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@github-actions
Copy link

github-actions bot commented Dec 8, 2020

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch 2 times, most recently from e52f918 to 17a524e Compare December 8, 2020 12:28
@github-actions
Copy link

github-actions bot commented Dec 8, 2020

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from 17a524e to bbd7494 Compare December 8, 2020 15:11
@github-actions
Copy link

github-actions bot commented Dec 8, 2020

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from bbd7494 to cb7d376 Compare December 8, 2020 16:22
@github-actions
Copy link

github-actions bot commented Dec 8, 2020

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from cb7d376 to 072bb4d Compare December 8, 2020 19:09
@github-actions
Copy link

github-actions bot commented Dec 8, 2020

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from 072bb4d to 94e247d Compare December 9, 2020 06:27
@github-actions
Copy link

github-actions bot commented Dec 9, 2020

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from 94e247d to 27b2f6e Compare December 9, 2020 09:59
@github-actions
Copy link

github-actions bot commented Dec 9, 2020

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from 27b2f6e to 995ff1b Compare December 31, 2020 14:12
@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@github-actions
Copy link

The Workflow run is cancelling this PR. Building image for the PR has been cancelled

Some parts of our CI are installing (for tests) provider packages
inside the CI container. That has proven to be rather brittle in
terms of keeping the requirements in a consistent state (and is
totally out of our control as described in apache#12692, however in our
tests we can continue using constraints to make sure we use
consistent set of requirements.
@potiuk potiuk force-pushed the install-provider-packages-with-constraints branch from 995ff1b to a06e691 Compare December 31, 2020 15:28
@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@potiuk
Copy link
Member Author

potiuk commented Jan 1, 2021

closing. The functionality (in a better way) is implemented in #13409

@potiuk potiuk closed this Jan 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants