Skip to content

Conversation

@adoroszlai
Copy link
Contributor

@adoroszlai adoroszlai commented Jun 19, 2025

What changes were proposed in this pull request?

  1. Fail checks if output.log is missing.
    • Save output.log for rat check to prevent unintended failure.
  2. Remove clean from test execution, since it removes $REPORT_DIR including output.log. It is no longer necessary after HDDS-13179.
  3. Fix the error pattern, BUILD FAILURE cannot happen due to --fail-never (HDDS-12500)

https://issues.apache.org/jira/browse/HDDS-13296

How was this patch tested?

Verified (1) with only a58a717:

$ ./hadoop-ozone/dev-support/checks/integration.sh -Dtest='TestConfigFileAppender' -am -pl :hdds-config
...
[INFO] BUILD SUCCESS
...

$ echo $?
1

$ cat target/integration/summary.txt 
Unknown failure, output.log missing

CI correctly reports integration test failures:
https://github.com/adoroszlai/ozone/actions/runs/15752841577

@adoroszlai
Copy link
Contributor Author

Merged as part of #8662.

@adoroszlai adoroszlai closed this Jun 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant