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

io_uring test fails with newer kernel (arm?) #33829

Closed
phlax opened this issue Apr 27, 2024 · 3 comments · Fixed by #33833
Closed

io_uring test fails with newer kernel (arm?) #33829

phlax opened this issue Apr 27, 2024 · 3 comments · Fixed by #33833
Assignees

Comments

@phlax
Copy link
Member

phlax commented Apr 27, 2024

I rebuilt the arm VM image we use to provision CI pools with in azp.

This created an error:

test/common/io/io_uring_impl_test.cc:366: Failure
Expected equality of these values:
  static_cast<char*>(iov1.iov_base)[0]
    Which is: '\0'
  'a'
    Which is: 'a' (97, 0x61)
Stack trace:
  0x988eb8: testing::internal::HandleExceptionsInMethodIfSupported<>()
  0x988d54: testing::Test::Run()
  0x989fc4: testing::TestInfo::Run()
... Google Test internal frames ...

test/common/io/io_uring_impl_test.cc:383: Failure
Expected equality of these values:
  static_cast<char*>(iov3.iov_base)[0]
    Which is: '\0'
  'e'
    Which is: 'e' (101, 0x65)
Stack trace:
  0x988eb8: testing::internal::HandleExceptionsInMethodIfSupported<>()
  0x988d54: testing::Test::Run()
  0x989fc4: testing::TestInfo::Run()
... Google Test internal frames ...

https://dev.azure.com/cncf/envoy/_build/results?buildId=169003&view=logs&j=767be981-567e-57d8-68c3-2140ede0a0bd&t=2181edf2-f610-59f2-c43a-04bb9d0bca00&l=219

in order to workaround the problem we disabled the failing test on arm (#33822)

i suspect this is not actually related to arm and is just the updated VM - ie the same thing will most likely happen when we update the x86 image

@phlax phlax added bug triage Issue requires triage labels Apr 27, 2024
@phlax
Copy link
Member Author

phlax commented Apr 27, 2024

cc @soulxu

@phlax
Copy link
Member Author

phlax commented Apr 27, 2024

the update (to Ubuntu jammy) switched kernel 6.2 -> 6.5

@soulxu
Copy link
Member

soulxu commented Apr 27, 2024

let me check

@soulxu soulxu self-assigned this Apr 27, 2024
@adisuissa adisuissa added arm and removed triage Issue requires triage labels Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants