Skip to content

[Lens] fix toEsql test use all-year UTC timezone#216425

Merged
stratoula merged 1 commit intoelastic:mainfrom
markov00:2025_03_31-fix_dst_esql
Mar 31, 2025
Merged

[Lens] fix toEsql test use all-year UTC timezone#216425
stratoula merged 1 commit intoelastic:mainfrom
markov00:2025_03_31-fix_dst_esql

Conversation

@markov00
Copy link
Contributor

@markov00 markov00 commented Mar 31, 2025

Summary

The test should verify if a timezone that falls under UTC offset can correctly transform a formBased lens configuation to an ESQL query.
Unfortunately the choosen timezone (Europe/London) falls under UTC only when not in DST.
I've selected one of the few countries that falls under UTC all year round to have this test pass correctly independently on when it is executed.

@markov00 markov00 added test Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor labels Mar 31, 2025
@markov00 markov00 requested a review from a team as a code owner March 31, 2025 08:59
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-visualizations (Team:Visualizations)

@stratoula
Copy link
Contributor

Force merging as it unlocks main

@stratoula stratoula merged commit 7bea047 into elastic:main Mar 31, 2025
17 of 19 checks passed
@markov00 markov00 added backport:skip This PR does not require backporting and removed backport:prev-minor labels Mar 31, 2025
@elastic elastic deleted a comment from kibanamachine Mar 31, 2025
@elastic elastic deleted a comment from kibanamachine Mar 31, 2025
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #18 / should filter the table correctly

Metrics [docs]

✅ unchanged

cqliu1 pushed a commit to cqliu1/kibana that referenced this pull request Mar 31, 2025
## Summary

The test should verify if a timezone that falls under UTC offset can
correctly transform a formBased lens configuation to an ESQL query.
Unfortunately the choosen timezone (Europe/London) falls under UTC only
when not in DST.
I've selected one of the few countries that falls under UTC all year
round to have this test pass correctly independently on when it is
executed.
@markov00
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.19

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

markov00 added a commit to markov00/kibana that referenced this pull request Apr 28, 2025
## Summary

The test should verify if a timezone that falls under UTC offset can
correctly transform a formBased lens configuation to an ESQL query.
Unfortunately the choosen timezone (Europe/London) falls under UTC only
when not in DST.
I've selected one of the few countries that falls under UTC all year
round to have this test pass correctly independently on when it is
executed.

(cherry picked from commit 7bea047)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// test v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants