Skip to content

Commit aaf8e86

Browse files
dpgasparsfirke
authored andcommitted
fix(ci): ephemeral env build and up dependency (apache#26919)
1 parent b2572d9 commit aaf8e86

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

.github/workflows/ephemeral-env.yml

+7-7
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ jobs:
1919
echo "has-secrets=1" >> "$GITHUB_OUTPUT"
2020
fi
2121
22-
ephemeral_env_comment:
22+
ephemeral-env-comment:
2323
needs: config
2424
if: needs.config.outputs.has-secrets
2525
name: Evaluate ephemeral env comment trigger (/testenv)
@@ -80,8 +80,8 @@ jobs:
8080
core.setFailed(errMsg)
8181
8282
ephemeral-docker-build:
83-
needs: ephemeral_env_comment
84-
name: docker-build
83+
needs: ephemeral-env-comment
84+
name: ephemeral-docker-build
8585
runs-on: ubuntu-latest
8686
steps:
8787
- name: Get Info from comment
@@ -148,9 +148,9 @@ jobs:
148148
docker tag $SHA $ECR_REGISTRY/$ECR_REPOSITORY:$SHA
149149
docker push -a $ECR_REGISTRY/$ECR_REPOSITORY
150150
151-
ephemeral_env_up:
152-
needs: ephemeral_env_comment
153-
if: needs.ephemeral_env_comment.outputs.slash-command == 'up'
151+
ephemeral-env-up:
152+
needs: [ephemeral-env-comment, ephemeral-docker-build]
153+
if: needs.ephemeral-env-comment.outputs.slash-command == 'up'
154154
name: Spin up an ephemeral environment
155155
runs-on: ubuntu-latest
156156
permissions:
@@ -207,7 +207,7 @@ jobs:
207207

208208
- name: Update env vars in the Amazon ECS task definition
209209
run: |
210-
cat <<< "$(jq '.containerDefinitions[0].environment += ${{ needs.ephemeral_env_comment.outputs.feature-flags }}' < ${{ steps.task-def.outputs.task-definition }})" > ${{ steps.task-def.outputs.task-definition }}
210+
cat <<< "$(jq '.containerDefinitions[0].environment += ${{ needs.ephemeral-env-comment.outputs.feature-flags }}' < ${{ steps.task-def.outputs.task-definition }})" > ${{ steps.task-def.outputs.task-definition }}
211211
212212
- name: Describe ECS service
213213
id: describe-services

0 commit comments

Comments
 (0)