Re: [VDSM] check-merge fails: error: [Errno 98] Address already in use

[Yaniv Bronheim]

do you still see that after merged? seems like broken version of lago in ci

On Mon, May 22, 2017 at 4:36 PM Nir Soffer  wrote:

> I see these failure in check-merge
>
> *13:26:59* Error occured, aborting*13:26:59* Traceback (most recent call 
> last):*13:26:59*   File "/usr/lib/python2.7/site-packages/ovirtlago/cmd.py", 
> line 360, in do_run*13:26:59* 
> self.cli_plugins[args.ovirtverb].do_run(args)*13:26:59*   File 
> "/usr/lib/python2.7/site-packages/lago/plugins/cli.py", line 184, in 
> do_run*13:26:59* self._do_run(**vars(args))*13:26:59*   File 
> "/usr/lib/python2.7/site-packages/lago/utils.py", line 501, in 
> wrapper*13:26:59* return func(*args, **kwargs)*13:26:59*   File 
> "/usr/lib/python2.7/site-packages/lago/utils.py", line 512, in 
> wrapper*13:26:59* return func(*args, prefix=prefix, **kwargs)*13:26:59*   
> File "/usr/lib/python2.7/site-packages/ovirtlago/cmd.py", line 166, in 
> do_deploy*13:26:59* prefix.deploy()*13:26:59*   File 
> "/usr/lib/python2.7/site-packages/lago/log_utils.py", line 635, in 
> wrapper*13:26:59* return func(*args, **kwargs)*13:26:59*   File 
> "/usr/lib/python2.7/site-packages/ovirtlago/reposetup.py", line 110, in 
> wrapper*13:26:59* with utils.repo_server_context(args[0]):*13:26:59*   
> File "/usr/lib64/python2.7/contextlib.py", line 17, in __enter__*13:26:59*
>  return self.gen.next()*13:26:59*   File 
> "/usr/lib/python2.7/site-packages/ovirtlago/utils.py", line 100, in 
> repo_server_context*13:26:59* 
> root_dir=prefix.paths.internal_repo(),*13:26:59*   File 
> "/usr/lib/python2.7/site-packages/ovirtlago/utils.py", line 76, in 
> _create_http_server*13:26:59* 
> generate_request_handler(root_dir),*13:26:59*   File 
> "/usr/lib64/python2.7/SocketServer.py", line 419, in __init__*13:26:59* 
> self.server_bind()*13:26:59*   File "/usr/lib64/python2.7/BaseHTTPServer.py", 
> line 108, in server_bind*13:26:59* 
> SocketServer.TCPServer.server_bind(self)*13:26:59*   File 
> "/usr/lib64/python2.7/SocketServer.py", line 430, in server_bind*13:26:59*
>  self.socket.bind(self.server_address)*13:26:59*   File 
> "/usr/lib64/python2.7/socket.py", line 224, in meth*13:26:59* return 
> getattr(self._sock,name)(*args)*13:26:59* error: [Errno 98] Address already 
> in use
>
>
> Failed build: 
> http://jenkins.ovirt.org/job/vdsm_master_check-merged-el7-x86_64/1794/console
>
> Other builds with same error:
> http://jenkins.ovirt.org/job/vdsm_master_check-merged-el7-x86_64/1793/console
> http://jenkins.ovirt.org/job/vdsm_master_check-merged-el7-x86_64/1790/console
>
> Nir
>
-- 
Yaniv Bronhaim.
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

permissions to https://hub.docker.com/u/ovirt

[Yaniv Bronheim]

Hi,

I need to push images to docker.io/ovirt. can you please attach my account
to ovirt admins? Currently I have to ovirtorg, but we plan to use the ovirt
one

Thank you
-- 
Yaniv Bronhaim.
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

request: Renaming gerrit project

[Yaniv Bronheim]

Hi,

short version: rename ovirt-container-engine project to ovirt-containers

long version: recently we decided to gather all ovirt container under
ovirt-containers project instead of splitting it to engine and node. can
you please rename the current ovirt-container-engine to ovirt-containers

Thanks.
-- 
Yaniv Bronhaim.
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

enable travis run on oc repos

[Yaniv Bronheim]

now when I have a sync in github can you enable the travis job on both
ovirt-container-* projects - https://travis-ci.org/oVirt/

https://travis-ci.org/oVirt/ovirt-container-engine/
https://travis-ci.org/oVirt/ovirt-container-node

thanks.
-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Re: [JIRA] (OVIRT-1024) enable github mirror to ovirt-container-node project

[Yaniv Bronheim]

last sync was in 2015 - I don't plan to merge something today, but I need
it to be updated


On Thu, Jan 19, 2017 at 11:21 AM, Evgheni Dereveanchin (oVirt JIRA) <
j...@ovirt-jira.atlassian.net> wrote:

