Skip to content

Conversation

@iRaindrop
Copy link
Contributor

@iRaindrop iRaindrop commented Sep 18, 2025

Improve installation guidance

iRaindrop = Bruce Hamilton bhamilton@expertsupport.com

Provide roadmap guidance for installing Knative, make improvements and consistencies.

Proposed Changes

  • Roadmap for installations
  • Improvements for accessing binaries (later push)

Improve installation guidance
@netlify
Copy link

netlify bot commented Sep 18, 2025

Deploy Preview for knative ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 84a9af2
🔍 Latest deploy log https://app.netlify.com/projects/knative/deploys/68d1daebace45e0008cc5fe5
😎 Deploy Preview https://deploy-preview-6395--knative.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@knative-prow knative-prow bot requested review from matzew and skonto September 18, 2025 06:41
@knative-prow knative-prow bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 18, 2025
@iRaindrop iRaindrop marked this pull request as draft September 18, 2025 06:41
@knative-prow knative-prow bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 18, 2025
Copy link
Member

@evankanderson evankanderson left a comment

Choose a reason for hiding this comment

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

I like the presentation. A few details to adjust:

  • We have 3 install options. Quickstart creates a local-only install, while you can either use YAML files or the Operator, depending on how you like to control installation on your cluster.
  • Install should work for all three platforms -- Mac, Windows, and Linux. We assume the target cluster is running Linux, but you can use kind/minikube locally with any platform to get a Linux Kubernetes cluster.

iRaindrop and others added 9 commits September 18, 2025 12:47
link fix

Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>
link fix

Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>
Misc edits
Minor edits
Consolidating CLI installations into this this topic.
Changing red bug alert to important
The old syntax was:
??? bug "Having issues upgrading `kn` to Homebrew?"
Removed alert formatting
Made Evan suggestions, table column test, spelling fixes
Replaced the table with a bulleted list approach.
@knative-prow knative-prow bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Sep 20, 2025
Put back the table
@knative-prow knative-prow bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Sep 20, 2025
Updated topics per effort - consolidating guidance
@knative-prow knative-prow bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Sep 21, 2025
Copy link
Member

@evankanderson evankanderson left a comment

Choose a reason for hiding this comment

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

This is looking quite improved, thank you!

A few notes -- do you have an idea when this will be ready to merge? (In particular, do you have more content you want to add to this PR, or mostly polishing the current contents?)

@iRaindrop
Copy link
Contributor Author

This is looking quite improved, thank you!

A few notes -- do you have an idea when this will be ready to merge? (In particular, do you have more content you want to add to this PR, or mostly polishing the current contents?)

Thanks Evan! Just a few more pushes to warp up this PR with the following:

  • Move the specific 'Install the Knative Operator CLI' section (an H2) from the 'Install Operator with CLI' topic into the 'Install CLI tools' topic.
  • Remove redundant prerequisite info from the Operator Install topics and point to the installation roadmap.
  • The 'Next steps' row is getting too busy so that will become a bulleted list with subitems.

Then in a following PR, I'll add that macro magic to get binaries more easily.

All files added for this PR.
@evankanderson
Copy link
Member

I'd like to merge #6398 soon, which will conflict with this PR. I'm happy to let this PR merge first if it will be done in the next day or so -- otherwise, I may ask you to handle the merge so that I can work on backporting those changes.

@iRaindrop
Copy link
Contributor Author

I'd like to merge #6398 soon, which will conflict with this PR. I'm happy to let this PR merge first if it will be done in the next day or so -- otherwise, I may ask you to handle the merge so that I can work on backporting those changes.

No problem. I'm in touch ups now and will be done within the hour.

@iRaindrop iRaindrop marked this pull request as ready for review September 22, 2025 22:53
@knative-prow knative-prow bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 22, 2025
@knative-prow knative-prow bot requested review from mmejia02 and pierDipi September 22, 2025 22:53
@iRaindrop
Copy link
Contributor Author

OK to merge.

Copy link
Member

@evankanderson evankanderson left a comment

Choose a reason for hiding this comment

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

/approve

I'd like to trim the operator stuff in the client/ section, personally, but I can understand how it got in there, since the operator subcommand is part of the kn plugin experience (as is func, which we don't mention, and a few other plugins).

---

# Install Knative using quickstart
# Install the Knative quickstart plugin
Copy link
Member

Choose a reason for hiding this comment

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

This topic covers both installing the plugin, and using the plugin to get a cluster with Knative installed on it.

Maybe just "quickstart" or "quickstart install"?


- Kubernetes CLI - `kubectl`
- Knative CLI - `kn`
- Knative Operator CLI - `kn`
Copy link
Member

Choose a reason for hiding this comment

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

The operator is part of the Knative CLI, but is primarily focused on administrators who have whole-cluster control. I think this documentation should be focused on developers, and we should leave out the operator bits.


