Skip to content

Commit fe21912

Browse files
authored
login to ecr public and use a locally build operator for smoke tests in PRs (#111)
* Login to ecr public for PR builds Logs in to ECR public for PR builds. We do this because authenticated image pulls have a much higher rate-limit, so we avoid failing smoke tests on pulling the image * also use a local operator version * build docker images --------- Co-authored-by: Rohan Desai <rohan@responsive.dev>
1 parent aac5fc7 commit fe21912

File tree

1 file changed

+15
-1
lines changed

1 file changed

+15
-1
lines changed

.github/workflows/github-pr.yaml

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,19 @@ jobs:
2020
java-version: '11'
2121
distribution: 'temurin'
2222

23+
- name: Configure AWS Credentials for Public ECR Login
24+
uses: aws-actions/configure-aws-credentials@v2
25+
with:
26+
role-to-assume: arn:aws:iam::292505934682:role/github-responsivedev-org
27+
role-session-name: github-publish-artifacts
28+
aws-region: us-east-1
29+
30+
- name: Login to Amazon ECR Public
31+
id: login-ecr-public
32+
uses: aws-actions/amazon-ecr-login@v1
33+
with:
34+
registry-type: public
35+
2336
- name: Configure AWS Credentials
2437
uses: aws-actions/configure-aws-credentials@v2
2538
with:
@@ -31,7 +44,7 @@ jobs:
3144
uses: gradle/gradle-build-action@v2
3245

3346
- name: Build & Test
34-
run: ./gradlew build kafka-client-examples:simple-example:buildDocker
47+
run: ./gradlew build buildDocker
3548

3649
- uses: actions/checkout@v3
3750
with:
@@ -55,6 +68,7 @@ jobs:
5568

5669
- name: Run Smoke Test
5770
run: |
71+
sindri/scripts/update-versions -u -s responsiveOperator -e local -i responsive-operator:`./gradlew operator:cV | grep Project.version | sed 's/Project version: //'`
5872
sindri/scripts/update-client-versions -s master -p false -i simple-example -t `./gradlew kafka-client:cV | grep "Project version" | sed 's/Project version: //'`
5973
sindri/scripts/run-smoke-test -l -w system-tests-pub-${GITHUB_HEAD_REF} -s master
6074
env:

0 commit comments

Comments
 (0)