>
> [ https://ovirt-jira.atlassian.net/browse/OVIRT-1024?page=
> com.atlassian.jira.plugin.system.issuetabpanels:comment-
> tabpanel=26010#comment-26010 ]
>
> Evgheni Dereveanchin commented on OVIRT-1024:
> -
>
> The mirror existed on GitHub, just replication didn't work. Adjusted
> gerrit config, let's see if it works after next push to the repo.
>
> > enable github mirror to ovirt-container-node project
> > 
> >
> > Key: OVIRT-1024
> > URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1024
> > Project: oVirt - virtualization made easy
> >  Issue Type: By-EMAIL
> >Reporter: eyal edri [Administrator]
> >Assignee: Evgheni Dereveanchin
> >
> > Because its not enabled by default, only by request as its a manual
> process.
> > Adding infra-support to create a ticket on it.
> > On Thu, Jan 12, 2017 at 5:41 PM, Yaniv Bronheim <ybron...@redhat.com>
> wrote:
> > > do you know maybe why doesn't ovirt github in sync with the gerrit? (
> > > https://github.com/ovirt/ovirt-container-node)
> > >
> > > Thanks.
> > >
> > > --
> > > *Yaniv Bronhaim.*
> > >
> > > ___
> > > Infra mailing list
> > > Infra@ovirt.org
> > > http://lists.ovirt.org/mailman/listinfo/infra
> > >
> > >
> > --
> > Eyal Edri
> > Associate Manager
> > RHV DevOps
> > EMEA ENG Virtualization R
> > Red Hat Israel
> > phone: +972-9-7692018
> > irc: eedri (on #tlv #rhev-dev #rhev-integ)
>
>
>
> --
> This message was sent by Atlassian JIRA
> (v1000.695.1#100025)
> ___
> Infra mailing list
> Infra@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/infra
>



-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

github ovirt project sync

[Yaniv Bronheim]

do you know maybe why doesn't ovirt github in sync with the gerrit? (
https://github.com/ovirt/ovirt-container-node)

Thanks.

-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

When trigged manually the check-merged job

[Yaniv Bronheim]

When trigged manually the check-merged job for specific patch we don't see
the link in the patch History - see in https://gerrit.ovirt.org/#/c/69802/3

it used to be different .. can we get the link back?

Thanks.

-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

changes in gerrit - backport does not include reviewers automatically

[Yaniv Bronheim]

A backport with same change-id as in master branch used to add the reviewer
of the original patch automatically. and also the reviewed-on label with
link to the original patch.

can we add it back?

-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Re: [JIRA] (OVIRT-900) Re: Rebase over other author's patch cannot be pushed to gerrit

[Yaniv Bronheim]

you can always ignore the patchset and push your version again. otherwise
you cant rebase your patch on it I think

On Sun, Dec 4, 2016 at 3:00 PM, eyal edri [Administrator] (oVirt JIRA) <
j...@ovirt-jira.atlassian.net> wrote:

>
> [ https://ovirt-jira.atlassian.net/browse/OVIRT-900?page=com.
> atlassian.jira.plugin.system.issuetabpanels:comment-
> tabpanel=23824#comment-23824 ]
>
> eyal edri [Administrator] commented on OVIRT-900:
> -
>
> So what you're saying is any 'registered user' should have 'forge author
> identity' permission?
> Isn't that a bit of a security issue?
>
> On Sun, Dec 4, 2016 at 2:58 PM, ybronhei (oVirt JIRA) <
>
>
>
>
> --
> Eyal Edri
> Associate Manager
> RHV DevOps
> EMEA ENG Virtualization R
> Red Hat Israel
>
> phone: +972-9-7692018
> irc: eedri (on #tlv #rhev-dev #rhev-integ)
>
>
> > Re: Rebase over other author's patch cannot be pushed to gerrit
> > ---
> >
> > Key: OVIRT-900
> > URL: https://ovirt-jira.atlassian.net/browse/OVIRT-900
> > Project: oVirt - virtualization made easy
> >  Issue Type: By-EMAIL
> >    Reporter: Barak Korren
> >Assignee: Shlomo Ben David
> >
> > Forwarding to infra-support.
> > On 1 December 2016 at 18:03, Yaniv Bronheim <ybron...@redhat.com> wrote:
> > > Not sure since when it was changed, but I noticed that I can't push
> patches
> > > if I'm not the author
> > >
> > > Counting objects: 50, done.
> > > Delta compression using up to 4 threads.
> > > Compressing objects: 100% (50/50), done.
> > > Writing objects: 100% (50/50), 36.66 KiB | 0 bytes/s, done.
> > > Total 50 (delta 34), reused 0 (delta 0)
> > > remote: Resolving deltas: 100% (34/34)
> > > remote: Processing changes: refs: 1, done
> > > remote:
> > > remote: ERROR:  In commit db14ec7c1555a9eb37a0fb931bbb4ebdfc674bb4
> > > remote: ERROR:  author email address rnach...@redhat.com
> > > remote: ERROR:  does not match your user account.
> > > remote: ERROR:
> > > remote: ERROR:  The following addresses are currently registered:
> > > remote: ERROR:bronh...@gmail.com
> > > remote: ERROR:ybron...@redhat.com
> > > remote: ERROR:
> > > remote: ERROR:  To register an email address, please visit:
> > > remote: ERROR:  https://gerrit.ovirt.org/#/settings/contact
> > > remote:
> > > remote:
> > > To ssh://ybron...@gerrit.ovirt.org:29418/vdsm
> > >  ! [remote rejected] HEAD -> refs/for/master (invalid author)
> > > error: failed to push some refs to
> > > 'ssh://ybron...@gerrit.ovirt.org:29418/vdsm'
> > >
> > > We must have permissions to do that, this is part of the rebasing
> part, and
> > > I think its fine to fix patches on behalf of someone else.. but its
> not the
> > > best practice for reviewing.
> > > anyway, please undo this change, unless its something that related
> only to
> > > my env.. let me know
> > >
> > > Thanks
> > >
> > > --
> > > Yaniv Bronhaim.
> > >
> > > ___
> > > Infra mailing list
> > > Infra@ovirt.org
> > > http://lists.ovirt.org/mailman/listinfo/infra
> > >
> > --
> > Barak Korren
> > bkor...@redhat.com
> > RHCE, RHCi, RHV-DevOps Team
> > https://ifireball.wordpress.com/
>
>
>
> --
> This message was sent by Atlassian JIRA
> (v1000.610.2#100023)
> ___
> Infra mailing list
> Infra@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/infra
>



-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Re: [JIRA] (OVIRT-900) Re: Rebase over other author's patch cannot be pushed to gerrit

[Yaniv Bronheim]

no.. this is not the issue here. I don't try to merge, I try to push a
change on a patch that im not the author of it. everyone should allow to do
so

On Sun, Dec 4, 2016 at 1:25 PM, eyal edri [Administrator] (oVirt JIRA) <
j...@ovirt-jira.atlassian.net> wrote:

>
> [ https://ovirt-jira.atlassian.net/browse/OVIRT-900?page=com.
> atlassian.jira.plugin.system.issuetabpanels:comment-
> tabpanel=23821#comment-23821 ]
>
> eyal edri [Administrator] commented on OVIRT-900:
> -
>
> OK, so as I suggested its best to verify the list of people in those
> groups and add/remove people who are missing.
>
> > Re: Rebase over other author's patch cannot be pushed to gerrit
> > ---
> >
> > Key: OVIRT-900
> > URL: https://ovirt-jira.atlassian.net/browse/OVIRT-900
> > Project: oVirt - virtualization made easy
> >  Issue Type: By-EMAIL
> >Reporter: Barak Korren
> >Assignee: Shlomo Ben David
> >
> > Forwarding to infra-support.
> > On 1 December 2016 at 18:03, Yaniv Bronheim <ybron...@redhat.com> wrote:
> > > Not sure since when it was changed, but I noticed that I can't push
> patches
> > > if I'm not the author
> > >
> > > Counting objects: 50, done.
> > > Delta compression using up to 4 threads.
> > > Compressing objects: 100% (50/50), done.
> > > Writing objects: 100% (50/50), 36.66 KiB | 0 bytes/s, done.
> > > Total 50 (delta 34), reused 0 (delta 0)
> > > remote: Resolving deltas: 100% (34/34)
> > > remote: Processing changes: refs: 1, done
> > > remote:
> > > remote: ERROR:  In commit db14ec7c1555a9eb37a0fb931bbb4ebdfc674bb4
> > > remote: ERROR:  author email address rnach...@redhat.com
> > > remote: ERROR:  does not match your user account.
> > > remote: ERROR:
> > > remote: ERROR:  The following addresses are currently registered:
> > > remote: ERROR:bronh...@gmail.com
> > > remote: ERROR:ybron...@redhat.com
> > > remote: ERROR:
> > > remote: ERROR:  To register an email address, please visit:
> > > remote: ERROR:  https://gerrit.ovirt.org/#/settings/contact
> > > remote:
> > > remote:
> > > To ssh://ybron...@gerrit.ovirt.org:29418/vdsm
> > >  ! [remote rejected] HEAD -> refs/for/master (invalid author)
> > > error: failed to push some refs to
> > > 'ssh://ybron...@gerrit.ovirt.org:29418/vdsm'
> > >
> > > We must have permissions to do that, this is part of the rebasing
> part, and
> > > I think its fine to fix patches on behalf of someone else.. but its
> not the
> > > best practice for reviewing.
> > > anyway, please undo this change, unless its something that related
> only to
> > > my env.. let me know
> > >
> > > Thanks
> > >
> > > --
> > > Yaniv Bronhaim.
> > >
> > > ___
> > > Infra mailing list
> > > Infra@ovirt.org
> > > http://lists.ovirt.org/mailman/listinfo/infra
> > >
> > --
> > Barak Korren
> > bkor...@redhat.com
> > RHCE, RHCi, RHV-DevOps Team
> > https://ifireball.wordpress.com/
>
>
>
> --
> This message was sent by Atlassian JIRA
> (v1000.610.2#100023)
> ___
> Infra mailing list
> Infra@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/infra
>



-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Re: [JIRA] (OVIRT-633) Re: [ovirt-devel] [ACTION REQUIRED] broken dependency with vdsm git6e84643

[Yaniv Bronheim]

On Wed, Jul 13, 2016 at 1:11 PM, sbonazzo (oVirt JIRA) <
j...@ovirt-jira.atlassian.net> wrote:

> sbonazzo created OVIRT-633:
> --
>
>  Summary: Re: [ovirt-devel] [ACTION REQUIRED] broken
> dependency with vdsm git6e84643
>  Key: OVIRT-633
>  URL: https://ovirt-jira.atlassian.net/browse/OVIRT-633
>  Project: oVirt - virtualization made easy
>   Issue Type: By-EMAIL
> Reporter: sbonazzo
> Assignee: infra
>
>
> On Wed, Jul 13, 2016 at 10:14 AM, Michal Skrivanek <
> michal.skriva...@redhat.com> wrote:
>
> >
> > > On 13 Jul 2016, at 09:42, Nir Soffer  wrote:
> > >
> > > On Wed, Jul 13, 2016 at 10:04 AM, Sandro Bonazzola <
> sbona...@redhat.com>
> > wrote:
> > >> DEBUG util.py:417:  Error: Package:
> > >> vdsm-4.18.999-232.git6e84643.el7.centos.x86_64 (ovirt-master-snapshot)
> > >> DEBUG util.py:417: Requires: safelease >= 1.0-7
> > >> DEBUG util.py:417: Available: safelease-1.0-5.el7.x86_64
> > >> (centos-epel)
> > >> DEBUG util.py:417: safelease = 1.0-5.el7
> > >
> > > Yaniv, 1.0-7 not available on centos?
>

available in epel 7 -
http://koji.fedoraproject.org/koji/buildinfo?buildID=747782


> >
> >
> >
> https://dl.fedoraproject.org/pub/epel/7/x86_64/s/safelease-1.0-7.el7.x86_64.rpm
> > Sandro, wrong repos set up?
> >
> >
> Maybe mirror not aligned or proxy issue. Adding infra.
>
>
>
> > >
> > >>
> > >>
> > >> --
> > >> Sandro Bonazzola
> > >> Better technology. Faster innovation. Powered by community
> > collaboration.
> > >> See how it works at redhat.com
> > >>
> > >> ___
> > >> Devel mailing list
> > >> de...@ovirt.org
> > >> http://lists.ovirt.org/mailman/listinfo/devel
> > > ___
> > > Devel mailing list
> > > de...@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
>
>
>
> --
> This message was sent by Atlassian JIRA
> (v1000.148.2#15)
> ___
> Infra mailing list
> Infra@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/infra
>



-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Re: problems with lago and check-merged ci script in vdsm

[Yaniv Bronheim]

found the issue -
http://resources.ovirt.org/repos/lago/stable/0.0/rpm/fc24/ is empty .
please build the repository for fc24. meanwhile I posted
https://gerrit.ovirt.org/#/c/61199/

On Mon, Jul 18, 2016 at 1:54 PM, Barak Korren <bkor...@redhat.com> wrote:

> Do you have Lago listed in your check-merged.packages file?
>
> On 18 July 2016 at 13:49, Yaniv Bronheim <ybron...@redhat.com> wrote:
> > Hi,
> > we get lago: command not found , eventhough we install lago as part of
> the
> > env packages
> >
> >
> http://jenkins.ovirt.org/job/vdsm_master_check-merged-fc24-x86_64/15/console
> >
> > can you check why ?
> >
> >
> > 08:37:20 + lago init
> >
> /home/jenkins/workspace/vdsm_master_check-merged-fc24-x86_64/vdsm/automation/vdsm_functional
> > automation/lago-env.yml
> > 08:37:20 ./automation/check-merged.sh: line 42: lago: command not found
> >
> >
> >
> > Thanks
> >
> >
> > --
> > Yaniv Bronhaim.
> >
> > ___
> > Infra mailing list
> > Infra@ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/infra
> >
>
>
>
> --
> Barak Korren
> bkor...@redhat.com
> RHEV-CI Team
>



-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

problems with lago and check-merged ci script in vdsm

[Yaniv Bronheim]

Hi,
we get lago: command not found , eventhough we install lago as part of the
env packages

http://jenkins.ovirt.org/job/vdsm_master_check-merged-fc24-x86_64/15/console

can you check why ?


*08:37:20* + lago init
/home/jenkins/workspace/vdsm_master_check-merged-fc24-x86_64/vdsm/automation/vdsm_functional
automation/lago-env.yml*08:37:20* ./automation/check-merged.sh: line
42: lago: command not found



Thanks


-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Error in vdsm functional tests

[Yaniv Bronheim]

Hi,
Recently we're getting the following error, even-though we require
python-mock in check-merged.packages.f23 (https://gerrit.ovirt.org/59797)

*09:52:43*   # Deploy environment: Success (in 0:09:06)*09:52:44* @
Deploy oVirt environment: Success (in 0:09:07)*09:52:44* + lago shell
vdsm_functional_tests_host-fc23 -c 'mount -t tmpfs tmpfs
/sys/kernel/mm/ksm'*09:52:44* + tee
/home/jenkins/workspace/vdsm_master_check-merged-fc23-x86_64/vdsm/exported-artifacts/functional_tests_stdout.fc23.log*09:52:44*
current session does not belong to lago group.*09:52:46* + lago shell
vdsm_functional_tests_host-fc23 -c ' cd
/usr/share/vdsm/tests*09:52:46* ./run_tests.sh
--with-xunit
--xunit-file=/tmp/nosetests-fc23.xml -s
  functional/supervdsmFuncTests.py '*09:52:47*
current session does not belong to lago group.*09:52:49* Traceback
(most recent call last):*09:52:49*   File "../tests/testrunner.py",
line 42, in *09:52:49* import testlib*09:52:49*   File
"/usr/share/vdsm/tests/testlib.py", line 44, in *09:52:49*
import mock*09:52:49* ImportError: No module named mock*09:52:49* +
failed=1*09:52:49* + lago copy-from-vm vdsm_functional_tests_host-fc23
/tmp/nosetests-fc23.xml
/home/jenkins/workspace/vdsm_master_check-merged-fc23-x86_64/vdsm/exported-artifacts/nosetests-fc23.xml*09:52:50*
current session does not belong to lago group.*09:52:51* Error
occured, aborting*09:52:51* Traceback (most recent call
last):*09:52:51*   File
"/usr/lib/python2.7/site-packages/lago/cmd.py", line 691, in
main*09:52:51* cli_plugins[args.verb].do_run(args)*09:52:51*
File "/usr/lib/python2.7/site-packages/lago/plugins/cli.py", line 180,
in do_run*09:52:51* self._do_run(**vars(args))*09:52:51*   File
"/usr/lib/python2.7/site-packages/lago/utils.py", line 488, in
wrapper*09:52:51* return func(*args, **kwargs)*09:52:51*   File
"/usr/lib/python2.7/site-packages/lago/utils.py", line 499, in
wrapper*09:52:51* return func(*args, prefix=prefix,
**kwargs)*09:52:51*   File
"/usr/lib/python2.7/site-packages/lago/cmd.py", line 514, in
do_copy_from_vm*09:52:51* host.copy_from(remote_path,
local_path)*09:52:51*   File
"/usr/lib/python2.7/site-packages/lago/plugins/vm.py", line 319, in
copy_from*09:52:51* local_path=local_path,*09:52:51*   File
"/usr/lib/python2.7/site-packages/scp.py", line 125, in get*09:52:51*
   self._recv_all()*09:52:51*   File
"/usr/lib/python2.7/site-packages/scp.py", line 257, in
_recv_all*09:52:51* raise SCPException(str(msg).strip())*09:52:51*
SCPException:  scp: /tmp/nosetests-fc23.xml: No such file or
directory*09:52:51* + :*09:52:51* + lago stop
vdsm_functional_tests_host-fc23


-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

4.0 branch rights

[Yaniv Bronheim]

Hi,
I need rights to merge and create tags in vdsm ovirt-4.0 branch.

Thanks in advance

-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

jenkins pep8 package

[Yaniv Bronheim]

Hi,
Jenkins vms used to use pep8 1.5.7 only. we bumped in a case that it ran
older version than that -
http://jenkins.ovirt.org/job/vdsm_master_check-patch-el7-x86_64/767/console
can you fix that? vdsm supports 1.5.7 and above.

Thanks,

-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Re: CI does not run tests post merge

[Yaniv Bronheim]

Yes, we never ran check_patch for vdsm

Can we add back our el7,f22,f21 runs for our current check_patch? I asked
for that few weeks ago, and I think its important to avoid regressions
Thanks.


On Mon, Jan 25, 2016 at 11:20 AM, Barak Korren  wrote:

> >
> > Yes sorry, check_merge is the one.
> > Was it removed intentionally?
> >
> Looks like there aren't check_merged jobs for vdsm, were there?
> Did anyone made any at some point?
>
>
> --
> Barak Korren
> bkor...@redhat.com
> RHEV-CI Team
> ___
> Infra mailing list
> Infra@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/infra
>



-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Re: [ovirt-devel] Automation CI for vdsm

[Yaniv Bronheim]

OK. so for now the patch [1] skips all storage and OVSNetwork tests
Which still doesn't help much, single run takes more than 3hrs :/ (1:17hr,
for the setup :\ the rest for the tests)

The output looks pretty good though :)

[1] https://gerrit.ovirt.org/#/c/48268/

On Thu, Dec 17, 2015 at 12:13 PM, Petr Horacek <phora...@redhat.com> wrote:

> Hi,
>
> OVSNetworkTest requires vdsm-hook-ovs installed (tests are not skipped
> if the package is not installed) and openvswitch service running (not
> started automatically). NetworkTest are not able to run together with
> OVSNetworkTest since we are doing ugly inheritance hacking there. My
> apologize for network tests difficulties, it will be resolved soon. If
> you want your life easier, temporary skip OVSNetworkTest suite and it
> should be OK.
>
> Best regards,
> Petr
>
> 2015-12-16 17:25 GMT+01:00 Yaniv Bronheim <ybron...@redhat.com>:
> > exactly - just posted a patch again that sings all fails as broken .
> > we'll get the report soon and I'll publish it as well. hope the run will
> > take less time now
> >
> > On Wed, Dec 16, 2015 at 6:19 PM, Nir Soffer <nsof...@redhat.com> wrote:
> >>
> >> Nice, but we cannot enable this until all the tests pass or disabled.
> >>
> >> There is no point in broken or flaky functional tests.
> >>
> >> On Wed, Dec 16, 2015 at 5:34 PM, Yaniv Bronheim <ybron...@redhat.com>
> >> wrote:
> >> > So its not stable. It won't block merges and at least give us report
> >> > after
> >> > each merge. It takes really long time to run it (because of the tests
> >> > themselves. Lago things takes maximum 15minutes, but the run last for
> >> > more
> >> > than 2hrs right now and I suspect functional/storageTests.py gets
> stuck)
> >> >
> >> > Bellow you can see where we stand (before I added python-rtslib
> >> > package).
> >> >
> >> > Now, I still want to merge the patch
> https://gerrit.ovirt.org/#/c/48268/
> >> > -
> >> > which enables this run after merges, and I still want you to consider
> >> > the
> >> > addition of Automation CI flag to our gerrit so that developer will be
> >> > able
> >> > to use it as a trigger for the check-merged.sh script run, just to see
> >> > if
> >> > their patch fixes\brakes something realted to the functional tests
> >> >
> >> >
> http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc23-x86_64/1480/ -
> >> > is
> >> > an example of how the run looks like. I still work to improve the
> output
> >> >
> >> >
> >> > Please reply and let me know if the idea around the automation flag is
> >> > acceptable by you.. and please review the patch for comments and acks.
> >> > We can ask dcaro to add the flag until Friday, otherwise we'll need to
> >> > delay
> >> > this effort after the holiday..
> >> >
> >> >
> >> > functional.sosPluginTests.SosPluginTest
> >> > testSosPlugin   OK
> >> > functional.vmRecoveryTests.RecoveryTests
> >> > test_vm_recoveryFAIL
> >> > functional.vmQoSTests.VMQosTests
> >> > testSmallVMBallooning   FAIL
> >> > functional.virtTests.VirtTest
> >> > testComplexVm   FAIL
> >> > testHeadlessVm  OK
> >> > testSimpleVmFAIL
> >> > testVmDefinitionGraphics('spice')   FAIL
> >> > testVmDefinitionGraphics('vnc') OK
> >> > testVmDefinitionLegacyGraphics('qxl')   FAIL
> >> > testVmDefinitionLegacyGraphics('vnc')   OK
> >> > testVmDefinitionMultipleGraphics('spice', 'vnc')FAIL
> >> > testVmDefinitionMultipleGraphics('vnc', 'spice')FAIL
> >> > testVmWithCdrom('self') FAIL
> >> > testVmWithCdrom('specParams')   FAIL
> >> > testVmWithCdrom('vmPayload')FAIL
> >> > testVmWithDevice('hotplugDisk') FAIL
> >> > testVmWithDevice('hotplugNic')

Re: Automation CI for vdsm

[Yaniv Bronheim]

exactly - just posted a patch again that sings all fails as broken .
we'll get the report soon and I'll publish it as well. hope the run will
take less time now

On Wed, Dec 16, 2015 at 6:19 PM, Nir Soffer <nsof...@redhat.com> wrote:

> Nice, but we cannot enable this until all the tests pass or disabled.
>
> There is no point in broken or flaky functional tests.
>
> On Wed, Dec 16, 2015 at 5:34 PM, Yaniv Bronheim <ybron...@redhat.com>
> wrote:
> > So its not stable. It won't block merges and at least give us report
> after
> > each merge. It takes really long time to run it (because of the tests
> > themselves. Lago things takes maximum 15minutes, but the run last for
> more
> > than 2hrs right now and I suspect functional/storageTests.py gets stuck)
> >
> > Bellow you can see where we stand (before I added python-rtslib package).
> >
> > Now, I still want to merge the patch https://gerrit.ovirt.org/#/c/48268/
> -
> > which enables this run after merges, and I still want you to consider the
> > addition of Automation CI flag to our gerrit so that developer will be
> able
> > to use it as a trigger for the check-merged.sh script run, just to see if
> > their patch fixes\brakes something realted to the functional tests
> >
> > http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc23-x86_64/1480/
> - is
> > an example of how the run looks like. I still work to improve the output
> >
> >
> > Please reply and let me know if the idea around the automation flag is
> > acceptable by you.. and please review the patch for comments and acks.
> > We can ask dcaro to add the flag until Friday, otherwise we'll need to
> delay
> > this effort after the holiday..
> >
> >
> > functional.sosPluginTests.SosPluginTest
> > testSosPlugin   OK
> > functional.vmRecoveryTests.RecoveryTests
> > test_vm_recoveryFAIL
> > functional.vmQoSTests.VMQosTests
> > testSmallVMBallooning   FAIL
> > functional.virtTests.VirtTest
> > testComplexVm   FAIL
> > testHeadlessVm  OK
> > testSimpleVmFAIL
> > testVmDefinitionGraphics('spice')   FAIL
> > testVmDefinitionGraphics('vnc') OK
> > testVmDefinitionLegacyGraphics('qxl')   FAIL
> > testVmDefinitionLegacyGraphics('vnc')   OK
> > testVmDefinitionMultipleGraphics('spice', 'vnc')FAIL
> > testVmDefinitionMultipleGraphics('vnc', 'spice')FAIL
> > testVmWithCdrom('self') FAIL
> > testVmWithCdrom('specParams')   FAIL
> > testVmWithCdrom('vmPayload')FAIL
> > testVmWithDevice('hotplugDisk') FAIL
> > testVmWithDevice('hotplugNic')  FAIL
> > testVmWithDevice('smartcard')   FAIL
> > testVmWithDevice('virtioNic')   FAIL
> > testVmWithDevice('virtioRng')   FAIL
> > testVmWithSla   FAIL
> > testVmWithStorage('iscsi')  SKIP:
> > python-rtslib is not installed.
> > testVmWithStorage('localfs')FAIL
> > testVmWithStorage('nfs')FAIL
> > functional.storageTests.StorageTest
> > testCreatePoolErrorsOK
> > testStorage('glusterfs', 0) ERROR
> > testStorage('glusterfs', 3) ERROR
> > testStorage('iscsi', 0) SKIP:
> > python-rtslib is not installed.
> > testStorage('iscsi', 3) SKIP:
> > python-rtslib is not installed.
> > testStorage('localfs', 0)   FAIL
> > testStorage('localfs', 3)   FAIL
> > testStorage('nfs', 0)   FAIL
> > testStorage('nfs', 3)   FAIL
> > functional.networkTests.NetworkTest
> > testAddVlanedBridgeless ERROR
> > testAddVlanedBridgeless_one

Fwd: Two gerrit acounts raised conflict

[Yaniv Bronheim]

Hi,
I had a problem to authenticate with my gmail id to gerrit, so I used my
github id and now I have a split brain thing review in -
https://gerrit.ovirt.org/#/c/43724/4
now the link to
https://gerrit.ovirt.org/login/?id=gerrit-oauth-provider_-github-oauth
doesn't work at all..

Any suggestion how to fix that?

Thanks.

-- 
*Yaniv Bronhaim.*
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Re: vdsm fails again becasue cpopen is not found

[Yaniv Bronheim]

python-cpopen should be installed there before running the make 

- Original Message -
 From: Nir Soffer nsof...@redhat.com
 To: infra@ovirt.org
 Sent: Monday, December 30, 2013 6:10:26 PM
 Subject: vdsm fails again becasue cpopen is not found
 
 Hi infra,
 
 checking python module: cpopen... no
 configure: error: failed to find required module cpopen
 Build step 'Execute shell' marked build as failure
 Build step 'Groovy Postbuild' marked build as failure
 Recording test results
 Finished: FAILURE
 
 See http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/6425/console
 
 Any idea why it broke again?
 
 Nir
 ___
 Infra mailing list
 Infra@ovirt.org
 http://lists.ovirt.org/mailman/listinfo/infra
 
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Re: vdsm_pep8_gerrit is broken

[Yaniv Bronheim]

- Original Message -
 From: Kiril Nesenko knese...@redhat.com
 To: Dan Kenigsberg dan...@redhat.com
 Cc: infra@ovirt.org
 Sent: Wednesday, December 25, 2013 8:54:49 AM
 Subject: Re: vdsm_pep8_gerrit is broken
 
 
 
 - Original Message -
  From: Dan Kenigsberg dan...@redhat.com
  To: Nir Soffer nsof...@redhat.com
  Cc: Kiril Nesenko knese...@redhat.com, infra@ovirt.org
  Sent: Wednesday, December 25, 2013 12:31:00 AM
  Subject: Re: vdsm_pep8_gerrit is broken
  
  On Tue, Dec 24, 2013 at 04:47:07PM -0500, Nir Soffer wrote:
   - Original Message -
From: Kiril Nesenko knese...@redhat.com
To: Nir Soffer nsof...@redhat.com
Cc: infra@ovirt.org
Sent: Tuesday, December 24, 2013 11:33:24 PM
Subject: Re: vdsm_pep8_gerrit is broken

What pep8 version are you using ?
   
   We use http://danken.fedorapeople.org/python-pep8-1.4.5-2.el6.noarch.rpm
   
   You can check what we use here:
   http://www.ovirt.org/Vdsm_Developers#Installing_required_packages
   
- Original Message -
 From: Nir Soffer nsof...@redhat.com
 To: infra@ovirt.org
 Sent: Tuesday, December 24, 2013 11:27:54 PM
 Subject: vdsm_pep8_gerrit is broken
 
 Hi infra,
 
 I added a patch [1], breaking pep8 intentionally - it is passed the
 pep8
 job
 [2]
 
 [1] http://gerrit.ovirt.org/22724
 [2] http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/6260/
 
 This explains why lately jenkins was rather liberal.
  
  Kiril, could you remind me where the code for the Jenkins jobs can be
  seen? Does it have it's own gerrit repo? I'd like vdsm_pep8_gerrit to
  call `pep8 --version`.
 

we call pep8 --version already (http://gerrit.ovirt.org/16424)
you can see that in the job's build output

 Currently the code is just inside the job. I assume it should be better to
 move into our jenkins git repo.
 
 Here is the code:
 
 rm -f ${WORKSPACE}/violations.txt
 pushd $WORKSPACE/vdsm
 git diff HEAD~ `git ls-files | grep \\.py$\|\\.py\\.in$` | pep8 --diff |
 sed 's/^.\/*//'  ${WORKSPACE}/violations.txt
 
 if [[ ${PIPESTATUS[1]} -ne 0 ]]; then
echo PEP8 Failed
 else
echo PEP8 is ok
 fi
 
 - Kiril
 
  
  Nir, I suspect that `pep --diff` simply ignores changes in files lacking
  the *py suffix.
  
 ___
 Infra mailing list
 Infra@ovirt.org
 http://lists.ovirt.org/mailman/listinfo/infra
 
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Re: python-cpopen is needed on jenkins slaves

[Yaniv Bronheim]

Any news about it ? sorry to miss that topic
python-cpopen exists in fedora19 official repository, so I don't understand the 
problem

- Original Message -
 From: Ewoud Kohl van Wijngaarden ewoud+ov...@kohlvanwijngaarden.nl
 To: Dan Kenigsberg dan...@redhat.com
 Cc: Yaniv Bronheim ybron...@redhat.com, infra infra@ovirt.org
 Sent: Wednesday, October 16, 2013 6:57:18 PM
 Subject: Re: python-cpopen is needed on jenkins slaves
 
 On Wed, Oct 16, 2013 at 06:32:59PM +0300, Dan Kenigsberg wrote:
  On Sun, Oct 13, 2013 at 07:39:28AM -0400, Yaniv Bronheim wrote:
   Hey,
   
   In the last days cpopen packages was released as part of EPEL and fedora
   formal repository,
   I proposed vdsm patch [1] that uses this package instead of our old
   method using that code as part of vdsm,
   
   For it to work, please install it over the jenkins slaves.
   
   I set this package as BuildRequires, so without it make rpm for vdsm
   doesn't work.
   
   [1] http://gerrit.ovirt.org/#/c/20142/
  
  Unfortunately, I've mis-interpreted
  http://gerrit.ovirt.org/#/c/20216/1
  as an ACK that all our jenkins slaves have python-cpopen.
  
  Apparently, at least fedora19 does not.
  http://jenkins.ovirt.org/job/vdsm_create_rpms/label=fedora19/754/console
 
 There was a problem installing python-cpopen:
 http://lists.ovirt.org/pipermail/infra/2013-October/004142.html
 
 I didn't specify it before, but removing vdsm-python-cpopen also removes
 vdsm so I haven't done that yet.
 
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Re: fake vdsm repo on oVirt

[Yaniv Bronheim]

Hey,
I recall wiki page that explains the usage of vdsmfake 
(http://www.ovirt.org/VDSM_Fake)
Can you share it in the README file and locate it in the root folder of the 
repository ? 
or even add short instructions summary in a README of how to run and work with 
the utility?
might be helpful

Thanks
Yaniv Bronhaim.

- Original Message -
 From: David Caro Estevez dcaro...@redhat.com
 To: Itamar Heim ih...@redhat.com
 Cc: Tomas Jelinek tjeli...@redhat.com, infra@ovirt.org
 Sent: Wednesday, September 18, 2013 6:50:20 PM
 Subject: Re: fake vdsm repo on oVirt
 
 Done:
 
 git clone git://gerrit.ovirt.org/ovirt-vdsmfake
 http://gerrit.ovirt.org/gitweb?p=ovirt-vdsmfake.git;a=shortlog;h=HEAD
 
 Let me know if need anything else!
 
 - Original Message -
  From: Itamar Heim ih...@redhat.com
  To: Tomas Jelinek tjeli...@redhat.com
  Cc: dcaro...@redhat.com, michal skrivanek michal.skriva...@redhat.com,
  infra@ovirt.org
  Sent: Wednesday, September 18, 2013 2:13:59 PM
  Subject: Re: fake vdsm repo on oVirt
  
  On 09/18/2013 03:13 PM, Tomas Jelinek wrote:
   Hi all,
  
   I would like to ask you to create a new repo on ovirt to host the fake
   VDSM
   project [1].
   The ovirt-vdsmfake would be a good name for it. I would say it should be
   enough to move the project
   from github to ovirt (e.g. fork it) and not replicate the changes back as
   that project does not make
   much sense alone.
  
   I would be the maintainer of it for now.
  
   Thank you,
   Tomas
  
   [1] https://github.com/lspevak/ovirt-vdsmfake
  
  
  david, please create.
  
 ___
 Infra mailing list
 Infra@ovirt.org
 http://lists.ovirt.org/mailman/listinfo/infra
 
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Re: [Engine-devel] fake VDSM as oVirt project?

[Yaniv Bronheim]

+1

- Original Message -
 From: Itamar Heim ih...@redhat.com
 To: Liran Zelkha liran.zel...@gmail.com
 Cc: engine-devel engine-de...@ovirt.org, infra infra@ovirt.org
 Sent: Sunday, September 15, 2013 12:57:29 PM
 Subject: Re: [Engine-devel] fake VDSM as oVirt project?
 
 On 09/13/2013 09:52 AM, Liran Zelkha wrote:
  +1 I use it constantly.
 
 +1
 adding infra, where new git repo's are usually requested.
 (we also ask board if a new project scope, but this seems like just a
 repo for a help/test program)
 
 if more +1's and no objections, ping next week to create  repo.
 
 thanks,
 Itamar
 
 
 
  On Fri, Sep 13, 2013 at 8:48 AM, Tomas Jelinek tjeli...@redhat.com
  mailto:tjeli...@redhat.com wrote:
 
  Hi all,
 
  some time ago Libor Spevak created a simple web app called vdsm fake:
  documented: http://www.ovirt.org/VDSM_Fake
  published: https://github.com/lspevak/ovirt-vdsmfake
 
  It is basically a simple hackable java web application which can
  emulate the VDSM so you can connect the
  engine to it. It is especially useful for:
  - having tons of cheap fake hosts on one machine to stress your engine
  - doing some experiments with VDSM API (e.g. vfeenstr proposes a new
  VDSM API to lower the network traffic between
 engine - VDSM and uses the vdsm fake to implement it and do
  some tests to get some numbers on how does this change the things)
 
  Omer came up with an idea of making this app as one of oVirt's
  project (http://www.ovirt.org/Subprojects) maybe with repository on
  oVirt's gerrit making it more accessible for getting/contributing
  for the whole community.
 
  What do you think about it?
 
  Tomas
  ___
  Engine-devel mailing list
  engine-de...@ovirt.org mailto:engine-de...@ovirt.org
  http://lists.ovirt.org/mailman/listinfo/engine-devel
 
 
 
 
  ___
  Engine-devel mailing list
  engine-de...@ovirt.org
  http://lists.ovirt.org/mailman/listinfo/engine-devel
 
 
 ___
 Engine-devel mailing list
 engine-de...@ovirt.org
 http://lists.ovirt.org/mailman/listinfo/engine-devel
 
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Re: Odd errors on vdsm_unit_tests

[Yaniv Bronheim]

Right, 
If supervdsmServer.py could not start when executing (wrong import for 
example), testIsSuperUp (first ut of supervdsmTests) does not return and you 
end up with killed..
It happens because we had logic that after 3 retries of communicating with 
supervdsmServer fail we kill the running process http://gerrit.ovirt.org/11932 
(Normally vdsm process, in this case the testrunner process is killed).

First, http://gerrit.ovirt.org/#/c/13804/ - this fix the issue that causes 
supervdsmServer to fail.
Second, http://gerrit.ovirt.org/#/c/13878/ - check if supervdsmServer starts as 
expected, if not the test will straightly fail.
Third, I'll submit a change for the logic of communicating with 
supervdsmServer.. I prefer to avoid the panic call. at least i can monkeyPatch 
it during the tests.

Thanks,
Yaniv.


- Original Message -
 From: Dan Kenigsberg dan...@redhat.com
 To: David Caro dcaro...@redhat.com, Yaniv Bronheim ybron...@redhat.com
 Cc: Giuseppe Vallarelli gvall...@redhat.com, infra@ovirt.org
 Sent: Sunday, April 14, 2013 11:28:54 AM
 Subject: Re: Odd errors on vdsm_unit_tests
 
 On Thu, Apr 11, 2013 at 11:47:52PM +0200, David Caro wrote:
  On Thu 11 Apr 2013 11:36:27 PM CEST, David Caro wrote:
   On Thu 11 Apr 2013 11:22:00 PM CEST, Dan Kenigsberg wrote:
   Jenkins used to send email notifications about unit tests failures to
   vdsm-patc...@lists.fedorahosted.org at least until March 20.
  
   Come to think of it, the emails seized to have content on March 5.
  
   Now they are not sent at all.
  
  
   I think I'm not on that list, I'll sign up and look at it, see if I can
   figure out something.
  
   Beyond this, we have a very odd non-deterministic failure
   http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1976/console
   (except below).
  
   Could the test be run with an unusual locale?
   Does anybody have another idea?
  
   Could someone (Giuseppe? David?) log into this fedora18-vm01 slave and
   try to
   reproduce the issue from the command line?
  
   Regards,
   Dan.
  
   ==
   ERROR: test_deviceCustomProperties (hooksTests.TestHooks)
   --
   Traceback (most recent call last):
 File /jenkins-workspaces/vdsm_unit_tests_gerrit/tests/hooksTests.py,
 line 125, in test_deviceCustomProperties
   params={'customProperty': ' rocks!'})
 File /jenkins-workspaces/vdsm_unit_tests_gerrit/vdsm/hooks.py, line
 72, in _runHooksDir
   scriptenv[k] = unicode(v).encode('utf-8')
   UnicodeDecodeError: 'ascii' codec can't decode byte 0xc5 in position 7:
   ordinal not in range(128)
  
   ==
   ERROR: test_runHooksDir (hooksTests.TestHooks)
   --
   Traceback (most recent call last):
 File /jenkins-workspaces/vdsm_unit_tests_gerrit/tests/hooksTests.py,
 line 72, in test_runHooksDir
   res = hooks._runHooksDir(DOMXML, dirName)
 File /jenkins-workspaces/vdsm_unit_tests_gerrit/vdsm/hooks.py, line
 72, in _runHooksDir
   scriptenv[k] = unicode(v).encode('utf-8')
   UnicodeDecodeError: 'ascii' codec can't decode byte 0xc5 in position 7:
   ordinal not in range(128)
  
   About that error, yes, Me and Antoni Segura (apuimedo) troubleshot it,
   the problem is that one of the developers has non ascii letter on it's
   name, passing it to the author tag on gerrit, that loads it on
   GERRIT_*_AUTHOR env variable. Not sure if antoni is working on a patch
   or notified someone though.
  
   --
   David Caro
  
   Red Hat Czech s.r.o.
   Continuous Integration Engineer - EMEA ENG Virtualization RD
  
   Tel.: +420 532 294 605
   Email: dc...@redhat.com
   Web: www.cz.redhat.com
   Red Hat Czech s.r.o., Purkyňova 99/71, 612 45, Brno, Czech Republic
   RHT Global #: 82-62605
   ___
   Infra mailing list
   Infra@ovirt.org
   http://lists.ovirt.org/mailman/listinfo/infra
  
  Forgot to mention, the error that bugs me is this one:
  
  http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1962/console
  
  We had several of those today and it all seemed to stop when I set the
  core dump limit to 'unlimited' (ulimit -c). I saw that before with one
  patch but subsequent patches worked, so I did not investigate further.
  As far as I know, the problem is that the test testIsSuperUp somehow
  end up killing the jenkins slave process (if you run it manually you
  get a message that says 'killed' in the console). That test seems to
  mess with threads and processes and maybe it just identifies the slave
  process as the one to stop (just a wild guess :S).
 
 Yeah, I've noticed this bug in testIsSuperUp, and Yaniv Bronheim is
 aware of it.
 
 Dan.

___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org

Re: [vdsm] May I apply for a user account on jenkins.ovirt.org to run VDSM functional tests?

[Yaniv Bronheim]

Zhshzhou, It sounds very helpful and we must use functional tests already as 
you mentioned.
 
Please inform vdsm-devel how to run those tests manually and how we can 
distinguish errors with that.
XML-RPC functional test cases are still in progress and there's a lot to add 
there, it might be useful to add wiki about those tests, what is covered and 
what we want to add

- Original Message -
From: Eyal Edri ee...@redhat.com
To: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com
Cc: vdsm-de...@lists.fedorahosted.org, infra@ovirt.org
Sent: Wednesday, January 30, 2013 7:47:06 PM
Subject: Re: [vdsm] May I apply for a user account on jenkins.ovirt.org to run 
VDSM functional tests?



- Original Message -
 From: Adam Litke a...@us.ibm.com
 To: Eyal Edri ee...@redhat.com, Zhou Zheng Sheng 
 zhshz...@linux.vnet.ibm.com,
 vdsm-de...@lists.fedorahosted.org, infra@ovirt.org
 Sent: Wednesday, January 30, 2013 4:09:54 PM
 Subject: Re: [vdsm] May I apply for a user account on jenkins.ovirt.org to 
 run VDSM functional tests?
 
 On Tue, Jan 29, 2013 at 12:21:46PM +0100, Ewoud Kohl van Wijngaarden
 wrote:
  On Tue, Jan 29, 2013 at 06:15:08AM -0500, Eyal Edri wrote:
   - Original Message -
From: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com
To: infra@ovirt.org
Cc: ShaoHe Feng shao...@linux.vnet.ibm.com
Sent: Tuesday, January 29, 2013 12:24:27 PM
Subject: May I apply for a user account on jenkins.ovirt.org to
run VDSMfunctional tests?
   
Hi all,
   
I notice there is no VDSM functional tests running in oVirt
Jenkins.
Currently in VDSM we have some XML-RPC functional test cases
for
iSCSI,
localfs and glusterfs storage as well as creating and
destroying VMs
on
those storage. Functional tests through JSON-RPC are under
review. I
also submit a patch to Gerrit for running the tests easily
(http://gerrit.ovirt.org/#/c/11238/). More test cases will be
added
to
improve test coverage and reduce the chance of regression.
   
Some bugs that can not be covered by unit test can be caught by
functional tests. I think it would be helpful to run these
functional
tests continuously. We can also configure the Gerrit trigger in
Jenkins
to run functional tests when someone verifies the patch or when
it
gets
approved but not merged. This may be helpful to the maintainer.
   
I've setup a Jenkins job for VDSM functional tests in my lab
server.
You
can refer to the job configuration of my current setup
(https://github.com/edwardbadboy/vdsm-jenkins/blob/master/config.xml).
After my patch in Gerrit is accepted, the job configuration
will be
simpler and the hacks can be removed. May I apply a user
account for
creating job in the oVirt Jenkins?
   
  
   Hi Zhou,
   Basically there shouldn't be any problem with that.
   we have an option for giving a 'power-user' permissions for
   certain
   users on oVirt misc projects to add and and configure jobs for
   thier
   project.
  
   it requires knowledge in jenkins, which it seems that you have
   and
   recognition from the team/other developers from the relevant
   project
   (in this case, VDSM) that you are an active member of the
   project.
   (just a formality essentially)
  
   I've added engine-devel list to this thread so anyone from vdsm
   team
   can vote +1 for adding you as a power user for jenkins.
  
   once we'll receive a few +1 and not objections i'll create a user
   for
   you and send you the details.
  
  
  I think vdsm-devel is more relevant here.
 
 Also a big +1 from me.

Ok, i think it's enough +1's and general approval from the community.
Zhou, please send me in a private email your desired user  email and i'll set 
up an account for you.

Eyal.

 
 --
 Adam Litke a...@us.ibm.com
 IBM Linux Technology Center
 
 
___
vdsm-devel mailing list
vdsm-de...@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra