New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AUTHZ] Check generated policy file in test suite #4677
[AUTHZ] Check generated policy file in test suite #4677
Conversation
...hz/src/test/gen/scala/org/apache/kyuubi/plugin/spark/authz/gen/PolicyJsonFileGenerator.scala
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #4677 +/- ##
============================================
+ Coverage 57.92% 57.95% +0.03%
Complexity 13 13
============================================
Files 580 580
Lines 32218 32220 +2
Branches 4304 4304
============================================
+ Hits 18663 18674 +11
+ Misses 11761 11757 -4
+ Partials 1794 1789 -5 see 10 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
<activation> | ||
<!-- activated when Ranger version is identical to required--> | ||
<property> | ||
<name>ranger.version</name> | ||
<value>2.4.0</value> | ||
</property> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for ping me, So do we have to specify the profile manually now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, and the profile name changed to gen-policy
.
Thanks, merged to master. |
Why are the changes needed?
genpolicy
profile in CI builds, as the property based activation not auto-triggered as expectedly with propertyranger.version=2.4.0
set in project parent pomKYUUBI_UPDATE=1 build/mvn clean test -pl :kyuubi-spark-authz_2.12 -Dtest=none -DwildcardSuites=org.apache.kyuubi.plugin.spark.authz.gen.PolicyJsonFileGenerator -Pgenpolicy
How was this patch tested?
Add some test cases that check the changes thoroughly including negative and positive cases if possible
Add screenshots for manual tests if appropriate
Run test locally before make a pull request