See the [Tekton documentation](http://hub.tekton.dev/tekton/task/kn){target=_blank}.

## Install the Knative Operator CLI Plugin
Copy link
Member

Choose a reason for hiding this comment

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

I'm okay with leaving a note here (though it seems odd that we mention the administrator-focused operator plugin but not the developer-focused func plugin), but this section should make it clear that the operator subcommands are mostly useful for cluster administrators, rather than average developers.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed. Let's find a better place for the Operator content in a subsequent PR.

@knative-prow
Copy link

knative-prow bot commented Sep 22, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: evankanderson, iRaindrop

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow knative-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 22, 2025
iRaindrop and others added 2 commits September 22, 2025 16:22
Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>
Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>
@evankanderson
Copy link
Member

/lgtm

@knative-prow knative-prow bot added the lgtm Indicates that a PR is ready to be merged. label Sep 22, 2025
@knative-prow knative-prow bot merged commit 554d71d into knative:main Sep 22, 2025
18 of 19 checks passed
@iRaindrop iRaindrop deleted the bh_install-updates branch September 22, 2025 23:54
evankanderson added a commit to evankanderson/docs that referenced this pull request Sep 25, 2025
* Installation Doc Updates

Improve installation guidance

* Formatting fix

* Update docs/install/README.md

link fix

Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>

* Update docs/install/README.md

link fix

Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>

* link fix and table update

More writing

* Update README.md

Misc edits

* Update README.md

Minor edits

* Adding install-kn to PR

Consolidating CLI installations into this this topic.

* Update install-kn.md

Changing red bug alert to important
The old syntax was:
??? bug "Having issues upgrading `kn` to Homebrew?"

* Update install-kn (snippet)

Removed alert formatting

* Added quickstart-install.md

Various edits

* Fixes and reviewed edits

Made Evan suggestions, table column test, spelling fixes

* Update quickstart-install.md

link fix

* Update README.md

Replaced the table with a bulleted list approach.

* Update README.md

Put back the table

* Added serving and eventing install topics

Updated topics per effort - consolidating guidance

* Link fixes

* Made Evan's edits

* Various updates

All files added for this PR.

* Link fix

* Formatting fixes

* Formatting and consistency fix

* Update docs/install/operator/knative-with-operator-cli.md

Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>

* Update docs/client/install-kn.md

Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>

---------

Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>
knative-prow bot pushed a commit that referenced this pull request Sep 25, 2025
* Phase 1: Adding documentation metadata tags (#6274)

* Add metadata tags for documentation

* Add classification for about 2/3rds of docs (func + eventing)

* Add classification for remainder of docs (serving+install)

* Add mermaid support (#6327)

* Publish threat model in documentation (#6263)

* Publish threat model in documentation

* Separate security contents a bit more, update link to threat model, update nav

* Add a section on supply chain and SBOM/SLSA mitigation

* Update threat model with feedback from David Hadas

* Update introduction with content from davidhadas, add sections on controller and webhook functionality and update targets of threats

* content tab fixes, added success output for kn func (#6367)

* Add dry run section and take out old feature flag in serving (#6366)

* add dry run section

* drop mention of old feature flag

* fix casing on nav

* update docs to be more clear and include inline example

---------

Co-authored-by: Dave Protasowski <dprotaso@gmail.com>

* Attempt to rebuild docs build process, inspired by #6319 (#6371)

* Attempt to rebuild docs build process

* Use a more modern python version

* Fix strict verify, hide versions on unversioned pages

* Fix search with mkdocs typescript patches (vendored). (#6392)

Hopefully, this can be fixed upstream via PR shortly.

* Installation Doc Updates (#6395)

* Installation Doc Updates

Improve installation guidance

* Formatting fix

* Update docs/install/README.md

link fix

Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>

* Update docs/install/README.md

link fix

Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>

* link fix and table update

More writing

* Update README.md

Misc edits

* Update README.md

Minor edits

* Adding install-kn to PR

Consolidating CLI installations into this this topic.

* Update install-kn.md

Changing red bug alert to important
The old syntax was:
??? bug "Having issues upgrading `kn` to Homebrew?"

* Update install-kn (snippet)

Removed alert formatting

* Added quickstart-install.md

Various edits

* Fixes and reviewed edits

Made Evan suggestions, table column test, spelling fixes

* Update quickstart-install.md

link fix

* Update README.md

Replaced the table with a bulleted list approach.

* Update README.md

Put back the table

* Added serving and eventing install topics

Updated topics per effort - consolidating guidance

* Link fixes

* Made Evan's edits

* Various updates

All files added for this PR.

* Link fix

* Formatting fixes

* Formatting and consistency fix

* Update docs/install/operator/knative-with-operator-cli.md

Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>

* Update docs/client/install-kn.md

Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>

---------

Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>

* Fix edit page links, move technical docs under sub-heading (#6398)

* Fix edit links by moving docs content under a dedicated subdirectory

* Fix edit links by moving docs content under a dedicated subdirectory

* Add High availability documentation section for eventing (#6401)

I have copy-pasted from the Knative Serving documentation page the block as I found it missing when configuring it.

* Update proc-running-function.md (#6400)

Undo separeate kn func output for invoke

* Add a note that Apache Kafka is required to use EKB (#6404)

* Move install docs to administration (#6403)

* Fix trailing newline complaints

* Fix redirects from #6398

---------

Co-authored-by: Bruce Hamilton <scarlettbernique@gmail.com>
Co-authored-by: Alexander-Kita <Alexander.Kita@ibm.com>
Co-authored-by: Dave Protasowski <dprotaso@gmail.com>
Co-authored-by: Aurélien Joga <aurelienjoga+ajoga@gmail.com>
Co-authored-by: Christoph Stäbler <cstabler@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants