Skip to content

Adding Apache Rat to ci/cd#4

Open
razinbouzar wants to merge 2 commits intoapache:masterfrom
razinbouzar:master
Open

Adding Apache Rat to ci/cd#4
razinbouzar wants to merge 2 commits intoapache:masterfrom
razinbouzar:master

Conversation

@razinbouzar
Copy link
Contributor

License headers auditing should be included as part of ci/cd builds. This includes adding the Apache Rat audit tool and adds license headers

Fixes #XXXX.

Description


This PR has:

  • been tested on a real K8S cluster to ensure creation of a brand new Druid cluster works.
  • been tested for backward compatibility on a real K*S cluster by applying the changes introduced here on an existing Druid cluster. If there are any backward incompatible changes then they have been noted in the PR description.
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added documentation for new or modified features or behaviors.

Key changed/added files in this PR
  • MyFoo
  • OurBar
  • TheirBaz

License headers auditing should be included as part of ci/cd builds. This includes adding the Apache Rat audit tool and adds license headers
All six e2e/*.sh scripts were stored in git with mode 100644 (not executable), which caused the CI failure. Fixed with git update-index --chmod=+x to set mode 100755 in the git index, and chmod +x locally to match.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant