kyuubi
kyuubi copied to clipboard
support list server configs by admin cli tool
:mag: Description
Issue References ๐
This pull request fixes #
Describe Your Solution ๐ง
- add command
list configin admin cli tool , for listing all the server configs, which is useful for checking runtime configs when before/after refreshing the configs of user-defaults or non-user-defaults
Types of changes :bookmark:
- [ ] Bugfix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
Test Plan ๐งช
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:
Related Unit Tests
Checklist ๐
- [ ] This patch was not authored or co-authored using Generative Tooling
Be nice. Be informative.
Codecov Report
Attention: Patch coverage is 0% with 45 lines in your changes missing coverage. Please review.
Project coverage is 0.00%. Comparing base (
8056235) to head (1c1d58d). Report is 99 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #6702 +/- ##
======================================
Coverage 0.00% 0.00%
======================================
Files 684 686 +2
Lines 42279 42324 +45
Branches 5765 5770 +5
======================================
- Misses 42279 42324 +45
| Flag | Coverage ฮ | |
|---|---|---|
0.00% <0.00%> (?) |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
listing configurations should be restricted to admin only. additionally, we may need to have another parameter to define the display mode for configuration values, e.g.
- ALL - display all configuration values as-is
- REDACTED - to avoid displaying password/secret values
PS: I remember there was a similar request to have a configuration display mode of configurations for retrieving batch info
Thanks for the PR! This PR is being closed due to inactivity. This isn't a judgement on the merit of the PR in any way. If this is still an issue with the latest version of Kyuubi, please reopen it and ask a committer to remove the Stale tag!
Thank you for using Kyuubi!