Vladik Romanovsky
Vladik Romanovsky
/approve Looks good!
Sorry, dropped the ball on this. Indeed, increasing the overhead is the way to go, unfortunately. This is a similar issue that we observed on multiple occasions when a large...
> @vladikr @mhenriks PTAL, I have updated the PR with the new APi Thinking about this.. Shouldn't we also prevent the VM from booting if ManualRecoveryRequired?
The API looks fine to me now. I'm only worried about the functionality. I don't believe users will look at the VM status and will run into an unbootable state.
> > The API looks fine to me now. I'm only worried about the functionality. I don't believe users will look at the VM status and will run into an...
/approve Thanks @dankenigsberg
ok, thank you @alancaldelas! This PR makes sense to me. /approve
/retest
> @vladikr are the automated checks stuck? anything I can do? thanks! @RobertoMachorro no worries, we just need a review from @aburdenthehand to proceed.