Repository navigation
Conversation
Refine WorkerPool and trust-bundle prerequisites, preserve installation records, and link related setup documentation. Refs google#487
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
|
@rich7420 the CLA check is failing on your commits. Could you sign the Google CLA at https://cla.developers.google.com/ with the email on your commits? Then it'll go green and we can review. |
|
Thanks for the heads-up, @mayawang. I’ll check the CLA registration for my commit email and follow up. |
|
Thanks, @mayawang. The CLA check now passes. |
|
Thanks @rich7420, and thanks for signing the CLA. One change in direction: we want the README to stay the low-bar path (ideally a single make deploy, and local mode from #504), so could you drop the README changes and link docs/installation.md from CONTRIBUTING.md instead? The installation guide and the docs/runner.md fixes are great as contributor and operator docs. |
|
Thanks, @mayawang. README changes removed; the guide is linked from CONTRIBUTING.md. |
| @@ -0,0 +1,239 @@ | |||
| # Version-pinned installation | |||
|
|
|||
| This source-install recipe uses the development baseline in the | |||
There was a problem hiding this comment.
Given this reads more like a script already, shall we turn it into a target like make deploy?
There was a problem hiding this comment.
Could I take a follow-up for configurable make deploy (snapshot storage, digest-pinned images, and a smoke check), coordinated with #508?
| - Git, Make, Go **1.27.1**, `kubectl`, [ko](https://ko.build/install/), and Docker | ||
| with a builder capable of producing `linux/amd64` images. | ||
| - A Linux/amd64 Kubernetes cluster configured for the pinned Substrate version. | ||
| It must support ClusterTrustBundle projection. Select its kube context before |
There was a problem hiding this comment.
Is it possible to say you need an Agent Substrate with X, Y, Z, in one sentence.
There was a problem hiding this comment.
Condensed the Substrate prerequisites into one sentence, with the setup details linked below.
Opened on behalf of @rich7420 while PR creation is limited to collaborators (#448). All commits are theirs.
Part of #487:
Docs only.