[DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-14 Thread Sean Busbey
Moving discussion here from HADOOP-14654. Short synopsis: * HADOOP-14654 updated commons-httplient to a new patch release in hadoop-project * Precommit checked the modules that changed (i.e. not many) * nightly had Azure support break due to a change in behavior. Is this just the cost of our app

Re: [DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-14 Thread Allen Wittenauer
> On Sep 14, 2017, at 8:03 AM, Sean Busbey wrote: > > * HADOOP-14654 updated commons-httplient to a new patch release in > hadoop-project > * Precommit checked the modules that changed (i.e. not many) > * nightly had Azure support break due to a change in behavior. OK, so it worked as c

Re: [DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-14 Thread Sean Busbey
> Committers MUST check the qbt output after a commit. They MUST make sure their commit didn’t break something new. How do we make this easier / more likely to happen? For example, I don't see any notice on HADOOP-14654 that the qbt post-commit failed. Is this a timing thing? Did Steve L just no

Re: [DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-14 Thread Brahma Reddy Battula
Thanks Sean busbey for starting the discussion.This was one of pain point even I notified to Allen couple of times.IIUC his point is committers and contributors should monitor qbt. IMHO,instead of post-commit try to fix in pre-commit only?? May be we can get dependcy module(dependency:list) and ex

Re: [DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-14 Thread Allen Wittenauer
> On Sep 14, 2017, at 11:01 AM, Sean Busbey wrote: > >> Committers MUST check the qbt output after a commit. They MUST make sure > their commit didn’t break something new. > > How do we make this easier / more likely to happen? > > For example, I don't see any notice on HADOOP-14654 that the

Re: [DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-14 Thread Chris Douglas
This has gotten bad enough that people are dismissing legitimate test failures among the noise. On Thu, Sep 14, 2017 at 1:20 PM, Allen Wittenauer wrote: > Someone should probably invest some time into integrating the HBase > flaky test code a) into Yetus and then b) into Hadoop. What do

Re: [DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-14 Thread Ray Chiang
On 9/14/17 1:36 PM, Chris Douglas wrote: This has gotten bad enough that people are dismissing legitimate test failures among the noise. On Thu, Sep 14, 2017 at 1:20 PM, Allen Wittenauer wrote: Someone should probably invest some time into integrating the HBase flaky test code a) in

Re: [DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-14 Thread Chris Douglas
On Thu, Sep 14, 2017 at 1:43 PM, Ray Chiang wrote: > The other solution I've seen (from Oozie?) is to re-run just the subset of > failing tests once more. That should help cut down the failures except for > the mostly flaky of flakies. Many of our unit tests generate random cases and report the

Re: [DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-14 Thread Sean Busbey
On 2017-09-14 15:36, Chris Douglas wrote: > This has gotten bad enough that people are dismissing legitimate test > failures among the noise. > > On Thu, Sep 14, 2017 at 1:20 PM, Allen Wittenauer > wrote: > > Someone should probably invest some time into integrating the HBase > > fla

Re: [DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-14 Thread Andrew Wang
On Thu, Sep 14, 2017 at 1:59 PM, Sean Busbey wrote: > > > On 2017-09-14 15:36, Chris Douglas wrote: > > This has gotten bad enough that people are dismissing legitimate test > > failures among the noise. > > > > On Thu, Sep 14, 2017 at 1:20 PM, Allen Wittenauer > > wrote: > > > Someone

Re: [DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-14 Thread Arun Suresh
I actually like this idea: > One approach: do a dependency:list of each module and for those that show a change with the patch we run tests there. Can 'jdeps' be used to prune the list of sub modules on which we do pre-commit ? Essentially, we figure out which classes actually use the modified cl

Re: [DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-14 Thread Sean Busbey
On Thu, Sep 14, 2017 at 4:23 PM, Andrew Wang wrote: > > > > > I discussed this on yetus-dev a while back and Allen thought it'd be > non-trivial: > > https://lists.apache.org/thread.html/552ad614d1b3d5226a656b60c01084 > 57bcaa1219fb9ad985f8750ba1@%3Cdev.yetus.apache.org%3E > > I unfortunately don

Re: [DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-15 Thread Steve Loughran
1. I think maybe we should have the special ability or process for patches which change dependencies. We know that they have the potential for damage way beyond their .patch size and I fear them 2. like allen says, we can't afford to have a full test run on every patch. Because t hen the infra

Re: [DISCUSS] Can we make our precommit test robust to dependency changes while staying usable?

2017-09-15 Thread Erik Krogen
I am +1 for a way to specify additional modules that should be run. Always running all module dependencies would be prohibitively expensive (especially for hadoop-common patches) but developers should have a good understanding of which patches are more high-risk for downstream consumers and can