mattcasters opened a new pull request, #7584:
URL: https://github.com/apache/hop/pull/7584

   ## What
   
   Fixes Jenkins integration-test permission failures after the Native Spark / 
surefire work, and restores ExecProcess single-command-line execution broken by 
#7565.
   
   Closes #7582
   
   ## Why
   
   Build [Hop-integration-tests 
#2247](https://ci-builds.apache.org/job/Hop/job/Hop-integration-tests/2247/) 
failed 14 tests with `AccessDeniedException` / `Could not delete` under 
bind-mounted project trees. Permission hardening only covered `output/` and 
surefire reports, while many older ITs still write under `files/` or the 
project root. Separately, #7565 changed `Runtime.exec(String)` to 
`Runtime.exec(String[])` for single-command mode, so command lines like 
`/bin/echo hop-single` no longer tokenize.
   
   ## How
   
   - **IT host/container prep** (`run-tests-docker.sh` / `run-tests.sh`): 
re-own `integration-tests/` to the container UID; world-writable `output/` and 
`files/`; writable project roots; residual cleanup that never deletes 
git-tracked fixtures.
   - **GCP key validation**: stricter check before running Google Sheets ITs; 
clear `SKIP_GOOGLE_SHEETS` logging.
   - **Generated artifacts → `output/`** for spreadsheet Excel writers, MDI 
JSON/Excel outputs, HTTP download target, transforms parquet prepare/read; 
update golden multi-file Excel dataset paths.
   - **Untrack residual seeds** (`http-action-output`, MDI generated outputs) 
and extend `.gitignore`.
   - **ExecProcess**: restore `Runtime.exec(String)` for 
non-arguments-in-fields mode; minor test setup hardening.
   
   ## Verification
   
   - Local full IT suite green after the final spreadsheet golden path fix.
   - `ExecProcessTest` (and related unit tests) pass after the `Runtime.exec` 
restore.


-- 
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]

Reply via email to