apupier commented on PR #29565: URL: https://github.com/apache/camel/pull/29565#issuecomment-6769673709
Thanks @Croway, @davsclaus, and @gnodet for the reviews and local reproduction! I have applied all fixes: 1. **License header definition**: Added `<Jenkinsfile.occupier>SLASHSTAR_STYLE</Jenkinsfile.occupier>` to root `pom.xml` (local `mvn license:check -N` is now passing). 2. **Relative job lookup**: Updated the default job path to `../camel-alt-arch-occupier` in `Jenkinsfile`, `Jenkinsfile.jdk26`, and `Jenkinsfile.jdk27`, and switched `catchError` to `stageResult: 'UNSTABLE'` (with `buildResult: 'SUCCESS'`) so setup errors are highlighted in the UI without failing the matrix build. 3. **Sandbox bypass**: Replaced Groovy's `JsonSlurperClassic` with a Python 3 parser inside the `sh` step. It queries and processes `/computer/api/json` in one step and outputs simple tab-separated lines (`<platform>\t<node>\t<idleCount>`), avoiding any Jenkins script approval. 4. **Job filter precision**: Refined the URL matching in the Python step to match only Camel Core builds while explicitly excluding the occupier job itself and non-core jobs (Kamelets, Quarkus, Spring-Boot, Karaf). 5. **Occupier startup**: Added `skipDefaultCheckout(true)` and switched the inspection agent label to `ubuntu`. 6. **CPS variable capture**: Marked loop variables `final` in `buildOccupierBranches`. _IBM Bob on behalf of apupier_ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
