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

Added --no-purge flag to scripts/localnet.sh to be able to save state when restarting a script. #675

Merged
merged 4 commits into from
Jul 31, 2024

Conversation

roman-opentensor
Copy link
Contributor

When working with localnet, there is often a need to restart the script. To save the state and not waste time on settings, you need to comment out some lines. I suggest introducing a flag that will bypass local state cleanup.

@roman-opentensor roman-opentensor self-assigned this Jul 23, 2024
@roman-opentensor roman-opentensor added blue team defensive programming, CI, etc devnet-skip labels Jul 23, 2024
@roman-opentensor roman-opentensor requested a review from a team July 23, 2024 16:28
unconst
unconst previously approved these changes Jul 23, 2024
@unconst unconst changed the base branch from main to devnet-ready July 23, 2024 18:42
@unconst unconst dismissed their stale review July 23, 2024 18:42

The base branch was changed.

@unconst unconst marked this pull request as ready for review July 23, 2024 18:42
@roman-opentensor roman-opentensor requested a review from a team July 24, 2024 21:49
@sam0x17
Copy link
Contributor

sam0x17 commented Jul 27, 2024

this PR should be into main as per https://github.com/opentensor/subtensor/blob/main/CONTRIBUTING.md

@distributedstatemachine distributedstatemachine merged commit a03ce30 into devnet-ready Jul 31, 2024
9 checks passed
@distributedstatemachine distributedstatemachine deleted the feat/roman/no-purge-flag branch July 31, 2024 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blue team defensive programming, CI, etc devnet-companion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants