Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to stable constraints #12636

Merged
merged 1 commit into from
Nov 27, 2020

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Nov 26, 2020

This change upgrades setup.py to provide non-conflicting
pip check valid set of constraints for CI image.

PIP check is run once the images are available.


^ 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
Copy link
Member Author

potiuk commented Nov 26, 2020

@turbaszek - I moved your changes to "Wait for images" step. As soon as #12635 gets merged, this PR (with "upgrade to latest dependencies" label added) should (I believe) cleanly build and successfully run pip-check (at least for the CI Images). Once the constraints are updated, we can also likely enable prod image check.

@potiuk
Copy link
Member Author

potiuk commented Nov 26, 2020

I can split it further into two PRs, but I want to make it pass and see if indeed PIP check works @turbaszek !

@potiuk potiuk force-pushed the upgrade-to-stable-constraints branch 3 times, most recently from 0bccbd9 to e57f60f Compare November 26, 2020 12:12
@github-actions github-actions bot added the full tests needed We need to run full set of tests for this PR to merge label Nov 26, 2020
@github-actions
Copy link

The PR needs to run all tests because it modifies core of Airflow! Please rebase it to latest master or ask committer to re-run it!

@potiuk
Copy link
Member Author

potiuk commented Nov 26, 2020

I still disable the result of the pip check - they will be printed but won't fail the build. Working on making sure all those are solved

@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 removed the full tests needed We need to run full set of tests for this PR to merge label Nov 26, 2020
@potiuk potiuk force-pushed the upgrade-to-stable-constraints branch from e57f60f to 9c6e941 Compare November 26, 2020 12:45
@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 upgrade-to-stable-constraints branch from 9c6e941 to 1d7aa92 Compare November 26, 2020 13:10
@potiuk potiuk added the full tests needed We need to run full set of tests for this PR to merge label Nov 26, 2020
@potiuk
Copy link
Member Author

potiuk commented Nov 26, 2020

@turbaszek -> final (????) test.

@potiuk
Copy link
Member Author

potiuk commented Nov 26, 2020

ERROR    airflow.models.dagbag.DagBag:dagbag.py:297 Failed to import: /opt/airflow/airflow/providers/google/ads/example_dags/example_ads.py
Traceback (most recent call last):
  File "/opt/airflow/airflow/models/dagbag.py", line 294, in _load_modules_from_file
    loader.exec_module(new_module)
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/opt/airflow/airflow/providers/google/ads/example_dags/example_ads.py", line 24, in <module>
    from airflow.providers.google.ads.operators.ads import GoogleAdsListAccountsOperator
  File "/opt/airflow/airflow/providers/google/ads/operators/ads.py", line 24, in <module>
    from airflow.providers.google.ads.hooks.ads import GoogleAdsHook
  File "/opt/airflow/airflow/providers/google/ads/hooks/ads.py", line 25, in <module>
    from google.ads.google_ads.v2.types import GoogleAdsRow
ModuleNotFoundError: No module named 'google.ads.google_ads.v2'

Python 3.7 and 3.8 only !

@potiuk potiuk force-pushed the upgrade-to-stable-constraints branch 2 times, most recently from 6f65d03 to ec7ee66 Compare November 26, 2020 21:31
potiuk added a commit that referenced this pull request Nov 27, 2020
This PR enables PIP check after constraints have been updated
to be stable and 'pip check' compliant in #12636
potiuk added a commit that referenced this pull request Nov 30, 2020
This change upgrades setup.py and setup.cfg to provide non-conflicting
`pip check` valid set of constraints for CI image.
@potiuk potiuk added this to the Airflow 1.10.14 milestone Nov 30, 2020
@potiuk potiuk added the type:improvement Changelog: Improvements label Nov 30, 2020
kaxil pushed a commit that referenced this pull request Dec 3, 2020
This change upgrades setup.py and setup.cfg to provide non-conflicting
`pip check` valid set of constraints for CI image.
kaxil pushed a commit that referenced this pull request Dec 3, 2020
This change upgrades setup.py and setup.cfg to provide non-conflicting
`pip check` valid set of constraints for CI image.

(cherry picked from commit 5370f3feb095e02704b0852fe630efdd118cb8f5)
potiuk added a commit that referenced this pull request Dec 3, 2020
This change upgrades setup.py and setup.cfg to provide non-conflicting
`pip check` valid set of constraints for CI image.

(cherry picked from commit 5370f3feb095e02704b0852fe630efdd118cb8f5)
ashb pushed a commit that referenced this pull request Dec 3, 2020
This change upgrades setup.py and setup.cfg to provide non-conflicting
`pip check` valid set of constraints for CI image.
ashb pushed a commit that referenced this pull request Dec 3, 2020
This change upgrades setup.py and setup.cfg to provide non-conflicting
`pip check` valid set of constraints for CI image.

(cherry picked from commit 5370f3feb095e02704b0852fe630efdd118cb8f5)
potiuk added a commit that referenced this pull request Dec 13, 2020
This PR enables PIP check after constraints have been updated
to be stable and 'pip check' compliant in #12636

(cherry picked from commit fa8af2d)
kaxil pushed a commit that referenced this pull request Jan 21, 2021
This PR enables PIP check after constraints have been updated
to be stable and 'pip check' compliant in #12636

(cherry picked from commit fa8af2d)
kaxil pushed a commit that referenced this pull request Jan 22, 2021
This PR enables PIP check after constraints have been updated
to be stable and 'pip check' compliant in #12636

(cherry picked from commit fa8af2d)
AntonyRileyAtVerto pushed a commit to vertoanalytics/incubator-airflow that referenced this pull request Feb 2, 2021
- BugFix: Tasks with ``depends_on_past`` or ``task_concurrency`` are stuck (apache#12663)
- Fix issue with empty Resources in executor_config (apache#12633)
- Fix: Deprecated config ``force_log_out_after`` was not used (apache#12661)
- Fix empty asctime field in JSON formatted logs (apache#10515)
- [AIRFLOW-2809] Fix security issue regarding Flask SECRET_KEY (apache#3651)
- [AIRFLOW-2884] Fix Flask SECRET_KEY security issue in www_rbac (apache#3729)
- [AIRFLOW-2886] Generate random Flask SECRET_KEY in default config (apache#3738)
- Add missing comma in setup.py (apache#12790)
- Bugfix: Unable to import Airflow plugins on Python 3.8 (apache#12859)
- Fix setup.py missing comma in ``setup_requires`` (apache#12880)
- Don't emit first_task_scheduling_delay metric for only-once dags (apache#12835)

- Update setup.py to get non-conflicting set of dependencies (apache#12636)
- Rename ``[scheduler] max_threads`` to ``[scheduler] parsing_processes`` (apache#12605)
- Add metric for scheduling delay between first run task & expected start time (apache#9544)
- Add new-style 2.0 command names for Airflow 1.10.x (apache#12725)
- Add Kubernetes cleanup-pods CLI command for Helm Chart (apache#11802)
- Don't let webserver run with dangerous config (apache#12747)
- Replace pkg_resources with importlib.metadata to avoid VersionConflict errors (apache#12694)

- Clarified information about supported Databases
cfei18 pushed a commit to cfei18/incubator-airflow that referenced this pull request Mar 5, 2021
)

This change upgrades setup.py and setup.cfg to provide non-conflicting
`pip check` valid set of constraints for CI image.
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 type:improvement Changelog: Improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants