search

solo-io / glooshot

Chaos engineering framework to help you Immunize your service mesh
https://glooshot.solo.io
52 stars 11 forks source link

bug in tutorial in docs #52

Open rickducott opened 5 years ago

rickducott commented 5 years ago

The tutorial says to install glooshot using this command:

supergloo install istio \
    --namespace glooshot \
    --name istio-istio-system \
    --installation-namespace istio-system \
    --mtls=true \
    --auto-inject=true

This writes the install CRD to the glooshot namespace. However, when the mesh CRD gets created, it gets written to istio-system. Either the docs should specify installing supergloo to the glooshot namespace, so that mesh discovery will write the discovered mesh CRD to glooshot. Or, it should properly specify the namespace for the mesh CRD:

supergloo set mesh stats \
    --target-mesh glooshot.istio-istio-system \
    --prometheus-configmap glooshot.glooshot-prometheus-server
lordofthejars commented 4 years ago

I am having some problems with this too.

I am getting: Error: converting output crd: reading crd spec into v1Mesh: unknown field "meshVersion" in v1.DiscoveryMetadata

mitchdraft commented 4 years ago

Hi @lordofthejars , could you tell me which version of Supergloo you are using? I know the docs say v0.3.18 or greater and that has been found not to be true.

I can confirm that v0.3.18 works, could you try this version?

lordofthejars commented 4 years ago

I am using the latest glooshot and supergloo

mitchdraft commented 4 years ago

ok I think that is the issue, the latest supergloo does not work with glooshot unfortunately