Closed akalenyu closed 2 months ago
/cc @alromeros @mhenriks
/test pull-containerized-data-importer-e2e-istio
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mhenriks
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/test pull-containerized-data-importer-e2e-upg
@akalenyu: The following test failed, say
/retest
to rerun all failed tests or/retest-required
to rerun all mandatory failed tests:Test name Commit Details Required Rerun command pull-containerized-data-importer-e2e-hpp-latest deb6d08 link unknown
/test pull-containerized-data-importer-e2e-hpp-latest
this is not good lol
"phase": "ImportScheduled",
"progress": "100.0%",
on the source-dv
/test pull-containerized-data-importer-e2e-hpp-latest
What this PR does / why we need it: These tests relied on tmpfs not supporting direct IO, but that has changed recently with https://github.com/torvalds/linux/commit/e88e0d366f9cfbb810b0c8509dc5d130d5a53e02. Refactored the direct IO utils in a way that allows testing without real filesystems.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged): Fixes #Special notes for your reviewer:
Release note: