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

ECK is leaking services on StatefulSet rename/replacement #1713

Closed
pebrc opened this issue Sep 11, 2019 · 1 comment · Fixed by #1730
Closed

ECK is leaking services on StatefulSet rename/replacement #1713

pebrc opened this issue Sep 11, 2019 · 1 comment · Fixed by #1730
Assignees
Labels
>bug Something isn't working v1.0.0-beta1

Comments

@pebrc
Copy link
Collaborator

pebrc commented Sep 11, 2019

Start with

apiVersion: elasticsearch.k8s.elastic.co/v1alpha1
kind: Elasticsearch
metadata:
  name: hello-eck
spec:
  version: 7.3.0
  nodes:
  - name: default
    nodeCount: 1

upgrade to

apiVersion: elasticsearch.k8s.elastic.co/v1alpha1
kind: Elasticsearch
metadata:
  name: hello-eck
spec:
  version: 7.3.0
  nodes:
  - name: masters
    config:
      node.master: true
      node.data: false
    nodeCount: 3
  - name: data
    config:
      node.master: false
      node.data: true
    nodeCount: 3

See that after a while we have, as expected,

 kubectl get sts
NAME                   DESIRED   CURRENT   AGE
hello-eck-es-data      3         3         11m
hello-eck-es-masters   3         3         11m

but the service sticks around:

kubectl get svc
NAME                   TYPE        CLUSTER-IP    EXTERNAL-IP   PORT(S)    AGE
hello-eck-es-data      ClusterIP   None          <none>        <none>     12m
hello-eck-es-default   ClusterIP   None          <none>        <none>     15m
hello-eck-es-http      ClusterIP   10.99.9.63    <none>        9200/TCP   15m
hello-eck-es-masters   ClusterIP   None          <none>        <none>     12m
@pebrc
Copy link
Collaborator Author

pebrc commented Sep 14, 2019

I realised it is not constrained to services, config secret seems to hang around as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug Something isn't working v1.0.0-beta1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants