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: fix docs upstream validation workflow #4343

Merged
merged 1 commit into from
Oct 16, 2023

Conversation

dvdksn
Copy link
Collaborator

@dvdksn dvdksn commented Oct 16, 2023

Signed-off-by: David Karlsson 35727626+dvdksn@users.noreply.github.com

Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
@crazy-max
Copy link
Member

close/open the PR to trigger the workflow that was disabled.

Copy link
Member

@crazy-max crazy-max left a comment

Choose a reason for hiding this comment

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

LGTM

@dvdksn In follow-up could you disable mode=max for gha cache export in the reusable workflow? https://github.com/moby/buildkit/actions/runs/6539953635/job/17759393799#step:6:532

#30 writing layer sha256:dcd3b03f5dc4c271c4a72c70bb3c27afea4359321210ac3d6a05a1ea5fd0711f 1.7s done
#30 writing layer sha256:ebb3fe13ab0e8f69bb60d0955ac4408e2ff7b0a49f75f7d15a98c4242ceb3a06 0.1s done
#30 preparing build cache for export 169.3s done
#30 DONE 169.3s

It takes more time to export cache than running the build.

@crazy-max crazy-max merged commit 2bf6e8f into moby:master Oct 16, 2023
106 of 113 checks passed
@dvdksn dvdksn deleted the fix-docs-upstream-workflow branch October 17, 2023 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants