Alex Kalenyuk

Use direct io with qemu-img convert if pod is OOMKilled

11

**What this PR does / why we need it**: For a while now we have been switching between cache=none (direct io) and cache=writeback (page cache) for qemu-img's writes. We have...

Clean up leftover snapshot sources from DataImportCron tests

10

**What this PR does / why we need it**: The dataimportcron tests may affect existing crons in the environment (in addition to the ones deployed by testing) so we are...

Update kubevirtci

6

**What this PR does / why we need it**: May help with nfs lane flaking: https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_containerized-data-importer/3017/pull-containerized-data-importer-e2e-nfs/1731266718799499264 Looks like the nfs server is working but mount commands from nfs csi against...

Errors when doing concurrent host assisted clones

8

**What happened**: Host-assisted clone source pods keep erroring when doing concurrent dumb clones. With 2 concurrent clones it seems to converge slowly to success. **What you expected to happen**: restarts...

Investigate WFFC ceph lane being significantly slower than immediate binding lane

5

**What happened**: As can be seen in this comparison: https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_containerized-data-importer/2815/pull-containerized-data-importer-e2e-ceph/1684474065181478912 https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_containerized-data-importer/2815/pull-containerized-data-importer-e2e-ceph-wffc/1684602514835509248 The same set of tests ran against a ceph wffc variation is resulting in a longer runtime for the...

Add goimports to golangci linters

3

**What this PR does / why we need it**: Recently we hit some unexpected changes in commits which could have been caught by goimports: https://github.com/kubevirt/containerized-data-importer/pull/3057#discussion_r1448696072. **Which issue(s) this PR fixes**...

[WIP] Try tackle log flakes by delaying log rotation

28

**What this PR does / why we need it**: Ultimately we probably want to shift away from tests that depend on logs **Which issue(s) this PR fixes** *(optional, in `fixes...

Add test for checking user-facing resources can be manipulated by non-cluster-admin

19

**What this PR does / why we need it**: This should keep us from introducing user-facing resources that cannot be manipulated by non-cluster-admin. **Which issue(s) this PR fixes** *(optional, in...

Common controller doesn't requeue on pvc finalizer remove err

7

**What happened**: The common snapshot controller will not requeue if intermittent errors prevent the removal of the PVC finalizer, such as ``` E1109 05:15:43.755196 1 snapshot_controller.go:191] error check and remove...

Handle lost sparseness in non http data sources

8

**What this PR does / why we need it**: Handle some non-http sources where we lose sparseness similarly to https://github.com/kubevirt/containerized-data-importer/pull/3219 For this to be complete, we must fix the broken...