solo-io / sqoop

The GraphQL Engine powered by Gloo
https://sqoop.solo.io
528 stars 42 forks source link

Example in Getting Started is not working. #38

Closed deorbit closed 5 years ago

deorbit commented 5 years ago

Following the steps at https://sqoop.solo.io/getting_started/ on a fresh minikube cluster, the query

{
  pets {
    id
  }
}

gives the following error

 "errors": [
    {
      "message": "executing resolver for field \"pets\": failed executing resolver for Query.pets: performing http post: Post http://127.0.0.1:9090/gloo-system.petstore.Query.pets: dial tcp 127.0.0.1:9090: connect: connection refused"
    }
  ]
}

Envoy logs (kubectl logs -n gloo-system sqoop-cdf94f envoy) reveal the following error repeated many times per second:

[2019-06-11 17:43:51.726][6][warning][config] [bazel-out/k8-opt/bin/external/envoy/source/common/config/_virtual_includes/grpc_mux_subscription_lib/common/config/grpc_mux_subscription_impl.h:70] gRPC config for type.googleapis.com/envoy.api.v2.Listener rejected: Error adding/updating listener sqoop-listener: Unable to parse JSON as proto (INVALID_ARGUMENT:normalize_path: Cannot find field.): {"upgrade_configs":[{"upgrade_type":"websocket"}],"http_filters":[{"name":"envoy.fault"},{"name":"envoy.cors"},{"name":"io.solo.transformation"},{"name":"envoy.router"}],"normalize_path":true,"rds":{"config_source":{"ads":{}},"route_config_name":"sqoop-listener-routes"},"stat_prefix":"http"}
sqoopctl --version
sqoopctl version 0.2.6

minikube version
minikube version: v1.1.0
ilackarms commented 5 years ago

thanks for reporting this issue! it's a real bug. i tested it locally and discovered the problem:

we recently updated the version of Gloo (envoy config server) that sqoop is using without updating the version of Envoy itself.

the quick fix for this is to update the sqoop deployment. the image for the envoy container should be quay.io/solo-io/gloo-envoy-wrapper:0.13.25.

you can quickly update this with the command

kubectl set image -n gloo-system deployment/sqoop envoy=quay.io/solo-io/gloo-envoy-wrapper:0.13.25

however we will definitely need to update our helm chart with the fix. thanks for reporting!