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

Remove backcompat code for stackdriver #36442

Merged
merged 1 commit into from
Dec 28, 2023

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Dec 27, 2023

Remove back-compatibility code for stackdriver now that we are
at >= 2.6.0.

This caused problems recently with flaky / leaking tests but
hopefully #36455 fixed the triggering mechanism for that flaky
test and this one should succeed now.


^ 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.rst or {issue_number}.significant.rst, in newsfragments.

Copy link
Contributor

@dirrao dirrao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@potiuk
Copy link
Member Author

potiuk commented Dec 27, 2023

Well. I am puzzled ... It seem the previously seen log error is gone... Let me run it few more times.

@potiuk potiuk closed this Dec 27, 2023
@potiuk potiuk reopened this Dec 27, 2023
@Taragolis
Copy link
Contributor

Well. I am puzzled ... It seem the previously seen log error is gone... Let me run it few more times.

There more you look on the flakey error the less chance that it appear again 🤣

@potiuk
Copy link
Member Author

potiuk commented Dec 27, 2023

There more you look on the flakey error the less chance that it appear again 🤣

Very much so: 🐻

@potiuk
Copy link
Member Author

potiuk commented Dec 27, 2023

Seems like main started to fail after merging the other PR - let's see if #36453 will fix it again.

@potiuk potiuk force-pushed the remove-backcompat-stackdriver-code branch from 6462d32 to 55b9fe9 Compare December 27, 2023 15:01
Remove back-compatibility code for stackdriver now that we are
at >= 2.6.0.

This caused problems recently with flaky / leaking tests but
hopefully apache#36455 fixed the triggering mechanism for that flaky
test and this one should succeed now.
@potiuk potiuk force-pushed the remove-backcompat-stackdriver-code branch from 55b9fe9 to 3d9ff56 Compare December 28, 2023 01:17
@potiuk potiuk changed the title Reproduce asyncio failure with stackdriver leaking pytest fixture tasks Remove backcompat code for stackdriver Dec 28, 2023
@potiuk
Copy link
Member Author

potiuk commented Dec 28, 2023

Right - we should be able to merge that one now after @Taragolis improved log filtering.

Copy link
Member

@hussein-awala hussein-awala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh finally

@potiuk potiuk merged commit 49ce3ed into apache:main Dec 28, 2023
74 checks passed
@potiuk potiuk deleted the remove-backcompat-stackdriver-code branch December 28, 2023 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:logging area:providers full tests needed We need to run full set of tests for this PR to merge provider:google Google (including GCP) related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants