Skip to content

Conversation

@iximeow
Copy link
Member

@iximeow iximeow commented Jan 20, 2026

only one change that's actually interesting for Omicron, but the full set:

  • run header-check in CI (propolis#1017)
  • small typo that I couldn't ignore (propolis#1018)
  • Indicator should allow stopping instances at Init (propolis#1007)

That last one fixes an issue where stopping a started-but-not-running VM crashes Propolis. I haven't tried to reach that issue via Neuxs, but I imagine it is reachable you stop an instance at the right moment after trying to start it. Either way, Propolis shouldn't die like that.

only one change that's actually interesting for Omicron, but the full
set:

* run header-check in CI (propolis#1017)
* small typo that I couldn't ignore (propolis#1018)
* Indicator should allow stopping instances at Init (propolis#1007)

That last one fixes an issue where stopping a started-but-not-running VM
crashes Propolis. I haven't tried to reach that issue via Omicron, but
I imagine it is reachable you stop an instance at the right moment after
trying to start it. Either way, Propolis shouldn't die like that.
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