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

Update dependencies - including zap (1.21) #14126

Merged
merged 2 commits into from
Jun 17, 2022

Conversation

ptabor
Copy link
Contributor

@ptabor ptabor commented Jun 16, 2022

Big changes are in separate commits. This is to be stashed on top of #14125.

@ptabor ptabor force-pushed the 20220615-update-grpc3 branch from d66fa53 to 79d0c3b Compare June 16, 2022 13:22
@ahrtr
Copy link
Member

ahrtr commented Jun 17, 2022

I suggest to revisit this PR after 14125 is merged and this PR is rebased afterwards.

@ptabor ptabor marked this pull request as draft June 17, 2022 06:59
@ptabor ptabor force-pushed the 20220615-update-grpc3 branch 3 times, most recently from 9ebfc73 to bb28fd8 Compare June 17, 2022 07:18
Signed-off-by: Piotr Tabor <[email protected]>
@ptabor ptabor force-pushed the 20220615-update-grpc3 branch from bb28fd8 to c5cce5b Compare June 17, 2022 08:05
@ptabor ptabor marked this pull request as ready for review June 17, 2022 08:08
@ptabor ptabor changed the title Update dependencies - including prometheous (1.17) & zap (1.21) Update dependencies - including zap (1.21) Jun 17, 2022
Signed-off-by: Piotr Tabor <[email protected]>
@ptabor ptabor force-pushed the 20220615-update-grpc3 branch from c5cce5b to 88cd0fe Compare June 17, 2022 08:23
Copy link
Member

@ahrtr ahrtr left a comment

Choose a reason for hiding this comment

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

LGTM

@ptabor ptabor merged commit 72b2a06 into etcd-io:main Jun 17, 2022
@ptabor ptabor deleted the 20220615-update-grpc3 branch June 17, 2022 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants