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.

Sign up for GitHub

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

Jump to bottom

[Python][CI] Hypothesis nightly test fails with pytz.exceptions.UnknownTimeZoneError: 'Factory' #36498

Closed
AlenkaF opened this issue Jul 6, 2023 · 0 comments · Fixed by #36508
Closed

[Python][CI] Hypothesis nightly test fails with pytz.exceptions.UnknownTimeZoneError: 'Factory' #36498

AlenkaF opened this issue Jul 6, 2023 · 0 comments · Fixed by #36508
Assignees
@AlenkaF
Labels
Component: Continuous Integration Component: Python Type: bug
Milestone
13.0.0

Comments

@AlenkaF
Copy link
Member

AlenkaF commented Jul 6, 2023

Describe the bug, including details regarding any error messages, version, and platform.

Nightly tests started failing again on test-conda-python-3.11-hypothesis
with a similar error to #36349

>               raise UnknownTimeZoneError(zone)
E               pytz.exceptions.UnknownTimeZoneError: 'Factory'
E               Falsifying example: test_python_datetime_with_pytz_timezone(
E                   self=<pyarrow.tests.test_pandas.TestConvertDateTimeLikeTypes object at 0x7fed9ef5d7d0>,
E                   tz=ZoneInfo('Factory'),
E               )

opt/conda/envs/arrow/lib/python3.11/site-packages/pytz/__init__.py:188: UnknownTimeZoneError

Component(s)

Continuous Integration, Python
@AlenkaF AlenkaF self-assigned this Jul 6, 2023
@github-actions github-actions bot mentioned this issue Jul 6, 2023
Merged
jorisvandenbossche pushed a commit that referenced this issue Jul 7, 2023
@AlenkaF
GH-36498: [Python][CI] Hypothesis nightly test fails with pytz.except…
375f3d9 
…ions.UnknownTimeZoneError: 'Factory' (#36508)

### What changes are included in this PR?

Skip the test if the timezone is `zoneinfo.ZoneInfo(key='Factory')`, because we don't support roundtripping such a timezone.

* Closes: #36498

Authored-by: AlenkaF <frim.alenka@gmail.com>
Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
@jorisvandenbossche jorisvandenbossche added this to the 13.0.0 milestone Jul 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AlenkaF AlenkaF

Labels
Component: Continuous Integration Component: Python Type: bug
Projects
None yet
Milestone
13.0.0
2 participants
@jorisvandenbossche @AlenkaF