Skip to content

Feat: set Peam devnet explicitly in local devnet - #156

Merged
ch4r10t33r merged 3 commits into
blockblaz:mainfrom
malik672:codex/fix-peam-devnet0-harness
Apr 10, 2026
Merged

ch4r10t33r merged 3 commits into
blockblaz:mainfrom
malik672:codex/fix-peam-devnet0-harness

Conversation

@malik672

Copy link
Copy Markdown
Contributor

fix harness startup issue

@ch4r10t33r can you please help me merge

@ch4r10t33r

Copy link
Copy Markdown
Contributor

Do we need this? local-devnet aren't used in devnet runs.

@malik672

Copy link
Copy Markdown
Contributor Author

Do we need this? local-devnet aren't used in devnet runs.

How does katya spin up devnets ?

@ch4r10t33r

Copy link
Copy Markdown
Contributor

They are spun up using the validator-config.yaml under ansible-devnet/ folder

@malik672
malik672 force-pushed the codex/fix-peam-devnet0-harness branch from 265a03e to 0b85b3f Compare April 10, 2026 14:35
@malik672

Copy link
Copy Markdown
Contributor Author

@ch4r10t33r made some changes

@ch4r10t33r

Copy link
Copy Markdown
Contributor

@ch4r10t33r made some changes

The flag appears to be non standard. I notice this is available only in ream and peam and in both cases, the data type of the field is different (ream uses an integer value, while peam uses a string).

@ch4r10t33r

Copy link
Copy Markdown
Contributor

As this is a non standard flag. Can you pls include this to peam-cmd.sh (instead of validator-config.yaml)? @malik672

@ch4r10t33r ch4r10t33r left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good.

@malik672

malik672 commented Apr 10, 2026 •

Copy link
Copy Markdown
Contributor Author

As this is a non standard flag. Can you pls include this to peam-cmd.sh (instead of validator-config.yaml)? @malik672

removed

@ch4r10t33r
ch4r10t33r merged commit c2fefa1 into blockblaz:main Apr 10, 2026
4 checks passed
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