<div dir="ltr">That is a good point, and one we are giving some thought to through convo on #pulp-dev and the issue [0]. The case of a plugin needing an unreleased change from core would fail with this change. It's a tradeoff though because if we go with nightlies as the version of core that is used, whenever the nightlies break, the unittest PR runners also will, which has been a reliability issue with the plugin unittest runner for a while.<br><br>I wrote some on the issue about it, but I see the 'plugin needs unreleased code from core' as a special case, not a normal case. It used to be common, but it's getting less common, which is good, because contributing to a plugin should not involve changes to the core as the norm. It will happen from time to time, so we can handle the special case, specially by running the unittests locally with the necessary unreleased version of platform and posting the results as evidence that its safe to merge.<br><br>[0]: <a href="https://pulp.plan.io/issues/2751">https://pulp.plan.io/issues/2751</a><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 2, 2017 at 4:43 PM, Michael Hrivnak <span dir="ltr"><<a href="mailto:mhrivnak@redhat.com" target="_blank">mhrivnak@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">What about cases where a plugin wants to use something that's new in the unreleased core? The master branch of a plugin will usually be released with the master branch of the core in the next 2.y release for example. That seems like a normal scenario; is it facilitated somehow with this testing change?</div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Fri, Jun 2, 2017 at 4:33 PM, Dennis Kliban <span dir="ltr"><<a href="mailto:dkliban@redhat.com" target="_blank">dkliban@redhat.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr"><div>In an effort to resolve issue 2751[0], I updated the PR builder job for plugins. Each PR for a plugin will now be tested against the latest stable release of the core found here[1]. This will ensure that the plugin is maintaining compatibility with the latest stable core and that we are only testing one change at a time. <br><br><br>[0] <a href="https://pulp.plan.io/issues/2751" target="_blank">https://pulp.plan.io/issues/27<wbr>51</a><br>[1] <a href="https://repos.fedorapeople.org/pulp/pulp/stable/latest/" target="_blank">https://repos.fedorapeople.org<wbr>/pulp/pulp/stable/latest/</a><span class="m_-7113407926791881581HOEnZb"><font color="#888888"><br><br><br></font></span></div><span class="m_-7113407926791881581HOEnZb"><font color="#888888">-Dennis<br></font></span></div>
<br></div></div>______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/pulp-dev</a><br>
<br></blockquote></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div class="m_-7113407926791881581gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><p style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"><span style="margin:0px!important;padding:0px!important">Michael</span> <span style="margin:0px!important;padding:0px!important">Hrivnak</span></p><p style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"></p><span style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"><span style="margin:0px!important;padding:0px!important">Principal Software Engineer</span><span style="margin:0px!important;padding:0px!important">, <span style="margin:0px!important;padding:0px!important">RHCE</span></span> </span><span style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px"></span><br style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"><p style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important">Red Hat</p></div></div>
</font></span></div>
<br>______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/pulp-dev</a><br>
<br></blockquote></div><br></div>