Skip to content

feat(cfg): add guardnet and ddosnet cfg presets#868

Merged
skylar-simoncelli merged 3 commits into
mainfrom
feat/add-guardnet-ddosnet-cfg-presets
Mar 5, 2026
Merged

feat(cfg): add guardnet and ddosnet cfg presets#868
skylar-simoncelli merged 3 commits into
mainfrom
feat/add-guardnet-ddosnet-cfg-presets

Conversation

@skylar-simoncelli

Copy link
Copy Markdown
Contributor

Summary

  • Adds res/cfg/guardnet.toml and res/cfg/ddosnet.toml so the binary recognizes CFG_PRESET=guardnet and CFG_PRESET=ddosnet
  • Without these files, nodes crash immediately with Failed to load config guardnet/ddosnet
  • The chain-spec data was added in feat: add guardnet and ddosnet chain configs #841 but the cfg presets were omitted — this is the missing piece

Context

res/cfg/<name>.toml is how the binary maps a CFG_PRESET env var to its chain config. All other networks (devnet, govnet, qanet, preview, preprod, mainnet) have these files. Guardnet and ddosnet did not.

The new tomls follow the same pattern as existing networks, referencing the files available in res/guardnet/ and res/ddosnet/. Fields for ics-config, reserve-config, permissioned-candidates-config, and registered-candidates-addresses are omitted as those files do not yet exist for these networks.

Add res/cfg/guardnet.toml and res/cfg/ddosnet.toml so the binary
recognizes the guardnet and ddosnet CFG_PRESET values. The chain-spec
data files were added in #841 but the cfg presets were omitted,
causing nodes to crash with "Failed to load config guardnet/ddosnet".

Omits ics-config, reserve-config, permissioned-candidates-config, and
registered-candidates-addresses as those files do not exist for these
networks yet.
@skylar-simoncelli skylar-simoncelli requested a review from a team as a code owner March 5, 2026 15:55
@github-actions

github-actions Bot commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

kics-logo

KICS version: v2.1.19

Category Results
CRITICAL CRITICAL 0
HIGH HIGH 0
MEDIUM MEDIUM 47
LOW LOW 3
INFO INFO 59
TRACE TRACE 0
TOTAL TOTAL 109
Metric Values
Files scanned placeholder 26
Files parsed placeholder 26
Files failed to scan placeholder 0
Total executed queries placeholder 73
Queries failed to execute placeholder 0
Execution time placeholder 12

auto-merge was automatically disabled March 5, 2026 19:01

Pull request was closed

@skylar-simoncelli skylar-simoncelli added this pull request to the merge queue Mar 5, 2026
Merged via the queue into main with commit 119306a Mar 5, 2026
49 of 62 checks passed
@skylar-simoncelli skylar-simoncelli deleted the feat/add-guardnet-ddosnet-cfg-presets branch March 5, 2026 22:00
m2ux added a commit that referenced this pull request Apr 23, 2026
Signed-off-by: Mike Clay <mike.clay@shielded.io>
m2ux added a commit that referenced this pull request Apr 23, 2026
Signed-off-by: Mike Clay <mike.clay@shielded.io>
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.

3 participants