-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
testutils: remove gocheck #25684
testutils: remove gocheck #25684
Conversation
/test |
/test |
/test |
/test |
ci-l4lb is #25892 |
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.
Thanks!
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.
Nit. Should these changes be in a separate commit?
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.
You mean, split removing checkmate from testutils from removing IntegrationCheck
?
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.
Yep, exactly (if that makes sense to you)
@tommyp1ckles could you give this PR a quick one over please? |
7b68378
to
7e01406
Compare
/test |
Both CI failures are known flakes. |
7e01406
to
aef811d
Compare
/test Job 'Cilium-PR-K8s-1.26-kernel-net-next' hit: #25958 (89.13% similarity) |
ci-ginkgo is #26009 |
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.
💯 💯 💯
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.
LGTM ✔️
test-1.26-net-next hits #25958 |
/test-1.26-net-next |
/test-ci-ginkgo |
gocheck tests can now call IntegrationTest, migrate them and remove IntegrationCheck. Signed-off-by: Lorenz Bauer <lmb@isovalent.com>
Remove uses of gocheck in favour of testing. Signed-off-by: Lorenz Bauer <lmb@isovalent.com>
aef811d
to
2e1f05f
Compare
/test |
runtime is #26222 |
sorry for the delay - looks good |
Use testing instead of gopkg.in/check.v1. Migrate users of IntegrationCheck to IntegrationTest.