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

Specify node-type during creating ClusterControlPlane #628

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

wenqiq
Copy link
Contributor

@wenqiq wenqiq commented Jul 8, 2024

No description provided.

@wenqiq wenqiq marked this pull request as ready for review July 8, 2024 07:18
go.mod Outdated Show resolved Hide resolved
pkg/nsx/client.go Outdated Show resolved Hide resolved
Copy link

@edwardbadboy edwardbadboy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A small comment on the variable name.

I think maybe you can invite Bin and Ran to review this patch, too.

pkg/nsx/client.go Outdated Show resolved Hide resolved
@wenqiq wenqiq requested review from liu4480 and gran-vmv July 11, 2024 07:11
go.mod Outdated
github.com/vmware/vsphere-automation-sdk-go/services/nsxt => github.com/zhengxiexie/vsphere-automation-sdk-go/services/nsxt v0.12.3-0.20240628090521-3ad1af047210
github.com/vmware/vsphere-automation-sdk-go/services/nsxt-mp => github.com/zhengxiexie/vsphere-automation-sdk-go/services/nsxt-mp v0.6.3-0.20240628090521-3ad1af047210
github.com/vmware/vsphere-automation-sdk-go/lib => github.com/wenqiq/vsphere-automation-sdk-go/lib v0.7.3-0.20240711034503-a008ff3b1f16
github.com/vmware/vsphere-automation-sdk-go/runtime => github.com/wenqiq/vsphere-automation-sdk-go/runtime v0.7.3-0.20240711034503-a008ff3b1f16
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what change does thei repo include? main branch is using xie's repo, did you fork from xie's repo?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It introduce a new field node_type in cluster-control-plane create request body. Now the change is merged to NSX 9.0.0, but I'm not sure if this change is available in the github.com/vmware/vsphere-automation-sdk-go , because the NSX 9.0.0 release is not GAed yet. So we need this new field in the sdk before NSX GA.

Signed-off-by: Wenqi Qiu <[email protected]>
Signed-off-by: Wenqi Qiu <[email protected]>
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 75.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 73.38%. Comparing base (1cf8cb7) to head (5223056).

Files with missing lines Patch % Lines
pkg/nsx/cluster.go 0.00% 3 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #628      +/-   ##
==========================================
- Coverage   73.38%   73.38%   -0.01%     
==========================================
  Files         109      109              
  Lines       15248    15257       +9     
==========================================
+ Hits        11190    11196       +6     
- Misses       3307     3310       +3     
  Partials      751      751              
Flag Coverage Δ
unit-tests 73.38% <75.00%> (-0.01%) ⬇️
Files with missing lines Coverage Δ
pkg/nsx/client.go 83.42% <ø> (ø)
pkg/nsx/services/nsxserviceaccount/cluster.go 79.90% <100.00%> (+0.28%) ⬆️
pkg/nsx/cluster.go 65.28% <0.00%> (-0.59%) ⬇️

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.

5 participants