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

[skip ci]Add validations for SubnetPort CRD #927

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

lxiaopei
Copy link
Contributor

Add validations for SubnetPort CR to specify one of subnet or subnetSet under spec.

Add validations for SubnetPort CR to specify one of subnet or subnetSet under spec.
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.83%. Comparing base (429879e) to head (71594ca).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           v4.2.0     #927      +/-   ##
==========================================
- Coverage   73.83%   73.83%   -0.01%     
==========================================
  Files         109      109              
  Lines       15252    15247       -5     
==========================================
- Hits        11262    11257       -5     
  Misses       3240     3240              
  Partials      750      750              
Flag Coverage Δ
unit-tests 73.83% <ø> (-0.01%) ⬇️
Files with missing lines Coverage Δ
pkg/apis/vpc/v1alpha1/subnetport_types.go 100.00% <ø> (ø)
...kg/controllers/subnetport/subnetport_controller.go 75.68% <ø> (-0.21%) ⬇️
---- 🚨 Try these New Features:

@lxiaopei
Copy link
Contributor Author

/e2e

1 similar comment
@lxiaopei
Copy link
Contributor Author

/e2e

@lxiaopei lxiaopei changed the title Add validations for SubnetPort CRD [skip ci]Add validations for SubnetPort CRD Nov 22, 2024
@lxiaopei lxiaopei merged commit dee2746 into vmware-tanzu:v4.2.0 Nov 22, 2024
2 of 3 checks 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.

5 participants