Skip to content
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

Support to create subnetport in subnetset #269

Merged

Conversation

heypnus
Copy link
Contributor

@heypnus heypnus commented Aug 9, 2023

  1. Adding index by subnet CR uid in subnetport store.
  2. Adding function GetPortsOfSubnet in service mediator -- moved several functions from
    service to controller to support it.
  3. Support to create subnetport in subnetset belongs to shared VPC or current namespace.
  4. Updating vsphere-automation-sdk-go/services/nsxt to 0.9.8 to fix the relization
    error for subnetport.
  5. Updating the real value to subnetport.Status.LogicalSwitchID.

@heypnus heypnus force-pushed the vpc/subnetset_port_integration branch 2 times, most recently from 923b058 to e5c00b6 Compare August 9, 2023 10:00
dantingl
dantingl previously approved these changes Aug 10, 2023
@heypnus heypnus force-pushed the vpc/subnetset_port_integration branch 4 times, most recently from c0cb5e0 to 3160a74 Compare August 10, 2023 13:24
1. Adding index by subnet CR uid in subnetport store.
2. Adding function GetPortsOfSubnet in service mediator -- moved several functions from
   service to controller to support it.
3. Support to create subnetport in subnetset belongs to shared VPC or current namespace.
4. Updating vsphere-automation-sdk-go/services/nsxt to 0.9.8 to fix the relization
   error for subnetport.
5. Updating the real value to subnetport.Status.LogicalSwitchID.
@heypnus heypnus force-pushed the vpc/subnetset_port_integration branch from 3160a74 to bcf0b67 Compare August 10, 2023 13:30
@heypnus heypnus merged commit 43f5db3 into vmware-tanzu:vpc_dev Aug 10, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants