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

replace github.com/cncf/udpa/go by github.com/cncf/xds/go #7001

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

mmorel-35
Copy link
Contributor

@mmorel-35 mmorel-35 commented Feb 23, 2024

replace github.com/cncf/udpa/go by github.com/cncf/xds/go

RELEASE NOTES: none

Signed-off-by: Matthieu MOREL matthieu.morel35@gmail.com

Copy link

codecov bot commented Feb 23, 2024

Codecov Report

Merging #7001 (a2b5080) into master (5ccf176) will decrease coverage by 0.14%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7001      +/-   ##
==========================================
- Coverage   82.43%   82.30%   -0.14%     
==========================================
  Files         296      296              
  Lines       31473    31473              
==========================================
- Hits        25946    25905      -41     
- Misses       4468     4497      +29     
- Partials     1059     1071      +12     
Files Coverage Δ
...ds/internal/xdsclient/xdsresource/unmarshal_lds.go 86.86% <100.00%> (ø)

... and 15 files with indirect coverage changes

@arvindbr8
Copy link
Member

IINM,github.com/cncf/udpa/go is in there for backwards compatibility for management servers that may send responses with deprecated types. @dfawley -- could you please confirm?

@dfawley
Copy link
Member

dfawley commented Feb 27, 2024

It looks like the two are equivalent now?

https://github.com/cncf/udpa/blob/c52dc94e7fbe/go/udpa/type/v1/typed_struct.go#L8

Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
@arvindbr8
Copy link
Member

@mmorel-35 -- thanks for the PR.

@arvindbr8 arvindbr8 merged commit eb880d5 into grpc:master Feb 27, 2024
14 checks passed
@mmorel-35 mmorel-35 deleted the udpa/xds branch February 27, 2024 18:56
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants