<div dir="ltr">Hi,<br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 2, 2015 at 11:12 PM, Jeremy Stanley <span dir="ltr"><<a href="mailto:fungi@yuggoth.org" target="_blank">fungi@yuggoth.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 2015-07-02 22:00:43 +0530 (+0530), sac wrote:<br>
> I have a patch[1] which should work only on Red Hat systems and<br>
> fails on non Red Hat systems due to non-availability of a package.<br>
<br>
</span>I suppose it's up to the puppet-manager-core group members to decide<br>
whether non-portable additions are something they want to support.<br>
<span class=""><br></span></blockquote><div><br></div><div>Agree. This currently supports only Red Hat because there is no corresponding package for nfs-ganesha in Debian/Ubuntu.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> However, jenkins runs the beaker tests on Ubuntu and by definition<br>
> this fails, which is quite intended. However, jenkins marks<br>
> FAILURE due to this error.<br>
><br>
> Due to this, patch stays in Verified-1 state incorrectly.<br>
<br>
</span>The job has been instructed to run against the entirety of the<br>
acceptance spec, and the contents of that spec aren't currently<br>
conditional for specific classes (besides repo selection and package<br>
providers).<br>
<span class=""><br></span></blockquote><div><br></div><div>Okay, makes sense.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> How to bypass or waive this failure? Can I request an exception?<br>
><br>
> [1] <a href="https://review.openstack.org/#/c/186699/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/186699/</a><br>
<br>
</span>Waiving your change past the job wouldn't solve this, since that job<br>
would just become instantly broken for the puppet-manila repo from<br>
that point on (and also we have no way to actually do that short of<br>
disabling the job or involving an admin for the code review system<br>
itself to bypass our gating safeties). If<br>
non-portable/platform-specific features are acceptable to the<br>
puppet-manila maintainers, I think your options are to either wrap<br>
your addition to the spec in a check for $::osfamily == 'RedHat' or<br>
modify your new class to not fail when applied on non-RedHat-family<br>
platforms.<br></blockquote><div><br></div><div>I think this is a good plan. Instead of `fail', I would go ahead with `warning'. Thanks.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Anyway, I suspect this is a discussion far better had in the change<br>
review itself rather than on the mailing list.<br></blockquote><div><br></div><div>Sure.</div><div><br></div><div>Thanks,</div><div>sac.</div></div></div></div>