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

ci(buildkit): set timeout to image job #4489

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

crazy-max
Copy link
Member

@crazy-max crazy-max commented Dec 14, 2023

related to #4477 (comment)

Image build seems to hang and job timed out after 6h (default on GitHub Actions) for the past few days:

  time="2023-12-11T06:13:24Z" level=debug msg="diff applied" d=2.806054ms digest="sha256:1f5f54dd06e9d17c845fb7c5fdf944c3160d6c4a7a8c645ee8b08e5618c0e65a" media=application/vnd.oci.image.layer.v1.tar+gzip size=942 spanID=62556bbbe3ab4d04 traceID=ca85fd7fcf30de02e721e83aa9c1c2e5
  time="2023-12-11T06:13:24Z" level=debug msg="Using single walk diff for /tmp/buildkit-mount4051339310"
  time="2023-12-11T06:13:24Z" level=debug msg="> creating xiytdlg99ck50ouj0ghtlxhtm [/bin/syft-scanner]" span="[linux/riscv64] generating sbom using docker.io/docker/buildkit-syft-scanner:stable-1" spanID=b1b778eff0601a70 traceID=ca85fd7fcf30de02e721e83aa9c1c2e5

Unfortunately we don't have a stacktrace in BuildKit logs because the worklflow is canceled by the runner.

This change sets a timeout of 15 minutes and send SIGQUIT to the BuildKit container so we can get a stacktrace.

@crazy-max crazy-max marked this pull request as ready for review December 14, 2023 14:49
@crazy-max crazy-max requested a review from tonistiigi December 14, 2023 14:49
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@AkihiroSuda AkihiroSuda merged commit b642b72 into moby:master Dec 15, 2023
60 checks passed
@crazy-max crazy-max deleted the ci-build-timeout branch December 15, 2023 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants