Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

tests: Explicitly set the containerd config file version #5353

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

liubin
Copy link
Member

@liubin liubin commented Dec 22, 2022

containerd config version 1 has been deprecated and will be removed in containerd v2.0

Fixes: #5352

Signed-off-by: Bin Liu [email protected]

containerd config version 1 has been deprecated and
will be removed in containerd v2.0

Fixes: kata-containers#5352

Signed-off-by: Bin Liu <[email protected]>
@liubin
Copy link
Member Author

liubin commented Dec 22, 2022

/test

@katacontainersbot katacontainersbot added the size/tiny Smallest and simplest task label Dec 22, 2022
Copy link
Member

@fidencio fidencio left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @liubin!

@fidencio
Copy link
Member

fidencio commented Jan 5, 2023

/test

@liubin
Copy link
Member Author

liubin commented Jan 9, 2023

/test

@GabyCT
Copy link
Contributor

GabyCT commented Jan 9, 2023

/test-ubuntu-pmem

@GabyCT
Copy link
Contributor

GabyCT commented Jan 9, 2023

It seems that containerd does not work properly

ctr: failed to dial "/run/containerd/containerd.sock": context deadline exceeded: connection error: desc = "transport: error while dialing: dial unix:///run/containerd/containerd.sock: timeout"

@liubin
Copy link
Member Author

liubin commented Jan 10, 2023

/test

@GabyCT
Copy link
Contributor

GabyCT commented Jan 10, 2023

/test-ubuntu

@liubin
Copy link
Member Author

liubin commented Jan 11, 2023

/test-vfio

@liubin
Copy link
Member Author

liubin commented Jan 12, 2023

/test

@GabyCT
Copy link
Contributor

GabyCT commented Jan 12, 2023

@liubin could you please test this PR only with /test-ubuntu? It seems that this brokes the metrics and VFIO CI which are baremetal as it impacts containerd and docker functionality. It seems that in all the rest of the CIs the containerd does not start properly.

@dborquez
Copy link
Contributor

/test

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/tiny Smallest and simplest task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Explicitly set the version property for containerd config file.
7 participants