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

[DEBUG] test sharing image on the host #5776

Draft
wants to merge 11 commits into
base: CCv0
Choose a base branch
from

Conversation

ChengyuZhu6
Copy link
Member

No description provided.

fidencio and others added 9 commits September 22, 2023 14:14
And let's hope for the buest. :-)

Signed-off-by: Fabiano Fidêncio <[email protected]>
nydus-snapshotter / nydusd are required for the CoCo effort.

Signed-off-by: ChengyuZhu6 <[email protected]>
Ensure nydus is setup to be used by containerd as part of our tests.

Signed-off-by: Fabiano Fidêncio <[email protected]>
Signed-off-by: ChengyuZhu6 <[email protected]>
The reason for that being:
```
The test case 5 failed due to a specific behavior when using containerd
and the snapshotter to download images. Containerd needs to fetch both
the manifest and configuration of the image. In this case, both
images`quay.io/kata-containers/confidential-containers:signed` and
`quay.io/kata-containers/confidential-containers:unsigned` have the same
IDs (sha256). Consequently, test case 4 downloaded image
`quay.io/kata-containers/confidential-containers:signed`. So, in test
case 5, when containerd detected that the image ID already existed, it
used the manifest and image name from
`quay.io/kata-containers/confidential-containers:signed` and passed it
to kata instead of
`quay.io/kata-containers/confidential-containers:unsigned`, resulting in
the use of image
`quay.io/kata-containers/confidential-containers:signed`. This explains
the error in test case 5. As a temporary measure, deleting the image
before running each test case should address this.
```
from:
https://cloud-native.slack.com/archives/C039JSH0807/p1695618313572309?thread_ts=1695591000.697989&cid=C039JSH0807

Signed-off-by: Fabiano Fidêncio <[email protected]>
Update image pull tests to provide registry creds on the host
for kubernetes and nydus snapshotter to use

Co-authored-by: ChengyuZhu6 <[email protected]>
Signed-off-by: stevenhorsman <[email protected]>
Signed-off-by: stevenhorsman <[email protected]>
Signed-off-by: stevenhorsman <[email protected]>
@ChengyuZhu6
Copy link
Member Author

/fidencio-test

@ChengyuZhu6
Copy link
Member Author

/fidencio-test

@ChengyuZhu6
Copy link
Member Author

/fidencio-test

@ChengyuZhu6
Copy link
Member Author

/fidencio-test

@ChengyuZhu6
Copy link
Member Author

/fidencio-test

1 similar comment
@ChengyuZhu6
Copy link
Member Author

/fidencio-test

@ChengyuZhu6
Copy link
Member Author

/fidencio-test

Depends-on: github.com/kata-containers/kata-containers#8076

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

/fidencio-test

3 similar comments
@ChengyuZhu6
Copy link
Member Author

/fidencio-test

@ChengyuZhu6
Copy link
Member Author

/fidencio-test

@ChengyuZhu6
Copy link
Member Author

/fidencio-test

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/large Task of significant size
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants