Skip to content

Commit 4231961

Browse files
davidgamerodavidgamero
authored
update status checks to always run, add deploytype check for manifests (#477)
1 parent 5bd702b commit 4231961

File tree

2 files changed

+112
-36
lines changed

2 files changed

+112
-36
lines changed

‎.github/workflows/integration-per-language.yml‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -484,8 +484,10 @@ jobs:
484484
curl -m 3 $SERVICEIP:${{env.serviceport}}
485485
sleep 5
486486
kill $tunnelPID
487-
- run: ./draft -v generate-workflow\
488-
--deploy-type manifests\
487+
- run: |
488+
./draft -v generate-workflow \
489+
-d ./langtest/ \
490+
--deploy-type manifests \
489491
--variable WORKFLOWNAME=someWorkflow \
490492
--variable BRANCHNAME=main \
491493
--variable ACRRESOURCEGROUP=someAcrResourceGroup \

‎.github/workflows/integration-tests.yml‎

Lines changed: 108 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -62,46 +62,120 @@ jobs:
6262
name: check_windows_addon_kustomize
6363
path: ./test/check_windows_addon_kustomize.ps1
6464
if-no-files-found: error
65-
language-integration-test:
66-
name: Matrix
65+
gomodule:
6766
needs:
6867
- build-linux
6968
- build-windows
70-
strategy:
71-
matrix:
72-
language: ["gomodule"] # we dont actually want a matrix here, so we include cases individually
73-
repo: ["davidgamero/go_echo"]
74-
include:
75-
- language: "go"
76-
repo: "davidgamero/go-echo-no-mod"
77-
- language: "python"
78-
repo: "OliverMKing/flask-hello-world"
79-
- language: "rust"
80-
repo: "OliverMKing/tiny-http-hello-world"
81-
- language: "javascript"
82-
repo: "davidgamero/express-hello-world"
83-
- language: "ruby"
84-
repo: "davidgamero/sinatra-hello-world"
85-
- language: "csharp"
86-
repo: "imiller31/csharp-simple-web-app"
87-
- language: "java"
88-
repo: "imiller31/simple-java-server"
89-
- language: "gradle"
90-
repo: "imiller31/simple-gradle-server"
91-
- language: "swift"
92-
repo: "OliverMKing/swift-hello-world"
93-
- language: "erlang"
94-
repo: "bfoley13/ErlangExample"
95-
- language: "clojure"
96-
repo: "imiller31/clojure-simple-http"
9769
uses: ./.github/workflows/integration-per-language.yml
9870
with:
99-
language: ${{ matrix.language }}
100-
repo: ${{ matrix.repo }}
101-
linux-integration-summary:
71+
language: gomodule
72+
repo: davidgamero/go_echo
73+
go:
74+
needs:
75+
- build-linux
76+
- build-windows
77+
uses: ./.github/workflows/integration-per-language.yml
78+
with:
79+
language: go
80+
repo: "davidgamero/go-echo-no-mod"
81+
python:
82+
needs:
83+
- build-linux
84+
- build-windows
85+
uses: ./.github/workflows/integration-per-language.yml
86+
with:
87+
language: "python"
88+
repo: "OliverMKing/flask-hello-world"
89+
rust:
90+
needs:
91+
- build-linux
92+
- build-windows
93+
uses: ./.github/workflows/integration-per-language.yml
94+
with:
95+
language: "rust"
96+
repo: "OliverMKing/tiny-http-hello-world"
97+
javascript:
98+
needs:
99+
- build-linux
100+
- build-windows
101+
uses: ./.github/workflows/integration-per-language.yml
102+
with:
103+
language: "javascript"
104+
repo: "davidgamero/express-hello-world"
105+
ruby:
106+
needs:
107+
- build-linux
108+
- build-windows
109+
uses: ./.github/workflows/integration-per-language.yml
110+
with:
111+
language: "ruby"
112+
repo: "davidgamero/sinatra-hello-world"
113+
csharp:
114+
needs:
115+
- build-linux
116+
- build-windows
117+
uses: ./.github/workflows/integration-per-language.yml
118+
with:
119+
language: "csharp"
120+
repo: "imiller31/csharp-simple-web-app"
121+
java:
122+
needs:
123+
- build-linux
124+
- build-windows
125+
uses: ./.github/workflows/integration-per-language.yml
126+
with:
127+
language: "java"
128+
repo: "imiller31/simple-java-server"
129+
gradle:
130+
needs:
131+
- build-linux
132+
- build-windows
133+
uses: ./.github/workflows/integration-per-language.yml
134+
with:
135+
language: "gradle"
136+
repo: "imiller31/simple-gradle-server"
137+
swift:
138+
needs:
139+
- build-linux
140+
- build-windows
141+
uses: ./.github/workflows/integration-per-language.yml
142+
with:
143+
language: "swift"
144+
repo: "OliverMKing/swift-hello-world"
145+
erlang:
146+
needs:
147+
- build-linux
148+
- build-windows
149+
uses: ./.github/workflows/integration-per-language.yml
150+
with:
151+
language: "erlang"
152+
repo: "bfoley13/ErlangExample"
153+
clojure:
154+
needs:
155+
- build-linux
156+
- build-windows
157+
uses: ./.github/workflows/integration-per-language.yml
158+
with:
159+
language: "clojure"
160+
repo: "imiller31/clojure-simple-http"
161+
linux-integration-fail:
102162
name: Integration Test Summary
103-
needs: language-integration-test
104163
runs-on: ubuntu-latest
164+
needs:
165+
- gomodule
166+
- go
167+
- python
168+
- rust
169+
- javascript
170+
- ruby
171+
- csharp
172+
- java
173+
- gradle
174+
- swift
175+
- erlang
176+
- clojure
177+
if: ${{ cancelled() || contains(needs.*.result, 'cancelled') || contains(needs.*.result, 'failure') }} # i cant believe this is necessary https://github.com/actions/runner/issues/2566
105178
steps:
106179
- run: |
107-
echo "Success"
180+
echo "Some workflows have failed!"
181+
exit 1

0 commit comments

Comments
 (0)