-
Notifications
You must be signed in to change notification settings - Fork 13
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
Draft for k8s support #66
base: main
Are you sure you want to change the base?
Conversation
…enShift/Kubernetes JUnit5 extension behavior
Skipping CI for Draft Pull Request. |
bdf275f
to
1e5f955
Compare
…rfoil test enabled. Missing parts: docs (limitations and operators based + prerequisited), CI
095071a
to
a660e86
Compare
/test all |
@@ -51,29 +53,30 @@ | |||
*/ | |||
@Slf4j | |||
@CleanBeforeAll | |||
public class OperatorSubscriptionTestCase { | |||
public class OpenShiftOperatorSubscriptionTestCase { |
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.
no @OpenShiftTest
on this one?
import lombok.extern.slf4j.Slf4j; | ||
|
||
@Slf4j | ||
public class Kubernetes extends DefaultKubernetesClient { |
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.
shouldn't the Kubernetes client (io.fabric8.kubernetes.client.KubernetesClient
) be provided through xtf
?
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.
6ba2b66
to
4d3c181
Compare
/test intersmash-e2e |
/test intersmash-e2e |
/test intersmash-e2e |
/test intersmash-e2e |
@fabiobrz: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Description
Fix #133
Type of change
test, version modification, documentation, etc.)
Checklist