Skip to content

docs: Add missing configuration documentation #1025

Merged
munishchouhan merged 3 commits intomasterfrom
added-missing-config
Apr 16, 2026
Merged

docs: Add missing configuration documentation #1025
munishchouhan merged 3 commits intomasterfrom
added-missing-config

Conversation

@munishchouhan
Copy link
Copy Markdown
Member

Summary

  • Document ~20 previously undocumented configuration options in docs/configuration.md including: wave.denyHosts, K8s DNS policy/servers, platform-specific node selectors, scan
    status duration, container mirror K8s resources, proxy cache settings, rate-limit timeout errors, license server URL, and advanced operational settings (job manager, message
    stream, thread monitor, trace slow endpoint)

Test plan

  • All documented config options verified to exist in the codebase with matching default values

Signed-off-by: munishchouhan <hrma017@gmail.com>
@christopher-hakkaart
Copy link
Copy Markdown
Member

christopher-hakkaart commented Apr 16, 2026

Hi @munishchouhan, your additions look good.

I've used this PR to update the page's style, standardize language and formatting, and fix a couple of indentation issues. I also moved some of the more instructional content to the configure-wave page, as it's guided content rather than a reference list. This wasn't a part of this PR, but while I was on the page, I moved it.

@christopher-hakkaart christopher-hakkaart changed the title Add missing configuration documentation docs: Add missing configuration documentation Apr 16, 2026
@munishchouhan munishchouhan merged commit ec29df8 into master Apr 16, 2026
5 checks passed
@munishchouhan munishchouhan deleted the added-missing-config branch April 16, 2026 05:57
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.

2 participants