Skip to content

fix(console): refine configuration views and editing - #881

Open
kitalkuyo-gita wants to merge 2 commits into
apache:masterfrom
kitalkuyo-gita:console-config-views
Open

kitalkuyo-gita wants to merge 2 commits into
apache:masterfrom
kitalkuyo-gita:console-config-views

Conversation

@kitalkuyo-gita

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

  • Mask sensitive configuration values in Console API responses, including nested configurations, while preserving internal configuration values.
  • Preserve saved credentials when masked values are submitted during edits; support explicit clearing and reject unresolved masks.
  • Require system administrator permissions for cluster/LLM management reads and system-session plugin operations.
  • Add minimal cluster/LLM selection endpoints returning only id, name, and type, and migrate frontend selectors.
  • Stabilize builds and tests by updating the Node download URL, excluding unused SIGAR dependencies, mocking heartbeat RPC calls, and preserving HBase startup exception causes.

How was this PR tested?

  • Tests have Added for the changes
  • Production environment verified

Added ConfigSecurityTest and ConfigAccessControlTest covering response masking, authorization, nested configurations, and credential update behavior.

Local validation on Linux ARM64:

  • JDK 8: Full 109-module clean test succeeded; 1,806 tests, zero failures/errors.
  • JDK 11: CI module set completed through resumed reactor runs; 1,810 tests, zero failures/errors.
  • Both matrices retained the repository’s seven default Doris integration-test skips. Required ClickHouse integration tests passed.
  • Console suites passed all 34 tests per JDK, including Checkstyle, RAT, and frontend production builds.
  • A locally running Console with disposable MySQL passed 37 HTTP/database checks covering authorization, tenant boundaries, masking, and credential updates.

Use the official Node distribution endpoint and exclude unused SIGAR dependencies from allocation benchmarks. Mock supervisor health RPCs in heartbeat tests and retain HBase startup causes for diagnostics.

This branch has not been deployed

No deployments
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