Skip to content

bugfix:Restrict Concurrency based on Runs#269

Closed
dharanikesav wants to merge 2 commits intojenkinsci:masterfrom
dharanikesav:master
Closed

bugfix:Restrict Concurrency based on Runs#269
dharanikesav wants to merge 2 commits intojenkinsci:masterfrom
dharanikesav:master

Conversation

@dharanikesav
Copy link
Copy Markdown

We have jenkins WorkFlowJobs that does not run on any specific nodes. When we run those pipelines, "Prevent multiple jobs with identical parameters from running concurrently" option is not working properly. To handle this scenario, have updated the logic to check ongoing builds based on runs in addition to nodes -> executors.

This change is critical for us.

Testing done

  • Have not written any test cases for this

@dharanikesav dharanikesav requested a review from a team as a code owner October 3, 2024 12:50
@dharanikesav dharanikesav marked this pull request as draft October 3, 2024 12:53
@dharanikesav dharanikesav marked this pull request as ready for review October 3, 2024 12:53
@basil
Copy link
Copy Markdown
Member

basil commented Oct 7, 2024

I am not really maintaining this plugin anymore, so feel free to take over:

https://www.jenkins.io/doc/developer/plugin-governance/adopt-a-plugin/

@dharanikesav
Copy link
Copy Markdown
Author

Hi @basil ,
I am happy to take over. Have created a pull request for permission updater repo -> jenkins-infra/repository-permissions-updater#4121

Please verify and approve. So that i can takeover from here.

Thanks,
Dharani

Copy link
Copy Markdown
Member

@basil basil left a comment

Choose a reason for hiding this comment

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

No description of automated or manual testing.

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