Skip to content

Conversation

@chewi
Copy link
Contributor

@chewi chewi commented Jan 7, 2026

What this PR does / why we need it:

Distributions need to be able to build offline using static sources. I didn't bother including the tarball in development releases to save space.

The vendoring is done just before the actual build so that the build can reuse these files, avoiding the need to download them again for each platform.

I have tested this as much as I can locally using act.

Please check the following list:

  • Does the affected code have corresponding tests, e.g. unit test, E2E test? -- N/A
  • Does this change require a documentation update? -- N/A
  • Does this introduce breaking changes that would require an announcement or bumping the major version? -- N/A
  • Do all new files have an appropriate license header? -- N/A

Distributions need to be able to build offline using static sources. I
didn't bother including the tarball in development releases to save
space.

The vendoring is done just before the actual build so that the build can
reuse these files, avoiding the need to download them again for each
platform.

Signed-off-by: James Le Cuirot <[email protected]>
@BigVan BigVan merged commit bd13e08 into containerd:main Jan 13, 2026
7 checks passed
@chewi chewi deleted the vendor branch January 13, 2026 09:28
@chewi
Copy link
Contributor Author

chewi commented Jan 13, 2026

Thank you very much for merging. If you do encounter any issues with this at release time, please let me know.

@chewi
Copy link
Contributor Author

chewi commented Jan 13, 2026

Hmm, was there a problem already? I can't see any actual error message though. 😕

@chewi
Copy link
Contributor Author

chewi commented Jan 13, 2026

Maybe not. This seems to happen when every PR is merged.

@BigVan
Copy link
Member

BigVan commented Jan 13, 2026

I think there is something different between actions/download-artifact@v6 and v4.
I have fixed it in the latest PR

@BigVan
Copy link
Member

BigVan commented Jan 13, 2026

still fails...let me check..😂

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