Skip to content

Pf 974 adds new update version workflow#27

Merged
sindrej merged 5 commits into
mainfrom
PF-974-adds-new-update-version-workflow
Dec 21, 2023
Merged

Pf 974 adds new update version workflow#27
sindrej merged 5 commits into
mainfrom
PF-974-adds-new-update-version-workflow

Conversation

@sindrej
Copy link
Copy Markdown
Contributor

@sindrej sindrej commented Dec 21, 2023

Same workflow as https://github.com/felleslosninger/github-workflows-internal/pull/34, just moved it to this repo

  1. Added job summary to help with troubleshooting
  2. Added product-name input to remove wildcard usage in find-and-replace
  3. Improved error messages to help pinpoint issue

This new workflow will be called from each app repo from the existing call-buildimage.yml like so:

call-update-image-version:
    uses: felleslosninger/github-workflows-internal/.github/workflows/update-image-version.yml@PF-974-improve-update-version
    needs: call-workflow-image-build-publish
    with:
      application-name: plattform-test-app
      product-name: plattform
      kubernetes-repo: plattform-test-cd
      image-version: ${{ needs.call-workflow-image-build-publish.outputs.image-version }}
      image-digest: ${{ needs.call-workflow-image-build-publish.outputs.image-digest }}
      image-name: plattform-test-app
      slack-channel-id: "U058DDWGPJT"
      default-environment: systest
    secrets: inherit

Test run: https://github.com/felleslosninger/plattform-test-app/actions/runs/7288360846

Copy link
Copy Markdown
Contributor

@jasaltvik jasaltvik left a comment

Choose a reason for hiding this comment

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

Veldig bra jobba!

@sindrej sindrej merged commit d247d52 into main Dec 21, 2023
@sindrej sindrej deleted the PF-974-adds-new-update-version-workflow branch December 21, 2023 13:36
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