Re: your patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs
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: - nearly identical checks/tests jenkins would running - doesn't care about IDE/editor - cons: - slower - can be annoying to commit (locally) broken code for later squashing If something is too anoying to be run (such as blocking every patch for 3 minute unit tests, when the poor developer only wants to post his patch and go home) - developer would find a way to skip it. - editor/IDE marriage with tests/checks running - pros: - dev has full control over what runs in checks/tests - allows to commit dirty commit - shorter == quicker than the quickest jenkins option - cons: - depends on IDE/editor support - less checks/tests = higher risk +1. It boils down to developer and maintainer prudence. I have such a plugin in my ViM for static testing; Ido (and everyone else) should have one, too. I'm less sure about auto-running `make check` at rundom points in time. I did both with: intelliJ/PyCharm and vim, almost 100% sure PyDev allows this. either allows ease of running tests - in 1st case upon git commit, in the latter - via a button/shortcut in the devtool. I can help with setting up either to an early adopter. Then give it a week or two to get some feedback later how well it goes. Besides, we're also trying to speedup jenkins response all the time I would not mind to BLOCK merging before jenkins hook has responded - assuming that I (as a branch maintainer) can remove the jenkins reviewer from gerrit. There could be emenrgencies that cannot wait for the response. And of course, as a maintainer, I must be able to override the decision of the robot (by removing it from the reviewer list). ___ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra
Re: your patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs
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 bit the local work - pros: - nearly identical checks/tests jenkins would running - doesn't care about IDE/editor - cons: - slower - can be annoying to commit (locally) broken code for later squashing If something is too anoying to be run (such as blocking every patch for 3 minute unit tests, when the poor developer only wants to post his patch and go home) - developer would find a way to skip it. - editor/IDE marriage with tests/checks running - pros: - dev has full control over what runs in checks/tests - allows to commit dirty commit - shorter == quicker than the quickest jenkins option - cons: - depends on IDE/editor support - less checks/tests = higher risk +1. It boils down to developer and maintainer prudence. I have such a plugin in my ViM for static testing; Ido (and everyone else) should have one, too. I'm less sure about auto-running `make check` at rundom points in time. I did both with: intelliJ/PyCharm and vim, almost 100% sure PyDev allows this. either allows ease of running tests - in 1st case upon git commit, in the latter - via a button/shortcut in the devtool. I can help with setting up either to an early adopter. Then give it a week or two to get some feedback later how well it goes. Besides, we're also trying to speedup jenkins response all the time I would not mind to BLOCK merging before jenkins hook has responded - assuming that I (as a branch maintainer) can remove the jenkins reviewer from gerrit. There could be emenrgencies that cannot wait for the response. And of course, as a maintainer, I must be able to override the decision of the robot (by removing it from the reviewer list). I'm actually working on adding a new flag 'Continuous Integration' that can only be set by maintainers and the ci bot, and that requires +1 to merge (where -1 does not block). Does that make sense to you? (that way you can't rebase and merge before ci runs and -1, it's easier to handle permissions, it's easier to spot on the ui, is clearer it's purpose and does not overload another flag). -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization RD Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 pgpc5HfLQhfHR.pgp Description: PGP signature ___ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra
Re: your patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs
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 - doesn't care about IDE/editor - cons: - slower - can be annoying to commit (locally) broken code for later squashing - editor/IDE marriage with tests/checks running - pros: - dev has full control over what runs in checks/tests - allows to commit dirty commit - shorter == quicker than the quickest jenkins option - cons: - depends on IDE/editor support - less checks/tests = higher risk I did both with: intelliJ/PyCharm and vim, almost 100% sure PyDev allows this. either allows ease of running tests - in 1st case upon git commit, in the latter - via a button/shortcut in the devtool. I can help with setting up either to an early adopter. Then give it a week or two to get some feedback later how well it goes. Besides, we're also trying to speedup jenkins response all the time WDYT? On Wed, Apr 29, 2015 at 10:57:14PM +0100, Dan Kenigsberg wrote: 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 did not pass for patchset #6 Please do not merge patches with failing tests! Barak, thanks for reporting this mistake of ours. https://gerrit.ovirt.org/#/c/40408/ would fix it momentarily. I believe that it stems from two reasons: - Ido did not run `make check` or `make pyflakes` before ticking verified on the patch - I failed to wait for the jenkins job to finish. To make sure that this does not repeat I should avoid merging freshly-posted patches. Ido should take better care for pep8 and pyflakes. I have vim plugins that help me avoid such mistakes I hear that http://www.vim.org/scripts/script.php?script_id=4440 is better than what I actually have. Regards, Dan. ---end quoted text--- -- Max Kovgan Senior Software Engineer Red Hat - EMEA ENG Virtualization RD Tel.: +972 9769 2060 Email: mkovgan [at] redhat [dot] com Web: http://www.redhat.com RHT Global #: 82-72060 ___ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization RD Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 pgpyShd7PFMfv.pgp Description: PGP signature ___ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra
Re: your patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs
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 running - locally Maybe pre-push instead, that will leverage a bit the local work - pros: - nearly identical checks/tests jenkins would running - doesn't care about IDE/editor - cons: - slower - can be annoying to commit (locally) broken code for later squashing If something is too anoying to be run (such as blocking every patch for 3 minute unit tests, when the poor developer only wants to post his patch and go home) - developer would find a way to skip it. - editor/IDE marriage with tests/checks running - pros: - dev has full control over what runs in checks/tests - allows to commit dirty commit - shorter == quicker than the quickest jenkins option - cons: - depends on IDE/editor support - less checks/tests = higher risk +1. It boils down to developer and maintainer prudence. I have such a plugin in my ViM for static testing; Ido (and everyone else) should have one, too. I'm less sure about auto-running `make check` at rundom points in time. I did both with: intelliJ/PyCharm and vim, almost 100% sure PyDev allows this. either allows ease of running tests - in 1st case upon git commit, in the latter - via a button/shortcut in the devtool. I can help with setting up either to an early adopter. Then give it a week or two to get some feedback later how well it goes. Besides, we're also trying to speedup jenkins response all the time I would not mind to BLOCK merging before jenkins hook has responded - assuming that I (as a branch maintainer) can remove the jenkins reviewer from gerrit. There could be emenrgencies that cannot wait for the response. And of course, as a maintainer, I must be able to override the decision of the robot (by removing it from the reviewer list). I'm actually working on adding a new flag 'Continuous Integration' that can only be set by maintainers and the ci bot, and that requires +1 to merge (where -1 does not block). Does that make sense to you? (that way you can't rebase and merge before ci runs and -1, it's easier to handle permissions, it's easier to spot on the ui, is clearer it's purpose and does not overload another flag). Yes, it does! ___ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra
Re: your patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs
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) broken code for later squashing - editor/IDE marriage with tests/checks running - pros: - dev has full control over what runs in checks/tests - allows to commit dirty commit - shorter == quicker than the quickest jenkins option - cons: - depends on IDE/editor support - less checks/tests = higher risk I did both with: intelliJ/PyCharm and vim, almost 100% sure PyDev allows this. either allows ease of running tests - in 1st case upon git commit, in the latter - via a button/shortcut in the devtool. I can help with setting up either to an early adopter. Then give it a week or two to get some feedback later how well it goes. Besides, we're also trying to speedup jenkins response all the time WDYT? On Wed, Apr 29, 2015 at 10:57:14PM +0100, Dan Kenigsberg wrote: 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 did not pass for patchset #6 Please do not merge patches with failing tests! Barak, thanks for reporting this mistake of ours. https://gerrit.ovirt.org/#/c/40408/ would fix it momentarily. I believe that it stems from two reasons: - Ido did not run `make check` or `make pyflakes` before ticking verified on the patch - I failed to wait for the jenkins job to finish. To make sure that this does not repeat I should avoid merging freshly-posted patches. Ido should take better care for pep8 and pyflakes. I have vim plugins that help me avoid such mistakes I hear that http://www.vim.org/scripts/script.php?script_id=4440 is better than what I actually have. Regards, Dan. ---end quoted text--- -- Max Kovgan Senior Software Engineer Red Hat - EMEA ENG Virtualization RD Tel.: +972 9769 2060 Email: mkovgan [at] redhat [dot] com Web: http://www.redhat.com RHT Global #: 82-72060 pgpVvWjD8Pjxc.pgp Description: PGP signature ___ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra
Re: your patch https://gerrit.ovirt.org/#/c/40346/ broke oVirt vdsm jobs
- Original Message - From: Dan Kenigsberg dan...@redhat.com To: Barak Korren bkor...@redhat.com Cc: Ido Barkan ibar...@redhat.com, Francesco Romani from...@redhat.com, Infra infra@ovirt.org, rhev-ci rhev...@redhat.com Sent: Thursday, April 30, 2015 12:57:14 AM Subject: Re: your 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 make: *** [pyflakes] Error 1 You could clearly see that the tests did not pass for patchset #6 Please do not merge patches with failing tests! Barak, thanks for reporting this mistake of ours. https://gerrit.ovirt.org/#/c/40408/ would fix it momentarily. I believe that it stems from two reasons: - Ido did not run `make check` or `make pyflakes` before ticking verified on the patch - I failed to wait for the jenkins job to finish. We will configure Jenkins to set -1 to make such mistakes harder to make. To make sure that this does not repeat I should avoid merging freshly-posted patches. Ido should take better care for pep8 and pyflakes. I have vim plugins that help me avoid such mistakes I hear that http://www.vim.org/scripts/script.php?script_id=4440 is better than what I actually have. I recommend https://github.com/scrooloose/syntastic its the one ring to rule them all... Regards, Dan. ___ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra
Re: your 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 make: *** [pyflakes] Error 1 You could clearly see that the tests did not pass for patchset #6 Please do not merge patches with failing tests! Barak, thanks for reporting this mistake of ours. https://gerrit.ovirt.org/#/c/40408/ would fix it momentarily. I believe that it stems from two reasons: - Ido did not run `make check` or `make pyflakes` before ticking verified on the patch - I failed to wait for the jenkins job to finish. To make sure that this does not repeat I should avoid merging freshly-posted patches. Ido should take better care for pep8 and pyflakes. I have vim plugins that help me avoid such mistakes I hear that http://www.vim.org/scripts/script.php?script_id=4440 is better than what I actually have. Regards, Dan. ___ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra
