Allow to use Jenkins label parser to allocate lockable-resources in more sophisticated way#475
Allow to use Jenkins label parser to allocate lockable-resources in more sophisticated way#475mPokornyETM merged 11 commits intojenkinsci:masterfrom mPokornyETM:use-jenkins-labels-parser
Conversation
|
@g3n35i5 do you want to provide review? |
|
@mPokornyETM The PR looks good to me so far, I have tested a few other examples at my local. What I don't understand are the labels in your test example. Did you give the lockable resources the labels "osDetail:Debian[11]" or "os:Linux" or does that refer to any agents configured in Jenkins? |
Co-authored-by: Jan-Frederik Schmidt <janfschmidt@mailbox.org>
Good question: when I create mnew node, I create also new resource with exact same name and labels. Therefore it looks as node labels. I will change the example, to eliminate this mistunderstaning. PS: Currently i work on automatic nodes-resources synchronisation. After them it will be really usefull. |
jimklimov
left a comment
There was a problem hiding this comment.
Looks beautiful and elegant, but did not test yet.
| buildPlugin(useContainerAgent: true, configurations: [ | ||
| // Test the common case (i.e., a recent LTS release) on both Linux and Windows. | ||
| [ platform: 'linux', jdk: '11', jenkins: '2.361.4' ], | ||
| [ platform: 'windows', jdk: '11', jenkins: '2.361.4' ], |
There was a problem hiding this comment.
"recent LTS release" was there for a reason ;)
Did something not work if you just bumped it, same way as below?
There was a problem hiding this comment.
"recent LTS release" was there for a reason ;)
Did something not work if you just bumped it, same way as below?
- yes there was compiling exceptions in java
- I need latest LTS (current one) to integrate some ideas rom data-tables-api.
- No body need to change jenkins version in 2 different files (this one + pom.xml) which leads to less mistakes.
Therefore I decide it will be fine to test it in this way also in future.
|
Thank you SO MUCH !!!! I can't test it yet but it is exactly what we needed. |
Yes you can test it. But how? Curently it is merged into master, but not released. You build it manually. Or wait for official release. |
|
or you can download it from here https://ci.jenkins.io/job/Plugins/job/lockable-resources-plugin/job/master/ |
I meant that I don't have time to test it right now, and I only have a production instance to test it so I'll rather wait for the official release. But thanks for the work and the links ! |
|
Technically, master builds are also in "incrementals" repo: https://repo.jenkins-ci.org/ui/packages?name=lockable-resources&type=packages => https://repo.jenkins-ci.org/ui/packages/gav:%2F%2Forg.6wind.jenkins:lockable-resources?name=lockable-resources&type=packages |
|
Thank you for this! When is the next release? I'm trying to figure out if I want to wait for that or install this from master. |
|
Th release interwal es every week, when something changed. The last one was brokne ( I think 1 sporadic failed testcase is the reason). |
Allow to use jenkins label parser to allocate lockbale-resources in more sofisticate way.
Example:
fix #428
close #309
close #474 107
see #341
see #455
Testing done
My local tests (exact with the example bellow) looks good.
Submitter checklist
- [ ] New public functions for internal use only are annotated with@NoExternalUse. In case it is used by non java code theUsed by {@code <panel>.jelly}Javadocs are annotated.- [ ] New or substantially changed JavaScript is not defined inline and does not callevalto ease the future introduction of Content Security Policy (CSP) directives (see documentation).- [ ] For dependency updates, there are links to external changelogs and, if possible, full differentials.- [ ] For new APIs and extension points, there is a link to at least one consumer.- [ ] Any localizations are transferred to *.properties files.Maintainer checklist
Before the changes are marked as
ready-for-merge:upgrade-guide-neededlabel is set and there is a Proposed upgrade guidelines section in the pull request title (see example).