On 04/27/2016 11:01 AM, Nick Boldt wrote:
One PR in Base will not automatically cause the results of that PR
to
be picked up by downstream projects (eg., Server, Openshift) or
JBT/JBDS aggregation -- to do that would require passing in a
commandline override for
-Djbosstools-base-site=http://path/to/output/of/the/pull-request/build/repo/
... for all the downstream projects of Base, and ensuring that the
output was published to a /pulls/ folder instead of the usual /builds/
folder.
So if I'm going down this road I wanted to make sure we think a little
about how the future might unfold, and how we might get there. Rather
not be painted into a corner. :D
I think it is beyond overkill to even try for b). wildfly doesn't
rebuild and run the test suite for a given change in jasper, for
example. The entire idea that not even an actual change, but a potential
change / PR, to base, could end up kicking off the entire product build
and their associated test suites. I think this is beyond overkill:
it's dangerous and will sap computing power and lock up slaves doing
what's essentially no-ops.
PR's change frequently. Imagine a change in base that gets a few
different revisions? Suddenly I can't even update my PR to change some
text string without it kicking off a potential entire build chain and
test suite.
Not to mention, what if the upstream tests were already failing? You'd
have to build all the upstream plugins with the OLD base, then build
base, build everything, run all the tests, and somehow compare the new
test results to the previous test results.
I think this is folly, and it should not be pursued at all.
Start with a), and finish with a).
On Tue, Apr 26, 2016 at 6:59 PM, Alexey Kazakov <alkazako(a)redhat.com>
wrote:
> +100 to start with a)
>
> On 04/26/2016 02:20 PM, Nick Boldt wrote:
>>
https://issues.jboss.org/browse/JBIDE-21657
>>
>> On Tue, Apr 26, 2016 at 1:13 PM, Mickael Istria <mistria(a)redhat.com>
>> wrote:
>>> On 04/26/2016 05:50 PM, Max Rydahl Andersen wrote:
>>>
>>> in other words, start with a
>>>
>>>> a) does the PR break my tests / want a green light on the PR page
>>> +1000. Is there a Jira for that?
>>> --
>>> Mickael Istria
>>> Eclipse developer at JBoss, by Red Hat
>>> My blog - My Tweets
>>>
>>> _______________________________________________
>>> jbosstools-dev mailing list
>>> jbosstools-dev(a)lists.jboss.org
>>>
https://lists.jboss.org/mailman/listinfo/jbosstools-dev
>>
> _______________________________________________
> jbosstools-dev mailing list
> jbosstools-dev(a)lists.jboss.org
>
https://lists.jboss.org/mailman/listinfo/jbosstools-dev