[WIP] e2e multicluster test combined PR set #6639
[WIP] e2e multicluster test combined PR set #6639tiswanso wants to merge 17 commits intoistio:masterfrom
Conversation
|
/test istio-pilot-multicluster-e2e |
Codecov Report
@@ Coverage Diff @@
## master #6639 +/- ##
=======================================
+ Coverage 71% 71% +1%
=======================================
Files 368 366 -2
Lines 31874 31388 -486
=======================================
- Hits 22455 22149 -306
+ Misses 8490 8352 -138
+ Partials 929 887 -42
Continue to review full report at Codecov.
|
|
/test istio-pilot-multicluster-e2e |
There was a problem hiding this comment.
why do we need ingress gateway on the remote?
There was a problem hiding this comment.
I see that the ingressgateway_test.go needs it.
9f346e0 to
27836d9
Compare
|
/test istio-pilot-multicluster-e2e |
2 similar comments
|
/test istio-pilot-multicluster-e2e |
|
/test istio-pilot-multicluster-e2e |
Runs multicluster e2e test. Will use mason to get GKE resources in the same GCP project. The resource_type will need to be updated after the test-infra commit of the boskos config change to create the new resource_type for multiple GKE clusters. Script can be run with pre-allocated clusters by having a pre-setup kubeconfig for multiple clusters. It will choose the pilot cluster to be the first cluster in the get-contexts or the PILOT_CLUSTER env var. example: PILOT_CLUSTER=gke_istio-ci-testing_us-east1-c_cluster-1 \ KUBE_USER=tiswanso@cisco.com ROOT=$(pwd) USE_MASON_RESOURCE=False \ prow/istio-pilot-multicluster-e2e.sh
-Add functions for multi-cluster setup to cluster_lib
gcloud IAM roles seem to have instance-template.* as a higher privilidge than instance.list|get.
- this aligns with current docs & preferred remote cluster config reference.
…mplate - Add istio-ingressgateway service and endpoints option to istio-remote charts - Make the multicluster tests always setup istio-remote via helm template with passed args. - Fix crash in getEndpointIPForService method
beefd57 to
0ee2cac
Compare
|
/test istio-pilot-multicluster-e2e |
0ee2cac to
0a57378
Compare
|
/test istio-pilot-multicluster-e2e |
…figmap
- istio-remote changes remove selectorless service/endpoint setup so DNS
for istio control plane services won't resolve on remote. Sidecar
injection configmap setup on remote uses the IP addresses of services
directly.
- change istioctl in e2e framework to allow use of sidecar inject
configmap on remote rather than default settings.
- enable/disable sidecar injection in remote based on test flag
- misc. fix zipkin address in istio-remote configmap
0a57378 to
cac3c7e
Compare
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: tiswanso Assign the PR to them by writing The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/test istio-pilot-multicluster-e2e |
|
@tiswanso: The following test failed, say
DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
|
This pull request has been automatically marked as stale because it has not had activity in the last 2 weeks. It will be closed in 2 weeks if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions! |
Using this PR to test several PRs together for multicluster.
Testing the following PRs together
#5516
#6454
#6679