Skip to content

HDDS-6606. Rat check failure ignored for ozone-annotation-processing#3321

Merged
adoroszlai merged 3 commits intoapache:masterfrom
adoroszlai:HDDS-6606
Apr 21, 2022
Merged

HDDS-6606. Rat check failure ignored for ozone-annotation-processing#3321
adoroszlai merged 3 commits intoapache:masterfrom
adoroszlai:HDDS-6606

Conversation

@adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

  • Collect rat failures from the new dev-support/annotations directory, too.
  • Add missing license header.
  • Reduce duplication by looping over the dirs to be checked.

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

How was this patch tested?

Verified that the check properly fails due to license violation:

$ ./hadoop-ozone/dev-support/checks/rat.sh
...
dev-support/annotations/target/rat.txt: !????? src/main/java/org/apache/ozone/annotations/package-info.java

Then added the missing header and re-ran the check:

$ ./hadoop-ozone/dev-support/checks/rat.sh
...

$ echo $?
0

https://github.com/adoroszlai/hadoop-ozone/runs/6081893903

@adoroszlai adoroszlai self-assigned this Apr 19, 2022
@adoroszlai adoroszlai requested a review from fapifta April 19, 2022 20:13
@adoroszlai adoroszlai added bug Something isn't working build Pull request that modifies the build process labels Apr 20, 2022
Copy link
Contributor

@fapifta fapifta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @adoroszlai for this improvement, and catching the fact that I missed to add the results of rat in the annotation-processing module to the output of the whole check, with that missing the license info in the package-info file...

I think the changes are great, and makes it more simpler to extend, while it is also easier to overview what is happening. +1

@adoroszlai adoroszlai merged commit dc9da7d into apache:master Apr 21, 2022
@adoroszlai adoroszlai deleted the HDDS-6606 branch April 21, 2022 13:28
@adoroszlai
Copy link
Contributor Author

Thanks @fapifta for the kind review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working build Pull request that modifies the build process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants