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

ci(deps): containerd to v1.7.9 and OpenTelemetry to v1.19.0 / v0.45.0 #4434

Closed

Conversation

santiago-cordoba-form3
Copy link

@santiago-cordoba-form3 santiago-cordoba-form3 commented Nov 22, 2023

Signed-off-by: santiago-cordoba-form3 <santiago.cordoba@form3.tech>
Signed-off-by: santiago-cordoba-form3 <santiago.cordoba@form3.tech>
Signed-off-by: santiago-cordoba-form3 <santiago.cordoba@form3.tech>
Signed-off-by: santiago-cordoba-form3 <santiago.cordoba@form3.tech>
@gerhard
Copy link

gerhard commented Nov 27, 2023

I look forward to getting this merged so that this CVE will get addressed: GHSA-rcjv-mgp8-qvmr

Thank you @crazy-max for picking this one up earlier

Does this PR look good to merge?


cc @sipsma @jedevc

@jedevc
Copy link
Member

jedevc commented Nov 28, 2023

Heya @santiago-cordoba-form3, thanks for the PR!

There's already a discussion for this in #4318 - so I'm going to close this PR in preference of that one, adding more PRs to this just complicates the discussion and spreads it out.

Thanks though! If there's anything else you'd like to help with, PRs are always appreciated 🎉

@jedevc jedevc closed this Nov 28, 2023
@jedevc
Copy link
Member

jedevc commented Nov 28, 2023

@gerhard I've updated #4318 to fix CI and also bump containerd, which should unblock that one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants