<div dir="ltr">I'm not a Cinder dev, but I could not help looking at this suggested change. To me at least, it's an insta-merge, but I think it might get overlooked because the Commit Message doesn't detail "What you would now see" versus "What you should see", and the cognitive load on reviewers is high enough that it gets ignored.<div><br></div><div>+1 from someone who has no right to say so.<br></div><div><br></div><div>--</div><div>MC</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 19, 2021 at 3:43 PM Sam Morrison <<a href="mailto:sorrison@gmail.com">sorrison@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hi Cinder cores,</div><div><br></div><div>I've had a simple review [1] waiting for several months, could I please get some feedback on this.</div><div><br></div><div>Thanks!</div><div>Sam<br></div><div><br></div><div>[1] <a href="https://review.opendev.org/c/openstack/cinder/+/764875" target="_blank">https://review.opendev.org/c/openstack/cinder/+/764875</a></div></div>
</blockquote></div>