Skip to content

feat(workflows): Add go-sdk auto bump workflow#351

Open
craciunoiuc wants to merge 1 commit into
prod-stagingfrom
craciunoiuc/dispatch_bump_sdk
Open

feat(workflows): Add go-sdk auto bump workflow#351
craciunoiuc wants to merge 1 commit into
prod-stagingfrom
craciunoiuc/dispatch_bump_sdk

Conversation

@craciunoiuc

Copy link
Copy Markdown
Contributor

If we add this, we will get PRs for sdk bumps instantly, instead of waiting 1 week for dependabot to kick in.
Also, dependabot doesn't work with branch bumps I believe.

We need to merge this before we merge the one below to be clean:

Depends on: unikraft-cloud/go-sdk#196

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a GitHub Actions workflow to bump unikraft.com/cloud/sdk in go.mod/go.sum and open a PR against the staging branch, aiming to shorten the time-to-update compared to waiting for Dependabot.

Changes:

  • Introduces a new sync-sdk workflow runnable via workflow_dispatch with a selectable SDK channel (prod-stable / prod-staging).
  • Updates the Go module dependency via go get and conditionally opens a PR (and closes older related bump PRs).

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

Comment thread .github/workflows/sync-sdk.yaml
Comment thread .github/workflows/sync-sdk.yaml Outdated
Comment thread .github/workflows/sync-sdk.yaml
Comment thread .github/workflows/sync-sdk.yaml
@craciunoiuc craciunoiuc force-pushed the craciunoiuc/dispatch_bump_sdk branch from aba12b2 to c0202eb Compare June 3, 2026 10:01
shell: bash
run: |
echo "Bumping Go SDK to latest from ${{ inputs.channel }}..."
go get unikraft.com/cloud/sdk@${{ inputs.channel }}

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.

I think we probably want to make sure that this is set using a commit hash. I know that go get on a branch can sometimes annoyingly cache.

git diff --stat go.mod go.sum || true
fi

- name: Close existing related unmerged bump PRs

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.

No need to do this. peter-evans/create-pull-request will update an existing branch if it's already there 🎉

@@ -0,0 +1,117 @@
name: sync sdk

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.

Suggested change
name: sync sdk
name: sync-sdk

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.

4 participants