Skip to content

Commit d854f77

Browse files
authored
Merge pull request #43 from agoric-labs/mfig-ci-stop
2 parents ae52154 + 8cfaec6 commit d854f77

File tree

2 files changed

+3
-13
lines changed

2 files changed

+3
-13
lines changed

.github/workflows/test.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ on:
1010

1111
jobs:
1212
test-with-values:
13+
timeout-minutes: 15
1314
runs-on: ubuntu-latest
1415

1516
steps:
@@ -55,7 +56,6 @@ jobs:
5556
- uses: ./
5657
id: starship
5758
with:
58-
name: test-with-values
5959
config: config.yaml
6060

6161
- name: Check kubectl pods
@@ -87,6 +87,7 @@ jobs:
8787

8888
test-with-remote-cluster:
8989
runs-on: ubuntu-latest
90+
timeout-minutes: 15
9091
steps:
9192

9293
- uses: actions/checkout@v4
@@ -134,7 +135,6 @@ jobs:
134135
- uses: ./
135136
id: starship
136137
with:
137-
name: test-kubeconfig
138138
kubeconfig: ${{ steps.kubeconfig.outputs.content }}
139139
config: config.yaml
140140

@@ -153,6 +153,7 @@ jobs:
153153

154154
test-with-no-ports:
155155
runs-on: ubuntu-latest
156+
timeout-minutes: 15
156157
steps:
157158

158159
- uses: actions/checkout@v4
@@ -175,7 +176,6 @@ jobs:
175176
- uses: ./
176177
id: starship
177178
with:
178-
name: test-no-portforward
179179
config: config.yaml
180180

181181
- name: Check kubectl pods

action.yaml

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,6 @@ inputs:
2424
description: "Helm repo to fetch the chart from (default: https://hyperweb-io.github.io/starship)"
2525
required: false
2626
default: "https://hyperweb-io.github.io/starship"
27-
name:
28-
description: "Helm chart release name for installing helm chart"
29-
required: false
30-
default: "starship"
3127
namespace:
3228
description: "Kubernetes namespace to deploy helm charts on (default: ci-{github.repository}-{github.workflow}-{github.ref} )"
3329
required: false
@@ -41,9 +37,6 @@ outputs:
4137
namespace:
4238
description: "Kubernetes namespace to which helm charts were deployed"
4339
value: ${{ steps.set-namespace.outputs.namespace }}
44-
name:
45-
description: "Helm chart release name for installing helm chart"
46-
value: ${{ inputs.name }}
4740

4841
runs:
4942
using: composite
@@ -146,7 +139,6 @@ runs:
146139
sleep 5
147140
starship start \
148141
--config ${{ inputs.config }} \
149-
--name ${{ inputs.name }} \
150142
--namespace ${{ steps.set-namespace.outputs.namespace }} \
151143
--repoUrl ${{ inputs.repo }} \
152144
--chart ${{ inputs.chart }} \
@@ -179,7 +171,6 @@ runs:
179171
kubectl get pods --namespace ${{ steps.set-namespace.outputs.namespace }}
180172
starship start \
181173
--config ${{ inputs.config }} \
182-
--name ${{ inputs.name }} \
183174
--namespace ${{ steps.set-namespace.outputs.namespace }} \
184175
--repoUrl ${{ inputs.repo }} \
185176
--chart ${{ inputs.chart }} \
@@ -195,7 +186,6 @@ runs:
195186
kubectl get pods --namespace ${{ steps.set-namespace.outputs.namespace }}
196187
starship start \
197188
--config ${{ inputs.config }} \
198-
--name ${{ inputs.name }} \
199189
--namespace ${{ steps.set-namespace.outputs.namespace }} \
200190
--repoUrl ${{ inputs.repo }} \
201191
--chart ${{ inputs.chart }} \

0 commit comments

Comments
 (0)