ReWrite target annotation not working

[owenmorgan]

Hello, im testing out Istio, but i'm having issues with rewriting requests from ingress.

e.g.

I have Istio running from the Helm chart, all default configuration.

I have a demo application, which accepts requests from root.. so locally i can

docker run -p 80:80 my.registry.com/one-demo

Request: curl localhost/
Response: 200

I have this application running on my k8s, with an ingress record using the rewrite-target to /

apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: one-demo
  namespace: "{{.Values.namespace}}"
  annotations:
    kubernetes.io/ingress.class: istio
    ingress.kubernetes.io/rewrite-target: /
spec:
  rules:
  - http:
      paths:
      - path: /demo
        backend:
          serviceName: one-demo
          servicePort: 8080

Request: curl istiolb.mydomain.com/demo
Response: 404

Looking at the access log for my demo app, i see

request: "GET /demo HTTP/1.1"
Is there a way i can ensure that the request is re-written to the root path?

Cheers

[kyessenov]

Hi, we haven't added support for rewriting paths using this annotation for ingresses directly. What you could do is use a route rule to the backend service and add a rewrite action (see example here https://github.com/istio/api/blob/master/proxy/v1/config/route_rule.proto#L265)

[ldemailly]

we should maybe have a doc/example ?

[tjquinno]

Is this really "won't fix" as suggested by the label, despite being re-opened?

The presence of a workaround (route rules) helps but it's rather awkward for istio to not support the full k8s ingress syntax.

[ldemailly]

This is a reasonable request we should prioritize for 0.3

In the meantime we should more clearly document the rule based option
(and in the meantime you can find a working example of basic prefix based rewrite in
istio/old_pilot_repo#1288 and

istio/tests/e2e/tests/simple/servicesToBeInjected.yaml

Line 42 in 7495ff9

prefix: /fortio/ # prefix

)

[rshriram]

Hi, supporting annotations is not really in Istio's roadmap. Ingress annotation are not part of the official ingress spec. Different controllers have different ways of extending the ingress controller.

[sslavic]

@rshriram thanks for heads up. Wish Ingress Kubernetes documentation like examples using rewrite annotation would include a note about those features not really being part of the Ingress spec, that they are just nginx or other implementation specific. It's also misleading that there's nothing implementation specific in annotation name.

[ldemailly]

there is ongoing work for a better ingress spec for both istio and k8s so there is hope mid term

[ldemailly]

issue should either be closed or open but have "closed/wontfix" label

[Sabiha-Syed]

Hi, we haven't added support for rewriting paths using this annotation for ingresses directly. What you could do is use a route rule to the backend service and add a rewrite action (see example here https://github.com/istio/api/blob/master/proxy/v1/config/route_rule.proto#L265)

https://github.com/istio/api/blob/master/proxy/v1/config/route_rule.proto
This Link isn't working, could you please send the working link.