-
Notifications
You must be signed in to change notification settings - Fork 20
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
Fix AccessMode name in NSX request body #693
Conversation
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
/e2e |
b1fdabe
to
016bc83
Compare
fix subnet e2e test Signed-off-by: Wenqi Qiu <[email protected]>
016bc83
to
e5c3315
Compare
Rename AccessMode `PrivateTGW` to `Private_TGW` in the NSX request body and fix the subnet e2e test Signed-off-by: Wenqi Qiu <[email protected]>
Rename AccessMode `PrivateTGW` to `Private_TGW` in the NSX request body and fix the subnet e2e test Signed-off-by: Wenqi Qiu <[email protected]>
AccessModePrivate string = "Private" | ||
AccessModeProject string = "PrivateTGW" | ||
) | ||
|
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.
Revert these changes?
@@ -1,7 +1,7 @@ | |||
apiVersion: crd.nsx.vmware.com/v1alpha1 | |||
kind: SubnetPort | |||
metadata: | |||
name: port-1 | |||
name: port-e2e-test-1 |
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.
What's the reason to change the name?
Fix AccessMode name in NSX request body when creating subnet and fix subnet e2e test