<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div><br></div><div><br>On Sep 3, 2015, at 19:48, Morgan Fainberg <<a href="mailto:morgan.fainberg@gmail.com">morgan.fainberg@gmail.com</a>> wrote:<br><br></div><blockquote type="cite"><div><meta http-equiv="content-type" content="text/html; charset=utf-8"><div><br><br></div><div><br>On Sep 3, 2015, at 19:28, David Stanek <<a href="mailto:dstanek@dstanek.com">dstanek@dstanek.com</a>> wrote:<br><br></div><blockquote type="cite"><div><div dir="ltr"><br><div class="gmail_quote"><div dir="ltr">On Thu, Sep 3, 2015 at 3:44 PM Henry Nash <<a href="mailto:henryn@linux.vnet.ibm.com">henryn@linux.vnet.ibm.com</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><br></div><div>I would like to request an FFE for the remaining two patches that are already in review (<a href="https://review.openstack.org/#/c/153897/" target="_blank">https://review.openstack.org/#/c/153897/</a> and <a href="https://review.openstack.org/#/c/154485/" target="_blank">https://review.openstack.org/#/c/154485/</a>). These contain only test code and no functional changes, and increase our test coverage - as well as enable other items to be re-use the list_role_assignment backend method.</div></div></blockquote><div><br></div><div>Do we need a FFE for changes to tests?</div><div><br></div></div></div></div></blockquote><div><br></div>I would say "no". <br></div></blockquote><br><div>To clarify "no" a FFE is not needed here. Not "no" to the additional testing. </div></body></html>