Skip to content

Conversation

@moonchen
Copy link
Contributor

@moonchen moonchen commented Aug 16, 2023

  • Add tests to verify handling a disk failure while initializing cache. You can turn on building these tests with --enable-disk-failure-tests on Autotools, or -DENABLE_DISK_FAILURE_TESTS on CMake.
  • Add tests to verify loading of existing cache volumes.
  • Add a way to inject I/O errors in AIO for tests. This is compiled out by default.

@moonchen moonchen marked this pull request as draft August 21, 2023 15:51
@ezelkow1
Copy link
Member

[approve ci]

@moonchen moonchen marked this pull request as ready for review August 27, 2023 17:06
@bryancall bryancall added this to the 10.0.0 milestone Aug 28, 2023
@bryancall bryancall added the AIO label Aug 28, 2023
@bryancall bryancall requested a review from ywkaras August 28, 2023 22:55
@ywkaras
Copy link
Contributor

ywkaras commented Aug 29, 2023

@moonchen There is a merge conflict.

* Add tests to verify handling a disk failure while initializing cache
* Add tests to verify loading of existing cache volumes
* Add a way to inject I/O errors in AIO for tests.  This is compiled out
  by default.
* parametrically generate single failure tests
* expect disks being marked bad
-Report disk I/O failure synchronously.
-Fix a signed/unsigned bug when checking disk I/O result.
-Rename a couple of symbols for clarity.
@moonchen
Copy link
Contributor Author

@ywkaras Thanks for the heads up. I've resolved the merge conflict. The tests require the following two PRs to pass:

#10317

and

#10286

@moonchen moonchen merged commit 3a5707f into apache:master Sep 5, 2023
cmcfarlen pushed a commit to cmcfarlen/trafficserver that referenced this pull request Jun 3, 2024
* asf/master: (22 commits)
  fix: check whether a protocol is enabled during the length calculation in create_npn_advertisement (apache#10381)
  Coverity 1518612: Remove dead code (apache#10384)
  prefetch_cmcd: make autests more robust by removing need for gold file wildcard (apache#10382)
  Give a chance to send a response before receiving next request on H2 (apache#9997)
  CID 1516688: Fix uninitialized member of AcceptOptions (apache#10152)
  Fix slice head request memory issue (apache#10285)
  Fixes the TSMgmt metrics APIs for new API metrics (apache#10379)
  Minor parent.config a/an change (apache#10372)
  Allow DbgCtl tag to be set after instance construction. (apache#10375)
  Fix more build dep issues, for later PRs to work (apache#10376)
  money_trace cid 1518569: string not null terminated (apache#10373)
  Fix a couple of Coverity issues in health check plugin, around filenames (apache#10371)
  Fixes some build issues that happens with  other changes (apache#10374)
  Eliminate unreachable code covered by switch default (apache#10370)
  Add tests for disk failure (apache#10192)
  Disable copying/moving for DbgCtl. (apache#10321)
  Cmake autest (apache#10327)
  cmake: add unit tests from mgmt/rpc (apache#10366)
  Adjust CMakeLists with git worktree (apache#10298)
  Fix example plugins build (apache#10326)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants