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

Remove images from Docker Hub from tests #3499

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

apostasie
Copy link
Contributor

Fix #3498

@apostasie
Copy link
Contributor Author

Note that this here is highlighting another of our test issues: we do have a number of platform specific files, but we ALSO do mark certain tests as skipped if we are on windows.

Using the two mechanisms simultaneously is problematic, and leading to situations like this here, where we have to add non-sensical definitions.

Suggestion is to stop using platform specific files for tests entirely, and instead just use appropriate Skip.

  • easier to know how many / which tests are being skipped on a certain platform
  • easier to enable / disable a single test for a platform without having to move a bunch of files around

@apostasie
Copy link
Contributor Author

TestRunWithTtyAndDetached #3437

Besides that ^, it is ready.

@AkihiroSuda at your convenience.

@apostasie apostasie marked this pull request as ready for review October 5, 2024 01:25
@AkihiroSuda AkihiroSuda added this to the v2.0.0 milestone Oct 5, 2024
@AkihiroSuda AkihiroSuda added the area/ci e.g., CI failure label Oct 5, 2024
cmd/nerdctl/image/image_list_test.go Outdated Show resolved Hide resolved
@fahedouch
Copy link
Member

Hi @apostasie, there is also a rate limiting on ghcr and everywhere!
A non-expensive solutions would be :

  • to check if image exist in content storage before making a pull otherwise pull
  • cache image list in mem struct for each pull and make a mem check for each pull.

unfortunately there is no flag that does this check in the pull subcommand

@apostasie
Copy link
Contributor Author

apostasie commented Oct 6, 2024

Hi @apostasie, there is also a rate limiting on ghcr and everywhere! A non-expensive solutions would be :

  • to check if image exist in content storage before making a pull otherwise pull
  • cache image list in mem struct for each pull and make a mem check for each pull.

unfortunately there is no flag that does this check in the pull subcommand

At the start of tests, we could spin a local registry and push there the images we need.

Then we could leverage hosts.toml to use the local registry as cache during the tests.

That would limit the number of round trips to the average to once per test suite and per image...

@apostasie
Copy link
Contributor Author

@AkihiroSuda updated to remove the fake "busybox" name and added a comment.

@apostasie
Copy link
Contributor Author

apostasie commented Oct 6, 2024

TestRunWithTtyAndDetached
I never thought I would say that in my life, but I actually want to buy a windows laptop (just to fix this)...

@apostasie
Copy link
Contributor Author

Hi @apostasie, there is also a rate limiting on ghcr and everywhere! A non-expensive solutions would be :

I am sure there is - we just never hit it on the CI, or debugging locally...

@apostasie
Copy link
Contributor Author

Latest failures are the usual - TestIPFSComposeUp and TestRunWithTtyAndDetached

@apostasie apostasie mentioned this pull request Oct 7, 2024
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda AkihiroSuda merged commit bc038d8 into containerd:main Oct 8, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci e.g., CI failure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docker Hub: 429s keep breaking the builds
3 participants