PVCs in migration guide looks incorrect

[dkirrane]

In the migration example I can see a couple of issues:

• Client node defines a PVC
• Data node doesn't define a PVC

Also, even there doesn't seem to be a way to disable PVC creation for a Client node. PVC always gets created.

[Crazybus]

Client node defines a PVC

We had an issue discussing this but it looks like it was closed for some reason. Right now it isn't possible to disable the PVC but is as simple as adding a value to disable it. If no one else picks it up I'll get around to it shortly.

#58 (comment)

Data node doesn't define a PVC

It does, although right now it uses the default

helm-charts/elasticsearch/values.yaml

Lines 72 to 76 in edf0517

	volumeClaimTemplate:
	accessModes: [ "ReadWriteOnce" ]
	resources:
	requests:
	storage: 30Gi

values that are specified

I'll make this the new "allow creation of pvcs to be disabled" issue

[Crazybus]

Opened #114 which makes it possible to disable the persistent volume and updated the migration guide.