Skip to content

fix: update README.md for installtion add Dockerfile version and labels#9

Open
chifu1234 wants to merge 2 commits into
mainfrom
dev-kkl
Open

fix: update README.md for installtion add Dockerfile version and labels#9
chifu1234 wants to merge 2 commits into
mainfrom
dev-kkl

Conversation

@chifu1234

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings June 10, 2026 13:47

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the project’s installation guidance and container packaging to support in-cluster plugin installation workflows, while also improving the Docker image’s reproducibility and metadata.

Changes:

  • Update the release-asset name in the install instructions and add in-cluster installation options (initContainer + image volume).
  • Refactor the Dockerfile into multi-stage builds with a pinned BusyBox source image and add OCI labels.
  • Ensure the final image contains only the runtime tools needed for the documented installation approach (cp) plus the /plugins payload.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
README.md Updates install instructions and adds in-cluster deployment examples.
Dockerfile Pins the BusyBox source image, restructures build stages, and adds OCI labels.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
Comment thread README.md Outdated
Signed-off-by: Kevin Klopfenstein <kk@sudo-i.net>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comment thread README.md Outdated
Comment thread Dockerfile

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread Dockerfile
…nd labels

Signed-off-by: Kevin Klopfenstein <kk@sudo-i.net>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

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