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

[CI] DateMathExpressionResolverTests.testExpression_CustomTimeZoneInIndexName failure #78783

Closed
rjernst opened this issue Oct 6, 2021 · 0 comments · Fixed by #78796
Closed

[CI] DateMathExpressionResolverTests.testExpression_CustomTimeZoneInIndexName failure #78783

rjernst opened this issue Oct 6, 2021 · 0 comments · Fixed by #78796
Labels
>test-failure Triaged test failures from CI

Comments

@rjernst
Copy link
Member

rjernst commented Oct 6, 2021

Build scan:
https://gradle-enterprise.elastic.co/s/h7kmtpcgqm4mi

Repro line:
./gradlew ':server:test' --tests "org.elasticsearch.cluster.metadata.DateMathExpressionResolverTests.testExpression_CustomTimeZoneInIndexName" -Dtests.seed=9E38B51CB7D2E6F -Dtests.locale=es-CO -Dtests.timezone=Europe/Ulyanovsk -Druntime.java=11

Reproduces locally?:
Yes

Applicable branches:
master

Failure excerpt:

  2> java.time.DateTimeException: Zone offset minutes and seconds must be negative because hours is negative
        at __randomizedtesting.SeedInfo.seed([9E38B51CB7D2E6F:99FD35C51101D220]:0)
        at java.base/java.time.ZoneOffset.validate(ZoneOffset.java:373)
        at java.base/java.time.ZoneOffset.ofHoursMinutesSeconds(ZoneOffset.java:317)
        at java.base/java.time.ZoneOffset.ofHoursMinutes(ZoneOffset.java:300)
        at org.elasticsearch.cluster.metadata.DateMathExpressionResolverTests.testExpression_CustomTimeZoneInIndexName(DateMathExpressionResolverTests.java:140)
@rjernst rjernst added the >test-failure Triaged test failures from CI label Oct 6, 2021
rjernst added a commit that referenced this issue Oct 6, 2021
@rjernst
Must date math test failure
038a82f 
see #78783
@rjernst rjernst mentioned this issue Oct 6, 2021
Merged
rjernst added a commit to rjernst/elasticsearch that referenced this issue Oct 6, 2021
@rjernst
Fix date math zone test to use negative minutes
858a82b 
Joda classes for timezones accepted positive minutes when a negative
hours was used, but Java time expects negative minutes when negative
hours are used. This commit fixes tests to ensure negative minutes are
used with negative hours.

closes elastic#78783
@rjernst rjernst mentioned this issue Oct 6, 2021
Merged
elasticsearchmachine pushed a commit that referenced this issue Oct 6, 2021
@rjernst
Fix date math zone test to use negative minutes (#78796)
d85195b 
Joda classes for timezones accepted positive minutes when a negative
hours was used, but Java time expects negative minutes when negative
hours are used. This commit fixes tests to ensure negative minutes are
used with negative hours.

closes #78783
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
>test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix date math zone test to use negative minutes rjernst/elasticsearch
1 participant
@rjernst