<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Dec 5, 2017 at 4:37 AM, Scott Marlow <span dir="ltr"><<a href="mailto:smarlow@redhat.com" target="_blank">smarlow@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">It would be great if we could have a branch that includes all of the commits that we are considering to merge at a particular time of day, such that we would run the TCK against that branch, only once a day. </blockquote><div><br></div><div>Can this be done that often? I had in my mind that if we did one of these it would amount to stealing one of the regular runs, but perhaps that's not the case.</div><div><br></div><div>Now, I don't think we'd want to do these anywhere near that often, but it's good to know what the limits would be. For example, I could imagine doing 10 of these over the course of a WF release, but by luck or whatever 3 of them come in the same week.</div><div><br></div><div>+1 to using a branch. We have a branch like that, master-ignore, that we use for batching up PRs to test as a group before merging. I wouldn't want to use master-ignore for this, but a differently named branch run the same way sounds good.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">If one of the changes cause a TCK failure, none of them get merged (investigation follows that to determine which change caused the failure(s)), if the test succeeds, we can then merge that batch of changes into WildFly master.<br>
<br>
We likely would want to avoid running the testing, on days when we haven't merged any changes to the WF testing branching.<br>
<br></blockquote><div><br></div><div>Can the TCK be set up to run based on a check for a change in the sha of the head of a branch? So every day at a fixed time it checks the branch, and does nothing if there is no change. If we want a run, we force push the branch before that time. We have CI jobs that check master-ignore that way, except they poll regularly, not just once a day. That works for those as they aren't so resource intensive that we worry a lot about limiting how often they run.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Would that approach help how we merge PRs on master?<br>
<br></blockquote><div><br></div><div>I think it could be helpful earlier in the release cycle before merging big changes, and then perhaps late in the release cycle if we're worried about possible regressions.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Scott<br>
<br>
On 12/04/2017 09:33 PM, Stuart Douglas wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">
<br>
<br>
On Tue, Dec 5, 2017 at 3:40 AM, Brian Stansberry <<a href="mailto:brian.stansberry@redhat.com" target="_blank">brian.stansberry@redhat.com</a> <mailto:<a href="mailto:brian.stansberry@redhat.com" target="_blank">brian.stansberry@redha<wbr>t.com</a>>> wrote:<br>
<br>
Great. :)<br>
<br>
One thing I think we need to do is figure out how to get custom TCK<br>
runs for PR branches. The TCK is a big part of our test coverage,<br>
and one way to not "use master as a test bed" is to get a check of a<br>
branch on the TCK before we merge it.<br>
<br>
I know we've gotten TCK runs of ad-hoc branches before, so by<br>
"figure out" I mean work out how to make that not overly painful,<br>
come to some sort of consensus on when it's worthwhile, etc.<br>
<br>
<br>
I think if we were going to do this it should probably be something reviewers can ask for on specific PR. The TCK uses a *lot* more resources than a standard CI run, so we need to make sure we limit it to cases where it is required.<br>
<br>
Stuart<br>
<br>
<br>
On Fri, Dec 1, 2017 at 10:04 AM, Alessio Soldano<br></span><span class="gmail-">
<<a href="mailto:asoldano@redhat.com" target="_blank">asoldano@redhat.com</a> <mailto:<a href="mailto:asoldano@redhat.com" target="_blank">asoldano@redhat.com</a>>> wrote:<br>
<br>
There you go... PR updated to consume the same api jar now<br>
released as final.<br>
<br>
Cheers<br>
Alessio<br>
<br>
On Fri, Dec 1, 2017 at 3:30 PM, David Lloyd<br></span><span class="gmail-">
<<a href="mailto:david.lloyd@redhat.com" target="_blank">david.lloyd@redhat.com</a> <mailto:<a href="mailto:david.lloyd@redhat.com" target="_blank">david.lloyd@redhat.com</a><wbr>>> wrote:<br>
<br>
On Thu, Nov 30, 2017 at 5:50 PM, Alessio Soldano<br></span><div><div class="gmail-h5">
<<a href="mailto:asoldano@redhat.com" target="_blank">asoldano@redhat.com</a> <mailto:<a href="mailto:asoldano@redhat.com" target="_blank">asoldano@redhat.com</a>>> wrote:<br>
> As suggested by Brian, I'd like to draw attention to the discussion on<br>
> <a href="https://github.com/wildfly/wildfly/pull/10604" rel="noreferrer" target="_blank">https://github.com/wildfly/wil<wbr>dfly/pull/10604</a><br>
<<a href="https://github.com/wildfly/wildfly/pull/10604" rel="noreferrer" target="_blank">https://github.com/wildfly/wi<wbr>ldfly/pull/10604</a>> .<br>
> The PR is an upgrade of the webservices stack, including JBossWS, Apache<br>
> CXF, JAXB-RI and JAXB API. In particular, the JAXB upgrade is for EE8 and<br>
> better JDK 9 compatibility.<br>
> Now, due to the upgrade of the JAXB API spec jar, the PR is essentially<br>
> stalled since 20 days; the new spec is released as an alpha (as it's been<br>
> tested within JBossWS only) and that does not satisfy a rule that requires<br>
> any artifact being pulled to be Final.<br>
> We're talking about a spec jar, we could simply re-tag that as Final,<br>
> chances are we won't need changes any time soon there anyway, but as Tomaz<br>
> pointed out, in principle that would be dishonest.<br>
<br>
My opinion is that you should go ahead and make a .Final<br>
tag. In the<br>
(unlikely?) event that the spec has to be modified for some<br>
reason, I<br>
think you could make a 1.0.1.Final tag and call it a "bug fix".<br>
<br>
The alternative is to simply wait. I don't think there is<br>
any middle position.<br>
<br>
> While I see the point in requiring that only sufficiently stable upgrades<br>
> are applied to the codebase, I'm wondering whether, maybe, we're going a bit<br>
> too far with the rules. Brian wrote on this topic: "how to determine that<br>
> something is good enough to go in without using master as a test bed" ?<br>
<br>
I don't think we are; I agree with the policy as it stands. If you<br>
look at it in terms of being able to release at any time,<br>
then it<br>
follows that everything _must_ be stable.<br>
<br>
--<br>
- DML<br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
wildfly-dev mailing list<br></div></div>
<a href="mailto:wildfly-dev@lists.jboss.org" target="_blank">wildfly-dev@lists.jboss.org</a> <mailto:<a href="mailto:wildfly-dev@lists.jboss.org" target="_blank">wildfly-dev@lists.jbos<wbr>s.org</a>><br>
<a href="https://lists.jboss.org/mailman/listinfo/wildfly-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailma<wbr>n/listinfo/wildfly-dev</a><span class="gmail-"><br>
<<a href="https://lists.jboss.org/mailman/listinfo/wildfly-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailm<wbr>an/listinfo/wildfly-dev</a>><br>
<br>
<br>
<br>
<br>
-- Brian Stansberry<br>
Manager, Senior Principal Software Engineer<br>
Red Hat<br>
<br>
______________________________<wbr>_________________<br>
wildfly-dev mailing list<br></span>
<a href="mailto:wildfly-dev@lists.jboss.org" target="_blank">wildfly-dev@lists.jboss.org</a> <mailto:<a href="mailto:wildfly-dev@lists.jboss.org" target="_blank">wildfly-dev@lists.jbos<wbr>s.org</a>><br>
<a href="https://lists.jboss.org/mailman/listinfo/wildfly-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailma<wbr>n/listinfo/wildfly-dev</a><br>
<<a href="https://lists.jboss.org/mailman/listinfo/wildfly-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailm<wbr>an/listinfo/wildfly-dev</a>><span class="gmail-"><br>
<br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
wildfly-dev mailing list<br>
<a href="mailto:wildfly-dev@lists.jboss.org" target="_blank">wildfly-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/wildfly-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailma<wbr>n/listinfo/wildfly-dev</a><br>
<br>
</span></blockquote>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Brian Stansberry<div>Manager, Senior Principal Software Engineer</div><div>Red Hat</div></div></div>
</div></div>