Skip to content

Create GitHub action for bumping patch versions#1259

Merged
efiacor merged 10 commits into
kptdev:mainfrom
nokia:patch-bump-action
Jun 25, 2026
Merged

Create GitHub action for bumping patch versions#1259
efiacor merged 10 commits into
kptdev:mainfrom
nokia:patch-bump-action

Conversation

@mozesl-nokia

Copy link
Copy Markdown
Contributor

Closes kptdev/kpt#4588

Mostly written by Cursor's composer-2.5 with some much needed manual simplifying. Thus, the script itself looks a bit esoteric, so feel free to suggest some more readability improvements!

Copilot AI review requested due to automatic review settings June 12, 2026 09:54
@netlify

netlify Bot commented Jun 12, 2026

Copy link
Copy Markdown

Deploy Preview for krm-function-catalog ready!

Name Link
🔨 Latest commit 6f52e57
🔍 Latest deploy log https://app.netlify.com/projects/krm-function-catalog/deploys/6a3bc71620dc320008a3f45b
😎 Deploy Preview https://deploy-preview-1259--krm-function-catalog.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@dosubot dosubot Bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jun 12, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a manual, workflow-dispatch GitHub Action for performing patch releases of functions/go KRM functions in this catalog, including building/pushing images, tagging, and creating GitHub Releases with generated notes.

Changes:

  • Add scripts/manual-patch-release.sh to bump patch versions based on the latest existing functions/go/<fn>/vX.Y.Z tag, push images, create tags, and publish releases.
  • Add .github/workflows/manual-patch-release.yaml to run the script via workflow_dispatch with function selection and dry-run support.
  • Add list-functions targets to the root Makefile and functions/go/Makefile to enumerate Go function names for validation and bulk release selection.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
scripts/manual-patch-release.sh Implements patch bump resolution, image push, tag creation, and GitHub Release creation for selected functions.
.github/workflows/manual-patch-release.yaml Provides a manual workflow entry point to run the patch-release script in CI with required permissions/tooling.
Makefile Adds a top-level list-functions target used by the release script/workflow.
functions/go/Makefile Adds list-functions target to output the canonical FUNCTIONS list one per line.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Makefile
Comment thread scripts/manual-patch-release.sh Outdated
Comment thread scripts/manual-patch-release.sh Outdated
Comment thread scripts/manual-patch-release.sh Outdated
Comment thread .github/workflows/manual-patch-release.yaml Outdated
Comment thread .github/workflows/manual-patch-release.yaml Outdated
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jun 15, 2026
Comment thread scripts/manual-patch-release.sh
@dosubot dosubot Bot removed the lgtm This PR has been approved by a maintainer label Jun 17, 2026
@mozesl-nokia mozesl-nokia requested a review from Copilot June 18, 2026 11:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

Comment thread scripts/manual-patch-release.sh Outdated
Comment thread scripts/manual-patch-release.sh
Comment thread Makefile
@mozesl-nokia mozesl-nokia added do not merge-hold On hold because some other task must be performed before this PR can be merged. and removed do not merge-hold On hold because some other task must be performed before this PR can be merged. labels Jun 23, 2026
mozesl-nokia and others added 8 commits June 24, 2026 14:00
Signed-off-by: Mózes László Máté <laszlo.mozes@nokia.com>
Assisted-by: Cursor:composer-2.5
Signed-off-by: Mózes László Máté <laszlo.mozes@nokia.com>
Assisted-by: Cursor:composer-2.5
Signed-off-by: mozesl-nokia <laszlo.mozes@nokia.com>

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: mozesl-nokia <laszlo.mozes@nokia.com>

Co-authored-by: Aravindhan Ayyanathan <aravindhan.a@est.tech>
Signed-off-by: Mózes László Máté <laszlo.mozes@nokia.com>
Assisted-by: Cursor:Auto
Signed-off-by: Mózes László Máté <laszlo.mozes@nokia.com>
Assisted-by: Cursor:composer-2.5
Signed-off-by: Mózes László Máté <laszlo.mozes@nokia.com>
Assisted-by: Cursor:composer-2.5
Signed-off-by: mozesl-nokia <laszlo.mozes@nokia.com>

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Mózes László Máté <laszlo.mozes@nokia.com>
Assisted-by: Cursor:Opus-4.7
Signed-off-by: Mózes László Máté <laszlo.mozes@nokia.com>
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jun 25, 2026
@efiacor efiacor merged commit 61b44fa into kptdev:main Jun 25, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GitHub action for bumping up the patch versions of catalog functions

5 participants