Skip to content

fix: race conditions in the plugin logic #4091

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 9 commits into from
Apr 24, 2024
Merged

fix: race conditions in the plugin logic #4091

merged 9 commits into from
Apr 24, 2024

Conversation

Pantani
Copy link
Collaborator

@Pantani Pantani commented Apr 20, 2024 •
edited
Loading

close #4074

Description

Fix race condition in the plugin logic

@Pantani Pantani self-assigned this Apr 20, 2024
@Pantani Pantani changed the title fix: race condition in the plugin logic fix: race conditions in the plugin logic Apr 20, 2024
@github-actions github-actions bot added component:ci CI/CD workflow and automated jobs. component:configs type:services Service-related issues. type:internal component:packages labels Apr 20, 2024
@Pantani Pantani marked this pull request as draft April 20, 2024 01:20
@Pantani Pantani marked this pull request as ready for review April 20, 2024 01:51
@Pantani Pantani enabled auto-merge (squash) April 22, 2024 10:06
julienrbrt
julienrbrt reviewed Apr 22, 2024
View reviewed changes
@Pantani Pantani requested a review from julienrbrt April 22, 2024 12:53
jeronimoalbi
jeronimoalbi previously approved these changes Apr 23, 2024
View reviewed changes
@Pantani Pantani mentioned this pull request Apr 24, 2024
Closed
@Pantani Pantani requested a review from jeronimoalbi April 24, 2024 00:47
@Pantani Pantani merged commit 71750ba into main Apr 24, 2024
@Pantani Pantani deleted the fix/test-race branch April 24, 2024 11:09
julienrbrt pushed a commit that referenced this pull request May 29, 2024
@Pantani @julienrbrt
fix: race conditions in the plugin logic (#4091)
849f000 
* fix race condition for unit tests

* add changelog

* increase TestScaffoldedTests timeout

---------

Co-authored-by: Pantani <Pantani>
@julienrbrt julienrbrt mentioned this pull request Mar 5, 2025
Merged
@julienrbrt julienrbrt mentioned this pull request May 30, 2025
Merged
@BrewTestBot BrewTestBot mentioned this pull request May 31, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeronimoalbi jeronimoalbi jeronimoalbi approved these changes

@ilgooz ilgooz Awaiting requested review from ilgooz ilgooz is a code owner

@salmad3 salmad3 Awaiting requested review from salmad3

@toschdev toschdev Awaiting requested review from toschdev

@julienrbrt julienrbrt Awaiting requested review from julienrbrt julienrbrt is a code owner

Assignees

@Pantani Pantani

Labels
component:ci CI/CD workflow and automated jobs. component:configs type:internal type:services Service-related issues.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Resolve Race Conditions
3 participants
@Pantani @jeronimoalbi @julienrbrt