lockable-resources-plugin icon indicating copy to clipboard operation
lockable-resources-plugin copied to clipboard

Published 20 hours ago • verified publisher icon jenkinsci

Feature request: configurable priorities

Open mhier opened this issue 5 years ago • 1 comments
trafficstars

Please add a parameter "priority" to the lock() function, which takes an integer number that defines the order in which concurrent jobs waiting for the same resource are served. The job with the highest number would get the resource first. If the priority is equal, the current precedence (first comes first, unless inversePrecedence is true) would be applied.

mhier avatar Apr 27 '20 09:04 mhier

This feature was removed in version 1.7 of this plugin (see https://github.com/jenkinsci/lockable-resources-plugin/commit/a07a64244e3ee2b65a67d5111a6e8aa90c2b3249), long before I took over maintainership. The commit suggests this could be done with external plugins, but I currently see no extension point to facilitate this...

TobiX avatar Apr 27 '20 10:04 TobiX

This feature was removed in version 1.7 of this plugin (see a07a642), long before I took over maintainership. The commit suggests this could be done with external plugins, but I currently see no extension point to facilitate this...

In that case shall be reverted. I will like this feature too.

mPokornyETM avatar Feb 07 '23 15:02 mPokornyETM

See also #223

mPokornyETM avatar Feb 07 '23 15:02 mPokornyETM

@rjarry had you any idea why it was removed?

mPokornyETM avatar Feb 09 '23 22:02 mPokornyETM

This was a long time ago. I think it was to allow using lockable-resources with https://plugins.jenkins.io/PrioritySorter/

rjarry avatar Feb 10 '23 20:02 rjarry

Hey everyone! I'm interested in prioritizing production branches over PRs. This seems to do it, any news?

lppedd avatar May 05 '23 18:05 lppedd

@lppedd Currently not. Let me see what will be possible. Or do you want to participate. Any help are welcome.

mPokornyETM avatar May 06 '23 07:05 mPokornyETM

@mPokornyETM it's just strange this was removed in the commit mentioned above.
The alternative plugins do not do the same at all.

lppedd avatar May 06 '23 09:05 lppedd

@mPokornyETM it's just strange this was removed in the commit mentioned above. The alternative plugins do not do the same at all.

even better, It shall be easy to revert the commit. I hope.

mPokornyETM avatar May 06 '23 12:05 mPokornyETM