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

Fix the kernel config to pass tests in the x86_64 CI Environment #165

Merged
merged 4 commits into from
Oct 29, 2024

Conversation

Code-Hex
Copy link
Owner

@Code-Hex Code-Hex commented Oct 29, 2024

We encountered an issue where tests were failing in x86_64 environments on macOS 11 and later versions.
After investigation, I identified that some EFI-related settings were missing from the configuration of the x86_64 Linux kernel used in the tests. This caused booting issues. Although the tests remain somewhat flaky, I confirmed that they pass if the job is retried.

Additional documentation

CleanShot 2024-10-30 at 00 07 52@2x

@Code-Hex Code-Hex requested a review from AkihiroSuda October 29, 2024 15:08
@Code-Hex Code-Hex changed the title Fix the Kernel config to Pass Tests in the x86_64 CI Environment Fix the Kernel config to pass tests in the x86_64 CI Environment Oct 29, 2024
@Code-Hex Code-Hex changed the title Fix the Kernel config to pass tests in the x86_64 CI Environment Fix the kernel config to pass tests in the x86_64 CI Environment Oct 29, 2024
@Code-Hex
Copy link
Owner Author

Code-Hex commented Oct 29, 2024

Although a few tests have started passing, the tests are still highly flaky 😭

  • The timeout set in actions does not seem to be effective.
  • The failure rate of TestSingleDirectoryShare/read-write seems high.
  • There are instances where tests hang after starting with === RUN Testxxx.

For now, I'll go ahead and merge this PR and continue investigating separately at https://github.com/Code-Hex/vz-ci-test

@Code-Hex Code-Hex merged commit 8aa9759 into main Oct 29, 2024
7 of 8 checks passed
@Code-Hex Code-Hex deleted the fix/ci-kernel branch October 29, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants