-
Notifications
You must be signed in to change notification settings - Fork 299
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
Legacy api e2e #822
Legacy api e2e #822
Conversation
pkg/e2e/legacy.go
Outdated
} | ||
|
||
func (crud *IngressCRUD) supportsNewAPI() (bool, error) { | ||
apiList, err := crud.C.Discovery().ServerResourcesForGroupVersion("networking/v1beta1") |
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.
Do you need to return error? I don't see anything but nil returned.
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.
done
cmd/e2e-test/draining_test.go
Outdated
ing := fuzz.NewIngressBuilder("", "ingress-1", ""). | ||
AddPath("test.com", "/", "service-1", intstr.FromInt(80)). | ||
Build() | ||
ing.Namespace = s.Namespace |
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.
Could you just set namespace in the builder? Same for other files too.
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.
done
c45afa0
to
786851b
Compare
IngressCRUD makes our e2e tests portable across older k8s API versions. e2e tests have been changed to use IngressCRUD.
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
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bowei, rramkumar1 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 |
Make the e2e framework use an adapter so it can use both the old (extensions) and new (networking) APIs