Skip to content

update config_schema to add dns settings#4713

Closed
JC-wk wants to merge 2 commits into
mainfrom
jc-wk/add-dns-to-config-schema
Closed

update config_schema to add dns settings#4713
JC-wk wants to merge 2 commits into
mainfrom
jc-wk/add-dns-to-config-schema

Conversation

@JC-wk

@JC-wk JC-wk commented Oct 15, 2025

Copy link
Copy Markdown
Collaborator

Resolves #4712

What is being addressed

The schema is missing the newly added dns settings

How is this addressed

@JC-wk JC-wk marked this pull request as ready for review October 15, 2025 08:42
@JC-wk JC-wk requested a review from a team as a code owner October 15, 2025 08:42
@github-actions

Copy link
Copy Markdown

Unit Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 9c47331.

@JC-wk

JC-wk commented Oct 15, 2025

Copy link
Copy Markdown
Collaborator Author

If we go ahead with #4715 then this one would not be needed

@JC-wk JC-wk marked this pull request as draft October 15, 2025 11:48
@martinpeck

Copy link
Copy Markdown
Member

Should we close this PR, or move it to Draft, if we'd rather apply #4715 instead?

@martinpeck

Copy link
Copy Markdown
Member

Doh! It is draft!!!! Sorry.

@JC-wk

JC-wk commented Oct 15, 2025

Copy link
Copy Markdown
Collaborator Author

Yeah happy to close it in favour of #4715

@JC-wk

JC-wk commented Oct 15, 2025

Copy link
Copy Markdown
Collaborator Author

closed in favour of #4715

@JC-wk JC-wk closed this Oct 15, 2025
@JC-wk JC-wk deleted the jc-wk/add-dns-to-config-schema branch October 16, 2025 08:39
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.

config_schema.json missing enable_dns_policy and allowed_dns settings

2 participants