istio-config-validator

istio-config-validator copied to clipboard

Smeb

Team `getyourguide`! Thank you all for what you've written here! I'm a huge fan. Your tool (and [blog post](https://inside.getyourguide.com/blog/2020/10/30/preventing-traffic-routing-regressions-for-istio-virtual-services)) are addressing the _exact_ problem I'm struggling with right now! While...

bittelc

Hi, For a VS rewrite rule, VS would rewrite the path (or the prefix) portion of the URI with the uri value provided. If the original URI was matched based...

ianrenUipath

Hi, For a VS rule, we may wanna use '', like a wildcard, to match the requests with all uri. But the validator seems not recognize '' in VS.

ianrenUipath

Consider using envoy route check tool

6

https://www.envoyproxy.io/docs/envoy/latest/install/tools/route_table_check_tool It would be pretty interesting to directly use the envoy check tool mentioned above. That should allow much richer/lower maintenance test cases as you can cover all of Istio...

howardjohn

Test coverage

1

To help find untested configurations, it would be helpful to have test coverage reports. This could be a simple synthetic value to start (number of tested routing entries vs total...

x-way

fix(deps): update go minor dependencies updates (auto-merge)

2

This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/envoyproxy/go-control-plane](https://redirect.github.com/envoyproxy/go-control-plane) | `v0.12.1-0.20240719165848-f888b4f71207` -> `v0.13.0` | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/)...

renovate[bot]

fix(deps): update go minor dependencies updates

1

This PR contains the following updates: | Package | Change | Age | Confidence | |---|---|---|---| | [github.com/spf13/cobra](https://redirect.github.com/spf13/cobra) | `v1.9.1` -> `v1.10.1` | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [github.com/stretchr/testify](https://redirect.github.com/stretchr/testify) |...

renovate[bot]