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

integration-cli: Update plugin test install image error #46902

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dmcgowan
Copy link
Member

@dmcgowan dmcgowan commented Dec 6, 2023

Check for the error returned by the plugin pull logic based off the containerd pull code. This code is used regardless of graphdriver or snapshotter.

Locally this test is failing for me on overlay2 and overlayfs snapshotter. The message which was previously being checked is in the old pull code which the plugin logic moved away from a few years ago. Let's see is CI handles it differently.

Check for the error returned by the plugin pull logic based off the
containerd pull code. This code is used regardless of graphdriver or
snapshotter.

Signed-off-by: Derek McGowan <derek@mcg.dev>
@rumpl rumpl added the containerd-integration Issues and PRs related to containerd integration label Dec 6, 2023
@rumpl rumpl closed this Dec 6, 2023
@rumpl rumpl reopened this Dec 6, 2023
@dmcgowan
Copy link
Member Author

dmcgowan commented Dec 6, 2023

I don't see any failures but I also don't know which job would be running TestDockerPluginSuite/TestPluginInstallImage. It definitely should have been failing before this change as well.

@thaJeztah
Copy link
Member

I don't see any failures but I also don't know which job would be running TestDockerPluginSuite/TestPluginInstallImage. It definitely should have been failing before this change as well.

Heh.... I may know why (😢);

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/testing containerd-integration Issues and PRs related to containerd integration status/2-code-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants