Sync v2-2-stable with v2-2-test to release 2.2.5
#22477
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ephraimbuddy
requested review from
potiuk,
ashb,
mik-laj,
ryanahamilton,
bbovenzi,
kaxil,
vikramkoka and
XD-DENG
as code owners
boring-cyborg
bot
added
area:API
ephraimbuddy
force-pushed
the
v2-2-test
branch
3 times, most recently
from
6c59d51 to
7dd3fed
Compare
|
wowowow! |
|
Hey @ephraimbuddy - I thiink we should merge #22385 before we run 2.2.5 - we have some problems with the currently "latest" providers and we need that one and at least voting on the new providers to release 2.2.5. |
|
I'd really love to merge it and start new provider's wave. |
eladkal
reviewed
Mar 22, 2022
ephraimbuddy
force-pushed
the
v2-2-test
branch
4 times, most recently
from
19178b9 to
ef70b89
Compare
The celery documentation have been moved from https://docs.celeryproject.org/ to https://docs.celeryq.dev/. The old links now refer to a 404 error page, the new links to the actual documentation. (cherry picked from commit a8de170)
We had disabled this previously in (#22254) but now the website is up on a different domain as listed in celery/celeryproject#51 (comment) (cherry picked from commit 1f7836e)
This is a change in the workflow specific to v2-2-branch that brings build-ci-images and tests in the same workflow. This can only be done because v2-2 branch no longer accepts PRs from forks and workflows are only one with direct pushes from commmitters. Then the workflows have write access to the packages and can be simplified (and can also contain different set of images than the main workflow).
Fixed dask executor and tests, distributed package does not ship with tests folder and the certificates, added certificates to certs folder (cherry picked from commit d3c168c)
Timedelta.max used in tests is not realistic and in some circumstances, when it is added to date, it might cause date OverflowError. Using long (but not 999999999 days long) timedelta solves the problem. (cherry picked from commit 18da121)
(cherry picked from commit 96e880d)
We have new Trove Classifiers in PyPI for Apache Airflow: pypa/trove-classifiers#87 This PR adds it for Airflow. The next release of Providers will add the classifiers for providers. (cherry picked from commit d3c65b6)
A workaround was added (#5731) to handle the refreshing of EKS tokens. It was necessary because of an upstream bug. It has since been fixed (kubernetes-client/python-base@70b78cd) and released in v21.7.0 (https://github.com/kubernetes-client/python/blob/master/CHANGELOG.md#v2170). (cherry picked from commit 7bd165f)
Kubernetes 23.* is more picky when it comes to values passed to Pod Generator - it requires: * imagePullPolicy * dnsPolicy * restartPolicy to be not None. We are fixing it in the way, that we simply skip setting those if they are None. (cherry picked from commit e856aba)
Previously we pinned this version as v12 as a change to Kube library internals meant v1.Pod objects now have a logger object inside them, and couldn't be pickled on Python 3.6. To fix that we have "backported" the change in Python 3.7 to make Logger objects be pickled "by name". (In Python 3.7 the change adds `__reduce__` methods on to the Logger and RootLogger objects, but here we achieve it `copyreg` stdlib module so we don't monkeypatch anything.) This fix is also applied in to airflow core in a separate commit, but we also apply it here in the provider so that cncf.kubernetes client library can be updated but still used with older versions of Airflow that don't have this fix in. (cherry picked from commit 7222f68)
ephraimbuddy
force-pushed
the
v2-2-test
branch
2 times, most recently
from
a3e102a to
1ccac5a
Compare
potiuk
approved these changes
Mar 26, 2022
potiuk
approved these changes
Mar 26, 2022
|
Looks Great |
|
🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.