[ovirt-devel] Errors when upgrading from 4.0 to master
Hi everyone! We have been seeing for a few days errors in the upgrade jobs for the 4.0->master process, for example: http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-4.0_el7_merged/58/ The error happens when starting the engine again after the upgrade, it does not start up and shows this message in the engine.log: 2016-06-03 07:27:22,331 ERROR [org.ovirt.engine.core.bll.network.macpool.MacPoolPerCluster] (ServerService Thread Pool -- 50) [] Error initializing: PreparedStatementCallback; bad SQL grammar [select * from getallmacsbymacpoolid(?)]; nested exception is org.postgresql.util.PSQLException: ERROR: column c.mac_pool_id does not exist Where: PL/pgSQL function getallmacsbymacpoolid(uuid) line 3 at RETURN QUERY The question is, should we be doing something different to what we did before on the upgrade process? Is this a bug? Thanks! -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Ovirt system tests review recording
Here you have the recording of the ovirt system tests review meeting: https://bluejeans.com/s/9FaJ/ There will be another one with a hands-on focus on which you'll finish running the tests on your laptop yourself. Enjoy! -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Sonarqube reports for ovirt-engine
On 05/05 11:04, Roman Mohr wrote: > HI all, > > I have asked sonarqube if they would add ovirt-engine to their hosted sonar > instance https://nemo.sonarqube.org/. They would be happy to do that. It is > free and for open source projects only. > > For those which are not familiar with sonarqube, it is a code quality > testing tool which gives a lot of architectural insights into applications > (coverity like features, code coverage, coupling, bunch of other OOP code > metrics, trends, ...). > > To integrate with nemo I would add a travis.yml file which would do the > build and analysis of ovirt-engine on travis and upload it to nemo. There > are no risks four our CI if travis or nemo have issues. > +1 from me > Best Regards, > Roman -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Permission issues when trying to migrate vm through the api (ovirt system tests)
Hi everyone! I'm having some issues when trying to run the ovirt system tests from ovirt master branch, and I need some help from you guys. The issue is that when trying to migrate a vm through the api, I get the error: RequestError: status: 400 reason: Bad Request detail: User is not authorized to perform this action. That does not happen when doing the same through the ui, the vm is migrated correctly. The engine logs don't add much more details: 2016-04-18 06:04:15,393 INFO [org.ovirt.engine.core.bll.MigrateVmToServerCommand] (default task-15) [29237280] No permission found for user '001a-001a-001a-001a-02dd' or one of the groups he is member of, when running action 'MigrateVmToServer', Required permissions are: Action type: 'USER' Action group: 'CREATE_VM' Object type: 'Cluster' Object ID: 'null'. 2016-04-18 06:04:15,393 WARN [org.ovirt.engine.core.bll.MigrateVmToServerCommand] (default task-15) [29237280] Validation of action 'MigrateVmToServer' failed for user admin@internal-authz. Reasons: VAR__ACTION__MIGRATE,VAR__TYPE__VM,USER_NOT_AUTHORIZED_TO_PERFORM_ACTION 2016-04-18 06:04:15,413 ERROR [org.ovirt.engine.api.restapi.resource.AbstractBackendResource] (default task-15) [] Operation Failed: [User is not authorized to perform this action.] Something that looks odd to me too, is that in the roles, when you edit the 'SuperUser' role (the one the admin user belongs to) there there's one permission missing, the 'VM->Provisioning Operations->Create Instance', and can't be added (it's greyed out), not sure if it's related though, I can pass you a screenshot if you want. I can give you access to an environment where that happens and more details/logs/etc if you want to look deeper into it. Thanks! -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [ACTION REQUIRED] vdsm_master_build-artifacts-fc23-x86_64 is failing due to missing dep on fc23 .packages
On 04/14 12:43, Sandro Bonazzola wrote: > On Thu, Apr 14, 2016 at 12:26 PM, Nir Soffer <nsof...@redhat.com> wrote: > > > On Thu, Apr 14, 2016 at 1:19 PM, David Caro <dc...@redhat.com> wrote: > > > On 04/14 13:17, Nir Soffer wrote: > > >> On Thu, Apr 14, 2016 at 12:55 PM, Vinzenz Feenstra <vfeen...@redhat.com> > > wrote: > > >> > > > >> >> On Apr 14, 2016, at 11:52 AM, Nir Soffer <nsof...@redhat.com> wrote: > > >> >> > > >> >> On Thu, Apr 14, 2016 at 12:23 PM, David Caro <dc...@redhat.com> > > wrote: > > >> >>> On 04/14 12:06, Dan Kenigsberg wrote: > > >> >>>> On Thu, Apr 14, 2016 at 11:56:10AM +0300, Dan Kenigsberg wrote: > > >> >>>>> > > >> >>>>> I've added the package it check-patch.packages, but not to > > >> >>>>> build-artifacts.packages. should be fixed in a minutes. Sorry. > > >> >>>> > > >> >>>> Actually, it's not that, as build-artifacts.packages is a softlink > > >> >>>> > > >> >>>>build-artifacts.packages.fc23 -> check-merged.packages.fc23 > > >> >>>> > > >> >>>> despite that, > > >> >>>> > > >> >>>> > > http://jenkins.ovirt.org/job/vdsm_master_build-artifacts-fc23-x86_64/823/consoleText > > >> >>>> > > >> >>>>mock \ > > >> >>>> > > --configdir="/home/jenkins/workspace/vdsm_master_build-artifacts-fc23-x86_64/vdsm" > > \ > > >> >>>>--root="mocker-fedora-23-x86_64.fc23" \ > > >> >>>>--install "autoconf automake git lago lago-ovirt > > libguestfs-tools-c m2crypto make mom policycoreutils-python pyflakes > > python-blivet python-coverage python-devel python-inotify python-ioprocess > > python-netaddr python-nose python-pep8 python-pthreading python-rtslib > > python-six python3-nose python3-six rpm-build sudo yum yum-utils grubby" \ > > >> >>>> > > --resultdir="logs/mocker-fedora-23-x86_64.fc23.install_packages" > > >> >>>> > > >> >>>> does not list python3-netaddr. Any idea why? > > >> >>> > > >> >>> That package is not in the packages list of the > > check-merged.packages.fc23 file > > >> >>> for that patch: > > >> >> > > >> >> We have single place for build requirements - vdsm.spec. How about > > >> >> using yum builddep to > > >> >> install dependencies instead of duplicating the data? > > >> > > > >> > That’s not really feasible, since we are generating vdsm.spec during > > the run of configure, then again configure requires some packages to be > > available…. > > >> > Not really the best of ideas > > >> > > >> Suggest a better way? > > > > > > > > > Maybe you should try to avoid generating the spec? at least, not the > > > 'buildrequires' section? (so it would be usable to extract the > > requirements > > > even if some sections of it will be changed later by > > autotools/make/whatever) > > > > Or maybe we should remove the build requirements from the spec, since it is > > > > Removing build requirements from the spec file will make the build > impossible in koji. > I guess he meant removing them from the spec file as 'source' for them, and generating the specfile from wherever they are defined. Not completely removing them from the spec and leave the spec without them > > > > not a portable solution (e.g. debian), and have simple list of packages > > for each > > supported platform. > > > > Milan, what is the solution that will work best for debian? > > > > Nir > > ___ > > Devel mailing list > > Devel@ovirt.org > > http://lists.ovirt.org/mailman/listinfo/devel > > > > > > -- > Sandro Bonazzola > Better technology. Faster innovation. Powered by community collaboration. > See how it works at redhat.com -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [ACTION REQUIRED] vdsm_master_build-artifacts-fc23-x86_64 is failing due to missing dep on fc23 .packages
On 04/14 13:17, Nir Soffer wrote: > On Thu, Apr 14, 2016 at 12:55 PM, Vinzenz Feenstra <vfeen...@redhat.com> > wrote: > > > >> On Apr 14, 2016, at 11:52 AM, Nir Soffer <nsof...@redhat.com> wrote: > >> > >> On Thu, Apr 14, 2016 at 12:23 PM, David Caro <dc...@redhat.com> wrote: > >>> On 04/14 12:06, Dan Kenigsberg wrote: > >>>> On Thu, Apr 14, 2016 at 11:56:10AM +0300, Dan Kenigsberg wrote: > >>>>> > >>>>> I've added the package it check-patch.packages, but not to > >>>>> build-artifacts.packages. should be fixed in a minutes. Sorry. > >>>> > >>>> Actually, it's not that, as build-artifacts.packages is a softlink > >>>> > >>>>build-artifacts.packages.fc23 -> check-merged.packages.fc23 > >>>> > >>>> despite that, > >>>> > >>>> http://jenkins.ovirt.org/job/vdsm_master_build-artifacts-fc23-x86_64/823/consoleText > >>>> > >>>>mock \ > >>>> > >>>> --configdir="/home/jenkins/workspace/vdsm_master_build-artifacts-fc23-x86_64/vdsm" > >>>> \ > >>>>--root="mocker-fedora-23-x86_64.fc23" \ > >>>>--install "autoconf automake git lago lago-ovirt > >>>> libguestfs-tools-c m2crypto make mom policycoreutils-python pyflakes > >>>> python-blivet python-coverage python-devel python-inotify > >>>> python-ioprocess python-netaddr python-nose python-pep8 > >>>> python-pthreading python-rtslib python-six python3-nose python3-six > >>>> rpm-build sudo yum yum-utils grubby" \ > >>>>--resultdir="logs/mocker-fedora-23-x86_64.fc23.install_packages" > >>>> > >>>> does not list python3-netaddr. Any idea why? > >>> > >>> That package is not in the packages list of the > >>> check-merged.packages.fc23 file > >>> for that patch: > >> > >> We have single place for build requirements - vdsm.spec. How about > >> using yum builddep to > >> install dependencies instead of duplicating the data? > > > > That’s not really feasible, since we are generating vdsm.spec during the > > run of configure, then again configure requires some packages to be > > available…. > > Not really the best of ideas > > Suggest a better way? Maybe you should try to avoid generating the spec? at least, not the 'buildrequires' section? (so it would be usable to extract the requirements even if some sections of it will be changed later by autotools/make/whatever) > > Nir > > > > >> > >> Keeping multiple requirements lists is not maintainable. > >> > >> Nir > >> ___ > >> Devel mailing list > >> Devel@ovirt.org > >> http://lists.ovirt.org/mailman/listinfo/devel > > -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [ACTION REQUIRED] vdsm_master_build-artifacts-fc23-x86_64 is failing due to missing dep on fc23 .packages
On 04/14 12:06, Dan Kenigsberg wrote: > On Thu, Apr 14, 2016 at 11:56:10AM +0300, Dan Kenigsberg wrote: > > > > I've added the package it check-patch.packages, but not to > > build-artifacts.packages. should be fixed in a minutes. Sorry. > > Actually, it's not that, as build-artifacts.packages is a softlink > > build-artifacts.packages.fc23 -> check-merged.packages.fc23 > > despite that, > > http://jenkins.ovirt.org/job/vdsm_master_build-artifacts-fc23-x86_64/823/consoleText > > mock \ > > --configdir="/home/jenkins/workspace/vdsm_master_build-artifacts-fc23-x86_64/vdsm" > \ > --root="mocker-fedora-23-x86_64.fc23" \ > --install "autoconf automake git lago lago-ovirt libguestfs-tools-c > m2crypto make mom policycoreutils-python pyflakes python-blivet > python-coverage python-devel python-inotify python-ioprocess python-netaddr > python-nose python-pep8 python-pthreading python-rtslib python-six > python3-nose python3-six rpm-build sudo yum yum-utils grubby" \ > --resultdir="logs/mocker-fedora-23-x86_64.fc23.install_packages" > > does not list python3-netaddr. Any idea why? That package is not in the packages list of the check-merged.packages.fc23 file for that patch: https://gerrit.ovirt.org/gitweb?p=vdsm.git;a=blob;f=automation/check-merged.packages.fc23;h=dd5c7f28805acf6c3f4c60b2606fb242b93a38f9;hb=d46574376cda5015551c5015df7f9dcfcec28493 > _______ > Devel mailing list > Devel@ovirt.org > http://lists.ovirt.org/mailman/listinfo/devel -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [lago-devel] Lago + ovirt-system-tests run fail on collecting logs
On 04/13 15:01, Yaniv Kaul wrote: > On Wed, Apr 13, 2016 at 2:02 PM, David Caro <dc...@redhat.com> wrote: > > > Though you will not have any caching and zram execution (just as the > > jenkins > > slaves don't have it). > > > > That should change. The sooner the better. So as I said, there's an usync between what you do and everyone else, it should be synced, either one side or the other, in any case, please open a task and prioritize properly, if it only affects the test scripts (like running on ram) then to bugzilla, if it affects lago, then github. > > If they have enough ram, they can actually run in /dev/shm/something. > That's what I'm doing with the VDSM functional tests. > Y. -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [lago-devel] Lago + ovirt-system-tests run fail on collecting logs
On 04/10 20:52, Yaniv Kaul wrote: > On Sun, Apr 10, 2016 at 6:06 PM, Eyal Edri <ee...@redhat.com> wrote: > > > test_logs/ > > > This is an annoying change of behavior. In the past, I believe the logs > were under the deployment dir. Now, they are here. It requires cleaning > them manually every time. Before it also required manual cleanup every time, it just turned out, that while doing the manual cleanup of the prefix (with ./run_suite -c) the logs were removed too (that was also an issue on jenkins, as you had to extract the logs before the cleanup) > It's part of issues we'll have to fix if we want (and I believe we do) > support multiple execution. It supports multiple execution as long as you are not running the same suite, same as before, the issue here is that you are using a very specific flow that is not used anywhere else, and thus, facing issues and user cases that noone else has. I really recommend: * Moving to the same flow jenkins uses * Moving jenkins to the same flow you use > I consider it as a regression in a way, since it's a changed behavior - and > I'm not sure for the better. It changed behavior yes, and it improved the log collection and cleanup procedures on jenkins. To alleviate that issue, I sent a patch for that in the beginning of the lago project that was never merged, feel free to open a task for that too, should be relatively easy to implement some kind of log rotation if the destination directory already exists. > Y. > ___ > lago-devel mailing list > lago-de...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/lago-devel -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [ACTION REQUIRED] Project vdsm_master_check-merged-el7-x86_64 failing consistently
On 04/12 18:16, Sandro Bonazzola wrote: > Hi, > Project vdsm_master_check-merged-el7-x86_64 failing consistently at least > since April 7. > > *00:15:04.646* * Starting VM vdsm_functional_tests_host-fc23: > *00:15:04.649* libvirt: QEMU Driver error : Cannot check QEMU binary > /usr/libexec/qemu-kvm: No such file or directory > > > Please fix functional test ASAP. > > CC infra so they're aware of the issue as well. That's caused by the tests when they run el* chroot on top of fedora* host, it can be easily fixed with this snippet running on the automation/* scripts (extracted from the lago ones): # Ugly fix to be able to run el* on fc* if ! [[ -e /usr/bin/qemu-kvm ]]; then ln -s /usr/libexec/qemu-kvm /usr/bin/qemu-kvm fi > > -- > Sandro Bonazzola > Better technology. Faster innovation. Powered by community collaboration. > See how it works at redhat.com > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [vdsm] Running VDSM unit tests on Travis CI using Docker
On 04/06 14:31, Yaniv Kaul wrote: > On Wed, Apr 6, 2016 at 2:23 PM, David Caro <dc...@redhat.com> wrote: > > > On 04/06 14:19, Edward Haas wrote: > > > On Wed, Apr 6, 2016 at 1:41 PM, Milan Zamazal <mzama...@redhat.com> > > wrote: > > > > > > > Edward Haas <eh...@redhat.com> writes: > > > > > > > > > On Wed, Apr 6, 2016 at 11:39 AM, Milan Zamazal <mzama...@redhat.com> > > > > wrote: > > > > > > > > > > Thank you, Edward, this is useful not only for CI. I use docker > > for > > > > > building Vdsm and running its unit tests and this helped me to > > get > > > > the > > > > > proper updated set of packages after recent changes in Vdsm. > > > > > > > > > > BTW, it seems that the following packages should be additionally > > > > added > > > > > for `make check-all': psmisc, which, python-ioprocess > > > > > > > > > > > > > > > Are you saying that make check is passing on your local machine? > > > > > > > > When I add the packages given above, `make check-all' (as well as `make > > > > check') works for me except for 4 tests in lib/vdsm/schedule.py that > > > > produce the following errors with `make check-all': > > > > > > > > File "/home/pdm/ovirt/vdsm/vdsm-test/lib/vdsm/schedule.py", line > > 134, > > > > in schedule > > > > heapq.heappush(self._calls, (deadline, call)) > > > > nose.proxy.TypeError: unorderable types: ScheduledCall() < > > > > ScheduledCall() > > > > > > > > File "/home/pdm/ovirt/vdsm/vdsm-test/tests/scheduleTests.py", line > > > > 160, in test_latency > > > > med = ticker.latency[len(ticker.latency) / 2] > > > > nose.proxy.TypeError: list indices must be integers, not float > > > > > > > > Those are probably Python 3 failures that should be fixed in Vdsm. > > > > The docker environment works fine for running the unit tests on my > > > > machine. > > > > > > > > > > I ran it on Travis CI with your recommended addition, and I am getting > > this > > > result: FAILED (SKIP=107, errors=14): > > > You can view the run here: > > https://travis-ci.org/EdDev/vdsm/builds/121117253 > > > > > > Afaik, you won't be able to run any tests that touch networking, or kernel > > modules (bonding and such). That is as much a limitation of travis as of > > docker, that was one of the points why we started using chroots instead of > > docker containers on ovirt ci. > > > > All of those should run fine in Lago, right? Yep, they run ok on chroot (with some mount magic) and they should run without any magic on lago > Y. > > > > > > > > > ___ > > > Devel mailing list > > > Devel@ovirt.org > > > http://lists.ovirt.org/mailman/listinfo/devel > > > > > > -- > > David Caro > > > > Red Hat S.L. > > Continuous Integration Engineer - EMEA ENG Virtualization R > > > > Tel.: +420 532 294 605 > > Email: dc...@redhat.com > > IRC: dcaro|dcaroest@{freenode|oftc|redhat} > > Web: www.redhat.com > > RHT Global #: 82-62605 > > > > ___ > > Devel mailing list > > Devel@ovirt.org > > http://lists.ovirt.org/mailman/listinfo/devel > > > ___ > Devel mailing list > Devel@ovirt.org > http://lists.ovirt.org/mailman/listinfo/devel -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [vdsm] Running VDSM unit tests on Travis CI using Docker
On 04/06 14:19, Edward Haas wrote: > On Wed, Apr 6, 2016 at 1:41 PM, Milan Zamazal <mzama...@redhat.com> wrote: > > > Edward Haas <eh...@redhat.com> writes: > > > > > On Wed, Apr 6, 2016 at 11:39 AM, Milan Zamazal <mzama...@redhat.com> > > wrote: > > > > > > Thank you, Edward, this is useful not only for CI. I use docker for > > > building Vdsm and running its unit tests and this helped me to get > > the > > > proper updated set of packages after recent changes in Vdsm. > > > > > > BTW, it seems that the following packages should be additionally > > added > > > for `make check-all': psmisc, which, python-ioprocess > > > > > > > > > Are you saying that make check is passing on your local machine? > > > > When I add the packages given above, `make check-all' (as well as `make > > check') works for me except for 4 tests in lib/vdsm/schedule.py that > > produce the following errors with `make check-all': > > > > File "/home/pdm/ovirt/vdsm/vdsm-test/lib/vdsm/schedule.py", line 134, > > in schedule > > heapq.heappush(self._calls, (deadline, call)) > > nose.proxy.TypeError: unorderable types: ScheduledCall() < > > ScheduledCall() > > > > File "/home/pdm/ovirt/vdsm/vdsm-test/tests/scheduleTests.py", line > > 160, in test_latency > > med = ticker.latency[len(ticker.latency) / 2] > > nose.proxy.TypeError: list indices must be integers, not float > > > > Those are probably Python 3 failures that should be fixed in Vdsm. > > The docker environment works fine for running the unit tests on my > > machine. > > > > I ran it on Travis CI with your recommended addition, and I am getting this > result: FAILED (SKIP=107, errors=14): > You can view the run here: https://travis-ci.org/EdDev/vdsm/builds/121117253 Afaik, you won't be able to run any tests that touch networking, or kernel modules (bonding and such). That is as much a limitation of travis as of docker, that was one of the points why we started using chroots instead of docker containers on ovirt ci. > ___ > Devel mailing list > Devel@ovirt.org > http://lists.ovirt.org/mailman/listinfo/devel -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [ATN] [ACTION REQUIRED] Tweaking engine CI flow
On 03/30 12:13, Eyal Edri wrote: > On Wed, Mar 30, 2016 at 11:56 AM, David Caro <dc...@redhat.com> wrote: > > > On 03/30 11:45, Roy Golan wrote: > > > Hi all, > > > > > > Eyal and I sat together to analyse and tweak the engine CI and this is > > what > > > we came up with: > > > > > > - dao test excludes updated - exclude dal/src/main/bundles from invoking > > > dao tests > > > A trivial update of validation messages without any db change would > > > trigger dao test without any need. This is one less job run for lots of > > > patches. > > > This change is effective now. > > > > I really recommend merging this into the standard ci scripts and doing the > > trigger/don't trigger stuff there > > > > Yea, thats the plan (discussed on another thread) but since it requires > some logic added of excluding certain path > we'll need to think how best to write it in check-patch.sh. vdsm is already doing something similar, you can look to their scripts > > > > > > > > > > - Spare CI re-run on trivial rebases > > > Gerrit trigger supports suppressing itself if the change to the tree > > was > > > trivial. Most of the waste of resources(time and IO :) ) is around > > > rebasing a change and waiting for CI to rerun. if Change1 is ci+1 and > > > Change2 is ci+1 the chances that they will break CI together is very > > small > > > and taking that risk is most probably worth it due to the huge resources > > > waste > > > > This is not exactly how it works, trivial rebases don't care if the > > previous > > patch had ci+1 or not, it just means that the rebase did not get any > > conflicts, > > that most of the time is not related to the tests working or not (it might > > be > > that someone change a method on another file, that your patch is using and > > it > > will be a trivial rebase and break the tests/compile). > > > > Even so, I think its worth doing a test for a limited amount of time (1 > week?) > while everyone is aware of it and to see if it cause errors, the chances > of 2 people working on the same file exactly during the same time and one > change failing another > is slim (If someone thinks otherwise and can give an example I'd be happy > to hear), > But since this is a gray area which we don't really know, I think its worth > the test. Imo the best approach here right now, in non-automatic tests, that is, run the tests on demand, that will avoid running on unneeded patches, on rebases or anything but still require the tests to be run before merging. That's what the workflow flag was meant to solve (as it's a lot easier than adding a new gerrit plugin that adds a button or similar to the ui). > > > > > > I don't really recommend skipping them. What you might meant is non-code > > changes, those are changes that only change the commit message, for > > example, if > > you have a patch, and you want to fix a typo in the commit message, > > currently > > that small commit message fix will require and trigger a ci run, while when > > skipping the non-code changes, it will not needed. > > > > > This change isn't effective yet - *Please reply here* if you agree or > > not > > > to make this change available. > > > > > > All of this is 'master' - 3.6 will follow if we will agree on activating > > > that change. > > > > > > Thanks, > > > Roy > > > > > ___ > > > Infra mailing list > > > in...@ovirt.org > > > http://lists.ovirt.org/mailman/listinfo/infra > > > > > > -- > > David Caro > > > > Red Hat S.L. > > Continuous Integration Engineer - EMEA ENG Virtualization R > > > > Tel.: +420 532 294 605 > > Email: dc...@redhat.com > > IRC: dcaro|dcaroest@{freenode|oftc|redhat} > > Web: www.redhat.com > > RHT Global #: 82-62605 > > > > ___ > > Devel mailing list > > Devel@ovirt.org > > http://lists.ovirt.org/mailman/listinfo/devel > > > > > > -- > Eyal Edri > Associate Manager > RHEV DevOps > EMEA ENG Virtualization R > Red Hat Israel > > phone: +972-9-7692018 > irc: eedri (on #tlv #rhev-dev #rhev-integ) -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [ATN] [ACTION REQUIRED] Tweaking engine CI flow
On 03/30 11:45, Roy Golan wrote: > Hi all, > > Eyal and I sat together to analyse and tweak the engine CI and this is what > we came up with: > > - dao test excludes updated - exclude dal/src/main/bundles from invoking > dao tests > A trivial update of validation messages without any db change would > trigger dao test without any need. This is one less job run for lots of > patches. > This change is effective now. I really recommend merging this into the standard ci scripts and doing the trigger/don't trigger stuff there > > - Spare CI re-run on trivial rebases > Gerrit trigger supports suppressing itself if the change to the tree was > trivial. Most of the waste of resources(time and IO :) ) is around > rebasing a change and waiting for CI to rerun. if Change1 is ci+1 and > Change2 is ci+1 the chances that they will break CI together is very small > and taking that risk is most probably worth it due to the huge resources > waste This is not exactly how it works, trivial rebases don't care if the previous patch had ci+1 or not, it just means that the rebase did not get any conflicts, that most of the time is not related to the tests working or not (it might be that someone change a method on another file, that your patch is using and it will be a trivial rebase and break the tests/compile). I don't really recommend skipping them. What you might meant is non-code changes, those are changes that only change the commit message, for example, if you have a patch, and you want to fix a typo in the commit message, currently that small commit message fix will require and trigger a ci run, while when skipping the non-code changes, it will not needed. > This change isn't effective yet - *Please reply here* if you agree or not > to make this change available. > > All of this is 'master' - 3.6 will follow if we will agree on activating > that change. > > Thanks, > Roy > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [VDSM] check_merged failures
On 03/08 18:42, Sandro Bonazzola wrote: > Adding infra This was already discussed on the lago-devel list, it has been fixed already (later than 147th build) > Il 08/Mar/2016 15:34, "Nir Soffer" <nsof...@redhat.com> ha scritto: > > > Hi all, > > > > The new check_merged job is failing randomly. I have see this error today: > > > > 14:20:16 @ Start specified VMs: > > 14:20:16 # Start nets: > > 14:20:16 * Create network vdsm_functional_tests_lago: > > 14:20:16 libvirt: error : internal error: Network is already in use > > by interface 7f0c-bc616ef > > 14:20:16 * Create network vdsm_functional_tests_lago: ERROR (in > > 0:00:00) > > 14:20:16 # Start nets: ERROR (in 0:00:00) > > 14:20:16 @ Start specified VMs: ERROR (in 0:00:00) > > 14:20:16 Error occured, aborting > > 14:20:16 Traceback (most recent call last): > > 14:20:16 File "/usr/lib/python2.7/site-packages/lago/cmd.py", line > > 539, in main > > 14:20:16 cli_plugins[args.verb].do_run(args) > > 14:20:16 File > > "/usr/lib/python2.7/site-packages/lago/plugins/cli.py", line 180, in > > do_run > > 14:20:16 self._do_run(**vars(args)) > > 14:20:16 File "/usr/lib/python2.7/site-packages/lago/cmd.py", line > > 132, in wrapper > > 14:20:16 return func(*args, > > prefix=lago_prefix.Prefix(prefix_path), **kwargs) > > 14:20:16 File "/usr/lib/python2.7/site-packages/lago/cmd.py", line > > 141, in wrapper > > 14:20:16 return func(*args, prefix=prefix, **kwargs) > > 14:20:16 File "/usr/lib/python2.7/site-packages/lago/cmd.py", line > > 191, in do_start > > 14:20:16 prefix.start(vm_names=vm_names) > > 14:20:16 File "/usr/lib/python2.7/site-packages/lago/prefix.py", > > line 819, in start > > 14:20:16 self.virt_env.start(vm_names=vm_names) > > 14:20:16 File "/usr/lib/python2.7/site-packages/lago/virt.py", line > > 187, in start > > 14:20:16 net.start() > > 14:20:16 File "/usr/lib/python2.7/site-packages/lago/virt.py", line > > 354, in start > > 14:20:16 self._env.libvirt_con.networkCreateXML(self._libvirt_xml()) > > 14:20:16 File "/usr/lib64/python2.7/site-packages/libvirt.py", line > > 4093, in networkCreateXML > > 14:20:16 if ret is None:raise libvirtError('virNetworkCreateXML() > > failed', conn=self) > > 14:20:16 libvirtError: internal error: Network is already in use by > > interface 7f0c-bc616ef > > > > Hopefully someone is owning the infrastructure and can look into this. > > > > Nir > > ___ > > Devel mailing list > > Devel@ovirt.org > > http://lists.ovirt.org/mailman/listinfo/devel > > > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] ovirt-3.6 Iscsi restart ends up on bad storage domains
Hi! I'm working on getting the ovirt system tests working for 3.6 with lago, and I have a setup with one vm for the engine, two for hosts, and two other vms for storage, one of them is nfs (that works well) and the other is for iscsi. The issue is with the iscsi one, it's an el7.2 vm, I use this script to set it up: https://gerrit.ovirt.org/#/c/53534/1/common/deploy-scripts/setup_storage_iscsi.sh That works well, I can add the storage domain, but if I reboot the iscsi vm, moving the domain into maintenance or not, when I start the vm again and try to activate the domain it complains with: VDSM command failed: Storage domain does not exist: (u'63af2399-f37d-441a-ac5d-09b65e64ed89',) VDSM lago_basic_suite_3_6_host0 command failed: Volume Group does not exist: (u'vg_uuid: rFDadB-GfrW-hHTw-Y5v5-zkWO-HOuC-Y3HAcV',) If I remove the domain, and manually add it again, it works well, until the next restart. Is there anything I'm not doing ok? I can provide logs if needed (just did not want to spam the list if it's a known issue). -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] PHX storage outage - all jenkins slaves affected
On 02/08 20:28, David Caro wrote: > > Hi everyone! > > There has been a storage outage today, it started around 17:30 CEST and > spanned > until ~20:15. All the services are back up now and running, but a bunch of > jenkins jobs failed due to the outage (all the slaves are using that storage) > so you might see some false positives in your ci runs. To retrigger you can > use > this job: > > http://jenkins.ovirt.org/gerrit_manual_trigger/ > > And/or submit a new patchset (rebasing should work). In any case, if you have > any issues or doubts, please respond to this email or ping me (dcaro/dcaroest) > on irc. > > Sorry for the inconvenience, we are gathering logs to find out what happend > and > prevent it from happening in the future. So the source of the issue has been sorted out, there was some uncoordinated effort that ended up changing the LACP settings in the switches for all the hosts, what caused a global network outage (all the hosts were affected) and that in turn caused the clustering to freeze as none of the nodes was able to contact the network both went down. Then, once the network came up, the master of the cluster tried to remount the drdb storage but was unable to due to some process keeping it busy, and did not fully start up. That is a scenario that we did not test (we tested one node failure, not both) so will have to investigate that failure case and find a solution for the clustering. We are also talking with the hosting to properly sync with us on that type of interventions so this will not happen again. Thanks for your patience > > -- > David Caro > > Red Hat S.L. > Continuous Integration Engineer - EMEA ENG Virtualization R > > Tel.: +420 532 294 605 > Email: dc...@redhat.com > IRC: dcaro|dcaroest@{freenode|oftc|redhat} > Web: www.redhat.com > RHT Global #: 82-62605 -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Proposing Martin Betak as oVirt UI maintainer
On 02/03 09:00, Vojtech Szocs wrote: > Hello, UI folks! > > I'd like to propose Martin Betak as oVirt UI maintainer. I've added him to the ovirt-engine-webadmin gerrit group, let me know if there's anything else needed > > Over time, Martin made some significant UI contributions: > > - improving UiCommon by utilizing Java generics [1] > > - adding GinUiBinder [2] to allow @Inject'ing GIN-managed > objects into GWT widgets created in ui.xml templates > > - upgrade GWT version to 2.6.1 [3] for both oVirt webapps > > - providing excellent feedback and ideas on oVirtJS project > [4] while demonstrating outstanding JavaScript knowledge > > [1] https://gerrit.ovirt.org/#/c/32907/ > [2] https://gerrit.ovirt.org/#/c/34954/ > [3] https://gerrit.ovirt.org/#/c/32135/ > [4] https://gerrit.ovirt.org/#/c/49466/ > > Personal note: Martin is familiar with oVirt GWT UI infra > as a whole (GWT-P component model, advanced GWT features, > UiCommon integration etc). His grasp on JS and its current > eco-system is very solid. > > Regards, > Vojtech > ___ > Devel mailing list > Devel@ovirt.org > http://lists.ovirt.org/mailman/listinfo/devel -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] PHX storage outage - all jenkins slaves affected
Hi everyone! There has been a storage outage today, it started around 17:30 CEST and spanned until ~20:15. All the services are back up now and running, but a bunch of jenkins jobs failed due to the outage (all the slaves are using that storage) so you might see some false positives in your ci runs. To retrigger you can use this job: http://jenkins.ovirt.org/gerrit_manual_trigger/ And/or submit a new patchset (rebasing should work). In any case, if you have any issues or doubts, please respond to this email or ping me (dcaro/dcaroest) on irc. Sorry for the inconvenience, we are gathering logs to find out what happend and prevent it from happening in the future. -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Automation CI for vdsm
I have an unstable temporary repo with el6, el7, fc22 and fc23 images that weight ~ 300 mb and are updated to latest packages, working on get it properly generated and be able to replace all the current images, that should ease the first init, but the reposetup is still an issue, you might consider not doing it if you have internet connection and don't care slowing ever run a bit (what it takes to download fron the repos whatever you want to install). We are also looking int being able to generate templates locally and/or upload them to the repos (so you install everything you need once, and then just reuse the locally generated image) David Caro El 31/1/2016 17:54, Barak Korren <bkor...@redhat.com> escribió: On 31 January 2016 at 14:15, Eyal Edri <ee...@redhat.com> wrote: > Adding lago-devel. > Anyone from Lago project can help debug why it is taking so long to setup > Lago (1:17 hours?) > Simple, Lago downloads some big images and does a full sync of some large package repos... Most of that should be a lot faster the 2nd time around, and also there is some work being done to shrink down the images. @Yaniv, you did the test on a machine that didn't run Lago before and was in TLV while using the repo in PHX right? -- Barak Korren bkor...@redhat.com RHEV-CI Team ___ Infra mailing list in...@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Jenkins overload and cancellation of ~1000 jobs
Hi everyone, Earlier today there was a huge load peak in jenkins that ended up with a job queue > 1000, we just removed from the queue most of the jobs to avoid stop servicing other projects (like for example the jobs related to releases and similar). That means that a lot of patches (mostly ovirt-engine) might get -1 or at least an error from jenkins, feel free to manually retrigger those patches here [1] [1] http://jenkins.ovirt.org/gerrit_manual_trigger/ Sorry for the inconveniences, -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Actively triggering of CI jobs
On 12/09 14:09, Amit Aviram wrote: > On Wed, Dec 9, 2015 at 12:15 PM, David Caro <dc...@redhat.com> wrote: > > > On 12/09 12:00, Amit Aviram wrote: > > > On Wed, Dec 9, 2015 at 11:37 AM, Barak Korren <bkor...@redhat.com> > > wrote: > > > > > > > >>> > [1]: http://www.ovirt.org/CI/Build_and_test_standards > > > > >>> > > > > > > > > > > > That's nice, but most of us are not aware of all that.. > > > > > > > > > > > > > Well, we can do a better job advocating that, I try to mention this > > > > almost in any infra/devel thread where 'CI' is mentioned. > > > > I'm open to suggestions about how to make developers more aware of the > > > > fact that the ultimate power to determine what happens in CI had > > > > mostly been placed in their hands... > > > > > > > > > What I'm offering is not letting us a choice, exactly because what you > > are > > > saying regarding the fact that most of the influence of what happens in > > CI > > > is in our hands. otherwise what happens is the current situation where > > > some/most of the developers are not aware of their options, or misses the > > > mails or whatever.. > > > > > > > > > > > > > > > > From what I'm seeing, most of the developers here don't make their > > > > patches > > > > > drafts.. moreover, > > > > > - personally I didn't even know that it will not trigger jobs if it > > is a > > > > > draft. (and I'm not the only one) > > > > > > > > Well, now you know... Adding 'devel' with hope more devs will read > > this. > > > > > > > > > - sometimes I need to label my patches, therefor can't make it a > > draft > > > > > > > > > By 'label' you mean set topic? > > > > Not sure those are mutually exclusive, 'git review' options seem to > > > > indicate they are not. I will look deeper into that. > > > > > > > > > nowadays we are waiting for the jobs too much to finish. and the > > reality > > > > is > > > > > that too much jobs shouldn't run at all- despite all of the nice > > things > > > > you > > > > > guys show here.. > > > > > > > > I which cases besides the patch not being "ready" (=draft...) should > > > > jobs not run? > > > > > > > > > > Most of the review process doesn't need the jobs to run. a patch has > > 5-10, > > > and sometimes much more sets until it is being merged- you don't need to > > > run the jobs every single time you are updating your patch.. > > > > > > > > > > > > > > > > > > > > I still think that it will be a better solution to force the > > developer to > > > > > activate the tests manually (by adding a flag when pushing or even > > doing > > > > it > > > > > with the jenkins client..) > > > > > > > > > We tried to add the 'workflow' flag for that at some point (It is used > > > > by most infra projects), but it was not accepted with any enthusiasm > > > > by the devs, you can search back the discussion on 'devel'. > > > > > > > > > The workflow makes job DISABLING optional. > > > I'm suggesting making job ENABLING optional, with some other flag.. > > > As we must run it to merge- it won't be missed, and will be triggered > > only > > > when needed. > > > > That's not true, the workflow make job enabling optional, if you don't set > > it, > > it will not run > > > > So I think we should use it :) > Why was it rejected? > Most people that answered just did not like the idea, they though that it was too much extra process for the developer, and that they had enough with the drafts for now. > > > > > > > > > > > > > > > > > > -- > > > > Barak Korren > > > > bkor...@redhat.com > > > > RHEV-CI Team > > > > > > > > > ___ > > > Infra mailing list > > > in...@ovirt.org > > > http://lists.ovirt.org/mailman/listinfo/infra > > > > > > -- > > David Caro > > > > Red Hat S.L. > > Continuous Integration Engineer - EMEA ENG Virtualization R > > > > Tel.: +420 532 294 605 > > Email: dc...@redhat.com > > IRC: dcaro|dcaroest@{freenode|oftc|redhat} > > Web: www.redhat.com > > RHT Global #: 82-62605 > > -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Request permissions for otopi maintainership
On 01/12 15:33, Eyal Edri wrote: > +1 > Done! > On Tue, Jan 12, 2016 at 3:26 PM, Tal Nisan <tni...@redhat.com> wrote: > > > You got my +1 > > > > On Tue, Jan 12, 2016 at 3:18 PM, Yedidyah Bar David <d...@redhat.com> > > wrote: > > > >> Hi, > >> > >> otopi access lists have as unique member Alon Bar-Lev who recently > >> left the project. > >> Since I work a lot on engine-setup which is based on otopi, I have > >> some good experience with it, and also worked on otopi itself in the > >> past, I'm stepping in, proposing myself as maintainer for that > >> project. > >> > >> Thanks, > >> -- > >> Didi > >> ___ > >> Devel mailing list > >> Devel@ovirt.org > >> http://lists.ovirt.org/mailman/listinfo/devel > >> > > > > > > ___ > > Devel mailing list > > Devel@ovirt.org > > http://lists.ovirt.org/mailman/listinfo/devel > > > > > > -- > Eyal Edri > Associate Manager > EMEA ENG Virtualization R > Red Hat Israel > > phone: +972-9-7692018 > irc: eedri (on #tlv #rhev-dev #rhev-integ) > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Request permissions for ovirt-host-deploy maintainership
On 01/12 13:17, Sandro Bonazzola wrote: > Hi, > ovirt-host-deploy access lists have as unique member Alon Bar-Lev which > recently left the project. > Since the python part is otopi based and I already had worked on it on the > past, I'm stepping in, proposing myself as maintainer for that project. > +1 (if my vote counts) > Thanks, > > -- > Sandro Bonazzola > Better technology. Faster innovation. Powered by community collaboration. > See how it works at redhat.com > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Request permissions for ovirt-host-deploy maintainership
On 01/12 08:02, Martin Perina wrote: > > > - Original Message - > > From: "Yedidyah Bar David" <d...@redhat.com> > > To: "Eyal Edri" <ee...@redhat.com> > > Cc: "David Caro" <dc...@redhat.com>, "infra" <in...@ovirt.org>, "devel" > > <devel@ovirt.org> > > Sent: Tuesday, January 12, 2016 2:00:54 PM > > Subject: Re: [ovirt-devel] Request permissions for ovirt-host-deploy > > maintainership > > > > On Tue, Jan 12, 2016 at 2:22 PM, Eyal Edri <ee...@redhat.com> wrote: > > > +1 > > > > > > On Tue, Jan 12, 2016 at 2:19 PM, David Caro <dc...@redhat.com> wrote: > > >> > > >> On 01/12 13:17, Sandro Bonazzola wrote: > > >> > Hi, > > >> > ovirt-host-deploy access lists have as unique member Alon Bar-Lev which > > >> > recently left the project. > > >> > Since the python part is otopi based and I already had worked on it on > > >> > the > > >> > past, I'm stepping in, proposing myself as maintainer for that project. > > >> > > > >> > > >> +1 (if my vote counts) > > > > +1 (same) > > +1 I suppose that's enough, access granted :) > > > > > >> > > >> > Thanks, > > >> > > > >> > -- > > >> > Sandro Bonazzola > > >> > Better technology. Faster innovation. Powered by community > > >> > collaboration. > > >> > See how it works at redhat.com > > >> > > >> > ___ > > >> > Infra mailing list > > >> > in...@ovirt.org > > >> > http://lists.ovirt.org/mailman/listinfo/infra > > >> > > >> > > >> -- > > >> David Caro > > >> > > >> Red Hat S.L. > > >> Continuous Integration Engineer - EMEA ENG Virtualization R > > >> > > >> Tel.: +420 532 294 605 > > >> Email: dc...@redhat.com > > >> IRC: dcaro|dcaroest@{freenode|oftc|redhat} > > >> Web: www.redhat.com > > >> RHT Global #: 82-62605 > > >> > > >> ___ > > >> Infra mailing list > > >> in...@ovirt.org > > >> http://lists.ovirt.org/mailman/listinfo/infra > > >> > > > > > > > > > > > > -- > > > Eyal Edri > > > Associate Manager > > > EMEA ENG Virtualization R > > > Red Hat Israel > > > > > > phone: +972-9-7692018 > > > irc: eedri (on #tlv #rhev-dev #rhev-integ) > > > > > > ___ > > > Devel mailing list > > > Devel@ovirt.org > > > http://lists.ovirt.org/mailman/listinfo/devel > > > > > > > > -- > > Didi > > ___ > > Devel mailing list > > Devel@ovirt.org > > http://lists.ovirt.org/mailman/listinfo/devel > > > ___ > Devel mailing list > Devel@ovirt.org > http://lists.ovirt.org/mailman/listinfo/devel -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] New git repositories ovirt-engine-api-model, ovirt-engine-api-metamodel and ovirt-engine-api-explorer
On 12/18 14:49, Sandro Bonazzola wrote: > +1 > Can you please create requested repositories? Repos created! For now, you have the gerrit projects, with you (Juan) as maintainer. They will update any bugs on the commits but will not check anything on the bugs (product or whatever). Let me know if you need bz-check hooks There's also github mirrors for each of them And the explorer has been populated with the contents of the github repo that you passed. Enjoy!! > > On Mon, Dec 14, 2015 at 8:02 PM, Juan Hernández <jhern...@redhat.com> wrote: > > > Hello, > > > > As part of the changes of the engine REST API for 4.0 two now components > > have been introduced: > > > > - The API model. This is the specification of the API, and it is > > currently part of the engine repository, in the > > backend/manager/modules/restapi/model directory. > > > > - The API metamodel. This is a set of tools that read the model and > > generate different artifacts from it, like the XML schema, the the > > JAX-RS interfaces, etc. It is also currently part of the engine > > repository, in the backend/manager/modules/restapi/metamodel. > > > > In order to support multiple versions of the specification inside the > > same engine the API model needs to be separated to a different git > > repository, its releases managed independently of the engine, and the > > artifacts uploaded to Maven central. > > > > The metamodel also needs to be separated to a different repository, and > > have its own relase cycle, and the artifacts uploaded to Maven central, > > as they will be required by the engine and by the generators of the SDKs. > > > > Thus I'm requesting the creation of two new git repositories in > > gerrit.ovirt.org to hold these components: > > > > ovirt-engine-api-model > > ovirt-engine-api-metamodel > > > > In addition version 4.0 will also include a web application used to > > explore the automatically generated reference documentation. This > > application is currently hosted here: > > > > https://github.com/jhernand/ovirt-api-explorer > > > > But I think it should be hosted in gerrit.ovirt.org, thus I'm also > > requesting the creation of another git repository: > > > > ovirt-engine-api-explorer > > > > All in call, can you please create these three git repositories and add > > me as maintainer, please? > > > > Thanks in advance, > > Juan Hernandez > > > > -- > > Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta > > 3ºD, 28016 Madrid, Spain > > Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat S.L. > > ___ > > Infra mailing list > > in...@ovirt.org > > http://lists.ovirt.org/mailman/listinfo/infra > > > > > > -- > Sandro Bonazzola > Better technology. Faster innovation. Powered by community collaboration. > See how it works at redhat.com -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Lago testing environment contribution
On 12/15 15:29, Roman Mohr wrote: > Hi, > > On Tue, Dec 15, 2015 at 3:19 PM, Yaniv Bronheim <ybron...@redhat.com> wrote: > > > Hi list, > > > > If you're not aware yet about Lago you should. This our engine to test > > functional flows over variant distributions for ovirt-engine and vdsm > > This project is currently maintained by dcaro but it will love to have > > more contribution in reviews, tests and more. > > > > If you want to follow it I encourage to join the mailing list > > lago-de...@ovirt.org and help with reviews if possible ([2]) > > > > You can also checkout following links for more info > > In [3] you can find full readme about how to use it to run ovirt-engine > > tests. should be intuitive and quick. If not, please raise > > > > > I would love to see a short paragraph on github and on readthedocs on what > this project is about and what the aims are. > It is really hard to even find out what it is for when you are reading the > documentation. Agree, though it's not yet so clear what it is for or what the aims are... But for now one could say something like: """ Provide a simple and easy way to create virtual environments aimed for functional/system tests, being able to run them on a standard development laptop. """ If that makes sense I'll add it to the docs > > > > [1] https://gerrit.ovirt.org/#/admin/projects/lago > > [2] https://gerrit.ovirt.org/#/q/project:+lago+status:+open > > [3] https://github.com/ovirt/lago > > > > Hope you'll find it interesting and Lago will get more attention by our > > community > > > > Greetings, > > > > -- > > *Yaniv Bronhaim.* > > > > ___ > > Devel mailing list > > Devel@ovirt.org > > http://lists.ovirt.org/mailman/listinfo/devel > > -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Lago testing environment contribution
On 12/15 15:46, David Caro Estevez wrote: > On 12/15 15:29, Roman Mohr wrote: > > Hi, > > > > On Tue, Dec 15, 2015 at 3:19 PM, Yaniv Bronheim <ybron...@redhat.com> wrote: > > > > > Hi list, > > > > > > If you're not aware yet about Lago you should. This our engine to test > > > functional flows over variant distributions for ovirt-engine and vdsm > > > This project is currently maintained by dcaro but it will love to have > > > more contribution in reviews, tests and more. > > > > > > If you want to follow it I encourage to join the mailing list > > > lago-de...@ovirt.org and help with reviews if possible ([2]) > > > > > > You can also checkout following links for more info > > > In [3] you can find full readme about how to use it to run ovirt-engine > > > tests. should be intuitive and quick. If not, please raise > > > > > > > > I would love to see a short paragraph on github and on readthedocs on what > > this project is about and what the aims are. > > It is really hard to even find out what it is for when you are reading the > > documentation. > > Agree, though it's not yet so clear what it is for or what the aims are... > > But for now one could say something like: > > """ > Provide a simple and easy way to create virtual environments aimed for > functional/system tests, being able to run them on a standard development > laptop. > """ > > If that makes sense I'll add it to the docs I see barak ever opened a bug on it :) https://bugzilla.redhat.com/show_bug.cgi?id=1291753 > > > > > > > > > [1] https://gerrit.ovirt.org/#/admin/projects/lago > > > [2] https://gerrit.ovirt.org/#/q/project:+lago+status:+open > > > [3] https://github.com/ovirt/lago > > > > > > Hope you'll find it interesting and Lago will get more attention by our > > > community > > > > > > Greetings, > > > > > > -- > > > *Yaniv Bronhaim.* > > > > > > ___ > > > Devel mailing list > > > Devel@ovirt.org > > > http://lists.ovirt.org/mailman/listinfo/devel > > > > > -- > David Caro > > Red Hat S.L. > Continuous Integration Engineer - EMEA ENG Virtualization R > > Tel.: +420 532 294 605 > Email: dc...@redhat.com > IRC: dcaro|dcaroest@{freenode|oftc|redhat} > Web: www.redhat.com > RHT Global #: 82-62605 > ___ > Devel mailing list > Devel@ovirt.org > http://lists.ovirt.org/mailman/listinfo/devel -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] About jenkins load
While trying to figure out a solution to the current jenkins load issue, I got some stats on the current build execution in jenkins, and just so we all are aware of the numbers. In the last month, the number of builds per day has been multiplied by 1750%, that is, from ~500 build/day tops to more than 8750 build/day The percentage of failures has gone down too (around 5% in the last week). That does not mean that the false negatives is going down too, but for sure it means that it's not raising a lot if raising at all. For the last 7 days (including weekend) we ran 25858 builds, from which only 928 were failures (combined real and false negatives, no easy way to know yet) The daily average build time is at around 10min, that does not include the time waiting in the queue though. For the queues, we had a big peak of >750 queued jobs on tuesday (lasting until thursday), with 4 smaller peaks of 300-200 jobs on last week's monday,tuesday and wednesday and on this week's monday. Will keep digging into this, cheers! -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Actively triggering of CI jobs
On 12/09 12:00, Amit Aviram wrote: > On Wed, Dec 9, 2015 at 11:37 AM, Barak Korren <bkor...@redhat.com> wrote: > > > >>> > [1]: http://www.ovirt.org/CI/Build_and_test_standards > > >>> > > > > > > > That's nice, but most of us are not aware of all that.. > > > > > > > Well, we can do a better job advocating that, I try to mention this > > almost in any infra/devel thread where 'CI' is mentioned. > > I'm open to suggestions about how to make developers more aware of the > > fact that the ultimate power to determine what happens in CI had > > mostly been placed in their hands... > > > What I'm offering is not letting us a choice, exactly because what you are > saying regarding the fact that most of the influence of what happens in CI > is in our hands. otherwise what happens is the current situation where > some/most of the developers are not aware of their options, or misses the > mails or whatever.. > > > > > > > > From what I'm seeing, most of the developers here don't make their > > patches > > > drafts.. moreover, > > > - personally I didn't even know that it will not trigger jobs if it is a > > > draft. (and I'm not the only one) > > > > Well, now you know... Adding 'devel' with hope more devs will read this. > > > > > - sometimes I need to label my patches, therefor can't make it a draft > > > > > By 'label' you mean set topic? > > Not sure those are mutually exclusive, 'git review' options seem to > > indicate they are not. I will look deeper into that. > > > > > nowadays we are waiting for the jobs too much to finish. and the reality > > is > > > that too much jobs shouldn't run at all- despite all of the nice things > > you > > > guys show here.. > > > > I which cases besides the patch not being "ready" (=draft...) should > > jobs not run? > > > > Most of the review process doesn't need the jobs to run. a patch has 5-10, > and sometimes much more sets until it is being merged- you don't need to > run the jobs every single time you are updating your patch.. > > > > > > > > > > I still think that it will be a better solution to force the developer to > > > activate the tests manually (by adding a flag when pushing or even doing > > it > > > with the jenkins client..) > > > > > We tried to add the 'workflow' flag for that at some point (It is used > > by most infra projects), but it was not accepted with any enthusiasm > > by the devs, you can search back the discussion on 'devel'. > > > The workflow makes job DISABLING optional. > I'm suggesting making job ENABLING optional, with some other flag.. > As we must run it to merge- it won't be missed, and will be triggered only > when needed. That's not true, the workflow make job enabling optional, if you don't set it, it will not run > > > > > > > -- > > Barak Korren > > bkor...@redhat.com > > RHEV-CI Team > > > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Artifactory.ovirt.org dns entry changed
Hi everyone, In order to finish the migration to the new artifactory server, I've just changed the dns entry, the change should take effect in 10 min tops. There are no changes needed anywhere else so it should be in effect immediately, remember that if you have patches on ovirt-engine 3.6, 3.6.1 or 3.5 branches, and you have not rebased your patches in the last week, the ci for you patches will start failing, so please do so :) Enjoy!! -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] [2015-12-01] Weekly infra team status
Hi everyone, this week updates: * Jenkins slaves stabilized After last week's upgrade, some of the slaves got wrongly configured, during this week all the slaves have been checked and fixed any issues found. * Artifactory migration We are migrating to an artifactory server on PHX hosting, everything is ready but if you have any ovirt-engine patches and you haven't rebased lately, please do, or your patches will start failing the tests on thursday morning (when the final migration will be made). * Ticketing system for infra issues We have almost ready the setup to create infra tickets from emails, that will allow us to handle better the requests and gather statistics, to avoid requests being forgotten or unanswered. (Thanks Vishnu!) Will send a dedicated email once it's ready, for now you can still use the jira page [1] or infa mailing list * Ovirt system tests are now running on two dedicated slaves, ovirt-srv08 and ovirt-srv09, which were manually setup. * New docs for repoman project [2] They are just dumped from the code, need some formatting and proper structure, but it's a big advance [1] https://ovirt-jira.atlassian.net [2] http://repoman.readthedocs.org/en/latest -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Improving our github presense
On 11/30 20:46, John Hunter wrote: > Definitely this is a good idea, lots of people hang out on github, this will > attract more contributor :) You mean having the github org with people added or having to maintain teams also on github? Imo having people added to the org on github is enough visibility, adding teams there gives no extra visibility > > Cheers, > Zhao > > On Mon, Nov 30, 2015 at 6:08 PM, David Caro <dc...@redhat.com> wrote: > > > > > I don't think that's a good idea, we are already maintaining such a list > > in the > > gerrit groups, doing so also on github would duplicate the effort to > > maintain > > such a list in sync and there's no real usage of it anywhere. > > > > On 11/29 10:15, Barak Korren wrote: > > > Now that we have members, we could also create teams to provide some > > > transparency to who does what in oVirt. > > > > > > I've created a team of oVirt infra: > > > https://github.com/orgs/oVirt/teams/ovirt-infra and added the members > > > I could find. I suggest others follow suit and create their own teams. > > > > > > On 21 November 2015 at 01:40, Nir Soffer <nsof...@redhat.com> wrote: > > > > We have now 39 members - but only 13 are public. > > > > > > > > To make yourself public, visit > > > > https://github.com/orgs/oVirt/people > > > > ___ > > > > Devel mailing list > > > > Devel@ovirt.org > > > > http://lists.ovirt.org/mailman/listinfo/devel > > > > > > > > > > > > -- > > > Barak Korren > > > bkor...@redhat.com > > > RHEV-CI Team > > > > -- > > David Caro > > > > Red Hat S.L. > > Continuous Integration Engineer - EMEA ENG Virtualization R > > > > Tel.: +420 532 294 605 > > Email: dc...@redhat.com > > IRC: dcaro|dcaroest@{freenode|oftc|redhat} > > Web: www.redhat.com > > RHT Global #: 82-62605 > > > > ___ > > Devel mailing list > > Devel@ovirt.org > > http://lists.ovirt.org/mailman/listinfo/devel > > > > > > -- > Best regards > Junwang Zhao > Department of Computer Science > Peking University > Beijing, 100871, PRC -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [vdsm] strange network test failure on FC23
I'm retriggering on another slave to see if it fails there too, might be env related. But that would not discard the race issue (as it might be just a coincidence or the slave a bit faster) On 11/27 11:55, Francesco Romani wrote: > Hi, > > Jenkins doesn't like my (trivial) https://gerrit.ovirt.org/#/c/49271/ > > which is about moving one log line (!). > > The failure is > > 00:08:54.680 > == > 00:08:54.680 FAIL: testEnablePromisc (ipwrapperTests.TestDrvinfo) > 00:08:54.680 > -- > 00:08:54.680 Traceback (most recent call last): > 00:08:54.680 File > "/home/jenkins/workspace/vdsm_master_check-patch-fc23-x86_64/vdsm/tests/ipwrapperTests.py", > line 130, in testEnablePromisc > 00:08:54.680 "Could not enable promiscuous mode.") > 00:08:54.680 AssertionError: Could not enable promiscuous mode. > 00:08:54.680 >> begin captured logging << > > 00:08:54.680 root: DEBUG: /usr/bin/taskset --cpu-list 0-1 /usr/sbin/brctl > show (cwd None) > 00:08:54.680 root: DEBUG: SUCCESS: = ''; = 0 > 00:08:54.680 root: DEBUG: /usr/bin/taskset --cpu-list 0-1 /sbin/ip link add > name vdsm-HIRjJp type bridge (cwd None) > 00:08:54.680 root: DEBUG: SUCCESS: = ''; = 0 > 00:08:54.680 root: DEBUG: /usr/bin/taskset --cpu-list 0-1 /sbin/ip link set > dev vdsm-HIRjJp up (cwd None) > 00:08:54.680 root: DEBUG: SUCCESS: = ''; = 0 > 00:08:54.680 root: DEBUG: /usr/bin/taskset --cpu-list 0-1 /sbin/ip link set > dev vdsm-HIRjJp promisc on (cwd None) > 00:08:54.680 root: DEBUG: SUCCESS: = ''; = 0 > 00:08:54.680 - >> end captured logging << > - > > > Here in fullest: > http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc23-x86_64/638/console > > The command like looks OK, and can't think any reason it could fail, except > startup race. > Using taskset, the ip command now takes a little longer to complete. > > Maybe -just wild guessing- the code isn't properly waiting for the command to > complete? > Otherwise not the slightest clue :) > > Bests, > > -- > Francesco Romani > RedHat Engineering Virtualization R & D > Phone: 8261328 > IRC: fromani > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [vdsm] strange network test failure on FC23
I see though that is leaving a bunch of test interfaces in the slave: 2753: vdsmtest-gNhf3: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UNKNOWN group default link/ether 86:73:13:4c:e2:63 brd ff:ff:ff:ff:ff:ff 2767: vdsmtest-aX5So: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UNKNOWN group default link/ether 9e:fa:75:3e:a3:e6 brd ff:ff:ff:ff:ff:ff 2768: vdsmtest-crso1: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UNKNOWN group default link/ether 22:ce:cb:5c:42:3b brd ff:ff:ff:ff:ff:ff 2772: vdsmtest-JDc5P: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UNKNOWN group default link/ether ae:79:dc:e9:22:9a brd ff:ff:ff:ff:ff:ff Can we do a cleanup in the tests and remove those? That might collide with other tests and create failures. On 11/27 19:07, David Caro wrote: > > I'm retriggering on another slave to see if it fails there too, might be env > related. But that would not discard the race issue (as it might be just a > coincidence or the slave a bit faster) > > On 11/27 11:55, Francesco Romani wrote: > > Hi, > > > > Jenkins doesn't like my (trivial) https://gerrit.ovirt.org/#/c/49271/ > > > > which is about moving one log line (!). > > > > The failure is > > > > 00:08:54.680 > > == > > 00:08:54.680 FAIL: testEnablePromisc (ipwrapperTests.TestDrvinfo) > > 00:08:54.680 > > -- > > 00:08:54.680 Traceback (most recent call last): > > 00:08:54.680 File > > "/home/jenkins/workspace/vdsm_master_check-patch-fc23-x86_64/vdsm/tests/ipwrapperTests.py", > > line 130, in testEnablePromisc > > 00:08:54.680 "Could not enable promiscuous mode.") > > 00:08:54.680 AssertionError: Could not enable promiscuous mode. > > 00:08:54.680 >> begin captured logging << > > > > 00:08:54.680 root: DEBUG: /usr/bin/taskset --cpu-list 0-1 /usr/sbin/brctl > > show (cwd None) > > 00:08:54.680 root: DEBUG: SUCCESS: = ''; = 0 > > 00:08:54.680 root: DEBUG: /usr/bin/taskset --cpu-list 0-1 /sbin/ip link add > > name vdsm-HIRjJp type bridge (cwd None) > > 00:08:54.680 root: DEBUG: SUCCESS: = ''; = 0 > > 00:08:54.680 root: DEBUG: /usr/bin/taskset --cpu-list 0-1 /sbin/ip link set > > dev vdsm-HIRjJp up (cwd None) > > 00:08:54.680 root: DEBUG: SUCCESS: = ''; = 0 > > 00:08:54.680 root: DEBUG: /usr/bin/taskset --cpu-list 0-1 /sbin/ip link set > > dev vdsm-HIRjJp promisc on (cwd None) > > 00:08:54.680 root: DEBUG: SUCCESS: = ''; = 0 > > 00:08:54.680 - >> end captured logging << > > - > > > > > > Here in fullest: > > http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc23-x86_64/638/console > > > > The command like looks OK, and can't think any reason it could fail, except > > startup race. > > Using taskset, the ip command now takes a little longer to complete. > > > > Maybe -just wild guessing- the code isn't properly waiting for the command > > to complete? > > Otherwise not the slightest clue :) > > > > Bests, > > > > -- > > Francesco Romani > > RedHat Engineering Virtualization R & D > > Phone: 8261328 > > IRC: fromani > > ___ > > Infra mailing list > > in...@ovirt.org > > http://lists.ovirt.org/mailman/listinfo/infra > > -- > David Caro > > Red Hat S.L. > Continuous Integration Engineer - EMEA ENG Virtualization R > > Tel.: +420 532 294 605 > Email: dc...@redhat.com > IRC: dcaro|dcaroest@{freenode|oftc|redhat} > Web: www.redhat.com > RHT Global #: 82-62605 -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] [ACTION REQUIRED] Any ovirt-engine master/3.6/3.6.1 patches must be rebased to include automation changes
In order to make more generic and to allow migrating the artifactory server that currently is providing us with the artifacts to build and test ovirt-engine, the port of the url we were using has been changed: master -> https://gerrit.ovirt.org/#/c/49275/ 3.6 -> https://gerrit.ovirt.org/#/c/49276/ 3.6.1 -> https://gerrit.ovirt.org/#/c/49277/ The migration of the artifactory server will be done later next week, so if you don't want your ci tests to start failing at that point, please rebase any patches that you might have for those branches on top of those commits (they are merged already). This also allown to avoid having to change those scripts anymore for future migrations Thanks! -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Ovirt upgrade job failing
Hi! Since yesterday morning the upgrade job from 3.6 to master [1] has been failing on applying the schema.sh, specifically: 2015-11-26 08:12:54 DEBUG otopi.plugins.ovirt_engine_setup.ovirt_engine.db.schema plugin.execute:941 execute-output: ['/usr/share/ovirt-engine/dbscripts/schema.sh', '-s', 'localhost', '-p', '5432', '-u', 'engine', '-d', 'engine', '-l', '/var/log/ovirt-engine/setup/ovirt-engine-setup-20151126081101-884dt8.log', '-c', 'apply'] stderr: psql:/usr/share/ovirt-engine/dbscripts/create_views.sql:108: ERROR: column snapshots.memory_metadata_disk_id does not exist LINE 59: snapshots.memory_metadata_disk_id, ^ FATAL: Cannot execute sql command: --file=/usr/share/ovirt-engine/dbscripts/create_views.sql I see that the line was introduced by [2], though it did not fail back then (so the issue might not be there), it might also come from 3.6 branch, but I need a bit of help tracking that down from any of the db guys This is making all the builds of that job fail, and it might be masking other errors (and giving -1 to a lot of patches). Thanks! [1] http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-3.6_el7_merged/986/ [2] https://gerrit.ovirt.org/#/c/47262/ -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] [2015-11-23] Weekly infra team status
Hi everyone, last week updates: * Jenkins upgraded to 1.638 from 1.60 This change cause a little havoc on friday builds, as the upgrade was not as easy as it used to, there was an issue with the slaves config that reverted it to an old version, making most of the slaves unavailable and the available ones unstable, that was fixed on friday too, but due to another unrelated network issue, it persisted a bit more than expected. Thanks amachuk++! * Network outage on friday On friday there was an issue with the vendor that provides communication between the PHX datacenter and europe, causing a huge delay and instability in the network, affecting the upgrade process above, it was partially solved on friday evening, and it's totally solved now. * Repoproxy issues On sunday there was a few hours where one of the mirroring services we have on phx stopped working and made any jobs that use it fail (with unable to contact repo), that was solved on site and most of the patches affected were retriggered manually, thanks eedri for handling this. NOTE: So for all that have failed jobs from friday/saturday/sunday that were not retriggered, to get it retriggered just take note of the last patch in the change that changed something more than just the commit, then go to [1] and enter the change-id/commit-id, **make sure you hit the 'Inclued All Patchsets' checkbox, then tick the change with more than just commit changes, and hit 'trigger selected'. That will retrigger the tests only for that patch, but the result of that run will be propagated to all the patches that have no code changes from that one. [1]: http://jenkins.ovirt.org/gerrit_manual_trigger/ * Github group We started adding people to the Github organization to get more visibility in Github for the oVirt project, and so far we have 41 members :) Just make sure to set yourself as 'public' member there, so anyone can see you are part of it! If anyone has not received the invite, please send me an email or ping me on irc and I'll send you one. -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Improving our github presense
On 11/18 17:16, Roman Mohr wrote: > On Wed, Nov 18, 2015 at 2:47 PM, Ewoud Kohl van Wijngaarden < > ewoud+ov...@kohlvanwijngaarden.nl> wrote: > > > On Wed, Nov 18, 2015 at 12:49:44AM +0200, Nir Soffer wrote: > > > Hi all, > > > > > > Looking in ovirt github page, there are only 2 members: > > > https://github.com/orgs/oVirt/people > > > > > > Is is possible to add automatically all contributors to this list? > > > > Why is that needed? It implies those people can actually do something on > > the org while it's just a mirror. If you actually look at contributors, > > then github will find them. > > https://github.com/oVirt/vdsm/graphs/contributors > > for example. > > > > I would appreciate if we would have more people in the organization there, > although it is only a mirror. > We would then also mirror the liveliness of our project on one of the most > important software platforms. Fair enough, I can create a 'developers' group there with only read rights and add people, any of you that wants to be added there, please reply this message only to me (to avoid spamming the list) with your github user and I'll send you an invite. Let's get noticed :) > > > > > ___ > > Devel mailing list > > Devel@ovirt.org > > http://lists.ovirt.org/mailman/listinfo/devel > > > > Roman > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Improving our github presense
On 11/18 18:08, David Caro wrote: > On 11/18 17:16, Roman Mohr wrote: > > On Wed, Nov 18, 2015 at 2:47 PM, Ewoud Kohl van Wijngaarden < > > ewoud+ov...@kohlvanwijngaarden.nl> wrote: > > > > > On Wed, Nov 18, 2015 at 12:49:44AM +0200, Nir Soffer wrote: > > > > Hi all, > > > > > > > > Looking in ovirt github page, there are only 2 members: > > > > https://github.com/orgs/oVirt/people > > > > > > > > Is is possible to add automatically all contributors to this list? > > > > > > Why is that needed? It implies those people can actually do something on > > > the org while it's just a mirror. If you actually look at contributors, > > > then github will find them. > > > https://github.com/oVirt/vdsm/graphs/contributors > > > for example. > > > > > > > I would appreciate if we would have more people in the organization there, > > although it is only a mirror. > > We would then also mirror the liveliness of our project on one of the most > > important software platforms. > > > Fair enough, I can create a 'developers' group there with only read rights and > add people, any of you that wants to be added there, please reply this message > only to me (to avoid spamming the list) with your github user and I'll send > you > an invite. > > Let's get noticed :) A couple things: * I have sent a bunch of invites to those that had any commits on vdsm or engine (only the first pages of the list in github) and had their users linked there, if you don't want to accept it just ignore it * Anyone that has accepted the invite, pleas set yourself as public member, so everyone can see that we are a bunch behind oVirt :) > > > > > > > > > > ___ > > > Devel mailing list > > > Devel@ovirt.org > > > http://lists.ovirt.org/mailman/listinfo/devel > > > > > > > Roman > > > ___ > > Infra mailing list > > in...@ovirt.org > > http://lists.ovirt.org/mailman/listinfo/infra > > > -- > David Caro > > Red Hat S.L. > Continuous Integration Engineer - EMEA ENG Virtualization R > > Tel.: +420 532 294 605 > Email: dc...@redhat.com > IRC: dcaro|dcaroest@{freenode|oftc|redhat} > Web: www.redhat.com > RHT Global #: 82-62605 -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com IRC: dcaro|dcaroest@{freenode|oftc|redhat} Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Improving our github presense
On 11/18 00:49, Nir Soffer wrote: > Hi all, > > Looking in ovirt github page, there are only 2 members: > https://github.com/orgs/oVirt/people > > Is is possible to add automatically all contributors to this list? afaik we don't have a reliable way to match github user and gerrit user so far, only a few people use the github extenal id. And I don't see any option around the github admin pannel either, anyone has more experience/ideas? > > Nir > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [vdsm] merge rights on master branch
Done, let me know if you need me to add him any more permissions (vdsm permisson groups is a bit complicated, I might have missed something) On 11/17 12:47, Dan Kenigsberg wrote: > Nir Soffer has been contributing to Vdsm for more than two years now. > Beyond his storage-related expertise, he is well-known for his thorough > code reviews and his pains when the master branch is broken. It's time > for him to take even bigger responsibility. > > David, please grant Nir Soffer with merge rights on the master branch of > Vdsm. > > I'll keep merging network, virt, and infra patches myself, but much less > than before. Adam keeps his merge right as before (he tends to use them > on emergencies only). > > Regards, > Dan. -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Reported-By - giving credit to our testers
On 11/13 08:55, Sandro Bonazzola wrote: > On Thu, Nov 12, 2015 at 3:45 PM, Nir Soffer <nsof...@redhat.com> wrote: > > > Hi all, > > > > Our QE (and sometimes our users) are working hard testing and > > reporting bugs, but > > their effort is never mentioned in our code. > > > > Looking at kernel git history, I found that they are using the > > Reported-By header for > > giving credit to the person reporting a bug. I suggest we adapt this > > header. > > > > Here are some examples how we can use it: > > > > - https://gerrit.ovirt.org/#/c/48483/3//COMMIT_MSG > > - > > https://github.com/oVirt/vdsm/commit/fb4c72af5e4c200409c74834111d44d92959ebbd > > - > > https://github.com/oVirt/vdsm/commit/f8127d88add881a4775e7030dde2433125c7b598 > > > infra: looking at above commit messages, can we get automatically added the > following lines? > > Reviewed-on: https://gerrit.ovirt.org/46540 > on patch creation > > Reviewed-by: Francesco Romani <from...@redhat.com> > on patch comment There's no easy way to plug in extra stuff, that is done on vdsm by gerrit itself because the project is set to 'cherry-pick', but that has consequences, for starters, when you submit a patch, the commit that gets in is not the same that you where reviewing (as the commit message changed) and thus you will be forced to rebase all the patches that depended on it. There's no built-in way either to change the commit on push (when updating/creating the patch) without creating a new patch. That means that to have that added, each time you send a patch a new one will be created, and each time you add a comment for the first time to the patch, another patch will be created too. Forcing the devel to refetch the commit each time he/she pushes. Not sure that smoothes up the dev process... I think that the less painful path will be to use a commit hook locally, so the commit message is changed locally first, but that will require the dev to install it (we can ease that with a script or similar though) > > > > > > > > > Thanks, > > Nir > > ___ > > Devel mailing list > > Devel@ovirt.org > > http://lists.ovirt.org/mailman/listinfo/devel > > > > > > -- > Sandro Bonazzola > Better technology. Faster innovation. Powered by community collaboration. > See how it works at redhat.com > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] [2015-11-16] Weekly oVirt infa team status
Hi everyone! This week news: * oVirt mirrors back on track: Most of the mirrors are back up and mirroring, the rest amarchuk is working with them on restoring the up status too * fc23 builds! We now have fc23 builds in the snapshot repos, so any projects that needed them can start migrating to use them instead of fc22 ones (kudos sbonazzo!) * Fixed publishers issue with ova files Now ova files will not break the publishers anymore * 4 new el7 slaves As an ongoing improvement, we spun up 4 new el7 slaves, to help with the load, we are working on getting as many as possible as soon as possible * Solved a bunch of space issues in the slaves There were a bunch of failures due to a space leak on the mock runner, that was fixed and will avoid those issues from now on. * CI flag propagation There's a new hook that propagates the CI flag value in case of non-code changes (those that only change the commit message). So no more changes should go without CI flag. If you had one of those changes before the hook was set, you can trigger the change again in jenkins and the CI value will be propagated (remember that you must trigger the change that has code changes, hit the 'include all patchsets' checkbox in jenkins [1]) * Infra docs hosting up and running We started the migration last week, and now the infra-docs are now hosted live, and automatically updated at readthedocs.org [2] Thanks everyone for keeping the infra running! [1] http://jenkins.ovirt.org/gerrit_manual_trigger/ [2] http://ovirt-infra-docs.readthedocs.org -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Jenkins build fails - bad dependency?
On 11/12, Sandro Bonazzola wrote: > On Thu, Nov 12, 2015 at 11:33 AM, Dan Kenigsberg <dan...@redhat.com> wrote: > > > On Wed, Nov 11, 2015 at 03:41:11PM +0100, Sandro Bonazzola wrote: > > > On Wed, Nov 11, 2015 at 12:06 PM, Eyal Edri <ee...@redhat.com> wrote: > > > > > > > ccing devel. > > > > > > > > On Tue, Nov 10, 2015 at 6:32 PM, Nir Soffer <nsof...@redhat.com> > > wrote: > > > > > > > >> This currently breaks the build for > > > >> https://gerrit.ovirt.org/31162 > > > >> > > > >> 00:08:25.505 Last metadata expiration check performed 0:00:02 ago on > > > >> Tue Nov 10 16:24:12 2015. > > > >> > > > >> 00:08:30.247 Error: nothing provides ovirt-vmconsole >= 1.0.0-0 needed > > > >> by vdsm-4.17.999-118.git71e8041.fc23.noarch. > > > >> 00:08:30.247 nothing provides ovirt-vmconsole >= 1.0.0-0 needed by > > > >> vdsm-4.17.999-118.git71e8041.fc23.noarch. > > > >> 00:08:30.247 nothing provides ovirt-vmconsole >= 1.0.0-0 needed by > > > >> vdsm-4.17.999-118.git71e8041.fc23.noarch. > > > >> 00:08:30.247 nothing provides ovirt-vmconsole >= 1.0.0-0 needed by > > > >> vdsm-4.17.999-118.git71e8041.fc23.noarch. > > > >> 00:08:30.247 nothing provides ovirt-vmconsole >= 1.0.0-0 needed by > > > >> vdsm-4.17.999-118.git71e8041.fc23.noarch. > > > >> 00:08:30.247 nothing provides ovirt-vmconsole >= 1.0.0-0 needed by > > > >> vdsm-4.17.999-118.git71e8041.fc23.noarch. > > > >> 00:08:30.247 nothing provides ovirt-vmconsole >= 1.0.0-0 needed by > > > >> vdsm-4.17.999-118.git71e8041.fc23.noarch. > > > >> 00:08:30.247 nothing provides ovirt-vmconsole >= 1.0.0-0 needed by > > > >> vdsm-4.17.999-118.git71e8041.fc23.noarch. > > > >> 00:08:30.247 nothing provides ovirt-vmconsole >= 1.0.0-0 needed by > > > >> vdsm-4.17.999-118.git71e8041.fc23.noarch. > > > >> 00:08:30.247 nothing provides ovirt-vmconsole >= 1.0.0-0 needed by > > > >> vdsm-4.17.999-118.git71e8041.fc23.noarch. > > > >> 00:08:30.247 nothing provides ovirt-vmconsole >= 1.0.0-0 needed by > > > >> vdsm-4.17.999-118.git71e8041.fc23.noarch. > > > >> 00:08:30.247 nothing provides ovirt-vmconsole >= 1.0.0-0 needed by > > > >> vdsm-4.17.999-118.git71e8041.fc23.noarch. > > > >> 00:08:30.247 nothing provides ovirt-vmconsole >= 1.0.0-0 needed by > > > >> vdsm-4.17.999-118.git71e8041.fc23.noarch > > > >> 00:08:30.247 (try to add '--allowerasing' to command line to replace > > > >> conflicting packages) > > > >> > > > >> Can you take a look at this? > > > >> > > > > > > > > > > looks like the automation/check-patch.repos file is missing the snapshot > > > repo. > > > > heh, we simply have no automation/check-patch.repos.fc23, yet. > > Is that > > > > http//gerrit.ovirt.org/48479 > > > > all that is needed? > > > > I think so You can try rebasing the failing patch on top of it, and sending it for review, it will run with those changes on jenkins > > -- > Sandro Bonazzola > Better technology. Faster innovation. Powered by community collaboration. > See how it works at redhat.com > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Reported-By - giving credit to our testers
On 11/12, Francesco Romani wrote: > - Original Message - > > From: "Nir Soffer" <nsof...@redhat.com> > > To: "devel" <devel@ovirt.org> > > Cc: "Dan Kenigsberg" <dan...@redhat.com>, "Francesco Romani" > > <from...@redhat.com>, "Piotr Kliczewski" > > <pklic...@redhat.com>, "Adam Litke" <ali...@redhat.com>, "Allon Mureinik" > > <amure...@redhat.com> > > Sent: Thursday, November 12, 2015 3:45:13 PM > > Subject: Reported-By - giving credit to our testers > > > > Hi all, > > > > Our QE (and sometimes our users) are working hard testing and > > reporting bugs, but > > their effort is never mentioned in our code. > > > > Looking at kernel git history, I found that they are using the > > Reported-By header for > > giving credit to the person reporting a bug. I suggest we adapt this header. > > Strongly agreed, let's do this! +1 > > Bests, > > -- > Francesco Romani > RedHat Engineering Virtualization R & D > Phone: 8261328 > IRC: fromani > ___ > Devel mailing list > Devel@ovirt.org > http://lists.ovirt.org/mailman/listinfo/devel -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Jenkins - introducing F23 jobs, dropping F21 jobs
On 11/09, Barak Korren wrote: > On 9 November 2015 at 10:35, David Caro <dc...@redhat.com> wrote: > > On 11/09, Barak Korren wrote: > >> On 9 November 2015 at 00:27, David Caro <dc...@redhat.com> wrote: > >> > On 11/08, Barak Korren wrote: > >> >> If all jobs are using mock already, maybe we should drop all Fedora > >> >> slaves to have less frequent upgrade cycles? > >> > > >> > Not all the jobs are mocked, and there's some issues trying to run latest > >> > fedora mocks on el6/7, so the move should be all slaves to fedora instead > >> > (should have to recheck this on latest el7, but on 7.1 you were unable to > >> > create fc22 chroots without having issues with yum/dnf differences). > >> > >> Could we work around this by installing dnf in the chroot and then > >> running it with "mock chroot" rather then using the built-in "mock > >> install" commands? > >> Going to Fedora will not sufficiently decrease the amount of upgrade > >> cycles IMO. > > > > Not really as the base install is already made with the out-of-the-chroot > > tool > > like it or not > > > Maybe we could circumvent that by pre-seeding the chroot cache with a > chroot created on Fedora to skit that step? I don't think it's worth it complicating the process so much just to avoid having to rebuild the slaves twice a year (though once a year might be enough, or even less) > >> > >> > > >> > But essentially yes, once everything is in mock, we can have homogeneous > >> > slaves, that will not require updating so frequently and should be a lot > >> > simpler > >> > to deploy (just de few packages to run the jenkins slave, mock, and > >> > probably > >> > libvirt to allow creating vms there) > >> > > >> Amen. :) > >> >> > >> >> On 8 November 2015 at 09:38, Eyal Edri <ee...@redhat.com> wrote: > >> >> > +1. > >> >> > > >> >> > we should probably schedule upgrades for f21 slaves due to EOL, > >> >> > but its not a requirement for jobs as all are using mock > >> >> > (mock_runnner/chrooter) > >> >> > > >> >> > e. > >> >> > > >> >> > On Thu, Nov 5, 2015 at 9:00 AM, Sandro Bonazzola <sbona...@redhat.com> > >> >> > wrote: > >> >> >> > >> >> >> Hi, > >> >> >> Fedora 23 has been released and Fedora 21 is going EOL in 1 month. > >> >> >> It's time to switch Jenkins jobs to the new F23 release. > >> >> >> If you're still using F21 for developing oVirt you might consider to > >> >> >> upgrade now. > >> >> >> I'll start introducing F23 jobs today for yamlized jenkins jobs and > >> >> >> will drop F21 jobs in 2 weeks, starting from November 19th. > >> >> >> > >> >> >> -- > >> >> >> Sandro Bonazzola > >> >> >> Better technology. Faster innovation. Powered by community > >> >> >> collaboration. > >> >> >> See how it works at redhat.com > >> >> >> _______ > >> >> >> Devel mailing list > >> >> >> Devel@ovirt.org > >> >> >> http://lists.ovirt.org/mailman/listinfo/devel > >> >> > > >> >> > > >> >> > > >> >> > > >> >> > -- > >> >> > Eyal Edri > >> >> > Supervisor, RHEV CI > >> >> > EMEA ENG Virtualization R > >> >> > Red Hat Israel > >> >> > > >> >> > phone: +972-9-7692018 > >> >> > irc: eedri (on #tlv #rhev-dev #rhev-integ) > >> >> > > >> >> > ___ > >> >> > Infra mailing list > >> >> > in...@ovirt.org > >> >> > http://lists.ovirt.org/mailman/listinfo/infra > >> >> > > >> >> > >> >> > >> >> > >> >> -- > >> >> Barak Korren > >> >> bkor...@redhat.com > >> >> RHEV-CI Team > >> >> ___ > >> >> Infra mailing list > >> >> in...@ovirt.org > >> >> http://lists.ovirt.org/mailman/listinfo/infra > >> > > >> > -- > >> > David Caro > >> > > >> > Red Hat S.L. > >> > Continuous Integration Engineer - EMEA ENG Virtualization R > >> > > >> > Tel.: +420 532 294 605 > >> > Email: dc...@redhat.com > >> > Web: www.redhat.com > >> > RHT Global #: 82-62605 > >> > >> > >> > >> -- > >> Barak Korren > >> bkor...@redhat.com > >> RHEV-CI Team > > > > -- > > David Caro > > > > Red Hat S.L. > > Continuous Integration Engineer - EMEA ENG Virtualization R > > > > Tel.: +420 532 294 605 > > Email: dc...@redhat.com > > Web: www.redhat.com > > RHT Global #: 82-62605 > > > > -- > Barak Korren > bkor...@redhat.com > RHEV-CI Team -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Jenkins - introducing F23 jobs, dropping F21 jobs
On 11/09, Barak Korren wrote: > On 9 November 2015 at 00:27, David Caro <dc...@redhat.com> wrote: > > On 11/08, Barak Korren wrote: > >> If all jobs are using mock already, maybe we should drop all Fedora > >> slaves to have less frequent upgrade cycles? > > > > Not all the jobs are mocked, and there's some issues trying to run latest > > fedora mocks on el6/7, so the move should be all slaves to fedora instead > > (should have to recheck this on latest el7, but on 7.1 you were unable to > > create fc22 chroots without having issues with yum/dnf differences). > > Could we work around this by installing dnf in the chroot and then > running it with "mock chroot" rather then using the built-in "mock > install" commands? > Going to Fedora will not sufficiently decrease the amount of upgrade cycles > IMO. Not really as the base install is already made with the out-of-the-chroot tool like it or not > > > > > But essentially yes, once everything is in mock, we can have homogeneous > > slaves, that will not require updating so frequently and should be a lot > > simpler > > to deploy (just de few packages to run the jenkins slave, mock, and probably > > libvirt to allow creating vms there) > > > Amen. :) > >> > >> On 8 November 2015 at 09:38, Eyal Edri <ee...@redhat.com> wrote: > >> > +1. > >> > > >> > we should probably schedule upgrades for f21 slaves due to EOL, > >> > but its not a requirement for jobs as all are using mock > >> > (mock_runnner/chrooter) > >> > > >> > e. > >> > > >> > On Thu, Nov 5, 2015 at 9:00 AM, Sandro Bonazzola <sbona...@redhat.com> > >> > wrote: > >> >> > >> >> Hi, > >> >> Fedora 23 has been released and Fedora 21 is going EOL in 1 month. > >> >> It's time to switch Jenkins jobs to the new F23 release. > >> >> If you're still using F21 for developing oVirt you might consider to > >> >> upgrade now. > >> >> I'll start introducing F23 jobs today for yamlized jenkins jobs and > >> >> will drop F21 jobs in 2 weeks, starting from November 19th. > >> >> > >> >> -- > >> >> Sandro Bonazzola > >> >> Better technology. Faster innovation. Powered by community > >> >> collaboration. > >> >> See how it works at redhat.com > >> >> ___ > >> >> Devel mailing list > >> >> Devel@ovirt.org > >> >> http://lists.ovirt.org/mailman/listinfo/devel > >> > > >> > > >> > > >> > > >> > -- > >> > Eyal Edri > >> > Supervisor, RHEV CI > >> > EMEA ENG Virtualization R > >> > Red Hat Israel > >> > > >> > phone: +972-9-7692018 > >> > irc: eedri (on #tlv #rhev-dev #rhev-integ) > >> > > >> > ___ > >> > Infra mailing list > >> > in...@ovirt.org > >> > http://lists.ovirt.org/mailman/listinfo/infra > >> > > >> > >> > >> > >> -- > >> Barak Korren > >> bkor...@redhat.com > >> RHEV-CI Team > >> ___ > >> Infra mailing list > >> in...@ovirt.org > >> http://lists.ovirt.org/mailman/listinfo/infra > > > > -- > > David Caro > > > > Red Hat S.L. > > Continuous Integration Engineer - EMEA ENG Virtualization R > > > > Tel.: +420 532 294 605 > > Email: dc...@redhat.com > > Web: www.redhat.com > > RHT Global #: 82-62605 > > > > -- > Barak Korren > bkor...@redhat.com > RHEV-CI Team -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 signature.asc Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [ANN] [QE] Bugzilla updates for oVirt Product
What projects are considered for that? What should be done to add a new one? (for example, the repoman or similar are also there?) On 09/09, Sandro Bonazzola wrote: > The oVirt team is pleased to announce that today oVirt moved to its own > classification within our Bugzilla system as previously anticipated [1]. > No longer limited as a set of sub-projects, each building block > (sub-project) of oVirt will be a Bugzilla product. > This will allow tracking of package versions and target releases based on > their own versioning schema. > Each maintainer, for example, will have administrative rights on his or her > Bugzilla sub-project and will be able to change flags, > versions, targets, and components. > > As part of the improvements of the Bugzilla tracking system, a flag system > has been added to the oVirt product in order to ease its management [2]. > The changes will go into affect in stages, please review the wiki for more > details. > > We invite you to review the new tracking system and get involved with oVirt > QA [3] to make oVirt better than ever! > > [1] http://community.redhat.com/blog/2015/06/moving-focus-to-the-upstream/ > [2] http://www.ovirt.org/Bugzilla_rework > [3] http://www.ovirt.org/OVirt_Quality_Assurance > > -- > Sandro Bonazzola > Better technology. Faster innovation. Powered by community collaboration. > See how it works at redhat.com > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 pgpDEYt_kaJ24.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [ANN] [QE] Bugzilla updates for oVirt Product
On 09/09, Yaniv Dary wrote: > Yaniv Dary > Technical Product Manager > Red Hat Israel Ltd. > 34 Jerusalem Road > Building A, 4th floor > Ra'anana, Israel 4350109 > > Tel : +972 (9) 7692306 > 8272306 > Email: yd...@redhat.com > IRC : ydary > > > On Wed, Sep 9, 2015 at 12:11 PM, Sandro Bonazzola <sbona...@redhat.com> > wrote: > > > > > > > On Wed, Sep 9, 2015 at 11:09 AM, David Caro <dc...@redhat.com> wrote: > > > >> > >> What projects are considered for that? > > > > > > You can see the planning here: > > https://docs.google.com/spreadsheets/d/1SOJNw1WQHEhE2rVP26qNtu30aKLQIV8JEzPp8KqCbwU/edit?usp=sharing > > > > > > > >> What should be done to add a new one? > >> > >> > > Adding Yaniv > > > > For the first stage we will not be adding any additional sub-projects. > After the full migration on the existing sub-project we can consider adding > additional ones. mmm, ok, not renaming either? (ovirt-testing-framework is not longer there, it was moved to lago) > > > > > > > > > >> (for example, the repoman or similar are also there?) > >> > >> On 09/09, Sandro Bonazzola wrote: > >> > The oVirt team is pleased to announce that today oVirt moved to its own > >> > classification within our Bugzilla system as previously anticipated [1]. > >> > No longer limited as a set of sub-projects, each building block > >> > (sub-project) of oVirt will be a Bugzilla product. > >> > This will allow tracking of package versions and target releases based > >> on > >> > their own versioning schema. > >> > Each maintainer, for example, will have administrative rights on his or > >> her > >> > Bugzilla sub-project and will be able to change flags, > >> > versions, targets, and components. > >> > > >> > As part of the improvements of the Bugzilla tracking system, a flag > >> system > >> > has been added to the oVirt product in order to ease its management [2]. > >> > The changes will go into affect in stages, please review the wiki for > >> more > >> > details. > >> > > >> > We invite you to review the new tracking system and get involved with > >> oVirt > >> > QA [3] to make oVirt better than ever! > >> > > >> > [1] > >> http://community.redhat.com/blog/2015/06/moving-focus-to-the-upstream/ > >> > [2] http://www.ovirt.org/Bugzilla_rework > >> > [3] http://www.ovirt.org/OVirt_Quality_Assurance > >> > > >> > -- > >> > Sandro Bonazzola > >> > Better technology. Faster innovation. Powered by community > >> collaboration. > >> > See how it works at redhat.com > >> > >> > ___ > >> > Infra mailing list > >> > in...@ovirt.org > >> > http://lists.ovirt.org/mailman/listinfo/infra > >> > >> > >> -- > >> David Caro > >> > >> Red Hat S.L. > >> Continuous Integration Engineer - EMEA ENG Virtualization R > >> > >> Tel.: +420 532 294 605 > >> Email: dc...@redhat.com > >> Web: www.redhat.com > >> RHT Global #: 82-62605 > >> > > > > > > > > -- > > Sandro Bonazzola > > Better technology. Faster innovation. Powered by community collaboration. > > See how it works at redhat.com > > > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 pgp4JPBsmNxOJ.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Merging patches into vdsm without CI
Sent patch and verifyed: https://gerrit.ovirt.org/#/c/45281 On 08/25, Dan Kenigsberg wrote: On Mon, Aug 24, 2015 at 03:43:25PM +0200, David Caro wrote: I've enabled the job that the jenkins patch creates, created manually, but review the patch so I can merge it please: http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc22-x86_64/ Thanks. We've now taken automation/check-patch.sh into ovirt-3.6. Can you issue a similar job for 3.6? It would be nice to run in on el7. Regards, Dan -- 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 pgp9hGyXvuWLd.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Merging patches into vdsm without CI
On 08/23, Dan Kenigsberg wrote: On Fri, Aug 21, 2015 at 05:04:03PM -0400, Adam Litke wrote: Hi all, Recent breakage in the vdsm CI flows have caused the change upload trigger to be disabled. This means that CI scores are no longer being automatically applied to uploaded changes. This means that patches cannot be merged into vdsm. I have a queue of patches which are otherwise ready for merge (which have passed CI in the past but needed rebasing). These patches have been stalled for almost a week now. What can we to to unfreeze the vdsm development process in the short and long term? Earlier today I worked with Sandro and David on manually running CI on my dev machine but am getting 100s of failures (so it looks like this wont even be a good short-term solution). Francesco and David were supposed to revive the broken job on Friday, but I see that it is still disabled. What is the reason of the delay? Francesco wanted the approval of the network guys, as the job has to create the tun char device itself (because mock does not do it). If that's allowable, then it can be enabled right away (you'll have to rebase to pull the scripts in). -- 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 pgp5EtNuRmZQe.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Merging patches into vdsm without CI
I've enabled the job that the jenkins patch creates, created manually, but review the patch so I can merge it please: http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc22-x86_64/ On 08/24, Francesco Romani wrote: - Original Message - From: Dan Kenigsberg dan...@redhat.com To: Francesco Romani from...@redhat.com Cc: David Caro dc...@redhat.com, Adam Litke ali...@redhat.com, devel@ovirt.org, Sandro Bonazzola sbona...@redhat.com, David Caro Estevez dcaro...@redhat.com Sent: Monday, August 24, 2015 2:36:55 PM Subject: Re: Merging patches into vdsm without CI On Mon, Aug 24, 2015 at 03:32:53AM -0400, Francesco Romani wrote: - Original Message - From: Dan Kenigsberg dan...@redhat.com To: David Caro dc...@redhat.com Cc: Adam Litke ali...@redhat.com, from...@redhat.com, devel@ovirt.org, Sandro Bonazzola sbona...@redhat.com, David Caro Estevez dcaro...@redhat.com Sent: Monday, August 24, 2015 8:53:07 AM Subject: Re: Merging patches into vdsm without CI On Mon, Aug 24, 2015 at 12:01:58AM +0200, David Caro wrote: On 08/23, Dan Kenigsberg wrote: On Fri, Aug 21, 2015 at 05:04:03PM -0400, Adam Litke wrote: Hi all, Recent breakage in the vdsm CI flows have caused the change upload trigger to be disabled. This means that CI scores are no longer being automatically applied to uploaded changes. This means that patches cannot be merged into vdsm. I have a queue of patches which are otherwise ready for merge (which have passed CI in the past but needed rebasing). These patches have been stalled for almost a week now. What can we to to unfreeze the vdsm development process in the short and long term? Earlier today I worked with Sandro and David on manually running CI on my dev machine but am getting 100s of failures (so it looks like this wont even be a good short-term solution). Francesco and David were supposed to revive the broken job on Friday, but I see that it is still disabled. What is the reason of the delay? Francesco wanted the approval of the network guys, as the job has to create the tun char device itself (because mock does not do it). If that's allowable, then it can be enabled right away (you'll have to rebase to pull the scripts in). I'm not sure how this can be done. but in any case, I'd rather NOSE_EXCLUDE tcTests until we solve this issue. Hi all, this is true. I didn't want to rush in a possible half-baked solution without any ack from the owners of the tests. With this patch: https://gerrit.ovirt.org/#/c/45134/3 the CI jobs WORKSFORME. I used these instructions http://www.ovirt.org/index.php?title=CI/Build_and_test_standards#Testing_the_scripts_locally and tested on el7 and fc22 succesfully. I've merged these patches (despite little comments that can wait for a follow-up patch). David, can you now enable CI on the master branch? Later, Francesco, would you backport these two patches to 3.6? Sure thing! I'll just wait for Jenkins second check before to backport patches. Bests, -- Francesco Romani RedHat Engineering Virtualization R D Phone: 8261328 IRC: fromani -- 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 pgpFCshD4DIlq.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Merging patches into vdsm without CI
The patch for the jenkins job is here: https://gerrit.ovirt.org/#/c/42477/ let me know if that's ok and I'll merge it (you might want for example run it on more distros) On 08/24, Francesco Romani wrote: - Original Message - From: Dan Kenigsberg dan...@redhat.com To: Francesco Romani from...@redhat.com Cc: David Caro dc...@redhat.com, Adam Litke ali...@redhat.com, devel@ovirt.org, Sandro Bonazzola sbona...@redhat.com, David Caro Estevez dcaro...@redhat.com Sent: Monday, August 24, 2015 2:36:55 PM Subject: Re: Merging patches into vdsm without CI On Mon, Aug 24, 2015 at 03:32:53AM -0400, Francesco Romani wrote: - Original Message - From: Dan Kenigsberg dan...@redhat.com To: David Caro dc...@redhat.com Cc: Adam Litke ali...@redhat.com, from...@redhat.com, devel@ovirt.org, Sandro Bonazzola sbona...@redhat.com, David Caro Estevez dcaro...@redhat.com Sent: Monday, August 24, 2015 8:53:07 AM Subject: Re: Merging patches into vdsm without CI On Mon, Aug 24, 2015 at 12:01:58AM +0200, David Caro wrote: On 08/23, Dan Kenigsberg wrote: On Fri, Aug 21, 2015 at 05:04:03PM -0400, Adam Litke wrote: Hi all, Recent breakage in the vdsm CI flows have caused the change upload trigger to be disabled. This means that CI scores are no longer being automatically applied to uploaded changes. This means that patches cannot be merged into vdsm. I have a queue of patches which are otherwise ready for merge (which have passed CI in the past but needed rebasing). These patches have been stalled for almost a week now. What can we to to unfreeze the vdsm development process in the short and long term? Earlier today I worked with Sandro and David on manually running CI on my dev machine but am getting 100s of failures (so it looks like this wont even be a good short-term solution). Francesco and David were supposed to revive the broken job on Friday, but I see that it is still disabled. What is the reason of the delay? Francesco wanted the approval of the network guys, as the job has to create the tun char device itself (because mock does not do it). If that's allowable, then it can be enabled right away (you'll have to rebase to pull the scripts in). I'm not sure how this can be done. but in any case, I'd rather NOSE_EXCLUDE tcTests until we solve this issue. Hi all, this is true. I didn't want to rush in a possible half-baked solution without any ack from the owners of the tests. With this patch: https://gerrit.ovirt.org/#/c/45134/3 the CI jobs WORKSFORME. I used these instructions http://www.ovirt.org/index.php?title=CI/Build_and_test_standards#Testing_the_scripts_locally and tested on el7 and fc22 succesfully. I've merged these patches (despite little comments that can wait for a follow-up patch). David, can you now enable CI on the master branch? Later, Francesco, would you backport these two patches to 3.6? Sure thing! I'll just wait for Jenkins second check before to backport patches. Bests, -- Francesco Romani RedHat Engineering Virtualization R D Phone: 8261328 IRC: fromani -- 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 pgpAqPtWiafre.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Ovirt engine find bugs job
Hi everyone! Lately there have been found issues with the findbugs job: http://jenkins.ovirt.org/job/ovirt-engine_3.5_find-bugs_merged/1373/ It turns out that it was running only on fc20 hosts, and there it ran well, but fc20 hosts have been removed (not supported anymore) and the test started running on el6/el7 hosts, and the version there is slightly different and it started complaining about issues it did not complain before. The quick fix is to solve the issue: BackendApiResource.java:264, DE_MIGHT_IGNORE, Priority: Low org.ovirt.engine.api.restapi.resource.BackendApiResource.getSchema() might ignore java.io.IOException But that leaves another question, which plataforms should we run findbugs on? should we use a matrix of platforms? always the same? It's your call, so please let me know what you want to do (I'll just pin it to el7 host for now, to avoid flaky tests). Thanks!! -- 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 pgpFI5_vQN5kx.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Ovirt engine find bugs job
On 08/17, Max Kovgan wrote: *Also worth mentioning:* job1: http://jenkins.ovirt.org/job/ovirt-engine_3.5_dao-unit-tests_gerrit, does: 1. does some sql-foo to create a db with some data 2. runs maven as follows: export MAVEN_OPTS= -XX:MaxPermSize=512m mvn \ -P enable-dao-tests \ -D engine.db.username=engine \ -D engine.db.password=engine \ -s /home/jenkins/workspace/ovirt-engine_3.5_dao-unit-tests_gerrit/artifactory-ovirt-org-settings.xml \ -Dengine.db.url=jdbc:postgresql://localhost/ovirt_engine_3_5_dao_unit_tests_gerrit_369 \ install job2: http://jenkins.ovirt.org/job/ovirt-engine_3.5_unit_tests_gerrit, does: 1. Run maven (I've beautified this just to ease on comparison, and it's indirect run via wrapped up make target): mvn \ -f ovirt-engine/pom.xml \ -s /home/jenkins/workspace/ovirt-engine_3.5_unit_tests_gerrit/artifactory-ovirt-org-settings.xml \ -DGERRIT_HOST=gerrit.ovirt.org \ -DGERRIT_PATCHSET_UPLOADER=Alon Bar-Lev alo...@redhat.com \ -DGERRIT_EVENT_ACCOUNT_NAME=Alon Bar-Lev \ -DGERRIT_EVENT_TYPE=patchset-created \ -DGERRIT_CHANGE_ID=I97fa20e0f5205ebddb6aff1cbfebf3f2879fa34e \ -DGERRIT_TOPIC=host-deploy \ -DGERRIT_CHANGE_NUMBER=44875 \ -DGERRIT_VERSION=2.10.3.1 \ -DGERRIT_PATCHSET_UPLOADER_NAME=Alon Bar-Lev \ -DGERRIT_CHANGE_COMMIT_MESSAGE=aG9zdC1kZXBsb3k6IHJlbW92ZSB0aW1lc3RhbXAgY2FjaGUgY2hlY2sgZm9yIHRhcmJhbGwKCndoZW4gbXVsdGlwbGUgcGFja2FnZXMgbWFuYWdlIHRoZSBkaXJlY3RvcnkgdHJlZSwgdGhlIGxhc3QgZmlsZQp0aW1lc3RhbXAgY2Fubm90IGJlIHVzZWQgYXMgcmVmZXJlbmNlIHRvIGNhY2hlIGludGVncml0eSBhcyBhbiBvbGRlcgpwYWNrYWdlIG1heSBiZSBhcHBsaWVkIGFmdGVyIG5ld2VyIG9uZS4KCndlIGNvdWxkIGhhdmUgdXNlZCBoYXNoIG1lY2hhbmlzbSwgYnV0IHRoZW4gd2UgaGFkIHRvIHJlYWQgZmlsZXMgb25jZQpwZXIgaGFzaCBhbmQgb25jZSBwZXIgY3JlYXRpb24sIHNvIGJldHRlciB0byBqdXN0IGNyZWF0ZSB0aGUgdGFyYmFsbCBhdApkZXNpZ25hdGVkIGludGVydmFsLgoKQnVnLVVybDogaHR0cHM6Ly9idWd6aWxsYS5yZWRoYXQuY29tL3Nob3dfYnVnLmNnaT9pZD0xMjUyMzY0CkNoYW5nZS1JZDogSTk3ZmEyMGUwZjUyMDVlYmRkYjZhZmYxY2JmZWJmM2YyODc5ZmEzNGUKU2lnbmVkLW9mZi1ieTogQWxvbiBCYXItTGV2IDxhbG9uYmxAcmVkaGF0LmNvbT4K \ -DGERRIT_PATCHSET_UPLOADER_EMAIL=alo...@redhat.com \ -DGERRIT_BRANCH=ovirt-engine-3.5 \ -DGERRIT_EVENT_HASH=-1504677913 \ -DGERRIT_PORT=29418 \ -DGERRIT_NAME=gerrit.ovirt.org \ -DGERRIT_CHANGE_OWNER_NAME=Alon Bar-Lev \ -DGERRIT_EVENT_ACCOUNT=Alon Bar-Lev alo...@redhat.com \ -DGERRIT_EVENT_ACCOUNT_EMAIL=alo...@redhat.com \ -DGERRIT_CHANGE_OWNER=Alon Bar-Lev alo...@redhat.com \ -DGERRIT_CHANGE_URL=https://gerrit.ovirt.org/44875 \ -DGERRIT_SCHEME=ssh \ -DGERRIT_CHANGE_SUBJECT=host-deploy: remove timestamp cache check for tarball \ -DGERRIT_REFSPEC=refs/changes/75/44875/1 \ -DGERRIT_PATCHSET_NUMBER=1 \ -DGERRIT_PATCHSET_REVISION=fb26eba4f8ebe696a31c584e162edca9c17ef801 \ -DGERRIT_PROJECT=ovirt-engine \ -DGERRIT_CHANGE_OWNER_EMAIL=alo...@redhat.com \ -Dmaven.repo.local=/home/jenkins/workspace/ovirt-engine_3.5_unit_tests_gerrit/.repository \ test Maybe we could do as follows: sequential run of combined job doing 1. maybe in parallel: 1. run: sql foo 2. job2 run - general unit tests: mvn clean test with params of job2 2. wait for 1.1 and 1.2. to complete, and then job1 run:mvn test -P enable-dao-tests with params of job1 The above could theoretically reduce total runtime of job1+job2 by 1/3. Given: job1 runs ~16m, job2 runs ~15m [most time ~10m is spent waiting for artifactory downloads, IIRC] Expected outcome: we could save 10m, reaching 21m, or even more, if the 2nd mvn run could probably reuse some of already compiled .class files, we'd could be even faster than 20m. Possible Pitfals: 1. problems combining tests result reports [if it indeed happens. *devs*, please comment] 2. we might need to be able to run job1 and job2 separately: - Workaround: - split job1 into II parts: job1.sql-foo and job1.mvn - run workflow (sharing workspace): parallel(job1.sql-foo,job2) == run job1.mvn WDYT? I'd wait until we have a proper pipeline, and try to generalize the definition of the test ordering putting it in the same repo as the project. Adding this right now will increase the maintenance of the overall setup, and will complicate the migration to a pipeline. Though, if you can do all that inside a bash script, we can use it as check-patch.sh for the ovirt-engine project and use that, though I suppose that you might prefer letting jenkins do the job orchestration instead of having to rewrite that code in bash. On Mon, Aug 17, 2015 at 2:22 PM, David Caro dc...@redhat.com wrote: Hi everyone! Lately there have been found issues with the findbugs job: http://jenkins.ovirt.org/job/ovirt-engine_3.5_find-bugs_merged
Re: [ovirt-devel] Findbugs on master patches
On 08/12, Yevgeny Zaspitsky wrote: The findbugs failure is related to patch set 1, which was previously was marked by Jenkins as +1. If that wasn't +1'ed, we would've wait until Jenkins job is finished, but since it has +1 we were under impression that the patch is OK. You are right, the findbugs was triggered by the verified +1 on patch 1. Looks like findbugs job was triggered by Verified+1, whereas test were run upon patch set upload. IMHO the split between the jobs makes their behavior misleading. Agree, but so far there's no agreement on the flow for it, and there's no effort being done to reach that agreement (it's on hold) so it will not change any time soon. I think we should drop the triggering by flag, or make it generic but not mix both. That's why I wanted to introduce the workflow flag, to trigger only by that flag on demand of the reviewer. Feel free to try and start that discussion again BTW, what was merged is patch set 2, which wasn't checked by findbugs at all. On Wed, Aug 12, 2015 at 3:42 PM, David Caro dc...@redhat.com wrote: On 08/12, Yevgeny Zaspitsky wrote: My bad, sorry. The findbugs job runs indeed. However look at that patch https://gerrit.ovirt.org/#/c/44604/ . Patch set 1 got Continuous-Integration+1 from Jenkins after running unit tests and then, much later, That much later is after merge, the findbugs ran once the patch was merged. You should understand that each event triggers a tests run, and for each tests run you get only one review. In this case the tests that check the code when you send the patch passed, but the ones that check the code after it's merged failed. Jenkins changed its mind and gave -1 after findbugs job has finished. IMHO that's is a misleading behavior. Regards, Yevgeny On Wed, Aug 12, 2015 at 12:12 PM, Sandro Bonazzola sbona...@redhat.com wrote: On Tue, Aug 11, 2015 at 6:13 PM, Yevgeny Zaspitsky yzasp...@redhat.com wrote: Hi All, Seems like recently Jenkins has stopped running findbugs check on the patches before they get merged into master. Although that could save some Jenkins resources and make the its reaction faster, that makes the integration (with master) check less effective. Was that done on purpose? Can that be restored for master patches? AFAIK findbugs is still up and running: http://jenkins.ovirt.org/search/?q=find-bugs Have you any specific patch as example? Regards, Yevgeny ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com ___ Infra mailing list in...@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 -- 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 pgpUoszkTFRqZ.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Findbugs on master patches
On 08/12, Yevgeny Zaspitsky wrote: My bad, sorry. The findbugs job runs indeed. However look at that patch https://gerrit.ovirt.org/#/c/44604/ . Patch set 1 got Continuous-Integration+1 from Jenkins after running unit tests and then, much later, That much later is after merge, the findbugs ran once the patch was merged. You should understand that each event triggers a tests run, and for each tests run you get only one review. In this case the tests that check the code when you send the patch passed, but the ones that check the code after it's merged failed. Jenkins changed its mind and gave -1 after findbugs job has finished. IMHO that's is a misleading behavior. Regards, Yevgeny On Wed, Aug 12, 2015 at 12:12 PM, Sandro Bonazzola sbona...@redhat.com wrote: On Tue, Aug 11, 2015 at 6:13 PM, Yevgeny Zaspitsky yzasp...@redhat.com wrote: Hi All, Seems like recently Jenkins has stopped running findbugs check on the patches before they get merged into master. Although that could save some Jenkins resources and make the its reaction faster, that makes the integration (with master) check less effective. Was that done on purpose? Can that be restored for master patches? AFAIK findbugs is still up and running: http://jenkins.ovirt.org/search/?q=find-bugs Have you any specific patch as example? Regards, Yevgeny ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com ___ Infra mailing list in...@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 pgp10ZaBV1htj.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Vdsm jobs are failing
We were experiencing an issue on the phx datacenter (bad connectivity to the 8.8.8.8 dns server) that was causing net issues on all the jobs. The issue should be over and should not happen again. Can you retrigger your failed jobs? On 08/11, Piotr Kliczewski wrote: All, There is forth isssue [4]. Thanks, Piotr [4] http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/5924/console On Tue, Aug 11, 2015 at 3:19 PM, Piotr Kliczewski piotr.kliczew...@gmail.com wrote: All, Bunch of jobs which were triggered by my patches failed for vdsm. There are 3 different issues that I saw. Please take a look at [1][2][3]. Thanks, Piotr [1] http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/5923/console [2] http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-el7_created/1396/console [3] http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc22_created/604/console ___ Infra mailing list in...@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 pgpRFu7oXE0ni.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] ovirt-hosted-engine-setup: extending maintainers team
done On 08/05, Sandro Bonazzola wrote: Eyal, David, looks like we have enough acks, please add SImone to hosted-engine-setup maintainers. On Tue, Aug 4, 2015 at 5:38 PM, Nir Soffer nsof...@redhat.com wrote: - Original Message - From: Sandro Bonazzola sbona...@redhat.com To: devel@ovirt.org Sent: Tuesday, August 4, 2015 5:50:44 PM Subject: [ovirt-devel] ovirt-hosted-engine-setup: extending maintainers team If you follow ovirt-hosted-engine-setup development, you probably noticed that I'm the only existing maintainer there. During the 3.6 development Simone Tiraboschi contributed a huge amount of patches and is currently unofficially maintaining iSCSI and FC storage support and the virtual appliance configuration flow. I'd like to officially nominate Simone as ovirt-hosted-engine-setup co-maintainer. Please approve. +1 Nir -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com -- 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 pgpXmATCInvCk.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] ovirt-engine related fedora 21 builds in jenkins
On 07/08, Sandro Bonazzola wrote: Hi, while dropping fc20 builds from master as announced 2 weeks ago[1] I've seen that several packages implying ovirt-engine availability on fc21 were still built on jenkins also if we dropped fc21 support for 3.6 in favor of fc22. Since there's no commitment from integration team and from infrastructure / CI team to support FC21 I would have liked to drop them. I've been stopped doing that since it seems there are developers using fc21 as development environment. I'd like to understand why. I would have understood having developers stuck on fedora 20 for supporting 3.5 and I totally understand developers already on fedora 22. But being fedora 21 the only unsupported version I don't see any real reason for keep wasting CI resources on such distribution. Is there any serious motivation for keeping fedora 21 engine related builds in jenkins? I may understand keeping vdsm related builds since vdsm is supposed to work on fc21 also for 3.5 so i've no objection in keeping vdsm and its deps on fc21 as long as vdsm team supports it. Totally agree, that will also allow us to remove all the fc21 slaves and replace them with fc22 or el7/el6. There are though a few projecs that still use fc21, but luckilly they use mock already and don't care much of what system has installed. Thanks, -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com ___ Infra mailing list in...@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 pgpDfrFbCkhSb.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] ssh gerrit.ovirt.org Connection timed out
I can access it from all the places I've tried, more probably it's a firewall somewhere. You can clone and fetch through https but you will not be able to push. Can you try using a gateway from another place? On 06/23, Doron Fediuck wrote: Adding the infra team On 23/06/15 12:14, yangfeng wrote: Hi gerrit maintainter Now I cannot ssh gerrit.ovirt.org. It is always timeout. I have create SSH keys following page http://www.ovirt.org/Working_with_oVirt_Gerrit. But I remember ssh used to work. Following is the command output: # ssh -v gerrit.ovirt.org OpenSSH_5.3p1, OpenSSL 1.0.1e-fips 11 Feb 2013 debug1: Reading configuration data /root/.ssh/config debug1: Applying options for gerrit.ovirt.org debug1: Reading configuration data /etc/ssh/ssh_config debug1: Applying options for * debug1: Connecting to gerrit.ovirt.org [107.22.212.69] port 29418. debug1: connect to address 107.22.212.69 port 29418: Connection timed out ssh: connect to host gerrit.ovirt.org port 29418: Connection timed out Did anyone meet same issue with me? I am in Beijing, China. Thanks Feng Yang ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel ___ Infra mailing list in...@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 pgpAlUpgk26ae.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Automated builds and releases
On 06/08, Sandro Bonazzola wrote: Il 05/06/2015 14:06, David Caro ha scritto: Hi everyone! In an effort to improve the project workflow and ease the maintenance and improve the quality of the project releases I want to propose start working towards automated builds and releases, the main ideas are the following: * Stop building differently for release and non-release: - Building only once, testing what you build and release what you test - Don't use two different version strings, one for testing and one for release I'm not really comfortable in releasing rpms like ovirt-host-deploy-offline-1.4.0-0.0.master.20150528094853.git7428372.el7.x86_64.rpm as GA release. This can be automated to don't add the extra info to the release in case of a tag creation, but that means that you have to run all the testing again when pushing the tag, ideally it would block the tag creation, so if it fails it would not tag the build but we don't have any gating system so right now it would have been done after the tag is pushed, so you'd have to force tag again (creating rpms with the same version that contain different code) or bump the tag (making it feasible that the first released version is not the lowest tag, for example, that the first release 4.17.* vdsm to be 4.17.5 instead of 4.17.0 because it took 5 retaggings to pass the tests) * Automate the build process, and the release process, directly getting the code from the repos (no manual build tarballs) This is fine for me, provided that the automated build start from a tagged version and become something like ovirt-host-deploy-offline-1.4.0-1 * Adopt semantic versioning, it's a lot more meaningful than the current scheme and fits very well with the above points No much experience in using semantic versioning, will take a look. This will ease and lower the maintenance and the extra work required by maintainers, release engineers (sandro) and infra itself by making releases as easy as hitting a button at any time. That will allow us to lower the time features and fixes get to the users, and deliver packages and builds that have passed through all the tests we have, instead of rebuilding on another env, at another time, by someone else, and passing only manual testing. +1 wdyt? ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com -- 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 pgpGUbccoGP5U.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] gerrit+ci improvement proposal
-manually-verified bugs. Once it's finished and humans reviewed the logic of the patch, Workflow+1 should be triggered allowing automation to check the correctness of the patch. IMHO there's no reason for wasting CI time on patches that will be correct from an automation point of view but nacked by reviewers. Especially if the patches are part of a big patchset. And one final note, for Workflow+2 - this is a preparation for a gating system, like Zuul used by openstack, that in the future we might use as automatic merger pending passing a verification step. this will prevent errors that happen sometimes post merge due to conflicts or other issues, and will be another level of validation before final merge. But as max said, its all part of the plan and we'll test it of course before implementing to see its value. 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 ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com ___ Infra mailing list in...@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel ___ Infra mailing list in...@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra ___ Infra mailing list in...@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra ___ Infra mailing list in...@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra ___ Infra mailing list in...@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 pgpn9aiCCisdQ.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Automated builds and releases
Hi everyone! In an effort to improve the project workflow and ease the maintenance and improve the quality of the project releases I want to propose start working towards automated builds and releases, the main ideas are the following: * Stop building differently for release and non-release: - Building only once, testing what you build and release what you test - Don't use two different version strings, one for testing and one for release * Automate the build process, and the release process, directly getting the code from the repos (no manual build tarballs) * Adopt semantic versioning, it's a lot more meaningful than the current scheme and fits very well with the above points This will ease and lower the maintenance and the extra work required by maintainers, release engineers (sandro) and infra itself by making releases as easy as hitting a button at any time. That will allow us to lower the time features and fixes get to the users, and deliver packages and builds that have passed through all the tests we have, instead of rebuilding on another env, at another time, by someone else, and passing only manual testing. wdyt? -- 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 pgpybRXZ563JF.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] gerrit+ci improvement proposal
On 06/04, Nir Soffer wrote: - Original Message - From: Max Kovgan mkov...@redhat.com To: devel@ovirt.org Cc: in...@ovirt.org Sent: Wednesday, June 3, 2015 8:22:54 PM Subject: [ovirt-devel] gerrit+ci improvement proposal Hi everyone! We really want to have reliable and snappy CI: to allow short cycles and encourage developers to write tests. # Problem Many patches are neither ready for review nor for CI upon submission, which is OK. But running all the jobs on those patches with limited resources results in: overloaded resources, slow response time, unhappy developers. # Proposed Solution To run less jobs we know we don’t need to, thus making more resources for the jobs we need to run. We have been experimenting to make our CI stabler and quicker to respond by using gerrit flags. This has improved in both directions very well internally. Now it seems a good time to let all the oVirt projects to use this. This solution indirectly promotes reviews and quick tests - “to fail early”, yet full blown static code analysis and long tests to run “when ready”. # How it works 2 new gerrit independent flags are added to gerrit. ## CI flag Will express patch CI status. Values: * +1 CI passed * 0 CI did not run yet * -1 CI failed -1 You must have CI error state. Most of the errors I have seen, the CI fail to run, and the failure is not related to the tested patch. You'll have to prove this, as the last time we discussed it less than 10% of the failures from the previous 2 weeks were that case. Also that's why maintainers can set +1 here and -1 is not a blocker. Permissions for setting: project maintainers (for special cases) should be able to set/override (except Jenkins). ## Workflow flag Will express patch “workflow” state. Values: * 0 Work In Progress * +1 Ready For Review * +2 Ready For Merge Permissions for setting: Owner can set +1, Project Maintainers can set +2 ## Review + CI Integration: Merging [“Submit” button to appear] will require: Review+1, CI+1, Workflow+2 Patch lifecycle now is: --- patch state |owner |reviewer |maintainer |CI tests |pass --- added/updated |- |-|- |quick|CI+1 review|Workflow+1|Review+1 |- |heavy|CI+1 merge ready |- |-|Workflow+2 |gating |CI+1 merge |- |-|merge |merge|CI+1 -1 Cannot require CI+1, the CI is not reliable enough yet. Even if the CI will be reliable, a failed test which is not related to the submitted patch should not block unrelated changes. This situation makes no sense, if it's reliable theres no error due to non changed stuff (unless the issue that triggers the error is part of your patch history, because it's merged or you are based on a broken patch, in this case it should fail and block) Changes from current workflow: Owner only adds reviewers, now owner needs to set Workflow+1 for the patch to be reviewed, and heavily auto-tested. Maintainer now needs to set Workflow+2 and wait for Submit button to appear after CI has completed running gating tests. -1 This means more work for developers. Instead, make workflow default to +1. If the want to disable the CI, she can set it to 0. Developers need more power and less process overhead. We have to make some stats, but from the small sample of patches that we checked, most than half of them were not ready to pass any ci, it's a bit more work, if you keep using drafts, if you don't it's the same. It's not ment to be used alongside drafts but instead of it. So the only case where it increases the overhead is if you did not use drafts, in which case you should start using them, so no real downside there. Nir ___ Infra mailing list in...@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 pgpesBjALdb9r.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Access to ovirt-srv08.ovirt.org
On 06/04, Eyal Edri wrote: - Original Message - From: David Caro dcaro...@redhat.com To: Tolik Litvosky tlito...@redhat.com Cc: Eyal Edri ee...@redhat.com, in...@ovirt.org, devel@ovirt.org Sent: Thursday, June 4, 2015 12:01:28 PM Subject: Re: Access to ovirt-srv08.ovirt.org On 06/04, Tolik Litvosky wrote: On Thu, 2015-06-04 at 04:50 -0400, Eyal Edri wrote: - Original Message - From: Tolik Litvosky tlito...@redhat.com To: Eyal Edri ee...@redhat.com Cc: in...@ovirt.org, devel@ovirt.org Sent: Thursday, June 4, 2015 11:44:09 AM Subject: Re: Access to ovirt-srv08.ovirt.org On Thu, 2015-06-04 at 02:31 -0400, Eyal Edri wrote: - Original Message - From: Tolik Litovsky tlito...@redhat.com To: ee...@redhat.com Cc: in...@ovirt.org Sent: Thursday, June 4, 2015 9:28:27 AM Subject: RE: Access to ovirt-srv08.ovirt.org The question is the nesting depth. We run currently in depth 2 . VM inside VM. Will depth of 3 will work. VM slave, and another 2 VM nesting above it. KVM people? Plz help. adding devel, but please elaborate on the depth, does it have to be nested or you're doing it because you only had 1 bare metal host till now (think you'll have 2 nested vms for example) The test requires depth 2 nested . We create an engine with hosts and storage. And on top of them we run another VM to test the ovirt features. This is the test scenario. OK, then we need to run a test and see the performance for it, because i think we will we gain many benefits from it, in terms of management and stability. Eyal , this testenv will most likely not be in my responsibility longer. But I know that there are several bare metals dedicated to this jobs. Can we plz bring them up , I belive it will be very helpfull before we finish the nested slaves investigation. There's only one afaik, and you are already using it. there are 6 more servers, but they are not ready yet, we will start working on them soon, i hope they will be ready in a week's time, we will update. regardless the system test maintainer should run the nested test so we'll know if we can leverage this. I was not aware they were dedicated for those jobs Tolik. Original message From: Eyal Edri ee...@redhat.com Date: 03/06/2015 11:14 PM (GMT+02:00) To: tlito...@redhat.com Cc: in...@ovirt.org Subject: Re: Access to ovirt-srv08.ovirt.org I'd like to move the system testing framework to use nested vms instead of directly using the baremetal hosts. we did use a single host to POC the system, i think now that we're about to expand it will be much more managble and scalable if we'll start using nested VMs which will be managed via the hosted engine. e. - Original Message - From: Tolik Litvosky tlito...@redhat.com To: in...@ovirt.org Sent: Wednesday, June 3, 2015 3:59:36 PM Subject: Access to ovirt-srv08.ovirt.org Hi I need access to the ovirt-srv08.ovirt.org in order to update the testenv CI repo directory. -- Best regards Tolik Litovsky RHEV-H Team Red Hat Red Hat: trustworthy, transformative technology. Powered by the community. Connect at redhat.com ___ Infra mailing list in...@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra -- Best regards Tolik Litovsky RHEV-H Team Red Hat Red Hat: trustworthy, transformative technology. Powered by the community. Connect at redhat.com -- Best regards Tolik Litovsky RHEV-H Team Red Hat Red Hat: trustworthy, transformative technology. Powered by the community. Connect at redhat.com ___ Infra mailing list in...@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 ___ Infra mailing list in...@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
Re: [ovirt-devel] gerrit+ci improvement proposal
in stone, in case we find a better approach on solving those issues, we will change to it. And we will keep improving so don't be afraid that it will be enforced: if this does not work out we will discard it. P.S. Kudos to dcaro, most of the work was done by him, and most of this text too. +1 from me, releasing CI from running non critical and un-essential jobs will not only reduce load from ci, and shorted response time for developers, it will allow us to add much more powerful tests such as functional system tests that actually add hosts and run VMs, improving our ability to find regression much more effectively. Another benefit to consider is saving reviewers time. I.e not only jenkins benefits from Worklow+1, but also human reviewers. Instead of looking at a patch that is too early to be reviewed, the author can set the Workflow+1 when the code is ready to review (even if he didn't verified it yet), thus saving time to other reviewers - for example people can add an email rule to alert them only when they are added to patches that have Workflow+1, and not before. For human reviewers I suggest to keep using drafts until the patch is finished. keep using? how many developers do you know are working with drafts until their patch is ready? i agree if everyone would use drafts load on jenkins was already much lower, unfortunately its not the case. IMO we don't need the workflow flag. I'm okay with CI not running on drafts. And yes... we do use them. We can try and educate people to use them more where needed. Drafts should be widely used in first-phase development, and less on bug-fixes. In addition, I think the patch owners shouldn't add reviewers, unless they need their input in the stage of the development. Once they want input, they should add reviewers. 1. So, if the patch is draft then no CI runs on it. 2. Once it turns into non-draft, you can run light-CI on it. 3. Once the patch has at least one +1 from a (human) reviewer, then it should run the heavy CI. This is pointless, we will have to ask someone (or add another user) to +1 the patch to enable the CI :-) 4. Once the patch has +1 from heavy CI, and +2 from reviewer (maintainer), then it can be merged. That's the process we have today, with slight change on when to run the CI and what CI to run (no CI on drafts, light CI on non-draft, heavy CI on +1 patches). I think Oved proposal is simpler and more useful. However, we need a way to run *any* ci jobs even on a draft. If we cannot afford this automatically with current system, lets add a way to trigger this manually from gerrit. So this becomes simply: 1. drafts do not run the ci automatically, but the owner can run the ci manually There are two ways I can think of such a trigger, through flags (that are easier to manage) or comments. The flags approach is the one that we exposed and tested on a few projects. 2. published patches run the light ci jobs automatically, owner can run heavy jobs manually A big issue here is that there's currently no separation between light and heavy jobs, not even a list of what should be considered as such. So for starters any job is considered to be heavy. 3. run all ci jobs before merge, maintain can ignore ci results This works with the system David suggested in the past, where each project implement scripts for checking patches and checking patches before merge. Both work with that system that was never implemented on any major project so far due to reluctance to add ci scripts inside the code base (where I strongly think they belong, as the code base evolves, the scripts must change with it) So this is not yet implemented everywhere. Nir ___ Infra mailing list in...@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 pgpbXdRq0fZwR.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Access to ovirt-srv08.ovirt.org
On 06/04, Tolik Litvosky wrote: On Thu, 2015-06-04 at 04:50 -0400, Eyal Edri wrote: - Original Message - From: Tolik Litvosky tlito...@redhat.com To: Eyal Edri ee...@redhat.com Cc: in...@ovirt.org, devel@ovirt.org Sent: Thursday, June 4, 2015 11:44:09 AM Subject: Re: Access to ovirt-srv08.ovirt.org On Thu, 2015-06-04 at 02:31 -0400, Eyal Edri wrote: - Original Message - From: Tolik Litovsky tlito...@redhat.com To: ee...@redhat.com Cc: in...@ovirt.org Sent: Thursday, June 4, 2015 9:28:27 AM Subject: RE: Access to ovirt-srv08.ovirt.org The question is the nesting depth. We run currently in depth 2 . VM inside VM. Will depth of 3 will work. VM slave, and another 2 VM nesting above it. KVM people? Plz help. adding devel, but please elaborate on the depth, does it have to be nested or you're doing it because you only had 1 bare metal host till now (think you'll have 2 nested vms for example) The test requires depth 2 nested . We create an engine with hosts and storage. And on top of them we run another VM to test the ovirt features. This is the test scenario. OK, then we need to run a test and see the performance for it, because i think we will we gain many benefits from it, in terms of management and stability. Eyal , this testenv will most likely not be in my responsibility longer. But I know that there are several bare metals dedicated to this jobs. Can we plz bring them up , I belive it will be very helpfull before we finish the nested slaves investigation. There's only one afaik, and you are already using it. Tolik. Original message From: Eyal Edri ee...@redhat.com Date: 03/06/2015 11:14 PM (GMT+02:00) To: tlito...@redhat.com Cc: in...@ovirt.org Subject: Re: Access to ovirt-srv08.ovirt.org I'd like to move the system testing framework to use nested vms instead of directly using the baremetal hosts. we did use a single host to POC the system, i think now that we're about to expand it will be much more managble and scalable if we'll start using nested VMs which will be managed via the hosted engine. e. - Original Message - From: Tolik Litvosky tlito...@redhat.com To: in...@ovirt.org Sent: Wednesday, June 3, 2015 3:59:36 PM Subject: Access to ovirt-srv08.ovirt.org Hi I need access to the ovirt-srv08.ovirt.org in order to update the testenv CI repo directory. -- Best regards Tolik Litovsky RHEV-H Team Red Hat Red Hat: trustworthy, transformative technology. Powered by the community. Connect at redhat.com ___ Infra mailing list in...@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra -- Best regards Tolik Litovsky RHEV-H Team Red Hat Red Hat: trustworthy, transformative technology. Powered by the community. Connect at redhat.com -- Best regards Tolik Litovsky RHEV-H Team Red Hat Red Hat: trustworthy, transformative technology. Powered by the community. Connect at redhat.com ___ Infra mailing list in...@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 pgpOq6qLTotcS.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Jenkins datacenter outage
We are experiencing an outage on the network of the datacenter where all the jenkins slaves are located, and it's making most of the jobs to fail, usually with dns error messages, the hosting site is informed about it but might take some time to fix, meanwhile be aware that most of the jobs might fail so you'll have to manually verify each patch that you want to merge. Cheers, -- 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 pgpfDoN8NEr09.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] broken build list?
On 05/26, Sandro Bonazzola wrote: Il 26/05/2015 17:03, Greg Sheremeta ha scritto: Any interest in creating a broken build list? I frequently run across jobs failing (findbugs, upgrade, etc.) that aren't related to the patches I post, and I'm never sure whether the responsible person knows about it, whether it's being worked on, etc. I think many of us feel that way -- and, most times, we just wait and try again later. The idea is that all broken builds can be posted about on this dedicated list for higher visibility and knowledge. Then we can be more proactive about fixing them. Can't we use infra@ for build failing on infra issues and devel@ for the real issues? I thought we already were doing that :S Greg Sheremeta Red Hat, Inc. Sr. Software Engineer, RHEV Cell: 919-807-1086 gsher...@redhat.com ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com ___ Infra mailing list in...@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 pgpJWVymrmWcD.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Happening now! the oVirt infra Hackathon is starting
On 05/13, Sandro Bonazzola wrote: Il 12/05/2015 17:08, Eyal Edri ha scritto: Thanks for anyone of joined and helped! We managed to achieve some good process and more training on the way. Some highlights: 1. infra team migrate all tickets from old trac into newly JIRA instance, expect more exiting news on this soon and on new ways to open tickets Please be sure to update https://fedorahosted.org/ovirt/ clearly pointing to the new Jira instance :-) Done 2. various yaml improvement including adding support for default parallel runs (will reduce queue size and reduce waiting for jenkins jobs) 3. preparations for adding a 'workflow' flag, which will enable jenkins to run only on patches ready to be reviewed 4. mock was upgraded to latest version 5. some puppet fixes for infra related manifests 6. foreman cleanup on existing errors 7. removed engine-patches from error email from jenkins, reducing size of emails to developers a lot more are still in progress, and we might schedule a 2nd hackathon soon... follow on on infra list for more info.. thanks, Eyal. - Original Message - From: Eyal Edri ee...@redhat.com To: infra in...@ovirt.org, devel@ovirt.org, Vishnu Sreekumar vishnu.sr...@gmail.com, aung unixa...@gmail.com Sent: Monday, May 11, 2015 10:11:22 AM Subject: [ovirt-devel] Happening now! the oVirt infra Hackathon is starting Hey, the oVirt infra team is starting its 1st hackathon now, details are on [1] the infra team will be on #ovirt_infra_hack on OFTC, for any questions. fyi, for any infra related request plesae open a ticket on [2] and the infra team will handle it tomorrow once the hackathon is over. [1] http://www.ovirt.org/Infra/Hackathons [2] https://fedorahosted.org/ovirt/report/1 Eyal Edri Supervisor, RHEV CI EMEA ENG Virtualization RD Red Hat Israel phone: +972-9-7692018 irc: eedri ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com ___ Infra mailing list in...@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 pgpeM7eug5d74.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Mixed local and shared storage
Thanks Christopher! That will force me to have as many gluster storages as hosts? Or it will allow to have one gluster setup with each of the hosts accessing to the local storage? I will investigate, but I'd prefer if I did not have to use any external service at all (in the end, is just local storage) On 05/11, Christopher Pereira wrote: On 08-05-2015 8:31, David Caro wrote: [...] The case is when you want to spin up vms from a template, but use them only once and remove them afterwards (for example, running tests), if you had the option to have a shared storage where you store the templates, and then a local storage where you spin up the vms, that allows really fast (local) storage for the vms and avoids you having to setup a clustered storage (gluster, etc) for those vms that do not need any of those features. So is that already possible? (For other threads and what I've tested seems not) Is that viable? Can it be implemented on ovirt? Hi David, You could use a local replica-1 gluster storage domain for this. replica-1 = no split brains + local performance + no complexity + no overhead + gluster benefits It just works fine. PS: I'm currently researching the best way to live replicate local replica-1 volumes by using geo-replication, adding a replica-2 brick, using libvirt blockcopy or oVirt API. Best regards, Christopher. -- 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 pgpurpwfyzwrE.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Mixed local and shared storage
Hi everyone! There's a use case for the $subject that is really favorable for us (CI guys) and I want to ask you if it's possible, if it's viable and if not already there if I should open a rfe. The case is when you want to spin up vms from a template, but use them only once and remove them afterwards (for example, running tests), if you had the option to have a shared storage where you store the templates, and then a local storage where you spin up the vms, that allows really fast (local) storage for the vms and avoids you having to setup a clustered storage (gluster, etc) for those vms that do not need any of those features. So is that already possible? (For other threads and what I've tested seems not) Is that viable? Can it be implemented on ovirt? Thanks! -- 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 pgp4oSjmCWQAt.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Jenkins upgrade today
On 04/30, Eyal Edri wrote: Thanks for the great effort david! Lets hope for smoother upgrades in the future :) Any cool new features to look for? i saw there is a new Dynamic Single/Multi line Build History layout. (issue 25381, issue 25393, issue 24687, issue 24589), so might improve seeing build queue views (not exceeding the column boundaries hopefully...) No, not many new features, mostly bug fixes and security updates Going forward we'll migrate jenkins to be a VM on phx and we'll have more control over it via puppet classes + snapshots capabilities. Eyal. - Original Message - From: David Caro dcaro...@redhat.com To: in...@ovirt.org, devel@ovirt.org Sent: Wednesday, April 29, 2015 9:45:00 PM Subject: Re: [ovirt-devel] Jenkins upgrade today On 04/29, David Caro wrote: Ok, so finally the upgrade is finished... heres the summary and current status: First tried to upgrade to version 1.607, that fixes some extra security advisories, doing that the gerrit trigger stopped working, then tried to upgrade and test different versions of the gerrit trigger plugin, none of them finally worked. Opened a bug on jenkins jira [1] on it. After that, I downgraded to 1.605, that has the major security fixes, but seems more stable (more positive votes from the community), the downgrade went well, the gerrit trigger plugin started working again, but all the slaves were gone. Then I tried upgrading/downgrading the ssh slaves plugin without getting them back, so I started to manually add them all one by one. Result is, that the slaves are back online, gerrit trigger is working, and jenkins is upgraded to 1.605 [2] It can be that some of the plugins misbehave, or that the slaves were not exactly configured as before (maybe there were some special tags that have been lost), so please send an email to infra if you see anything suspicious. Forgot the links: [1] https://issues.jenkins-ci.org/browse/JENKINS-28161 [2] http://jenkins-ci.org/changelog Thanks! On 04/29, David Caro wrote: I'm having some issues with the upgrade, so I'm still working on stabilizing it, will notify you when it's back to stable. On 04/29, David Caro wrote: Hi! Today I will be upgrading jenkins to the latest stable version, the upgrade will start at 17:00 CEST (18:00 IST) and will take around 30min. I will set jenkins to not accept new jobs 30 minutes before that to let any pending jobs finish, but any job that does not finish in that time, will be aborted, if that happens feel free to retrigger your patch at [1]. Cheers! [1] http://jenkins.ovirt.org/gerrit_manual_trigger/ -- 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 -- 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 ___ Infra mailing list in...@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 -- 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 ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- 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 pgpXsC50ZzsJd.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Maintainer rights on vdsm - ovirt-3.5-gluster
Done On 04/20, Dan Kenigsberg wrote: On Mon, Apr 20, 2015 at 03:20:18PM +0530, Sahina Bose wrote: Hi! On the vdsm branch ovirt-3.5-gluster, could you provide merge rights to Bala (barum...@redhat.com) ? +1 from me. ovirt-3.5-gluster needs a rebase on top of the current ovirt-3.5 ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- 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 pgpt9FjWbTbxl.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Jenkins upgrade today
Hi! Today I will be upgrading jenkins to the latest stable version, the upgrade will start at 17:00 CEST (18:00 IST) and will take around 30min. I will set jenkins to not accept new jobs 30 minutes before that to let any pending jobs finish, but any job that does not finish in that time, will be aborted, if that happens feel free to retrigger your patch at [1]. Cheers! [1] http://jenkins.ovirt.org/gerrit_manual_trigger/ -- 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 pgpg6NkacMJWK.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Maintainer rights on vdsm - ovirt-3.5-gluster
You want also push rights? It's not the same submit, push, create reference, verify, code review. All those are different rights. I've added the submit, code-review and verify rights. Let me know if you want also the rest. Also note that pushing and submitting is not the same, pushing will skip gerrit and any gerrit checks, so it's usually something to avoid. While submitting is done through gerrit, what allows gerrit to check validity of the code to be merged (gerrit hooks, updating bugzillas, jenkins jobs, signed-by flabs). On 04/29, Sahina Bose wrote: On 04/20/2015 04:36 PM, Dan Kenigsberg wrote: On Mon, Apr 20, 2015 at 03:20:18PM +0530, Sahina Bose wrote: Hi! On the vdsm branch ovirt-3.5-gluster, could you provide merge rights to Bala (barum...@redhat.com) ? +1 from me. ovirt-3.5-gluster needs a rebase on top of the current ovirt-3.5 Hi! Is there a push right to be enabled on this branch as well? Currently, rebasing and pushing is failing with following: remote: Branch refs/heads/ovirt-3.5-gluster: remote: You are not allowed to perform this operation. remote: To push into this reference you need 'Push' rights. remote: User: bala remote: Please read the documentation and contact an administrator remote: if you feel the configuration is incorrect remote: Processing changes: refs: 1, done To ssh://b...@gerrit.ovirt.org:29418/vdsm.git ! [remote rejected] ovirt-3.5-gluster - ovirt-3.5-gluster (prohibited by Gerrit) error: failed to push some refs to 'ssh://b...@gerrit.ovirt.org:29418/vdsm.git' ___ Infra mailing list in...@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 pgp_ikTr0H4tL.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Jenkins upgrade today
On 04/29, David Caro wrote: Ok, so finally the upgrade is finished... heres the summary and current status: First tried to upgrade to version 1.607, that fixes some extra security advisories, doing that the gerrit trigger stopped working, then tried to upgrade and test different versions of the gerrit trigger plugin, none of them finally worked. Opened a bug on jenkins jira [1] on it. After that, I downgraded to 1.605, that has the major security fixes, but seems more stable (more positive votes from the community), the downgrade went well, the gerrit trigger plugin started working again, but all the slaves were gone. Then I tried upgrading/downgrading the ssh slaves plugin without getting them back, so I started to manually add them all one by one. Result is, that the slaves are back online, gerrit trigger is working, and jenkins is upgraded to 1.605 [2] It can be that some of the plugins misbehave, or that the slaves were not exactly configured as before (maybe there were some special tags that have been lost), so please send an email to infra if you see anything suspicious. Forgot the links: [1] https://issues.jenkins-ci.org/browse/JENKINS-28161 [2] http://jenkins-ci.org/changelog Thanks! On 04/29, David Caro wrote: I'm having some issues with the upgrade, so I'm still working on stabilizing it, will notify you when it's back to stable. On 04/29, David Caro wrote: Hi! Today I will be upgrading jenkins to the latest stable version, the upgrade will start at 17:00 CEST (18:00 IST) and will take around 30min. I will set jenkins to not accept new jobs 30 minutes before that to let any pending jobs finish, but any job that does not finish in that time, will be aborted, if that happens feel free to retrigger your patch at [1]. Cheers! [1] http://jenkins.ovirt.org/gerrit_manual_trigger/ -- 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 -- 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 ___ Infra mailing list in...@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 -- 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 pgpjeHcrf89J_.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Gerrit upgrade
Hi everyone, we will be upgrading gerrit this evening, if everything goes well, we might even have oauth auth (google and github). The upgrade will start at 17:00 CEST (18:00 IST) and should take ~15 min, downtime should be minimal, but some restarts might be required and cache flushes, so don't expect it to work properly until finishedf, will let you know what's the result after. Thanks! -- 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 pgpW6CltFKZtU.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Gerrit upgrade
Hi! Gerrit upgrade is finthed, and I have good and bad news: * The oauth plugin is up and running, it works with google and github oauth services correctly BUT the accounts are not detected automatically and there's no easy way of doing so manually (it requires a token that is generated on the first request). So anyone that did not wait for a magic solution will have to stop waiting and follow the steps I sent the last time [1] All the others make sure to do this to add the new ids: * log into gerrit with the old openid * go to settings-account-Identities * hit 'Link Another Identity' * you'll see the same page as logging in, not you have github and google links under the textbox, select the one you want to link and follow the instructions Note: you might see a red 'untrusted' tag on the new added oauth ids, I'm working on it, should not affect it's usage. Enjoy! pd. if you still have issues, feel free to contact me through the infra list [1] https://www.mail-archive.com/devel%40ovirt.org/msg03368.html On 04/27, David Caro wrote: Hi everyone, we will be upgrading gerrit this evening, if everything goes well, we might even have oauth auth (google and github). The upgrade will start at 17:00 CEST (18:00 IST) and should take ~15 min, downtime should be minimal, but some restarts might be required and cache flushes, so don't expect it to work properly until finishedf, will let you know what's the result after. Thanks! -- 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 -- 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 pgpZY8UVao8B0.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Gerrit upgrade
Quick update: The ui adding of the oauth is not yet working [1], so if you want link the google id and can't wait, send an email and will get you through. [1] https://github.com/davido/gerrit-oauth-provider/wiki/FAQ On 04/27, David Caro wrote: Hi! Gerrit upgrade is finthed, and I have good and bad news: * The oauth plugin is up and running, it works with google and github oauth services correctly BUT the accounts are not detected automatically and there's no easy way of doing so manually (it requires a token that is generated on the first request). So anyone that did not wait for a magic solution will have to stop waiting and follow the steps I sent the last time [1] All the others make sure to do this to add the new ids: * log into gerrit with the old openid * go to settings-account-Identities * hit 'Link Another Identity' * you'll see the same page as logging in, not you have github and google links under the textbox, select the one you want to link and follow the instructions Note: you might see a red 'untrusted' tag on the new added oauth ids, I'm working on it, should not affect it's usage. Enjoy! pd. if you still have issues, feel free to contact me through the infra list [1] https://www.mail-archive.com/devel%40ovirt.org/msg03368.html On 04/27, David Caro wrote: Hi everyone, we will be upgrading gerrit this evening, if everything goes well, we might even have oauth auth (google and github). The upgrade will start at 17:00 CEST (18:00 IST) and should take ~15 min, downtime should be minimal, but some restarts might be required and cache flushes, so don't expect it to work properly until finishedf, will let you know what's the result after. Thanks! -- 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 -- 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 -- 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 pgpD1XjthW6Gp.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Fwd: Facing issues while login into gerrit.ovirt.org
The change is not that simple, the issue is that google dropped OpenID at once, so to keep logging in with google we need to add oauth2, and that's not well supported yet, requires a gerrit upgrade and an external plugin compilation (not yet supported officially) and account migration that's potentially dangerous. So, for now, stop using the ui if you don't have a non-google OpenID associated already to your account, and ping me on irc, as adding the new id requires accessing the db directly. You can though use ssh to review/merge/send patches On 04/22, Martin Perina wrote: Hi, according to [1] our gerrit needs to be updated to 2.10.3. David, is it possible to do that soon? Thanks Martin [1] https://code.google.com/p/gerrit/issues/detail?id=2677 - Original Message - From: Shubhendu Tripathi shtri...@redhat.com To: devel@ovirt.org Sent: Wednesday, April 22, 2015 6:47:07 AM Subject: [ovirt-devel] Fwd: Facing issues while login into gerrit.ovirt.org Forwarded Message Subject:Facing issues while login into gerrit.ovirt.org Date: Wed, 22 Apr 2015 09:59:37 +0530 From: Shubhendu Tripathi shtri...@redhat.com To: infra in...@ovirt.org Hi, While login to gerrit.ovirt.org today I am facing an issue where it redirects to [1] if I select open id as my gmail account. Looks like gerrit needs to migrate to OpenID Connect now. Any idea how to resolve the same? Thanks and Regards, Shubhendu Tripathi [1] https://support.google.com/accounts/answer/6206245?p=openidrd=1 ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- 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 pgp961ev6eEJg.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] [oVirt Jenkins] vdsm_master_unit-tests_merged - Build # 4878 - Still Failing!
On the current old job yes, that's ignored, but on the new one that has been waiting for any vdsm devel to fix for quite some time, that's not an issue: http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created_staging/ On 04/21, Michal Skrivanek wrote: On Apr 21, 2015, at 06:14 , Eyal Edri ee...@redhat.com wrote: Adding max Sent from Nine From: Nir Soffer Sent: Apr 21, 2015 2:59 AM To: Dan Kenigsberg Cc: msi...@redhat.com;mpoled...@redhat.com;ee...@redhat.com;devel@ovirt.org;dc...@redhat.com;do...@redhat.com Subject: Re: [oVirt Jenkins] vdsm_master_unit-tests_merged - Build # 4878 - Still Failing! - Original Message - From: Dan Kenigsberg dan...@redhat.com To: msi...@redhat.com Cc: mpoled...@redhat.com, ee...@redhat.com, devel@ovirt.org, dc...@redhat.com, nsof...@redhat.com, do...@redhat.com Sent: Monday, April 20, 2015 8:52:15 AM Subject: Re: [oVirt Jenkins] vdsm_master_unit-tests_merged - Build # 4878 - Still Failing! On Sun, Apr 19, 2015 at 01:26:47PM +0100, Jenkins ci oVirt Server wrote: Project: http://jenkins.ovirt.org/job/vdsm_master_unit-tests_merged/ Build: http://jenkins.ovirt.org/job/vdsm_master_unit-tests_merged/4878/ Build Number: 4878 Build Status: Still Failing Triggered By: Triggered by Gerrit: https://gerrit.ovirt.org/39806 Hi Martin, I have merged your https://gerrit.ovirt.org/39411 prematurely, and provided an insufficient fix to it. This patch works for me, after installing mom. Not sure why it fails in the ci. well, seems because we ignore BuildRequires: mom apparently Nir ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- 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 pgpM_tL4uSXOv.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Agenda for today oVirt Weekly Sync
I want to add: - Security incident status On 04/15, Brian Proffitt wrote: As noted in users list, I will be unable to attend today's meeting due to a child's dr. appointment. I have laid out my suggestions for changing the format of the meeting [1], and hope they will be debated. For events; the most important thing is yet-another request for help at Grazer Linux Days in Graz, Austria this weekend[2]. Rene Koch is seeking backup help manning the booth at that event[2] All other event and conference news is the same as two weeks ago. Thanks, Brian [1] http://lists.ovirt.org/pipermail/users/2015-April/032389.html [2] http://lists.ovirt.org/pipermail/users/2015-April/032251.html - Original Message - From: Sandro Bonazzola sbona...@redhat.com To: us...@ovirt.org, devel@ovirt.org, Brian Proffitt bprof...@redhat.com, Yaniv Dary yd...@redhat.com, infra in...@ovirt.org Sent: Wednesday, April 15, 2015 7:04:07 AM Subject: Agenda for today oVirt Weekly Sync Hi, I would like to add to the oVirt Weekly Sync agenda the following topics - Packages review for 3.5.2 GA - Blockers review for 3.5.2 GA - Outages in Jenkins due to no space left on master - Next week is feature submission deadline: plans for reviewing submitted features - Support for JDK 1.8 / Wildfly progress updates / Contingency plan of building Java7 rpms for Fedora 22. -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com -- Brian Proffitt Community Liaison oVirt Open Source and Standards, Red Hat - http://community.redhat.com Phone: +1 574 383 9BKP IRC: bkp @ OFTC ___ Infra mailing list in...@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 pgpA2K6hZNIy7.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Gerrit, only infra can remove reviewers
On 04/13, Michal Skrivanek wrote: On Apr 8, 2015, at 19:24 , David Caro dcaro...@redhat.com wrote: Effective now, only infra team members can remove reviewers from engine master and vdsm master. That means that you'll have to poke one of us each time you want to prevent jenkins (or anyone) from blocking any patches. we have people working in various time zones, how do we make sure an urgent thing is addressed in time? what's the full contact list? escalation list? infra at ovirt dot org for any and all the issues. Prefferently if you opened a ticket at [1] 1 https://fedorahosted.org/ovirt/ Thanks, michal Enjoy! -- 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 ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- 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 pgpQfhgnYkMYG.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Gerrit, only infra can remove reviewers
On 04/13, Moti Asayag wrote: - Original Message - From: David Caro dcaro...@redhat.com To: devel@ovirt.org, in...@ovirt.org Sent: Wednesday, April 8, 2015 7:24:42 PM Subject: [ovirt-devel] Gerrit, only infra can remove reviewers Effective now, only infra team members can remove reviewers from engine master and vdsm master. Can't you limit the list of removed reviewers only for the relevant reviewer list ? specifically for gerrit-hooks and oVirt Jenkins CI Server ? No, that's not an option right now (unsupported) I'd like to have the ability to remove unintended reviewer from a patch (i.e. if was mistakenly added ). Also, can a user remove himself from the reviewer list of a given patch ? Yes, you should, have you tried? That means that you'll have to poke one of us each time you want to prevent jenkins (or anyone) from blocking any patches. Enjoy! -- 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 ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- 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 pgpuNFWwLVZZh.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] gerrit jenkins bot has gotten noisier
This was done because it was requested some time ago. That will also remind devels that they should not merge any patches that have not finished running the jenkins jobs yet. In a (hopefully) near future, that will not be needed as jobs will become a gateway to merge, meaning that the regular merge flow will require passing the jenkins jobs. In the meantime, it's just a remainder. On 03/25, Martin Perina wrote: - Original Message - From: Greg Sheremeta gsher...@redhat.com To: in...@ovirt.org, Eyal Edri ee...@redhat.com, David Caro dcaro...@redhat.com Cc: devel@ovirt.org Sent: Wednesday, March 25, 2015 1:40:35 PM Subject: [ovirt-devel] gerrit jenkins bot has gotten noisier Hey guys, Sometime last week, the jenkins bot upped the verbosity of his notifications on patches. Was this by design? Example: https://gerrit.ovirt.org/#/c/39098/ He now adds many build started messages. He used to only let us know about success / failure. Not a huge deal, but it's more noise to filter through -- both in gerrit and email. Agreed, especially for emails. Personally I don't care about build process details, I'd like to know only final result of the build. If the build failed, I could find details on jenkins. Martin Thanks, Greg Greg Sheremeta Red Hat, Inc. Sr. Software Engineer, RHEV Cell: 919-807-1086 gsher...@redhat.com ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- 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 pgpgB2xKpXkB2.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Gerrit update
On 03/02, Yaniv Dary wrote: The removal of publish comment and submit has been making my life harder. Can you redirect the comment to the gerrit forums? We don't have the manpower required to maintain a customized fork. Yaniv On 02/24/2015 01:00 PM, Yedidyah Bar David wrote: - Original Message - From: Eli Mesika emes...@redhat.com To: David Caro dcaro...@redhat.com Cc: devel@ovirt.org, Infra in...@ovirt.org Sent: Tuesday, February 24, 2015 12:53:44 PM Subject: Re: [ovirt-devel] Gerrit update Another nice feature added is full keyboard support so you can do anything with your keyboard. Most used by me is the ']' and '[' to got to Next/Prev file in the patch Very nice indeed, but not new... Press '?' to get a context-sensitive list of keyboard shortcuts. - Original Message - From: David Caro dcaro...@redhat.com To: Infra in...@ovirt.org, devel@ovirt.org Sent: Monday, February 23, 2015 8:29:03 PM Subject: Re: Gerrit update Some relevant new features that might be missed: == UI * Add My Groups menu entry that shows the list of own groups. * Added option to quickly add current user as reviewer of a change. An Add Me button is displayed next to the Add button when searching for reviewers to add to a change. This allows users to quickly add themselves as a reviewer on the change without having to type their name in the search box. == Side-by-Side Diff Screen * Allow the user to select the syntax highlighter. * Add Shift-a keybinding to show/hide left side. * Add syntax highlighting of the commit message. == Change List / Dashboards * Add option to show Change-ID in the change table. == Project Screens * Add branch actions to Projects Branches view. == User Preferences * Users can customize the contents of the My menu from the preferences screen. * Replace Display name in review category preference with a list of options. Including new options Show Abbreviated Name to display Abbreviated reviewer names and Show Username to show usernames in the change list. For a full list you can check the release notes: https://gerrit-documentation.storage.googleapis.com/ReleaseNotes/ReleaseNotes-2.10.html Enjoy! On 02/23, David Caro wrote: Gerrit has been upgraded successfully, I'll be monitoring it through the day to make sure nothing pops up, but if you see anything strange please ping me. Thanks! On 02/23, David Caro wrote: Hi everyone! As of right now, gerrit is being updated to 2.10 [1] I will notice when it's ready, in the meantime you (or jenkins) will not be able to use it. I forgot the link: [1] https://gerrit-documentation.storage.googleapis.com/ReleaseNotes/ReleaseNotes-2.10.html Cheers! -- 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 ___ Infra mailing list in...@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 -- 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 ___ Infra mailing list in...@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- Yaniv Dary Red Hat Israel Ltd. 34 Jerusalem Road Building A, 4th floor Ra'anana, Israel 4350109 Tel : +972 (9) 7692306 8272306 Email: yd...@redhat.com IRC : ydary ___ Infra mailing list in...@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 pgpZduMBn5t7n.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Gerrit restart
Can you provide more detailed information please? * Full command line you did run * Full ssh config for gerit.us * git remote -v * git log --graph --decorate --pretty=oneline --abbrev-commit -100 * git branch -a -v * git config --get-regexp user.* That way we can start looking into it. Thanks! On 02/24, Vered Volansky wrote: Hi, I can't push a patch: ! [remote rejected] HEAD - res/for/master (prohibited by Gerrit) error: failed to push some refs to 'gerrit.us:ovirt-engine' I also can't ssh gerrit.ovirt.org: Permission denied (publickey,gssapi-keyex,gssapi-with-mic). Obviously nothing changed in my .ssh/gerrit ssh public keys setting. Thanks, Vered - Original Message - From: David Caro dcaro...@redhat.com To: devel@ovirt.org Sent: Monday, February 23, 2015 12:31:24 PM Subject: [ovirt-devel] Gerrit restart Hi! After this morning upgrade, gerrit http access has stopped working, I'm trying to solve it but in the meantime I might have to restart gerrit a couple of times, you can still use git or ssh to clone repos. I'll update when finished. -- 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 ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- 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 pgpQRztVOAfgN.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Gerrit restart
Hi! After this morning upgrade, gerrit http access has stopped working, I'm trying to solve it but in the meantime I might have to restart gerrit a couple of times, you can still use git or ssh to clone repos. I'll update when finished. -- 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 pgp2dMo5P19sm.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Gerrit restart
I've done a few tests now and everything seems to work as expected, can anyone confirm that it's working for him? If noone replies, I'll consider it solved cheers On 02/23, David Caro wrote: Hi! After this morning upgrade, gerrit http access has stopped working, I'm trying to solve it but in the meantime I might have to restart gerrit a couple of times, you can still use git or ssh to clone repos. I'll update when finished. -- 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 -- 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 pgpLOgtJbEWwi.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Gerrit restart
On 02/23, Martin Perina wrote: Hi, this Cannot merge error shown in Strategy: Rebase if Necessary line is strange. For example patch [1] is rebased on top of master and locally I cannot see any path conflict, build was including tests was OK. Same as Nir patch, you can't cherry-pick that patch by itself: 01:36 PM ~/Work/redhat/ovirt/ovirt-engine (master|✔) dcaro@brete$ git fetch ssh://dc...@gerrit.ovirt.org:29418/ovirt-engine refs/changes/59/38059/1 git cherry-pick FETCH_HEAD remote: Counting objects: 19408, done remote: Finding sources: 100% (123/123) remote: Total 123 (delta 45), reused 115 (delta 45) Receiving objects: 100% (123/123), 40.06 KiB | 0 bytes/s, done. Resolving deltas: 100% (45/45), completed with 42 local objects. From ssh://gerrit.ovirt.org:29418/ovirt-engine * branchrefs/changes/59/38059/1 - FETCH_HEAD error: could not apply 07a90af... core: Refactor FenceActionType hint: after resolving the conflicts, mark the corrected paths hint: with 'git add paths' or 'git rm paths' hint: and commit the result with 'git commit' 01:37 PM ~/Work/redhat/ovirt/ovirt-engine (master|● 19✖ 1✚ 1) dcaro@brete$ Once you merge the patch it depends on, It should stop showing that message. Martin [1] http://gerrit.ovirt.org/38059 - Original Message - From: Nir Soffer nsof...@redhat.com To: David Caro dcaro...@redhat.com Cc: devel@ovirt.org Sent: Monday, February 23, 2015 12:33:24 PM Subject: Re: [ovirt-devel] Gerrit restart - Original Message - From: David Caro dcaro...@redhat.com To: devel@ovirt.org Sent: Monday, February 23, 2015 12:54:26 PM Subject: Re: [ovirt-devel] Gerrit restart I've done a few tests now and everything seems to work as expected, can anyone confirm that it's working for him? It seem to work but there are few issues: 1. Vertical and horizontal scrolling in the commit message pane The new version assume that the commit message is limited to about 70 characters, which is not true practically. The commit message must include sometimes much wider content, and the view should display correctly our data instead of trying to force the views of the tool authors. Example: http://gerrit.ovirt.org/36593 2. The changes view (http://gerrit.ovirt.org/#/dashboard/self) Show a new Status column, which has two issues: - Showing Draft or Merge Conflict - these are separate concepts - Showing Merge Conflict when there is no merge conflict Example: http://gerrit.ovirt.org/38044 3. The new Conflict with pane seems incorrect about patches which do not conflict (at least not when merged in the correct order of dependencies). Example: http://gerrit.ovirt.org/37727 So this seems to be a regression. Can we test such upgrades with our data before we deploy them? Nir If noone replies, I'll consider it solved cheers On 02/23, David Caro wrote: Hi! After this morning upgrade, gerrit http access has stopped working, I'm trying to solve it but in the meantime I might have to restart gerrit a couple of times, you can still use git or ssh to clone repos. I'll update when finished. -- 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 -- 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 ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- 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 pgpOyzS4FOdZH.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Gerrit restart
On 02/23, David Caro wrote: On 02/23, Nir Soffer wrote: - Original Message - From: David Caro dcaro...@redhat.com To: Martin Perina mper...@redhat.com Cc: devel@ovirt.org, Nir Soffer nsof...@redhat.com Sent: Monday, February 23, 2015 2:39:42 PM Subject: Re: [ovirt-devel] Gerrit restart On 02/23, Martin Perina wrote: Hi, this Cannot merge error shown in Strategy: Rebase if Necessary line is strange. For example patch [1] is rebased on top of master and locally I cannot see any path conflict, build was including tests was OK. Same as Nir patch, you can't cherry-pick that patch by itself: 01:36 PM ~/Work/redhat/ovirt/ovirt-engine (master|✔) dcaro@brete$ git fetch ssh://dc...@gerrit.ovirt.org:29418/ovirt-engine refs/changes/59/38059/1 git cherry-pick FETCH_HEAD remote: Counting objects: 19408, done remote: Finding sources: 100% (123/123) remote: Total 123 (delta 45), reused 115 (delta 45) Receiving objects: 100% (123/123), 40.06 KiB | 0 bytes/s, done. Resolving deltas: 100% (45/45), completed with 42 local objects. From ssh://gerrit.ovirt.org:29418/ovirt-engine * branchrefs/changes/59/38059/1 - FETCH_HEAD error: could not apply 07a90af... core: Refactor FenceActionType hint: after resolving the conflicts, mark the corrected paths hint: with 'git add paths' or 'git rm paths' hint: and commit the result with 'git commit' 01:37 PM ~/Work/redhat/ovirt/ovirt-engine (master|● 19✖ 1✚ 1) dcaro@brete$ Once you merge the patch it depends on, It should stop showing that message. Yea, but we do not suggest this patch alone, it is part of a series, which gerrit knows about. Maybe there is a new configuration to disable these messages? I don't see it in the config documentation. Feel free to open a bug to gerrit if it bothers you enough or if you want it changed. We can also do as openstack and have our own gerrit clone with custom patches, but as always, that requires someone/s to maintain it and develop it. That would allow us to have the nice jenkins side pannel that they have too: https://review.openstack.org/#/c/157452/ Martin [1] http://gerrit.ovirt.org/38059 - Original Message - From: Nir Soffer nsof...@redhat.com To: David Caro dcaro...@redhat.com Cc: devel@ovirt.org Sent: Monday, February 23, 2015 12:33:24 PM Subject: Re: [ovirt-devel] Gerrit restart - Original Message - From: David Caro dcaro...@redhat.com To: devel@ovirt.org Sent: Monday, February 23, 2015 12:54:26 PM Subject: Re: [ovirt-devel] Gerrit restart I've done a few tests now and everything seems to work as expected, can anyone confirm that it's working for him? It seem to work but there are few issues: 1. Vertical and horizontal scrolling in the commit message pane The new version assume that the commit message is limited to about 70 characters, which is not true practically. The commit message must include sometimes much wider content, and the view should display correctly our data instead of trying to force the views of the tool authors. Example: http://gerrit.ovirt.org/36593 2. The changes view (http://gerrit.ovirt.org/#/dashboard/self) Show a new Status column, which has two issues: - Showing Draft or Merge Conflict - these are separate concepts - Showing Merge Conflict when there is no merge conflict Example: http://gerrit.ovirt.org/38044 3. The new Conflict with pane seems incorrect about patches which do not conflict (at least not when merged in the correct order of dependencies). Example: http://gerrit.ovirt.org/37727 So this seems to be a regression. Can we test such upgrades with our data before we deploy them? Nir If noone replies, I'll consider it solved cheers On 02/23, David Caro wrote: Hi! After this morning upgrade, gerrit http access has stopped working, I'm trying to solve it but in the meantime I might have to restart gerrit a couple of times, you can still use git or ssh to clone repos. I'll update when finished. -- 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 -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization RD Tel.: +420 532 294 605
Re: [ovirt-devel] Gerrit restart
On 02/23, Nir Soffer wrote: - Original Message - From: David Caro dcaro...@redhat.com To: Martin Perina mper...@redhat.com Cc: devel@ovirt.org, Nir Soffer nsof...@redhat.com Sent: Monday, February 23, 2015 2:39:42 PM Subject: Re: [ovirt-devel] Gerrit restart On 02/23, Martin Perina wrote: Hi, this Cannot merge error shown in Strategy: Rebase if Necessary line is strange. For example patch [1] is rebased on top of master and locally I cannot see any path conflict, build was including tests was OK. Same as Nir patch, you can't cherry-pick that patch by itself: 01:36 PM ~/Work/redhat/ovirt/ovirt-engine (master|✔) dcaro@brete$ git fetch ssh://dc...@gerrit.ovirt.org:29418/ovirt-engine refs/changes/59/38059/1 git cherry-pick FETCH_HEAD remote: Counting objects: 19408, done remote: Finding sources: 100% (123/123) remote: Total 123 (delta 45), reused 115 (delta 45) Receiving objects: 100% (123/123), 40.06 KiB | 0 bytes/s, done. Resolving deltas: 100% (45/45), completed with 42 local objects. From ssh://gerrit.ovirt.org:29418/ovirt-engine * branchrefs/changes/59/38059/1 - FETCH_HEAD error: could not apply 07a90af... core: Refactor FenceActionType hint: after resolving the conflicts, mark the corrected paths hint: with 'git add paths' or 'git rm paths' hint: and commit the result with 'git commit' 01:37 PM ~/Work/redhat/ovirt/ovirt-engine (master|● 19✖ 1✚ 1) dcaro@brete$ Once you merge the patch it depends on, It should stop showing that message. Yea, but we do not suggest this patch alone, it is part of a series, which gerrit knows about. Maybe there is a new configuration to disable these messages? I don't see it in the config documentation. Feel free to open a bug to gerrit if it bothers you enough or if you want it changed. We can also do as openstack and have our own gerrit clone with custom patches, but as always, that requires someone/s to maintain it and develop it. Martin [1] http://gerrit.ovirt.org/38059 - Original Message - From: Nir Soffer nsof...@redhat.com To: David Caro dcaro...@redhat.com Cc: devel@ovirt.org Sent: Monday, February 23, 2015 12:33:24 PM Subject: Re: [ovirt-devel] Gerrit restart - Original Message - From: David Caro dcaro...@redhat.com To: devel@ovirt.org Sent: Monday, February 23, 2015 12:54:26 PM Subject: Re: [ovirt-devel] Gerrit restart I've done a few tests now and everything seems to work as expected, can anyone confirm that it's working for him? It seem to work but there are few issues: 1. Vertical and horizontal scrolling in the commit message pane The new version assume that the commit message is limited to about 70 characters, which is not true practically. The commit message must include sometimes much wider content, and the view should display correctly our data instead of trying to force the views of the tool authors. Example: http://gerrit.ovirt.org/36593 2. The changes view (http://gerrit.ovirt.org/#/dashboard/self) Show a new Status column, which has two issues: - Showing Draft or Merge Conflict - these are separate concepts - Showing Merge Conflict when there is no merge conflict Example: http://gerrit.ovirt.org/38044 3. The new Conflict with pane seems incorrect about patches which do not conflict (at least not when merged in the correct order of dependencies). Example: http://gerrit.ovirt.org/37727 So this seems to be a regression. Can we test such upgrades with our data before we deploy them? Nir If noone replies, I'll consider it solved cheers On 02/23, David Caro wrote: Hi! After this morning upgrade, gerrit http access has stopped working, I'm trying to solve it but in the meantime I might have to restart gerrit a couple of times, you can still use git or ssh to clone repos. I'll update when finished. -- 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 -- 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 ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel ___ Devel mailing list
Re: [ovirt-devel] Gerrit restart
Did you change anything? I've restated httpd to refresh the ssl cert (now it's a valid self-signed one). But that seems unrelated. On 02/23, Martin Betak wrote: Ok, not to jinx it, but login seems to be working now :-) - Original Message - From: Martin Betak mbe...@redhat.com To: David Caro dcaro...@redhat.com Cc: devel@ovirt.org Sent: Monday, February 23, 2015 1:19:04 PM Subject: Re: [ovirt-devel] Gerrit restart I have trouble signing-in. It returns 404: The requested URL /login/#/c/32907/ was not found on this server. Martin - Original Message - From: David Caro dcaro...@redhat.com To: devel@ovirt.org Sent: Monday, February 23, 2015 11:54:26 AM Subject: Re: [ovirt-devel] Gerrit restart I've done a few tests now and everything seems to work as expected, can anyone confirm that it's working for him? If noone replies, I'll consider it solved cheers On 02/23, David Caro wrote: Hi! After this morning upgrade, gerrit http access has stopped working, I'm trying to solve it but in the meantime I might have to restart gerrit a couple of times, you can still use git or ssh to clone repos. I'll update when finished. -- 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 -- 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 ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- 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 pgpTmeE7ZeWY5.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Gerrit restart
On 02/23, Nir Soffer wrote: - Original Message - From: David Caro dcaro...@redhat.com To: devel@ovirt.org Sent: Monday, February 23, 2015 12:54:26 PM Subject: Re: [ovirt-devel] Gerrit restart I've done a few tests now and everything seems to work as expected, can anyone confirm that it's working for him? It seem to work but there are few issues: 1. Vertical and horizontal scrolling in the commit message pane The new version assume that the commit message is limited to about 70 characters, which is not true practically. The commit message must include sometimes much wider content, and the view should display correctly our data instead of trying to force the views of the tool authors. Example: http://gerrit.ovirt.org/36593 This has always been like that as far as I remember. Not a new issue. Actually now it allows you to scroll instead of letting the text go all the way out the screen, I think it's an improvement. 2. The changes view (http://gerrit.ovirt.org/#/dashboard/self) Show a new Status column, which has two issues: - Showing Draft or Merge Conflict - these are separate concepts That's a design decision, nothing we can do about. Feel free to develop a ui plugin to hide it if it bothers you, this new version allows it (the old did not). - Showing Merge Conflict when there is no merge conflict Example: http://gerrit.ovirt.org/38044 You can't cherry-pick that patch into master directly (vdsm project submit type is cherry-pick): 01:16 PM ~/Work/redhat/ovirt/vdsm (master|✔) dcaro@brete$ git fetch ssh://dc...@gerrit.ovirt.org:29418/vdsm refs/changes/28/38028/1 remote: Counting objects: 745, done remote: Finding sources: 100% (29/29) remote: Total 29 (delta 3), reused 9 (delta 3) Unpacking objects: 100% (29/29), done. From ssh://gerrit.ovirt.org:29418/vdsm * branchrefs/changes/28/38028/1 - FETCH_HEAD 01:16 PM ~/Work/redhat/ovirt/vdsm (master|✔) dcaro@brete$ git cherry-pick FETCH_HEAD error: could not apply 53f59a7... virt: Extract Vm.__extendDriveReplica method hint: after resolving the conflicts, mark the corrected paths hint: with 'git add paths' or 'git rm paths' hint: and commit the result with 'git commit' 01:17 PM ~/Work/redhat/ovirt/vdsm (master|✖ 1✚ 1) dcaro@brete$ 3. The new Conflict with pane seems incorrect about patches which do not conflict (at least not when merged in the correct order of dependencies). Example: http://gerrit.ovirt.org/37727 I don't think it's taking into account the deps, the conflict seems to be calculated from path to path directly. Feel free to open a but to gerrit if you think any of those issues should not be there or it's a bug. https://code.google.com/p/gerrit/issues/list So this seems to be a regression. Can we test such upgrades with our data before we deploy them? Sure, I'll ping you to do the testing next time we plan an upgrade, I'll have to trim down some of the data but I'll gladly spend some time on it to provide you with a working dataset you can install and test before the upgrade. Nir If noone replies, I'll consider it solved cheers On 02/23, David Caro wrote: Hi! After this morning upgrade, gerrit http access has stopped working, I'm trying to solve it but in the meantime I might have to restart gerrit a couple of times, you can still use git or ssh to clone repos. I'll update when finished. -- 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 -- 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 ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- 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 pgpw7m6KjpCgf.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Fwd: Gerrit update
I sent it only to infra :/ - Forwarded message from David Caro dcaro...@redhat.com - From: David Caro dcaro...@redhat.com To: Infra in...@ovirt.org Date: Mon, 23 Feb 2015 07:30:27 +0100 Subject: Gerrit update User-Agent: Mutt/1.5.22.1 (2013-10-16) Hi everyone! As of right now, gerrit is being updated to 2.10 [1] I will notice when it's ready, in the meantime you (or jenkins) will not be able to use it. Cheers! -- 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 ___ Infra mailing list in...@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra - End forwarded message - -- 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 pgpmqrv8SNp4J.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Gerrit update
Gerrit has been upgraded successfully, I'll be monitoring it through the day to make sure nothing pops up, but if you see anything strange please ping me. Thanks! On 02/23, David Caro wrote: Hi everyone! As of right now, gerrit is being updated to 2.10 [1] I will notice when it's ready, in the meantime you (or jenkins) will not be able to use it. I forgot the link: [1] https://gerrit-documentation.storage.googleapis.com/ReleaseNotes/ReleaseNotes-2.10.html Cheers! -- 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 ___ Infra mailing list in...@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 pgpNATbtQ7HDP.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Help with issues with migration
On 02/04, Francesco Romani wrote: - Original Message - From: David Caro dcaro...@redhat.com To: devel@ovirt.org Sent: Wednesday, February 4, 2015 11:21:39 AM Subject: [ovirt-devel] Help with issues with migration Hi! Upstream phoenix lab has stabilized, but after the outages we are finding some issues probably caused by it. One of them is that th vm migration is not working, ew have two hosts srv05 and srv06, and we want to migrate vms from 06 to 05 but we find those errors on vdsm on 05: Feb 04 02:56:50 ovirt-srv05 vdsm[5170]: vdsm vm.Vm WARNING vmId=`43415276-a7bf-4c86-b0e9-70a5f6d39a40`::Unknown type found, device: '{'device': 'unix', 'alias': 'channel0', 'type': 'channel', 'address': {'bus': '0', 'controller': '0', 'type': 'virtio-serial', 'port': '1'}}' found Feb 04 02:56:50 ovirt-srv05 vdsm[5170]: vdsm vm.Vm WARNING vmId=`43415276-a7bf-4c86-b0e9-70a5f6d39a40`::Unknown type found, device: '{'device': 'unix', 'alias': 'channel1', 'type': 'channel', 'address': {'bus': '0', 'controller': '0', 'type': 'virtio-serial', 'port': '2'}}' found Feb 04 02:56:50 ovirt-srv05 vdsm[5170]: vdsm vm.Vm WARNING vmId=`43415276-a7bf-4c86-b0e9-70a5f6d39a40`::Unknown type found, device: '{'device': 'spicevmc', 'alias': 'channel2', 'type': 'channel', 'address': {'bus': '0', 'controller': '0', 'type': 'virtio-serial', 'port': '3'}}' found Feb 04 02:56:50 ovirt-srv05 vdsm[5170]: vdsm vm.Vm ERROR vmId=`43415276-a7bf-4c86-b0e9-70a5f6d39a40`::Alias not found for device type graphics during migration at destination host All of these are mostly noise. We have BZs basically to silence them, but they are harmless. After that sanlock complains: Feb 04 02:56:50 ovirt-srv05 sanlock[1055]: 2015-02-04 02:56:50-0700 1453 [1055]: cmd 9 target pid 6479 not found There is a BZ for this also, but likely not the root cause here but it seems that the vm is starting up: Feb 04 02:56:50 ovirt-srv05 systemd[1]: Starting Virtual Machine qemu-el6-vm03-phx-ovirt-org. -- Subject: Unit machine-qemu\x2del6\x2dvm03\x2dphx\x2dovirt\x2dorg.scope has begun with start-up -- Defined-By: systemd -- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel -- -- Unit machine-qemu\x2del6\x2dvm03\x2dphx\x2dovirt\x2dorg.scope has begun starting up. Feb 04 02:56:50 ovirt-srv05 systemd-machined[5642]: New machine qemu-el6-vm03-phx-ovirt-org. -- Subject: A virtual machine or container has been started -- Defined-By: systemd -- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel -- -- The virtual machine qemu-el6-vm03-phx-ovirt-org with its leader PID 6479 has been -- started is now ready to use. Feb 04 02:56:50 ovirt-srv05 systemd[1]: Started Virtual Machine qemu-el6-vm03-phx-ovirt-org. -- Subject: Unit machine-qemu\x2del6\x2dvm03\x2dphx\x2dovirt\x2dorg.scope has finished start-up -- Defined-By: systemd -- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel -- -- Unit machine-qemu\x2del6\x2dvm03\x2dphx\x2dovirt\x2dorg.scope has finished starting up. -- -- The start-up result is done. But it shuts down: Feb 04 02:56:51 ovirt-srv05 systemd-machined[5642]: Machine qemu-el6-vm03-phx-ovirt-org terminated. -- Subject: A virtual machine or container has been terminated -- Defined-By: systemd -- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel -- -- The virtual machine qemu-el6-vm03-phx-ovirt-org with its leader PID 6479 has been -- shut down. Feb 04 02:56:51 ovirt-srv05 vdsm[5170]: vdsm vm.Vm ERROR vmId=`43415276-a7bf-4c86-b0e9-70a5f6d39a40`::Failed to start a migration destination vm Traceback (most recent call last): File /usr/share/vdsm/virt/vm.py, line 2298, in _startUnderlyingVm self._completeIncomingMigration() File /usr/share/vdsm/virt/vm.py, line 4107, in _completeIncomingMigration self._incomingMigrationFinished.isSet(), usedTimeout) File /usr/share/vdsm/virt/vm.py, line 4160, in _attachLibvirtDomainAfterMigration raise MigrationError(e.get_error_message()) MigrationError: Domain not found