Skip to content

Conversation

TheLastCicada
Copy link
Contributor

Purpose:

TestnetA does not have a hard-fork height specified and can therefore not successfully farm plots with the latest version of Chia. This adds a hard fork height to the default config, which is also being applied on any existing testneta infrastructure.

@TheLastCicada TheLastCicada requested a review from a team as a code owner October 17, 2025 01:26
cmmarslender
cmmarslender previously approved these changes Oct 17, 2025
@cmmarslender cmmarslender added Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes labels Oct 17, 2025
@cmmarslender cmmarslender marked this pull request as draft October 17, 2025 03:07
@arvidn
Copy link
Contributor

arvidn commented Oct 17, 2025

in order to make this "just work" for existing farmers and nodes that already have a config.yaml, you also need to introduce this override if it doesn't exist in the config file. here: https://github.com/Chia-Network/chia-blockchain/blob/main/chia/consensus/default_constants.py#L112

@cmmarslender
Copy link
Member

in order to make this "just work" for existing farmers and nodes that already have a config.yaml, you also need to introduce this override if it doesn't exist in the config file. here: main/chia/consensus/default_constants.py#L112

I think we control the majority of the nodes on the network, so we had already added it to those node's configs, but if you think its ok to have something in default constants as well to make this work a bit better, we could also add something there.

@cmmarslender cmmarslender marked this pull request as ready for review October 18, 2025 17:05
@cmmarslender cmmarslender merged commit ab01147 into main Oct 18, 2025
406 of 407 checks passed
@cmmarslender cmmarslender deleted the ZB.TestnetA-hardfork-height branch October 18, 2025 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants