Skip to content

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

Closed
santiago-cordoba-form3 wants to merge 4 commits intomoby:masterfrom
santiago-cordoba-form3:master
Closed

ci(deps): containerd to v1.7.9 and OpenTelemetry to v1.19.0 / v0.45.0 #4434
santiago-cordoba-form3 wants to merge 4 commits intomoby:masterfrom
santiago-cordoba-form3:master

Conversation

@santiago-cordoba-form3
Copy link
Copy Markdown

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

@santiago-cordoba-form3 santiago-cordoba-form3 marked this pull request as draft November 22, 2023 16:56
@santiago-cordoba-form3 santiago-cordoba-form3 marked this pull request as ready for review November 23, 2023 09:57
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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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.

3 participants