Closed akalenyu closed 3 months ago
/cc @arnongilboa @alromeros
/hold cancel
/retest
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: awels
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-non-csi-hpp
/cherrypick release-v1.58
@akalenyu: #3293 failed to apply on top of branch "release-v1.58":
Applying: tests: fix regex in import progress test
Applying: Fix actual progress reporting for url->scratch
Using index info to reconstruct a base tree...
M pkg/importer/http-datasource.go
Falling back to patching base and 3-way merge...
Auto-merging pkg/importer/http-datasource.go
CONFLICT (content): Merge conflict in pkg/importer/http-datasource.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0002 Fix actual progress reporting for url->scratch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
/cherrypick release-v1.60
@jpeimer: only kubevirt org members may request cherry picks. If you are already part of the org, make sure to change your membership to public. Otherwise you can still do the cherry-pick manually.
Oh, we actually don't have 1.60 yet
What this PR does / why we need it: Progress reporting has been disabled across the board for a while since we stop using nbdkit/direct raw imports. This PR fixes the issue alongside a flawed test case that was supposed to catch this regression.
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: