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

time_SUITE: skip tz-specific tests when running out of Stockholm #5759

Merged
merged 1 commit into from
Mar 7, 2022

Conversation

max-au
Copy link
Contributor

@max-au max-au commented Mar 1, 2022

These tests always fail when running in any timezone other
than Stockholm.

These tests always fail when running in any timezone other
than Stockholm.
@max-au
Copy link
Contributor Author

max-au commented Mar 1, 2022

I have tested this PR on Linux and Windows in both timezones to ensure test does not get skipped when running in MET/CET, and skipped otherwise.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 1, 2022

CT Test Results

       3 files     125 suites   35m 29s ⏱️
1 432 tests 1 390 ✔️ 42 💤 0
1 730 runs  1 671 ✔️ 59 💤 0

Results for commit 9070492.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@bjorng bjorng self-assigned this Mar 2, 2022
@bjorng bjorng added team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI labels Mar 2, 2022
@bjorng
Copy link
Contributor

bjorng commented Mar 2, 2022

Thanks for pull request! Added to our daily builds.

@bjorng bjorng linked an issue Mar 2, 2022 that may be closed by this pull request
@max-au
Copy link
Contributor Author

max-au commented Mar 4, 2022

I think this PR does not fix y2k_SUITE issue, as it does not touch it at all.

@bjorng
Copy link
Contributor

bjorng commented Mar 7, 2022

Yes, you are right. I have unlinked this PR from the issue.

@bjorng bjorng merged commit 4034ed5 into erlang:master Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants