Moves airflow import in deprecated pod_generator to local #39062
Conversation
This file contains hidden or 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
The import might be invoked when K8S executor starts with sentry on and it might lead to circular imports Related: apache#31442
boring-cyborg
bot
added
area:providers
provider:cncf-kubernetes
2 tasks
hussein-awala
approved these changes
Apr 16, 2024
Taragolis
approved these changes
Apr 16, 2024
jedcunningham
approved these changes
Apr 16, 2024
eladkal
approved these changes
Apr 16, 2024
grrolland
pushed a commit
to grrolland/airflow
that referenced
this pull request
Apr 19, 2024
The import might be invoked when K8S executor starts with sentry on and it might lead to circular imports Related: apache#31442
Taragolis
pushed a commit
that referenced
this pull request
May 6, 2024
…sion (#39056) * Fixes #36629 * Fixes PR failed test * Remove an parametrize duplicate tests * Fix formatting * Fix formatting * Fixes #36629 * Fixes PR failed test * Remove an parametrize duplicate tests * update simple-salesforce type hints to support 1.12.6 (#39047) * Fix formatting * Add changelog for airflow python client 2.9.0 (#39060) * Upgrade to latest hatchling as build dependency (#39044) * Prepare docs 1st wave (RC3) + ad hoc April 2024 (#38995) (#39054) * Prepare docs 1st wave (RC3) + ad hoc April 2024 (#38995) * update databricks * [docs] update `DagBag` class docstring to include all params (#38814) * update docstring for DagBag class * break long line * fix space Signed-off-by: kalyanr <kalyan.ben10@live.com> --------- Signed-off-by: kalyanr <kalyan.ben10@live.com> * Data aware scheduling docs edits (#38687) * Moves airflow import in deprecated pod_generator to local (#39062) The import might be invoked when K8S executor starts with sentry on and it might lead to circular imports Related: #31442 * KPO xcom sidecar PodDefault usage (#38951) We should use the same, non deprecated, version of PodDefaults for the xcom sidecar when creating and reading xcom. * Fix formatting * Change date/time parsing method for newer_than parameter un SFTPSensor * Add examples in AWS auth manager documentation (#39040) * update document (#39068) * Update hatchling to version 1.24.0 (#39072) * Check that the dataset<>task exists before trying to render graph (#39069) * Change date/time parsing method for newer_than parameter un SFTPSensor * Fix utc timezone in unit tests * Fix utc timezone in unit tests --------- Signed-off-by: kalyanr <kalyan.ben10@live.com> Co-authored-by: Grégoire Rolland <gregoire.rolland@mymoneybank.com> Co-authored-by: Hussein Awala <hussein@awala.fr> Co-authored-by: Ephraim Anierobi <splendidzigy24@gmail.com> Co-authored-by: Jarek Potiuk <jarek@potiuk.com> Co-authored-by: Elad Kalif <45845474+eladkal@users.noreply.github.com> Co-authored-by: Kalyan <kalyan.ben10@live.com> Co-authored-by: Laura Zdanski <25642903+lzdanski@users.noreply.github.com> Co-authored-by: Jed Cunningham <66968678+jedcunningham@users.noreply.github.com> Co-authored-by: Vincent <97131062+vincbeck@users.noreply.github.com> Co-authored-by: humit <jhjang1005@naver.com> Co-authored-by: Brent Bovenzi <brent@astronomer.io>
pateash
pushed a commit
to pateash/airflow
that referenced
this pull request
May 13, 2024
…sion (apache#39056) * Fixes apache#36629 * Fixes PR failed test * Remove an parametrize duplicate tests * Fix formatting * Fix formatting * Fixes apache#36629 * Fixes PR failed test * Remove an parametrize duplicate tests * update simple-salesforce type hints to support 1.12.6 (apache#39047) * Fix formatting * Add changelog for airflow python client 2.9.0 (apache#39060) * Upgrade to latest hatchling as build dependency (apache#39044) * Prepare docs 1st wave (RC3) + ad hoc April 2024 (apache#38995) (apache#39054) * Prepare docs 1st wave (RC3) + ad hoc April 2024 (apache#38995) * update databricks * [docs] update `DagBag` class docstring to include all params (apache#38814) * update docstring for DagBag class * break long line * fix space Signed-off-by: kalyanr <kalyan.ben10@live.com> --------- Signed-off-by: kalyanr <kalyan.ben10@live.com> * Data aware scheduling docs edits (apache#38687) * Moves airflow import in deprecated pod_generator to local (apache#39062) The import might be invoked when K8S executor starts with sentry on and it might lead to circular imports Related: apache#31442 * KPO xcom sidecar PodDefault usage (apache#38951) We should use the same, non deprecated, version of PodDefaults for the xcom sidecar when creating and reading xcom. * Fix formatting * Change date/time parsing method for newer_than parameter un SFTPSensor * Add examples in AWS auth manager documentation (apache#39040) * update document (apache#39068) * Update hatchling to version 1.24.0 (apache#39072) * Check that the dataset<>task exists before trying to render graph (apache#39069) * Change date/time parsing method for newer_than parameter un SFTPSensor * Fix utc timezone in unit tests * Fix utc timezone in unit tests --------- Signed-off-by: kalyanr <kalyan.ben10@live.com> Co-authored-by: Grégoire Rolland <gregoire.rolland@mymoneybank.com> Co-authored-by: Hussein Awala <hussein@awala.fr> Co-authored-by: Ephraim Anierobi <splendidzigy24@gmail.com> Co-authored-by: Jarek Potiuk <jarek@potiuk.com> Co-authored-by: Elad Kalif <45845474+eladkal@users.noreply.github.com> Co-authored-by: Kalyan <kalyan.ben10@live.com> Co-authored-by: Laura Zdanski <25642903+lzdanski@users.noreply.github.com> Co-authored-by: Jed Cunningham <66968678+jedcunningham@users.noreply.github.com> Co-authored-by: Vincent <97131062+vincbeck@users.noreply.github.com> Co-authored-by: humit <jhjang1005@naver.com> Co-authored-by: Brent Bovenzi <brent@astronomer.io>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
The import might be invoked when K8S executor starts with sentry on and it might lead to circular imports
Related: #31442
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an 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 a newsfragment file, named
{pr_number}.significant.rstor{issue_number}.significant.rst, in newsfragments.