-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
fix(deps): upgrade go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc dependency #18285
Conversation
Signed-off-by: Justin Marquis <justin@akuity.io>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
@crenshaw-dev Safe to cherry-pick now? |
@34fathombelow you can do it , we already released |
/cherry-pick release-2.11 |
Signed-off-by: Justin Marquis <justin@akuity.io> Co-authored-by: Soumya Ghosh Dastidar <44349253+gdsoumya@users.noreply.github.com>
/cherry-pick release-2.10 |
Signed-off-by: Justin Marquis <justin@akuity.io> Co-authored-by: Soumya Ghosh Dastidar <44349253+gdsoumya@users.noreply.github.com>
/cherry-pick release-2.9 |
Cherry-pick failed with |
Signed-off-by: Justin Marquis <justin@akuity.io> Co-authored-by: Soumya Ghosh Dastidar <44349253+gdsoumya@users.noreply.github.com> Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
Signed-off-by: Justin Marquis <justin@akuity.io> Co-authored-by: Soumya Ghosh Dastidar <44349253+gdsoumya@users.noreply.github.com>
Signed-off-by: Justin Marquis <justin@akuity.io> Co-authored-by: Soumya Ghosh Dastidar <44349253+gdsoumya@users.noreply.github.com>
Signed-off-by: Justin Marquis <justin@akuity.io> Co-authored-by: Soumya Ghosh Dastidar <44349253+gdsoumya@users.noreply.github.com> Signed-off-by: Javier Solana <javier.solana@cabify.com> Signed-off-by: Javier Solana <javier.solana@cabify.com>
Fixes CVE-2023-47108
Opened #18258 to ensure we migrate from
interceptors
tostat handlers
in future versions of Argo CD.Upgraded to the lowest minor version that contained the patch. I had to exclude
StreamServerInterceptor
&UnaryServerInterceptor
from the linter checks to pass the depreciation warnings. Although migrating to the newer functions is preferred, going this route ensures no breaking changes when cherry picking the fix.Upstream has deprecated
StreamServerInterceptor
,UnaryServerInterceptor
, andStreamClientInterceptor
functions but has not been removed as of the latest release (0.51.0).Built and tested with image:
![Screenshot 2024-05-19 at 11 09 13 PM](https://private-user-images.githubusercontent.com/76892343/331961556-39bd6326-19b5-4a85-b20e-235345f09360.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjQ1NDg0MjcsIm5iZiI6MTcyNDU0ODEyNywicGF0aCI6Ii83Njg5MjM0My8zMzE5NjE1NTYtMzliZDYzMjYtMTliNS00YTg1LWIyMGUtMjM1MzQ1ZjA5MzYwLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA4MjUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwODI1VDAxMDg0N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTBlYWYwNTg2NmQwMDNjOGUyOGE4OWRiZWQzY2Q5ZTBhMmZmMzU2NmUyNzljOGIxODBjYjYzMWZiNTUyOWMzMjMmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.lUUfgvxxRp52sM0KE95wlcDDLImQMJHYRo0hhNER4r0)
quay.io/34fathombelow/argocd:latest-otel46.1
cherry-pick
v2.11
,v2.10
, andv2.9
?