Skip to content

Conversation

@dims
Copy link
Member

@dims dims commented Jun 1, 2021

No description provided.

dims added 2 commits June 1, 2021 18:04
This reverts commit 1acca8b.

Signed-off-by: Davanum Srinivas <[email protected]>
This reverts commit 79e3452.

Signed-off-by: Davanum Srinivas <[email protected]>
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jun 1, 2021

Build succeeded.

@dims dims closed this Jun 1, 2021
@dims dims reopened this Jun 1, 2021
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jun 1, 2021

Build succeeded.

@dims dims closed this Jun 1, 2021
@dims dims reopened this Jun 1, 2021
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jun 1, 2021

Build succeeded.

@dims dims closed this Jun 1, 2021
@dims dims reopened this Jun 1, 2021
@thaJeztah
Copy link
Member

Ah, hmm.. yes, I think #5519 is wrong, as this implementation was explicitly different than the one in libcontainer, because containerd (and moby/docker) need to set up / load the profile, whereas runc only has to apply it.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Jun 1, 2021

Build succeeded.

@dims dims closed this Jun 2, 2021
@dims dims reopened this Jun 2, 2021
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jun 2, 2021

Build succeeded.

Signed-off-by: Davanum Srinivas <[email protected]>
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jun 2, 2021

Build succeeded.

@estesp
Copy link
Member

estesp commented Jun 3, 2021

I assume we are done with this testing PR given the root issue is now fixed?

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