On Fri, Apr 29, 2022 at 06:35:52PM +0000, Cary Huang wrote:
> The patch applied and tested fine. I think for this kind of exception, it really is up to the administrator how he/she should proceed to resolve depending on his/her business application. Leaving things configurable by the user is generally a nice and modest change. I also like that you leave the parameters as enum entry so it is possible to extend other possible actions such as automatically adjust to match the new value.
Thanks for reviewing! Do you think this patch can be marked as
ready-for-committer?
--
Nathan Bossart
Amazon Web Services: https://aws.amazon.com