Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WX-1774] Push images during PR check using GHA #341

Merged
merged 13 commits into from
Sep 25, 2024

Conversation

salonishah11
Copy link
Contributor

@salonishah11 salonishah11 commented Sep 23, 2024

Jira: https://broadworkbench.atlassian.net/browse/WX-1774

This PR updates the build GHA to also

  • use standard bumper GHA to generate the next image version. Note as a result of this Agora will use semantic versioning of format v0.0.X similar to other services
  • push the new image to Dockerhub and GCR for each PR commit

Note: these changes are only related to build and push GHA for PR checks. There will be a follow up PR to create new GHA that pushes the images and reports the new version to Sherlock when merge to develop happens.

Example GHA run: https://github.com/broadinstitute/agora/actions/runs/11020083532/job/30604056905?pr=341
New tag generated in above run: v0.0.1-3d4fe7a
Link to v0.0.1-3d4fe7a in Dockerhub
Link to v0.0.1-3d4fe7a in GCR

@salonishah11
Copy link
Contributor Author

jenkins retest

@salonishah11 salonishah11 marked this pull request as ready for review September 24, 2024 18:58
@salonishah11 salonishah11 changed the title [WX-1774] Push SNAP images during PR check using GHA [WX-1774] Push images during PR check using GHA Sep 24, 2024
Comment on lines +20 to +39
- name: Git secrets setup
run: |
git clone https://github.com/awslabs/git-secrets.git ~/git-secrets
cd ~/git-secrets
git checkout ad82d68ee924906a0401dfd48de5057731a9bc84
sudo make install
shell: bash

- name: Secrets check
run: |
sudo ln -s "$(which echo)" /usr/local/bin/say
./minnie-kenny.sh --force
git secrets --scan-history
shell: bash

- name: Setup JDK
uses: actions/setup-java@v4
with:
distribution: temurin
java-version: 11
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't new code. It was already part of renamed docker_build.yml file

@salonishah11 salonishah11 merged commit 6a337ad into develop Sep 25, 2024
7 checks passed
@salonishah11 salonishah11 deleted the sps_build_push_PR_GHA branch September 25, 2024 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants