Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Istio: add gateway-virtualservice link #1674

Merged
merged 1 commit into from
Mar 5, 2019

Conversation

tomerlange
Copy link
Contributor

This PR adds links between gateways and virtualservices.
The relevant tests are TestBookInfoScenario and TestIstioGatewayVirtualServiceScenario, which can be found at tests/istio_test.go.

@tomerlange tomerlange force-pushed the istio-gateway branch 2 times, most recently from ba25b50 to b719d81 Compare February 21, 2019 22:20
@tomerlange
Copy link
Contributor Author

run skydive-functional-tests-backend-orientdb

tests/istio_test.go Outdated Show resolved Hide resolved
tests/istio_test.go Outdated Show resolved Hide resolved
@tomerlange tomerlange force-pushed the istio-gateway branch 4 times, most recently from 210a220 to efb1ae2 Compare February 25, 2019 21:56
@hunchback
Copy link
Collaborator

image

does not show the link connecting the gateway to virtualservice - I believe this is not consistent in all runs - kindly re-check

@hunchback
Copy link
Collaborator

image

does not show the link connecting the gateway to virtualservice - I believe this is not consistent in all runs - kindly re-check

I have checked this and this is not due to a bug in your code - rather this is a general issue effecting also other k8s resources (and possibly physical resources). so given that this issue will be solved outside the context of this PR.

@hunchback hunchback merged commit 6830dcb into skydive-project:master Mar 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants