Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

examples: Fix up standalone-etcd.yaml #17369

Merged

Conversation

joestringer
Copy link
Member

@joestringer joestringer commented Sep 11, 2021

This YAML was a bit out-of-date, using older etcd and v1beta1 StatefulSet.
Give it a refresh.

Used by:

./contrib/scripts/kind.sh
kubectl apply -f examples/kubernetes/addons/etcd/standalone-etcd.yaml
kubectl get pods
# Wait for the ETCD pod to become ready
cat <<EOF > cilium-etcd.yaml
identityAllocationMode: kvstore
nativeRoutingCIDR: "172.16.0.0/12"
etcd:
  enabled: true
  endpoints:
  - http://$(k get pod etcd-cilium-0 -o json | jq -r '.status.podIP'):32379
EOF
cd /tmp/
helm install cilium cilium/cilium --version 1.9.10 \
  --namespace kube-system -f cilium-etcd.yaml

@joestringer joestringer requested review from a team and nathanjsweet September 11, 2021 00:41
@joestringer joestringer added the release-note/misc This PR makes changes that have no direct user impact. label Sep 11, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Sep 11, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Sep 11, 2021
This YAML was a bit out-of-date, using older etcd and v1beta1 StatefulSet.
Give it a refresh.

Used by:

  ./contrib/scripts/kind.sh
  kubectl apply -f examples/kubernetes/addons/etcd/standalone-etcd.yaml
  kubectl get pods
  # Wait for the ETCD pod to become ready
  cat <<EOF > cilium-etcd.yaml
  identityAllocationMode: kvstore
  nativeRoutingCIDR: "172.16.0.0/12"
  etcd:
    enabled: true
    endpoints:
    - http://$(k get pod etcd-cilium-0 -o json | jq -r '.status.podIP'):32379
  EOF
  helm install cilium cilium/cilium --version 1.9.10 \
    --namespace kube-system -f cilium-etcd.yaml

Signed-off-by: Joe Stringer <joe@cilium.io>
@christarazi christarazi added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Sep 15, 2021
@christarazi
Copy link
Member

Reviews in and CI does not need to be run as the PR is just a change to an example file. Merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants