-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[release-v0.4] fix: Fix owner references in created resources #119
Conversation
Same comment as on #120: Can you put a list of conflicting files and a note explaining the required changes into the commit message? (e.g. what you wrote in the PR description) |
Now the resources will be correctly cleaned up after the VM is deleted. This is a cherry-pick of e1edce9. There were conflicts in the `tests/proxy_test.go` file, because this branch uses older versions of `gomega` and `kubevirt.io/client-go` modules. Signed-off-by: Andrej Krejcir <akrejcir@redhat.com>
ac3e584
to
2420db0
Compare
Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much!
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: 0xFelix The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
/lgtm |
Manual cherry-pick of: #101
The manual change is in the test code, so that it compiles with older
gomega
andkubevirt.io/client-go
.Release note: