Skip to content

chore(ci): Adds workflow to bump protocol buffers#243

Merged
dmihalcik-virtru merged 11 commits intomainfrom
DSPX-959-autobump
Apr 18, 2025
Merged

chore(ci): Adds workflow to bump protocol buffers#243
dmihalcik-virtru merged 11 commits intomainfrom
DSPX-959-autobump

Conversation

@dmihalcik-virtru
Copy link
Copy Markdown
Member

@dmihalcik-virtru dmihalcik-virtru commented Apr 17, 2025

Adds a dependabot-style workflow, update-platform-branch, which will bump the protocol buffer source tag, used during the generate-sources build phase for RPCs with the policy and key access services.

It runs on a schedule to create a dependabot/rennovate style PR, but if you need to bump faster it supports manual dispatch, or, if we get it setup, repository dispatch to make it slightly more automated

@dmihalcik-virtru dmihalcik-virtru requested review from a team as code owners April 17, 2025 15:42
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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

Comments suppressed due to low confidence (1)

.github/workflows/update-platform-branch.yaml:43

  • The sed command creates .bak backup files which might be unintentionally committed; consider removing the backup files after modification or adjusting the command to avoid generating backups.
find . -name "pom.xml" -exec sed -i.bak "s|<platform.branch>.*</platform.branch>|<platform.branch>${TAG}</platform.branch>|g" {} \;

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 introduces a new CI workflow for updating the platform.branch property in pom.xml files and creates a pull request with the changes, while the checks workflow has been updated to reference a new branch.

  • Adds update-platform-branch workflow to bump protocol buffers version in pom.xml files.
  • Updates the test workflow reference in checks.yaml to the new branch.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/update-platform-branch.yaml New workflow for updating platform.branch properties and automating PR creation.
.github/workflows/checks.yaml Updated xtest workflow reference to a branch specific to protogen changes.

Comment thread .github/workflows/update-platform-branch.yaml Outdated
dmihalcik-virtru and others added 2 commits April 17, 2025 13:32
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This reverts commit 2eab556.
Comment thread .github/workflows/update-platform-branch.yaml
elizabethhealy
elizabethhealy previously approved these changes Apr 17, 2025
Copy link
Copy Markdown
Member

@elizabethhealy elizabethhealy left a comment

Choose a reason for hiding this comment

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

other than the above question lgtm

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 introduces a new GitHub workflow to automatically update the platform.branch property in pom.xml files by bumping the protocol buffer tag.

  • Adds a workflow that triggers on schedule, manual dispatch, or via workflow_call.
  • Implements steps to fetch the latest protocol/go tag, check for existing PRs, and update or create a PR accordingly.

Comment thread .github/workflows/update-platform-branch.yaml Outdated
Comment thread .github/workflows/update-platform-branch.yaml
@sonarqubecloud
Copy link
Copy Markdown

@dmihalcik-virtru dmihalcik-virtru merged commit 48e9376 into main Apr 18, 2025
9 checks passed
@dmihalcik-virtru dmihalcik-virtru deleted the DSPX-959-autobump branch April 18, 2025 14:26
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