Closed mansam closed 3 years ago
@mansam: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.
@nunnatsa FYI
@mansam please take a look at failing test:
Operator resource test Test CRD schemas /home/travis/gopath/src/github.com/kubevirt/vm-import-operator/pkg/operator/resources/operator/operator_test.go:42 [It] Test CRD schemas /home/travis/gopath/src/github.com/kubevirt/vm-import-operator/pkg/operator/resources/operator/operator_test.go:42 • Failure [0.040 seconds]
@pkliczewski That's the erroneous test failure I mentioned in https://github.com/kubevirt/vm-import-operator/pull/446. It's due to operator-utils' GetMissingEntries not handling pointers to metav1.Time. The times in the VirtualMachineImportStatus also generate the same errors, but the test ignores the status subresource so it's not noticed.
Investigating the Prow failure. Retesting to see if happens consistently.
/retest
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: fdupont-redhat, jakub-dzon, mansam, pkliczewski
To complete the pull request process, please assign
You can assign the PR to them by writing /assign
in a comment when ready.
The full list of commands accepted by this bot can be found here.
@pkliczewski That's the erroneous test failure I mentioned in #446. It's due to operator-utils' GetMissingEntries not handling pointers to metav1.Time. The times in the VirtualMachineImportStatus also generate the same errors, but the test ignores the status subresource so it's not noticed.
@jakub-dzon Would mind taking a look ^^
@mansam Please push the new version of images
Signed-off-by: Sam Lucidi slucidi@redhat.com