adoroszlai opened a new pull request, #6913:
URL: https://github.com/apache/ozone/pull/6913

   ## What changes were proposed in this pull request?
   
   Acceptance check relies on a hack (`sudo chmod -R a+rwX 
hadoop-ozone/dist/target`) to let all tests pass, because some of those assume 
"current directory" is writable.  Developers may encounter the same problem 
locally, where such hack is not desired.
   
   Most acceptance tests already use `/tmp` for temporary files.  The goal of 
this change is to fix all remaining tests, which rely on writable working dir.
   
   - Create local files in `${TEMP_DIR}` instead of the working dir.
   - Configure `ozone.http.basedir`, because S3 Gateway also defaults to the 
working dir.
   - Simplify some tests to use built-in keywords.
   - Disable the test case `Adds optional dir entries`, which needs to create 
subdir in `share/ozone/lib/`.
   
   https://issues.apache.org/jira/browse/HDDS-11103
   
   ## How was this patch tested?
   
   Removed the `chmod` hack, so existing tests now run in "read-only" 
environment, thus validate the change.
   
   https://github.com/adoroszlai/ozone/actions/runs/9843024729


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to