Skip to content

[tmpnet] Enable runtime-specific restart behavior #3882

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

Merged
merged 3 commits into from
May 7, 2025

Conversation

maru-ava
Copy link
Contributor

@maru-ava maru-ava commented Apr 12, 2025

PR Chain: tmpnet+kube

This PR chain enables tmpnet to deploy temporary networks to Kubernetes. Early PRs refactor tmpnet to support the addition in #3615 of a new tmpnet node runtime for kube.

Why this should be merged

Delegate responsibility for restart to the node runtime to allow the process runtime to continue to start/stop and the kube runtime to scale down/scale up.

How this was tested

CI

Need to be documented in RELEASES.md?

N/A

TODO

@maru-ava maru-ava self-assigned this Apr 12, 2025
@maru-ava maru-ava added testing This primarily focuses on testing tooling Build, test and development tooling labels Apr 12, 2025
@github-project-automation github-project-automation bot moved this to Backlog 🗄️ in avalanchego Apr 12, 2025
@maru-ava maru-ava moved this from Backlog 🗄️ to In Progress 🏗 in avalanchego Apr 12, 2025
@maru-ava maru-ava force-pushed the tmpnet-final-refactor branch 3 times, most recently from 5b18b7a to fd9bace Compare April 12, 2025 04:31
@maru-ava maru-ava force-pushed the tmpnet-implicit-data-dir branch 3 times, most recently from ca83ff8 to 465cd68 Compare April 12, 2025 22:22
@maru-ava maru-ava force-pushed the tmpnet-final-refactor branch 2 times, most recently from b768168 to bcf4146 Compare April 12, 2025 23:09
@maru-ava maru-ava force-pushed the tmpnet-implicit-data-dir branch from 465cd68 to f0440ce Compare April 13, 2025 22:29
@maru-ava maru-ava force-pushed the tmpnet-final-refactor branch from bcf4146 to 11caceb Compare April 13, 2025 22:32
@maru-ava maru-ava force-pushed the tmpnet-implicit-data-dir branch from f0440ce to 1c1ab28 Compare April 14, 2025 18:10
@maru-ava maru-ava force-pushed the tmpnet-final-refactor branch from 11caceb to 3661417 Compare April 14, 2025 18:11
@maru-ava maru-ava force-pushed the tmpnet-implicit-data-dir branch from 1c1ab28 to 3b43707 Compare April 17, 2025 00:32
Base automatically changed from tmpnet-implicit-data-dir to master April 17, 2025 01:04
@maru-ava maru-ava force-pushed the tmpnet-final-refactor branch from 3661417 to 36f1837 Compare April 17, 2025 01:49
@maru-ava maru-ava changed the base branch from master to tmpnet-rename-node-process April 17, 2025 01:50
@maru-ava maru-ava moved this from In Progress 🏗 to In Review 👀 in avalanchego Apr 17, 2025
@maru-ava maru-ava removed the testing This primarily focuses on testing label Apr 17, 2025
Base automatically changed from tmpnet-rename-node-process to master April 17, 2025 14:51
@maru-ava maru-ava moved this from In Review 👀 to In Progress 🏗 in avalanchego Apr 17, 2025
@maru-ava maru-ava force-pushed the tmpnet-final-refactor branch from 36f1837 to fa68cad Compare April 17, 2025 15:44
@maru-ava maru-ava changed the base branch from master to tmpnet-flagsmap-string-values April 17, 2025 15:54
@maru-ava maru-ava force-pushed the tmpnet-final-refactor branch from 95041b1 to db7e645 Compare April 17, 2025 17:33
Copy link
Contributor

@qdm12 qdm12 left a comment

Choose a reason for hiding this comment

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

2 more PRs to go 💯 !

maru-ava added 3 commits May 5, 2025 15:48
Delegate responsibility for restart to the node runtime to allow the
process runtime to continue to start/stop and the kube runtime to
scale down/scale up.
This is intended to avoid the problem of ephemeral nodes not being
running when a restart is attempted.
@maru-ava maru-ava force-pushed the tmpnet-final-refactor branch from a61ac6e to 02662a6 Compare May 5, 2025 16:20
@maru-ava maru-ava requested review from qdm12 and joshua-kim May 5, 2025 16:43
@maru-ava
Copy link
Contributor Author

maru-ava commented May 5, 2025

Minor cleanup that suggests a quick re-review. The changes are isolated to the last 2 commits.

@joshua-kim joshua-kim added this pull request to the merge queue May 7, 2025
Merged via the queue into master with commit e221a0f May 7, 2025
24 checks passed
@joshua-kim joshua-kim deleted the tmpnet-final-refactor branch May 7, 2025 17:05
@github-project-automation github-project-automation bot moved this from In Progress 🏗️ to Done 🎉 in avalanchego May 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tooling Build, test and development tooling
Projects
Status: Done 🎉
Development

Successfully merging this pull request may close these issues.

4 participants