Re: your patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs

2015-05-05 Thread Dan Kenigsberg
On Tue, May 05, 2015 at 10:11:09AM +0200, David Caro wrote: On 05/05, Max Kovgan wrote: hi, Dan. makes sense to me to focus on 2 use cases: - pre-commit hook running everything jenkins is running - locally Maybe pre-push instead, that will leverage a bit the local work - pros:

Re: your patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs

2015-05-05 Thread David Caro
On 05/05, Dan Kenigsberg wrote: On Tue, May 05, 2015 at 10:11:09AM +0200, David Caro wrote: On 05/05, Max Kovgan wrote: hi, Dan. makes sense to me to focus on 2 use cases: - pre-commit hook running everything jenkins is running - locally Maybe pre-push instead, that will leverage a

Re: your patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs

2015-05-05 Thread David Caro
On 05/05, Max Kovgan wrote: hi, Dan. makes sense to me to focus on 2 use cases: - pre-commit hook running everything jenkins is running - locally Maybe pre-push instead, that will leverage a bit the local work - pros: - nearly identical checks/tests jenkins would running -

Re: your patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs

2015-05-05 Thread Dan Kenigsberg
On Tue, May 05, 2015 at 05:18:00PM +0200, David Caro wrote: On 05/05, Dan Kenigsberg wrote: On Tue, May 05, 2015 at 10:11:09AM +0200, David Caro wrote: On 05/05, Max Kovgan wrote: hi, Dan. makes sense to me to focus on 2 use cases: - pre-commit hook running everything jenkins is

Re: your patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs

2015-05-04 Thread Max Kovgan
hi, Dan. makes sense to me to focus on 2 use cases: - pre-commit hook running everything jenkins is running - locally - pros: - nearly identical checks/tests jenkins would running - doesn't care about IDE/editor - cons: - slower - can be annoying to commit (locally)

Re: your patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs

2015-04-30 Thread Barak Korren
patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs On Wed, Apr 29, 2015 at 11:16:37AM -0400, Barak Korren wrote: Patch does not pass pyflakes: ./tests/samplingTests.py:30: 'libvirtconnection' imported but unused ./tests/samplingTests.py:36: 'MonkeyPatch' imported but unused

Re: your patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs

2015-04-29 Thread Dan Kenigsberg
On Wed, Apr 29, 2015 at 11:16:37AM -0400, Barak Korren wrote: Patch does not pass pyflakes: ./tests/samplingTests.py:30: 'libvirtconnection' imported but unused ./tests/samplingTests.py:36: 'MonkeyPatch' imported but unused make: *** [pyflakes] Error 1 You could clearly see that the tests