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

[AN-425] Only update cromwell chart when merging to Dev #7697

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from
Draft
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
67 changes: 22 additions & 45 deletions .github/workflows/chart_update_on_merge.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ name: chart-update-on-merge

on:
pull_request:
branches: [ develop ]
types:
- closed

Expand All @@ -10,8 +11,19 @@ jobs:
name: Cromwell Version Auto Updater
if: github.event.pull_request.merged == true
runs-on: ubuntu-latest
outputs:
tag: ${{ steps.tag.outputs.tag }}
steps:
- uses: sbt/setup-sbt@v1

- name: Bump the tag to a new version
uses: databiosphere/github-actions/actions/[email protected]
id: tag
env:
DEFAULT_BUMP: patch
GITHUB_TOKEN: ${{ secrets.BROADBOT_GITHUB_TOKEN }}
RELEASE_BRANCHES: develop
WITH_V: true
- name: Clone Cromwell
uses: actions/checkout@v4
with:
Expand Down Expand Up @@ -95,49 +107,14 @@ jobs:
git push https://broadbot:[email protected]/broadinstitute/cromwhelm.git main
cd -

### WX-1836 Steps below here are disabled Azure CI
Copy link
Contributor Author

Choose a reason for hiding this comment

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

since we're deprecating Azure, I thought these could probably go...


- name: (DISABLED) Clone terra-helmfile
uses: actions/checkout@v3
if: false
with:
repository: broadinstitute/terra-helmfile
token: ${{ secrets.BROADBOT_GITHUB_TOKEN }} # Has to be set at checkout AND later when pushing to work
path: terra-helmfile

- name: (DISABLED) Update workflows-app in terra-helmfile
if: false
run: |
set -e
cd terra-helmfile
sed -i "s|image: broadinstitute/cromwell:.*|image: broadinstitute/cromwell:$CROMWELL_VERSION|" charts/workflows-app/values.yaml
cd -

- name: (DISABLED) Update cromwell-runner-app in terra-helmfile
if: false
run: |
set -e
cd terra-helmfile
sed -i "s|image: broadinstitute/cromwell:.*|image: broadinstitute/cromwell:$CROMWELL_VERSION|" charts/cromwell-runner-app/values.yaml
cd -

- name: (DISABLED) Make PR in terra-helmfile
if: false
env:
BROADBOT_TOKEN: ${{ secrets.BROADBOT_GITHUB_TOKEN }}
GH_TOKEN: ${{ secrets.BROADBOT_GITHUB_TOKEN }}
run: |
set -e
JIRA_ID=${{ steps.get-jira-id.outputs.jira-id }}
if [[ $JIRA_ID == "missing" ]]; then
echo "JIRA_ID missing, PR to terra-helmfile will not be created"
exit 0;
fi
cd terra-helmfile
git checkout -b ${JIRA_ID}-cromwell-update-$CROMWELL_VERSION
git config --global user.name "broadbot"
git config --global user.email "[email protected]"
git commit -am "${JIRA_ID}: Auto update Cromwell to $CROMWELL_VERSION in workflows-app and cromwell-runner-app"
git push -u origin ${JIRA_ID}-cromwell-update-$CROMWELL_VERSION
gh pr create --title "${JIRA_ID}: auto update Cromwell version to $CROMWELL_VERSION in workflows-app and cromwell-runner-app" --body "${JIRA_ID} helm chart update" --label "automerge"
cd -
report-to-sherlock:
# Report new cromwell version to Broad DevOps
uses: broadinstitute/sherlock/.github/workflows/client-report-app-version.yaml@main
needs: [ chart-update ]
with:
new-version: ${{ needs.leo-build-tag-publish-job.outputs.tag }}
chart-name: 'leonardo'
permissions:
contents: 'read'
id-token: 'write'
Loading