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

Do not attempt to download lifecycle binary for Windows #1833

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

natalieparellano
Copy link
Contributor

As of lifecycle 0.20.6, Windows support is removed, and no lifecycle binary is available for Windows on GitHub.

Without this change, we'd be unable to upgrade the lifecycle version in kpack beyond 0.20.5.

This change avoids attempting to download the Windows binary, but still produces an empty Windows layer in the lifecycle image, to minimize the surface area of changes.

This is a first step toward deprecating Windows support in kpack (see #1366), which requires further changes throughout the code base to remove Windows-specific logic entirely.

As of lifecycle 0.20.6, Windows support is removed,
and no lifecycle binary is available for Windows on GitHub.

Without this change, we'd be unable to upgrade the lifecycle version in kpack
beyond 0.20.5.

This change avoids attempting to download the Windows binary,
but still produces an empty Windows layer in the lifecycle image,
to minimize the surface area of changes.

This is a first step toward deprecating Windows support in kpack
(see #1366),
which requires further changes throughout the code base
to remove Windows-specific logic entirely.

Signed-off-by: Natalie Arellano <[email protected]>
@natalieparellano
Copy link
Contributor Author

I verified that go run ./hack/lifecycle/main.go creates a pullable image. Below is the Windows layer that gets created - I think this is what we want.

Screenshot 2025-02-26 at 2 19 27 PM

@natalieparellano natalieparellano marked this pull request as ready for review February 26, 2025 19:24
@natalieparellano natalieparellano requested a review from a team as a code owner February 26, 2025 19:24
@tomkennedy513 tomkennedy513 merged commit f5c8a87 into main Feb 26, 2025
3 checks passed
@tomkennedy513 tomkennedy513 deleted the remove-lifecycle-windows branch February 26, 2025 21:29
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