Skip to content

Conversation

@hammadsaedi
Copy link

Updates local build instructions in README.md with correct commands and environment setup needed to successfully build the k3s-upgrade image.

Changes

  • Fix build command: Replace deprecated docker build with docker buildx build including platform specification
  • Add missing prerequisite: Include make download-assets step to download k3s binaries
  • Update build arguments: Add required --build-arg ARTIFACT=k3s parameter
  • Correct repository reference: Change from k3s-io to rancher (proper upstream)
  • Document environment variables: Add clear explanation of ARCH, TAG, VERSION, and REPO variables
  • Update .gitignore: Exclude artifacts/ directory created by make download-assets

Test Plan

  • Updated build commands execute successfully on local environment
  • make download-assets properly downloads k3s artifacts to artifacts/ directory
  • docker buildx build completes without errors using updated parameters
Successful build execution

- Add make download-assets step for artifact preparation
- Update docker build to use buildx with platform specification
- Add required ARTIFACT=k3s build argument
- Update example version from v1.17.2 to v1.33.5+k3s1 throughout
- Fix repository reference from k3s-io to rancher
- Document environment variables (ARCH, TAG, VERSION, REPO)
- Add artifacts/ to .gitignore to exclude downloaded binaries
- Update Plan example version for consistency

Signed-off-by: hammadsaedi <[email protected]>
@hammadsaedi hammadsaedi force-pushed the docs/update-local-build-instructions branch from 7a34e3c to 292c75f Compare November 1, 2025 18: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.

1 participant