-
Notifications
You must be signed in to change notification settings - Fork 67
[sim] Optionally enable health monitor #9628
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
base: main
Are you sure you want to change the base?
Conversation
|
Cool. Does this cause the simulated sled agent to look at the actual SMF state wherever it's running? Wouldn't it be more useful to allow the reported state to be customized directly? |
Yes, but that was the use case I've been having 😄.
That would be really useful too. I wonder if there is a possibility to have one or the other. Do you think it' would be relatively straightforward to do that? |
|
@davepacheco, I've changed the approach. It's now possible to inject fake data via a config file. Let me know what you think! I updated the PR's description to show the new way this would work |
Adds the ability to enable the sled agent health monitor on simulated systems. This is and will be very useful for various types of testing.
Disabled:
With fake health monitor results
Enabled
Closes: #9517