Closed elfenpiff closed 2 months ago
Attention: Patch coverage is 91.56627%
with 14 lines
in your changes are missing coverage. Please review.
Project coverage is 78.54%. Comparing base (
aeb96a6
) to head (d25b7df
). Report is 1 commits behind head on main.
Can you increase the watchdog timeout of the bitset_tests bit_set_concurrent_set_and_reset_works
test to 100 seconds. It seems the performance in the debug builds on aarch64 is quite low and the test sometimes hits the 60 seconds limit.
Could you also add Corrupted
to the list of expected creation errors in service_tests.rs
. The main branch on Windows failed with that error code.
Can you increase the watchdog timeout of the
bitset_tests bit_set_concurrent_set_and_reset_works
test to 100 seconds. It seems the performance in the debug builds on aarch64 is quite low and the test sometimes hits the 60 seconds limit.Could you also add
Corrupted
to the list of expected creation errors inservice_tests.rs
. The main branch on Windows failed with that error code.
I doubled the watchdog timeout and increased the github CI timeout to 90 minutes since the main branch on windows hit that timeout.
I did not add Corrupted
since it was already listed in the errors - or maybe can you give me a pointer what and where exactly you would like to have?
Notes for Reviewer
Pre-Review Checklist for the PR Author
SPDX-License-Identifier: Apache-2.0 OR MIT
iox2-123-introduce-posix-ipc-example
)[#123] Add posix ipc example
)task-list-completed
)Checklist for the PR Reviewer
Post-review Checklist for the PR Author
References
Closes #173