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

Remove API code for deprecated DHCP config #981

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

yanjunz97
Copy link
Contributor

@yanjunz97 yanjunz97 commented Dec 31, 2024

The PR remove the old API code for deprecated DHCP config and update the Subnet e2e test with DHCP switch check.

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.

Project coverage is 73.36%. Comparing base (c729a64) to head (51496b3).

Files with missing lines Patch % Lines
pkg/nsx/services/subnetport/builder.go 66.66% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #981      +/-   ##
==========================================
+ Coverage   73.29%   73.36%   +0.06%     
==========================================
  Files         118      118              
  Lines       16397    16363      -34     
==========================================
- Hits        12019    12004      -15     
+ Misses       3591     3577      -14     
+ Partials      787      782       -5     
Flag Coverage Δ
unit-tests 73.36% <85.71%> (+0.06%) ⬆️
Files with missing lines Coverage Δ
pkg/nsx/services/subnet/builder.go 87.50% <100.00%> (+1.29%) ⬆️
pkg/nsx/services/subnet/subnet.go 65.89% <100.00%> (+1.10%) ⬆️
pkg/nsx/services/subnetport/subnetport.go 82.83% <ø> (+0.74%) ⬆️
pkg/nsx/services/subnetport/builder.go 80.00% <66.66%> (+3.48%) ⬆️

Copy link
Contributor

@dantingl dantingl left a comment

Choose a reason for hiding this comment

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

Do we need to update SDK repo or NSX API schema is not changed?

@yanjunz97
Copy link
Contributor Author

Do we need to update SDK repo or NSX API schema is not changed?

NSX API schema is not changed, and they still keep the dhcp_config field in nsx subnet.

@yanjunz97 yanjunz97 merged commit bbdbefb into vmware-tanzu:main Jan 3, 2025
3 checks passed
yanjunz97 added a commit to yanjunz97/nsx-operator that referenced this pull request Jan 3, 2025
The PR removes the old API code for deprecated DHCP config
and updates the Subnet e2e test with DHCP switch check.

Signed-off-by: Yanjun Zhou <[email protected]>
yanjunz97 added a commit that referenced this pull request Jan 3, 2025
The PR removes the old API code for deprecated DHCP config
and updates the Subnet e2e test with DHCP switch check.

Signed-off-by: Yanjun Zhou <[email protected]>
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