[GitHub] [ozone] adoroszlai commented on pull request #1928: HDDS-4741. Modularize upgrade test

[GitBox <gi...@apache.org>]

adoroszlai commented on pull request #1928:
URL: https://github.com/apache/ozone/pull/1928#issuecomment-784330923


   > Also it looks like there is an extra robot log and report getting placed in the upgrade directory (outside the results folder). Looks like log and report for 0.5.0 first gets put there, and then log and report for 1.0.0 overwrites it. Is this supposed to happen?
   
   Thanks for pointing this out.  It turns out not specific to the `upgrade` test or this change.  It happens for each "subtest" on `master`, too.  Example (from [here](https://github.com/apache/ozone/runs/1960981044#step:7:107)):
   
   ```
   Log:     /mnt/ozone/hadoop-ozone/dist/target/ozone-1.1.0-SNAPSHOT/compose/compatibility/result/log.html
   Report:  /mnt/ozone/hadoop-ozone/dist/target/ozone-1.1.0-SNAPSHOT/compose/compatibility/result/report.html
   Output:  /mnt/ozone/hadoop-ozone/dist/target/ozone-1.1.0-SNAPSHOT/compose/result/compatibility.xml
   Log:     /mnt/ozone/hadoop-ozone/dist/target/ozone-1.1.0-SNAPSHOT/compose/log.html
   Report:  /mnt/ozone/hadoop-ozone/dist/target/ozone-1.1.0-SNAPSHOT/compose/report.html
   ```
   
   (The second log and report are the ones created in the same way as you described.)
   
   Since the fix is [pretty simple](https://github.com/apache/ozone/pull/1928/commits/749d47121886d282e55a5e5b1f1de47031bfb330), I incuded it here, instead of creating a separate issue.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org