Rename someguy to needle, pin ipni/needle v1.0.0 - #1
Merged
Merged
Conversation
The fork shipped as ipni/needle v1.0.0 (NEEDLE_ env prefix, needle_ metric prefix, no aliases). This moves the whole deployment onto it: - git mv roles/someguy -> roles/needle, k8s/someguy -> k8s/needle (previous commit) - every someguy_* Ansible variable to needle_*, every SOMEGUY_* env var to NEEDLE_*, every someguy_ metric reference to needle_ - K8s resources: namespace, ConfigMap needle-env-vars, Deployments needle/-b/-c/-d, PVCs <instance>-data, Envoy cluster and endpoints - image pin to the v1.0.0 index digest (amd64 manifest sha256:f31bd19df98216b6418cb6c007fe5ce2559c7ed430d60bf18c0929e74d5671f3) - README, AGENTS.md and prose in every role; ports unchanged (API 8190-8193, libp2p 4004-4007) Verified: yamllint, ansible-lint (production profile), syntax-check on all playbooks, render-manifests + kubectl kustomize for every build (4 Deployments / 4 PVCs at needle_instances=4, 1/1 at =1), Envoy cluster and endpoints render as needle.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The fork shipped as ipni/needle v1.0.0: NEEDLE_ env prefix, needle_ metric prefix, no aliases (ipni/needle#8). This moves the whole deployment onto it.
Verified locally: yamllint, ansible-lint (production profile), syntax-check on all playbooks, render-manifests + kubectl kustomize for every build (4 Deployments / 4 PVCs at needle_instances=4, 1/1 at =1), Envoy cluster and endpoints render as needle.
NOTE: applying this to a box is a one-shot rename - the old someguy namespace/PVCs must be deleted first (local-path deletes data with the claim) or the new pods lose their address-book snapshots. That is a deliberate part of the cutover, done per box with -l .