Skip to content

Conversation

@iximeow
Copy link
Member

@iximeow iximeow commented Jan 20, 2026

only two changes that are actually interesting for Omicron, but the full set:

  • ASpace should allow spaces with size 1 (propolis#1020)
  • 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)

The first one fixes a Propolis panic when guests use fixed interrupts for VirtIO NICs (so far as we know the only currently-relevant case of this is iPXE)

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.
@iximeow iximeow merged commit 5eb1337 into main Jan 21, 2026
18 checks passed
@iximeow iximeow deleted the another-propolis-bump branch January 21, 2026 22:05
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