Commit bccdccf
committed
feat: Add Istio profile for E2E testing framework
Implements a new Istio profile to test Semantic Router integration with
Istio service mesh, addressing issue #656.
## Changes
### Istio Profile Implementation (e2e/profiles/istio/profile.go)
- Implements Profile interface for Istio integration
- 5-stage deployment:
1. Install Istio control plane via Helm (base, istiod, gateway)
2. Configure namespace with istio-injection label
3. Deploy Semantic Router with automatic sidecar injection
4. Create Istio Gateway, VirtualService, and DestinationRule
5. Verify environment (istiod, gateway, sidecar, service health)
- Helm-based installation (no istioctl required)
- Configurable Istio version via ISTIO_VERSION env var (default: 1.28.0)
- Comprehensive error handling and partial cleanup on failure
### Test Cases (e2e/testcases/istio_*.go)
- istio-sidecar-health-check: Verify Envoy sidecar injection and health
- istio-traffic-routing: Test routing through Istio ingress gateway
- istio-mtls-verification: Verify ISTIO_MUTUAL TLS configuration
- istio-tracing-observability: Check metrics and tracing headers
### Helper Function Refactoring (e2e/pkg/helpers/kubernetes.go)
- Created GetServiceByLabelInNamespace() for generic service lookup
- Maintained GetEnvoyServiceName() as backward-compatible wrapper
- Zero changes required to existing profiles (ai-gateway, aibrix, dynamic-config)
### CI Integration (.github/workflows/integration-test-k8s.yml)
- Added 'istio' to test matrix (runs in parallel with ai-gateway, aibrix)
- No additional dependencies required (Helm-based installation)
### Documentation (e2e/README.md)
- Added Istio profile section with prerequisites, setup steps, and usage
- Documented Helm-based installation approach
- Included troubleshooting guidance
## Design Decisions
**Helm vs istioctl:**
- Chose Helm-based installation for consistency with ai-gateway/aibrix profiles
- No CI dependencies required (istioctl not needed)
- Uses official Istio Helm charts from istio-release.storage.googleapis.com
**LoadBalancer Wait Flag:**
- Disabled Helm --wait for gateway to avoid LoadBalancer timeout in Kind
- Explicit deployment verification via waitForDeployment() instead
**Test Scope:**
- Focused on Istio-specific functionality (sidecar, gateway, mTLS, observability)
- Signal-decision tests intentionally excluded (can be added in follow-up PR)
## Testing
- ✅ Code compiles successfully
- ✅ Helm chart URLs verified accessible (base, istiod, gateway)
- ✅ Istio control plane deploys and runs successfully
- ✅ Sidecar injection and pod health verified
- ✅ Backward compatibility maintained for existing profiles
Resolves #656
Signed-off-by: Asaad Balum <[email protected]>1 parent 13db404 commit bccdccf
File tree
11 files changed
+1522
-10
lines changed- .github/workflows
- e2e
- cmd/e2e
- pkg/helpers
- profiles/istio
- testcases
- tools/make
11 files changed
+1522
-10
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
16 | 16 | | |
17 | 17 | | |
18 | 18 | | |
19 | | - | |
| 19 | + | |
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
17 | | - | |
| 17 | + | |
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
| |||
517 | 517 | | |
518 | 518 | | |
519 | 519 | | |
| 520 | + | |
| 521 | + | |
| 522 | + | |
| 523 | + | |
| 524 | + | |
| 525 | + | |
| 526 | + | |
| 527 | + | |
| 528 | + | |
| 529 | + | |
| 530 | + | |
| 531 | + | |
| 532 | + | |
| 533 | + | |
| 534 | + | |
| 535 | + | |
| 536 | + | |
| 537 | + | |
| 538 | + | |
| 539 | + | |
| 540 | + | |
| 541 | + | |
| 542 | + | |
| 543 | + | |
| 544 | + | |
| 545 | + | |
| 546 | + | |
| 547 | + | |
| 548 | + | |
| 549 | + | |
| 550 | + | |
| 551 | + | |
| 552 | + | |
| 553 | + | |
| 554 | + | |
| 555 | + | |
| 556 | + | |
| 557 | + | |
| 558 | + | |
| 559 | + | |
| 560 | + | |
| 561 | + | |
| 562 | + | |
| 563 | + | |
| 564 | + | |
| 565 | + | |
| 566 | + | |
| 567 | + | |
| 568 | + | |
| 569 | + | |
| 570 | + | |
| 571 | + | |
| 572 | + | |
| 573 | + | |
| 574 | + | |
| 575 | + | |
| 576 | + | |
| 577 | + | |
| 578 | + | |
| 579 | + | |
| 580 | + | |
| 581 | + | |
| 582 | + | |
| 583 | + | |
| 584 | + | |
| 585 | + | |
| 586 | + | |
| 587 | + | |
| 588 | + | |
| 589 | + | |
| 590 | + | |
| 591 | + | |
| 592 | + | |
| 593 | + | |
| 594 | + | |
| 595 | + | |
| 596 | + | |
| 597 | + | |
| 598 | + | |
| 599 | + | |
| 600 | + | |
| 601 | + | |
| 602 | + | |
| 603 | + | |
| 604 | + | |
| 605 | + | |
| 606 | + | |
| 607 | + | |
| 608 | + | |
| 609 | + | |
| 610 | + | |
| 611 | + | |
| 612 | + | |
| 613 | + | |
| 614 | + | |
| 615 | + | |
| 616 | + | |
| 617 | + | |
| 618 | + | |
| 619 | + | |
| 620 | + | |
| 621 | + | |
| 622 | + | |
| 623 | + | |
| 624 | + | |
| 625 | + | |
| 626 | + | |
| 627 | + | |
| 628 | + | |
| 629 | + | |
| 630 | + | |
| 631 | + | |
| 632 | + | |
| 633 | + | |
| 634 | + | |
| 635 | + | |
| 636 | + | |
| 637 | + | |
| 638 | + | |
| 639 | + | |
| 640 | + | |
| 641 | + | |
| 642 | + | |
| 643 | + | |
| 644 | + | |
| 645 | + | |
| 646 | + | |
| 647 | + | |
| 648 | + | |
| 649 | + | |
| 650 | + | |
| 651 | + | |
| 652 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
| 15 | + | |
15 | 16 | | |
16 | 17 | | |
17 | 18 | | |
18 | 19 | | |
| 20 | + | |
19 | 21 | | |
20 | 22 | | |
21 | 23 | | |
| |||
103 | 105 | | |
104 | 106 | | |
105 | 107 | | |
106 | | - | |
107 | | - | |
108 | | - | |
| 108 | + | |
| 109 | + | |
109 | 110 | | |
110 | 111 | | |
111 | 112 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
| 41 | + | |
41 | 42 | | |
42 | | - | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
43 | 49 | | |
44 | 50 | | |
45 | 51 | | |
46 | 52 | | |
47 | 53 | | |
48 | 54 | | |
49 | 55 | | |
50 | | - | |
| 56 | + | |
51 | 57 | | |
52 | 58 | | |
53 | 59 | | |
54 | 60 | | |
55 | 61 | | |
56 | | - | |
| 62 | + | |
57 | 63 | | |
58 | 64 | | |
59 | 65 | | |
| |||
0 commit comments