Skip to content

Commit

Permalink
ci: run cert-manager scenario with SMI policies enabled (openservicem…
Browse files Browse the repository at this point in the history
…esh#1549)

Only the nosplit integration test is meant to run without traffic
split. This change makes the cert-manager scenario testable with
SMI policies similar to the `tresor` scenario.

Signed-off-by: Shashank Ram <shashank08@gmail.com>
  • Loading branch information
shashankram authored Aug 17, 2020
1 parent 354230d commit 5e71156
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -227,7 +227,7 @@ jobs:
go run ./ci/cmd/maestro.go
integration-nosplit:
name: Integration Test with jetstack/cert-manager, SMI traffic policies (no Traffic Split), and egress disabled
name: Integration Test with jetstack/cert-manager, SMI traffic policies, and egress disabled
runs-on: ubuntu-latest
needs: [build]
steps:
Expand All @@ -254,14 +254,14 @@ jobs:
go-version: 1.14
id: go

- name: Run Simulation w/ jetstack/cert-manager, SMI policies (no Traffic Split), and egress disabled
- name: Run Simulation w/ jetstack/cert-manager, SMI policies, and egress disabled
env:
CERT_MANAGER: "cert-manager" # enables jetstack/cert-manager integration
BOOKSTORE_SVC: "bookstore-v1"
BOOKSTORE_SVC: "bookstore"
BOOKTHIEF_EXPECTED_RESPONSE_CODE: "404"
ENABLE_EGRESS: "false"
EGRESS_EXPECTED_RESPONSE_CODE: "404" # egress is disabled
DEPLOY_TRAFFIC_SPLIT: "false"
DEPLOY_TRAFFIC_SPLIT: "true"
run: |
touch .env
make kind-up
Expand Down

0 comments on commit 5e71156

Please sign in to comment.