Skip to content
This repository was archived by the owner on Jul 11, 2023. It is now read-only.

envoy: add helper to convert IP CIDR string to XDS object #4082

Merged
merged 1 commit into from
Sep 8, 2021

Conversation

shashankram
Copy link
Member

Description:
Adds a helper to convert IP CIDR represented as a string
to an XDS CidrRange object, to avoid code duplication.

Affected area:

Functional Area
Control Plane [X]

Please answer the following questions with yes/no.

  1. Does this change contain code from or inspired by another project? no

    • Did you notify the maintainers and provide attribution?
  2. Is this a breaking change? no

Adds a helper to conver IP CIDR represented as a string
to an XDS CidrRange object, to avoid code duplication.

Signed-off-by: Shashank Ram <[email protected]>
@shashankram shashankram requested a review from a team as a code owner September 7, 2021 22:56
@codecov-commenter
Copy link

Codecov Report

Merging #4082 (9f122e3) into main (ed71553) will increase coverage by 0.05%.
The diff coverage is 88.23%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4082      +/-   ##
==========================================
+ Coverage   69.02%   69.08%   +0.05%     
==========================================
  Files         205      205              
  Lines       11249    11242       -7     
==========================================
+ Hits         7765     7766       +1     
+ Misses       3432     3424       -8     
  Partials       52       52              
Flag Coverage Δ
unittests 69.08% <88.23%> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/envoy/lds/ingress.go 81.08% <0.00%> (+6.08%) ⬆️
pkg/envoy/lds/egress.go 80.64% <100.00%> (-1.71%) ⬇️
pkg/envoy/lds/inmesh.go 79.07% <100.00%> (-0.52%) ⬇️
pkg/envoy/xdsutil.go 89.04% <100.00%> (+0.57%) ⬆️
pkg/k8s/announcement_handlers.go 78.37% <0.00%> (+5.40%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ed71553...9f122e3. Read the comment docs.

@shashankram shashankram merged commit 9587d45 into openservicemesh:main Sep 8, 2021
@shashankram shashankram deleted the xds-cidr branch September 8, 2021 17:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants