Skip to content

Conversation

technicallyty
Copy link
Contributor

@technicallyty technicallyty commented Sep 25, 2025

Description

update that allows configuration to be added to the nodes in system test. see readme for example


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • tackled an existing issue or discussed with a team member
  • left instructions on how to review the changes
  • targeted the main branch

@technicallyty technicallyty marked this pull request as ready for review October 8, 2025 19:14

import (
"fmt"
"reflect"

Check notice

Code scanning / CodeQL

Sensitive package import Note test

Certain system packages contain functions which may be a possible source of non-determinism
@technicallyty technicallyty marked this pull request as draft October 8, 2025 21:57
@technicallyty
Copy link
Contributor Author

pausing for now as this became really complicated with cometbft v2 being a dependency. this should likely be removed in another PR.

```go
s := systemtest.Sut
s.ResetChain(t)
s.SetupChain("--config-changes=consensus.timeout_commit=10s")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this! 😀

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