Skip to content

docs: clarify kind prereqs and fix stale install-script references#32

Open
Davanum Srinivas (dims) wants to merge 1 commit into
agent-substrate:mainfrom
dims:docs/kind-local-dev-prereqs
Open

docs: clarify kind prereqs and fix stale install-script references#32
Davanum Srinivas (dims) wants to merge 1 commit into
agent-substrate:mainfrom
dims:docs/kind-local-dev-prereqs

Conversation

@dims
Copy link
Copy Markdown
Collaborator

  • README Quickstart now pins kind >= v0.31.0. Earlier releases ship a default Kubernetes node image without the PodCertificateRequest API the install scripts depend on, so kube-apiserver never reaches readiness on a fresh brew install kind machine.
  • internal/e2e/readme.md referenced a hack/install.sh that does not exist. Point readers at the two real entry points instead (hack/install-ate.sh for GKE, hack/install-ate-kind.sh for kind).
  • demos/counter/README.md claimed a GCS bucket was required, but the kind overlay provisions an in-cluster rustfs S3 store. Document both targets so the demo can run without provisioning GCP.

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

  • Tests pass
  • Appropriate changes to documentation are included in the PR

@dims Davanum Srinivas (dims) force-pushed the docs/kind-local-dev-prereqs branch from ceef546 to 97ff0d1 Compare May 21, 2026 07:50
- README Quickstart now pins kind >= v0.31.0. Earlier releases ship a
  default Kubernetes node image without the PodCertificateRequest API
  the install scripts depend on, so kube-apiserver never reaches
  readiness on a fresh `brew install kind` machine.
- README Quickstart also documents the 12 GiB Docker Desktop memory
  floor required for the full system bring-up.
- internal/e2e/readme.md referenced a `hack/install.sh` that does not
  exist. Point readers at the two real entry points instead
  (hack/install-ate.sh for GKE, hack/install-ate-kind.sh for kind).
- demos/counter/README.md claimed a GCS bucket was required, but the
  kind overlay provisions an in-cluster rustfs S3 store. Document
  both targets so the demo can run without provisioning GCP.

Signed-off-by: Davanum Srinivas <davanum@gmail.com>
@dims Davanum Srinivas (dims) force-pushed the docs/kind-local-dev-prereqs branch from 97ff0d1 to 6f2fae0 Compare May 21, 2026 08:47
@BenTheElder
Copy link
Copy Markdown
Collaborator

Thanks, one thing: I'd like to eliminate those pre-reqs as much as possible #46

@BenTheElder Benjamin Elder (BenTheElder) added the documentation Improvements or additions to documentation label May 21, 2026
Comment thread README.md

1. Make sure you have [`kind`](https://kind.sigs.k8s.io/), [`kubectl`](https://kubernetes.io/docs/tasks/tools/), and [`docker`](https://www.docker.com/) installed and configured on your dev machine.
1. Make sure the following tools are installed and configured on your development machine:
- [`kind`](https://kind.sigs.k8s.io/) **v0.31.0 or newer** (required for the `PodCertificateRequest` API the install scripts use; matches the CI-pinned `go install sigs.k8s.io/kind@v0.31.0`).
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

not required or used anymore

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Benjamin Elder (@BenTheElder) we still use (

run: go install sigs.k8s.io/kind@v0.31.0
)

should we reflect that? (and drop the note about PodCertificateRequest API)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

If you DIY install we don't inherently depend on kind, but our dev scripts will depend on the version in the tools module in the future, I think we don't need to specify the version here. Thanks.

I also think we will stop depending on the default node image in the future, but I want to ship some upstream kind fixes first.

Comment thread README.md
1. Make sure the following tools are installed and configured on your development machine:
- [`kind`](https://kind.sigs.k8s.io/) **v0.31.0 or newer** (required for the `PodCertificateRequest` API the install scripts use; matches the CI-pinned `go install sigs.k8s.io/kind@v0.31.0`).
- [`kubectl`](https://kubernetes.io/docs/tasks/tools/).
- [`docker`](https://www.docker.com/) with **at least 12 GiB of memory** allocated to the VM (Docker Desktop: Settings → Resources → Memory).
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

are we using 12GiB?

must be the image cache ... we're going to need to think about tiers there

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

12 seemed to work :) i did not try going lower

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hmm OK, I want to avoid overstating it if we can, this is also likely going to be very fluid as we iterate, most of it is probably various in-memory storage currently that doesn't strictly need to all be in memory in the future ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants