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

Replace process stop_time with shutdown_time and shutdown_signal #2860

Merged
merged 3 commits into from
Apr 11, 2023

Conversation

sporksmith
Copy link
Contributor

Progress on #2034, which is part of the planned breaking changes for 3.0: #2496

@sporksmith sporksmith self-assigned this Apr 10, 2023
@github-actions github-actions bot added Component: Build Build/install tools and dependencies Component: Documentation In-repository documentation, under docs/ Component: Main Composing the core Shadow executable Component: Testing Unit and integration tests and frameworks labels Apr 10, 2023
@sporksmith
Copy link
Contributor Author

Oops, nevermind for the moment; I need to make sure we still use SIGKILL at the simulation end time...

@sporksmith sporksmith requested review from stevenengler and removed request for stevenengler April 10, 2023 23:57
@sporksmith
Copy link
Contributor Author

Oops, nevermind for the moment; I need to make sure we still use SIGKILL at the simulation end time...

Ah no, I did keep the SIGKILL at simulation shutdown

docs/shadow_config_spec.md Outdated Show resolved Hide resolved
src/main/core/sim_config.rs Outdated Show resolved Hide resolved
src/test/socket/bind/test_bind_in_new_process.rs Outdated Show resolved Hide resolved
docs/shadow_config_spec.md Outdated Show resolved Hide resolved
docs/shadow_config_spec.md Outdated Show resolved Hide resolved
docs/shadow_config_spec.md Outdated Show resolved Hide resolved
docs/shadow_config_spec.md Outdated Show resolved Hide resolved
src/main/core/support/configuration.rs Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
src/test/config/shutdown/CMakeLists.txt Outdated Show resolved Hide resolved
@sporksmith sporksmith merged commit 23c3759 into shadow:main Apr 11, 2023
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Build Build/install tools and dependencies Component: Documentation In-repository documentation, under docs/ Component: Main Composing the core Shadow executable Component: Testing Unit and integration tests and frameworks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants