Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(cli/cmd): update operator init-nodes cmd #2703

Merged
merged 2 commits into from
Dec 16, 2024

Conversation

kc1116
Copy link
Contributor

@kc1116 kc1116 commented Dec 13, 2024

This PR updates the flags for the operator init-nodes cmd so that the cmd can be run directly from source without having to install the omni CLI. This PR adds a new flag --halo-tag, which is the halo tag to use for the new node. This is required because the current buildinfo.GitCommit() func will return an err if the cmd is not run from an binary. This PR also binds the current --feature-flags to the init-nodes command. Now we can run the command in CI and locally as follows

cd cli/cmd/omni
go run main.go operator init-nodes --network=staging \
--moniker=foo \
--home=/tmp \
--halo-tag=main \
--feature-flags=evmstaking

issue: none

@kc1116 kc1116 merged commit 609ff38 into main Dec 16, 2024
19 checks passed
@kc1116 kc1116 deleted the khalil/halotag-flag-initnodes branch December 16, 2024 15:17
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