<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, 7 Jul 2016 at 03:06 Matthew Treinish <<a href="mailto:mtreinish@kortar.org">mtreinish@kortar.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Jul 06, 2016 at 11:41:56AM -0500, Matt Riedemann wrote:<br>> I just wonder how many deployments are actually relying on this, since as<br>
> noted elsewhere in this thread we don't really enforce this for all things,<br>
> only what happens to get tested in our CI system, e.g. the virtuozzo<br>
> rootwrap filters that don't have grenade testing.<br>
<br>
Sure, our testing coverage here is far from perfect, that's never been in<br>
dispute. It's always been best effort (which there has been limited in this<br>
space) like I'm not aware of anything doing any upgrade testing with<br>
virtuozzo enabled, or any of the other random ephemeral storage backends,<br>
**cough** ceph **cough**.  But, as I said before just because we don't catch all<br>
the issues isn't a reason to throw everything out the window.<br></blockquote><div><br></div><div>So now we have identified some other examples recently added to the codebase, that where not noticed by grenade for one reason or another.</div><div><br></div><div>Do we:</div><div>A) revert+postpone the virtuozzo changes until the next release?</div><div>B) add a releasenote saying you need to update the rootwrap filter first?</div><div><br></div><div>(Yes, this is a test)<br></div><div><br></div><div>It's boring, but not that hard to manually diff filters between releases - I can do an audit if we'd like to build a list of other such changes.</div><div><br></div><div> - Gus</div></div></div>