Skip to content

Various Release/Workflow plumbing items#22

Merged
anthonysomerset merged 1 commit into
mainfrom
workflow-plumbing-updates
May 13, 2026
Merged

Various Release/Workflow plumbing items#22
anthonysomerset merged 1 commit into
mainfrom
workflow-plumbing-updates

Conversation

@anthonysomerset
Copy link
Copy Markdown
Collaborator

@anthonysomerset anthonysomerset commented Mar 18, 2026

Fixes #23

A number of small changes to optimise workflows and also to ideally align with how things are done within other repos within the project more generally - this will also simplify requirements for PR creators. e.g. not manually bumping the version and test versions for every single PR only for a release PR that bumps the version

note we likely will have some CI failures

  1. Configuring auto release note generation
  2. Release action no longer a part of PR action
  3. testing CI workflow is in its own workflow to be shared and callable from both release and PR actions
  4. action dependencies updated to current versions and cleaned up

@anthonysomerset anthonysomerset self-assigned this Mar 18, 2026
Comment thread .github/workflows/ci.yaml Fixed
Comment thread .github/workflows/pr.yaml Fixed
@anthonysomerset anthonysomerset marked this pull request as ready for review March 18, 2026 15:28
@anthonysomerset anthonysomerset added the enhancement New feature or request label Mar 18, 2026
@anthonysomerset
Copy link
Copy Markdown
Collaborator Author

This PR fixes #23 by updating the install-helm action to most current

@anthonysomerset anthonysomerset requested a review from rwaffen March 18, 2026 17:28
Copy link
Copy Markdown
Member

@rwaffen rwaffen left a comment

Choose a reason for hiding this comment

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

generally looks ok, but i've got questions :D

Comment thread .github/workflows/ci.yaml
Comment thread .github/workflows/release.yaml
Comment thread .github/workflows/release.yaml Outdated
- name: Make the release
id: create_release
uses: actions/create-release@v1
uses: softprops/action-gh-release@v2.6.1
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why use this action instead of github default one?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

the default one is long "archived" by GH and we want to use the auto-release notes functionality and this action allows for both this and the fact that we upload an asset to release (the packaged chart file) - i wish GH still maintained that create-release and kept it up to date

@anthonysomerset
Copy link
Copy Markdown
Collaborator Author

@rwaffen what questions remain at this point, would like to close this out so that i can start closing out other items

@anthonysomerset anthonysomerset marked this pull request as draft April 29, 2026 16:58
@anthonysomerset anthonysomerset marked this pull request as ready for review April 29, 2026 16:58
@anthonysomerset anthonysomerset requested a review from a team as a code owner April 29, 2026 16:58
@anthonysomerset
Copy link
Copy Markdown
Collaborator Author

@OpenVoxProject/container-maintainers can someone please review this part of this fixes current CI issues blocking closing of other PR's happy to discuss on slack or here more if any queries remain

@anthonysomerset anthonysomerset force-pushed the workflow-plumbing-updates branch from 3493816 to d17ad2d Compare May 13, 2026 17:28
@anthonysomerset anthonysomerset merged commit cf5137a into main May 13, 2026
1 of 2 checks passed
@anthonysomerset anthonysomerset deleted the workflow-plumbing-updates branch May 13, 2026 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: CI is failing due to action failing to install most current helm

4 participants