Skip to content

Commit b2f7ec0

Browse files
Merge #69
69: docs: Update node onboarding public ECR URL r=jwolski2 a=jwolski2 <!-- Thank you for your Pull Request. Please provide a description above and review the requirements below. Bug fixes and new features should include tests. Contributors guide: https://github.com/NillionNetwork/nillion/blob/master/CONTRIBUTING.md --> ## Motivation <!-- Explain the context and why you're making that change. What is the problem you're trying to solve? In some cases there is not a problem and this can be thought of as being the motivation for your change. --> We were using an obfuscated public ECR name to avoid easy discovery of nilvm Docker images prior to the nilvm repo being open sourced. ## Solution <!-- Summarize the solution and provide any necessary context needed to understand the code change. --> This change updates the references to the obfuscated repo name now that the repo is open. This repo lives in our centralized platform account. We will start to publish release candidates there starting this week. The v0.9.0 image already lives there. Fixes # Design discussion issue (if applicable) # ## Merge requirement checklist * [ ] [CONTRIBUTING](https://github.com/NillionNetwork/nillion/blob/main/CONTRIBUTING.md) guidelines followed * [ ] Unit tests added/updated (if applicable) * [ ] Breaking change analysis completed (if applicable). "Will this change require all network cluster operators to update? Does it break public APIs?" * [ ] For new features or breaking changes, created a documentation issue in [nillion-docs](https://github.com/NillionNetwork/nillion-docs/issues/new/choose) Co-authored-by: Jeff Wolski <[email protected]>
2 parents 5bcbfc4 + 253518a commit b2f7ec0

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

docs/node_onboarding/README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,15 +91,15 @@ be supplied to the node container to ensure authentication against MinIO is poss
9191
## Launch Your Node
9292

9393
Pull the network-compatible version of the node Docker image from Nillion's AWS ECR
94-
[repository](public.ecr.aws/f8o3t8b6/vkv1h9vlo); remember to append `-amd64` to the version number
94+
[repository](public.ecr.aws/k5d9x2g2/nilvm); remember to append `-amd64` to the version number
9595
tag when pulling, e.g.:
9696

9797
> [!IMPORTANT]
9898
> Substitute `${VERSION}`, below, with the Node/SDK version for the corresponding network found in
9999
> the networks list above.
100100

101101
```bash
102-
docker pull public.ecr.aws/f8o3t8b6/vkv1h9vlo:${VERSION}-amd64
102+
docker pull public.ecr.aws/k5d9x2g2/nilvm:${VERSION}-amd64
103103
```
104104

105105
Then, launch the node on your infrastructure using the certificates, Docker image, node
@@ -110,7 +110,7 @@ _This command is not meant to be the exact Docker `run` command to run on your
110110
illustrative for the environment variable and ECR URL._
111111

112112
```bash
113-
docker run -e CONFIG_PATH=/etc/nillion/node.yaml public.ecr.aws/f8o3t8b6/vkv1h9vlo:${VERSION}-amd64
113+
docker run -e CONFIG_PATH=/etc/nillion/node.yaml public.ecr.aws/k5d9x2g2/nilvm:${VERSION}-amd64
114114
```
115115

116116
All nodes present in `cluster.members` must be available before network functions can be validated.

0 commit comments

Comments
 (0)