Skip to content

ci: Reuse the open plugin catalog PR for each plugin - #256

Merged
vvoytovych merged 1 commit into
mainfrom
ci/reuse-plugin-catalog-pr
Oct 19, 2026
Merged

vvoytovych merged 1 commit into
mainfrom
ci/reuse-plugin-catalog-pr

Conversation

@vvoytovych

Copy link
Copy Markdown
Contributor

Summary

Each release opens a new plugin catalog PR in datum-cloud/datumctl-plugins and leaves the one before it open, so the bump PRs pile up unmerged and users install old plugins. Since v1.22.0 (datum-cloud/actions#98), the shared update-plugin-index action updates the open PR for the plugin instead, and this repository still pins v1.21.0.

Change Why
update-plugin-index@v1.21.0 to @v1.22.0 Each release updates the open catalog PR for its plugin. The action's inputs are unchanged.
A concurrency group on the catalog job, one per catalog, plugin and base branch Every release of a plugin now writes the same PR branch, so two releases close together must not write it at once. The action's README asks callers for this group.

Test plan

  • The next release's catalog job updates the open catalog PR for its plugin instead of opening a new one

Pin the shared update-plugin-index action at v1.22.0, which updates the open catalog PR for a plugin instead of opening one per release (datum-cloud/actions#98). Its inputs are unchanged.
@vvoytovych
vvoytovych requested a review from a team as a code owner October 19, 2026 15:36

@ecv ecv 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.

Yes. Thank you.

@vvoytovych
vvoytovych merged commit 8abbe30 into main Oct 19, 2026
12 checks passed
@vvoytovych
vvoytovych deleted the ci/reuse-plugin-catalog-pr branch October 19, 2026 16:50
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.

2 participants