Skip to content

fix: prevent concurrent plugin install conflicts - #794

Merged
linw1995 merged 2 commits into
mainfrom
linw/snp-623-prevent-concurrent-plugin-installation-failures-in-plugin
Aug 6, 2026
Merged

fix: prevent concurrent plugin install conflicts#794
linw1995 merged 2 commits into
mainfrom
linw/snp-623-prevent-concurrent-plugin-installation-failures-in-plugin

Conversation

@linw1995

@linw1995 linw1995 commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Description

  • atomically create and lock shared plugin records before updating installation state
  • keep tenant installations idempotent and plugin reference counts consistent under concurrency
  • cover concurrent installs for the same tenant and across independent tenants

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Performance improvement
  • Other

Essential Checklist

Testing

  • I have tested the changes locally and confirmed they work as expected
  • I have added unit tests where necessary and they pass successfully

Bug Fix (if applicable)

  • I have used GitHub syntax to close the related issue (e.g., Fixes #123 or Closes #123)

Additional Information

Tracked in Linear: SNP-623

@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. bug Something isn't working go Pull requests that update go code labels Aug 6, 2026
@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Aug 6, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Aug 6, 2026
@linw1995
linw1995 merged commit 08e9f62 into main Aug 6, 2026
7 checks passed
@linw1995
linw1995 deleted the linw/snp-623-prevent-concurrent-plugin-installation-failures-in-plugin branch August 6, 2026 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working go Pull requests that update go code 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.

2 participants