Skip to content

Conversation

@aajisaka
Copy link
Member

Why are the changes needed?

Add recommendation to protect credentials in Kyuubi server for multi-tenant use case.

Fixes #7265

How was this patch tested?

New test is not added because it's only documentation and logging change.

Was this patch authored or co-authored using generative AI tooling?

No

@github-actions github-actions bot added kind:documentation Documentation is a feature! module:server labels Dec 10, 2025
Copy link
Member

@turboFei turboFei left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (fba1f94) to head (477ad75).

Files with missing lines Patch % Lines
...g/apache/kyuubi/server/KinitAuxiliaryService.scala 0.00% 5 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##           master   #7269   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files         696     696           
  Lines       43530   43535    +5     
  Branches     5883    5885    +2     
======================================
- Misses      43530   43535    +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aajisaka aajisaka self-assigned this Dec 10, 2025
@aajisaka aajisaka added this to the v1.10.3 milestone Dec 10, 2025
@aajisaka aajisaka closed this in 4b180d4 Dec 10, 2025
aajisaka added a commit that referenced this pull request Dec 10, 2025
…ist for multi-tenant clusters

### Why are the changes needed?

Add recommendation to protect credentials in Kyuubi server for multi-tenant use case.

Fixes #7265

### How was this patch tested?

New test is not added because it's only documentation and logging change.

### Was this patch authored or co-authored using generative AI tooling?

No

Closes #7269 from aajisaka/multi-tenant-doc.

Closes #7265

477ad75 [Akira Ajisaka] Fix scalastyle check
5e947b9 [Akira Ajisaka] [KYUUBI #7265] Recommend to configure kyuubi.session.local.dir.allow.list for multi-tenant clusters.

Authored-by: Akira Ajisaka <[email protected]>
Signed-off-by: Akira Ajisaka <[email protected]>
(cherry picked from commit 4b180d4)
Signed-off-by: Akira Ajisaka <[email protected]>
@aajisaka
Copy link
Member Author

Merged into master and branch-1.10. Thank you @turboFei and @cxzl25 !

@aajisaka aajisaka deleted the multi-tenant-doc branch December 10, 2025 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind:documentation Documentation is a feature! module:server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

📝 Add instructions for multi-tenant cluster for security page

4 participants