[ovirt-devel] Errors when upgrading from 4.0 to master

2016-06-03 Thread David Caro

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

2016-05-18 Thread David Caro

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

2016-05-05 Thread David Caro Estevez
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)

2016-04-18 Thread David Caro

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

2016-04-14 Thread David Caro
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

2016-04-14 Thread David Caro
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

2016-04-14 Thread David Caro
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

2016-04-13 Thread David Caro
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

2016-04-13 Thread David Caro
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

2016-04-13 Thread David Caro
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

2016-04-06 Thread David Caro
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

2016-04-06 Thread David Caro
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

2016-03-30 Thread David Caro
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

2016-03-30 Thread David Caro
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

2016-03-08 Thread David Caro
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

2016-02-15 Thread David Caro

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

2016-02-09 Thread David Caro
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

2016-02-08 Thread David Caro
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

2016-02-08 Thread David Caro

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

2016-01-31 Thread David Caro Estevez
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

2016-01-19 Thread David Caro

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

2016-01-15 Thread David Caro
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

2016-01-12 Thread David Caro
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

2016-01-12 Thread David Caro
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

2016-01-12 Thread David Caro
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

2015-12-18 Thread David Caro Estevez
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

2015-12-15 Thread David Caro Estevez
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

2015-12-15 Thread David Caro Estevez
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

2015-12-11 Thread David Caro

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

2015-12-09 Thread David Caro
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

2015-12-04 Thread David Caro


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

2015-12-01 Thread David Caro

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

2015-11-30 Thread David Caro
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

2015-11-27 Thread David Caro

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

2015-11-27 Thread David Caro

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

2015-11-27 Thread David Caro

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

2015-11-27 Thread David Caro

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

2015-11-23 Thread David Caro

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

2015-11-18 Thread David Caro
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

2015-11-18 Thread David Caro
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

2015-11-18 Thread David Caro
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

2015-11-17 Thread David Caro

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

2015-11-17 Thread David Caro
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

2015-11-16 Thread David Caro

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?

2015-11-12 Thread David Caro
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

2015-11-12 Thread David Caro
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

2015-11-09 Thread David Caro
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

2015-11-09 Thread David Caro
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

2015-09-09 Thread David Caro

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

2015-09-09 Thread David Caro
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

2015-08-25 Thread David Caro
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

2015-08-24 Thread David Caro
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

2015-08-24 Thread David Caro

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

2015-08-24 Thread David Caro

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

2015-08-17 Thread David Caro

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

2015-08-17 Thread David Caro
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

2015-08-12 Thread David Caro
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

2015-08-12 Thread David Caro
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

2015-08-11 Thread David Caro

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

2015-08-05 Thread David Caro Estevez

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

2015-07-08 Thread David Caro
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

2015-06-23 Thread David Caro

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

2015-06-08 Thread David Caro
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

2015-06-07 Thread David Caro
-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

2015-06-05 Thread David Caro

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

2015-06-04 Thread David Caro
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

2015-06-04 Thread David Caro
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

2015-06-04 Thread David Caro
 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

2015-06-04 Thread David Caro
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

2015-05-26 Thread David Caro

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?

2015-05-26 Thread David Caro
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

2015-05-13 Thread David Caro
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

2015-05-11 Thread David Caro

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

2015-05-08 Thread David Caro

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

2015-04-30 Thread David Caro
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

2015-04-29 Thread David Caro
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

2015-04-29 Thread David Caro

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

2015-04-29 Thread David Caro
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

2015-04-29 Thread David Caro
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

2015-04-27 Thread David Caro

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

2015-04-27 Thread David Caro

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

2015-04-27 Thread David Caro

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

2015-04-22 Thread David Caro

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!

2015-04-21 Thread David Caro

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

2015-04-15 Thread David Caro

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

2015-04-13 Thread David Caro
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

2015-04-13 Thread David Caro


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

2015-03-25 Thread David Caro

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

2015-03-18 Thread David Caro
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

2015-02-24 Thread David Caro

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

2015-02-23 Thread David Caro

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

2015-02-23 Thread David Caro

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

2015-02-23 Thread David Caro
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

2015-02-23 Thread David Caro
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

2015-02-23 Thread David Caro
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

2015-02-23 Thread David Caro

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

2015-02-23 Thread David Caro
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

2015-02-22 Thread David Caro

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

2015-02-22 Thread David Caro

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

2015-02-04 Thread David Caro
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

  1   2   >