Skip to content

Conversation

@sunank200
Copy link
Collaborator

@sunank200 sunank200 commented Dec 22, 2025

The test was failing because it tried to patch a non-existent TIMEZONE attribute. The timezone module uses _Timezone.initialized_timezone to store the default timezone, which is what the parse function uses when parsing timestamps without timezone information.

(cherry picked from commit 3d33d6d)


^ 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 airflow-core/newsfragments.

@sunank200 sunank200 marked this pull request as ready for review December 22, 2025 06:31
@sunank200 sunank200 added the full tests needed We need to run full set of tests for this PR to merge label Dec 22, 2025
…apache#59698)

The test was failing because it tried to patch a non-existent TIMEZONE
attribute. The timezone module uses _Timezone.initialized_timezone
to store the default timezone, which is what the parse function uses
when parsing timestamps without timezone information.

Fixes failing tests:
- TestCLIDBClean.test_date_timezone_omitted[UTC]
- TestCLIDBClean.test_date_timezone_omitted[Europe/Berlin]
- TestCLIDBClean.test_date_timezone_omitted[America/Los_Angeles]

(cherry picked from commit 3d33d6d)
@sunank200 sunank200 force-pushed the backport-3d33d6d-v3-1-test branch from 226c803 to df77264 Compare December 22, 2025 07:51
@sunank200
Copy link
Collaborator Author

This can be closed because the original PR #59689 was not backported to this branch.

@sunank200 sunank200 closed this Dec 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:CLI 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.

3 participants