Change in vdsm[master]: build: Remove build requires needed for the tests

[eedri]

Eyal Edri has posted comments on this change.

Change subject: build: Remove build requires needed for the tests
..


Patch Set 1:

You're assuming you'll always have .packages file.
What will happen if you'll run vdsm build outside ovirt CI?

Won't it be better to enable a Boolean like 'enable_tests' same as we had for 
gluster, so a user can choose if to use it when building?

-- 
To view, visit https://gerrit.ovirt.org/65500
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3c385b0abeb2657a26215ce5b96976cc290734bb
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Barak Korren 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Irit Goihman 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Piotr Kliczewski 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Change in vdsm[master]: Adding build-artifacts-manual.sh script

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Adding build-artifacts-manual.sh script
..


Patch Set 7: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/62560
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie3962bfd45b651323a54c04cac6fdb5e844a78f7
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Sharon Naftaly 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Irit Goihman 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sharon Naftaly 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[ovirt-4.0]: tests: mom: create the socket under tmpdir

[eedri]

Eyal Edri has posted comments on this change.

Change subject: tests: mom: create the socket under tmpdir
..


Patch Set 1: Code-Review+2

-- 
To view, visit https://gerrit.ovirt.org/63129
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib41253a58e8abadeb99d57072ace75461a8e078b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Sharon Naftaly 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Irit Goihman 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák 
Gerrit-Reviewer: Sharon Naftaly 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: check-merged: switch fc24 to el7

[eedri]

Eyal Edri has posted comments on this change.

Change subject: check-merged: switch fc24 to el7
..


Patch Set 2: Code-Review+1

+1 I agree keeping el7 is the right thing for stabilization and less 
maintenance. 

fc24 can be used for short & simple tests using check-patch

-- 
To view, visit https://gerrit.ovirt.org/62352
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I18655ed38f695e7ffe3d340d7a12cbe22779ddd4
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Irit Goihman 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Irit Goihman 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: build: require only the same version, not release

[eedri]

Eyal Edri has posted comments on this change.

Change subject: build: require only the same version, not release
..


Patch Set 1:

Looks OK on x86_64:
http://jenkins.ovirt.org/job/vdsm_master_build-artifacts-el7-x86_64/717/

-- 
To view, visit https://gerrit.ovirt.org/62672
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibf5366b0d38471114ddd3ad82e1402ebd308338e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: build: require only the same version, not release

[eedri]

Eyal Edri has posted comments on this change.

Change subject: build: require only the same version, not release
..


Patch Set 1:

running a try on build artifacts also:
http://jenkins.ovirt.org/job/vdsm_master_build-artifacts-el7-x86_64/717/console

-- 
To view, visit https://gerrit.ovirt.org/62672
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibf5366b0d38471114ddd3ad82e1402ebd308338e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: enabling ppc64le builds for vdsm

[eedri]

Eyal Edri has posted comments on this change.

Change subject: enabling ppc64le builds for vdsm
..


Patch Set 16: Verified+1

works for el7, disabled fc24 build for ppc64le due to weird aarch64 deps.
not sure we support fc24 also, but even if we do, we can add it later.

-- 
To view, visit https://gerrit.ovirt.org/62123
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7edd684cf40f2122c4a3afe0091ac2eace1129ef
Gerrit-PatchSet: 16
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eyal Edri 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Gal Ben Haim 
Gerrit-Reviewer: Irit Goihman 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola 
Gerrit-Reviewer: Yaniv Bronhaim
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: enabling ppc64le builds for vdsm

[eedri]

Eyal Edri has posted comments on this change.

Change subject: enabling ppc64le builds for vdsm
..


Patch Set 16:

fc24 build fails on weird dependency to aarch64:
do we need to add aarch64 repo also? 

DEBUG util.py:421:  --> Processing Dependency: edk2-aarch64 for package: 
2:qemu-system-aarch64-2.6.0-4.fc24.ppc64le
DEBUG util.py:421:  ---> Package qemu-system-x86.ppc64le 2:2.6.0-4.fc24 will be 
installed
DEBUG util.py:421:  --> Processing Dependency: edk2-ovmf for package: 
2:qemu-system-x86-2.6.0-4.fc24.ppc64le
DEBUG util.py:421:  --> Finished Dependency Resolution
DEBUG util.py:421:  Error: Package: 2:qemu-system-aarch64-2.6.0-4.fc24.ppc64le 
(updates)
DEBUG util.py:421: Requires: edk2-aarch64
DEBUG util.py:421:   You could try using --skip-broken to work around the 
problem
DEBUG util.py:421:  Error: Package: 2:qemu-system-x86-2.6.0-4.fc24.ppc64le 
(updates)
DEBUG util.py:421: Requires: edk2-ovmf
DEBUG util.py:421:   You could try running: rpm -Va --nofiles --nodigest
DEBUG util.py:546:  Child return code was: 1

-- 
To view, visit https://gerrit.ovirt.org/62123
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7edd684cf40f2122c4a3afe0091ac2eace1129ef
Gerrit-PatchSet: 16
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eyal Edri 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Gal Ben Haim 
Gerrit-Reviewer: Irit Goihman 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola 
Gerrit-Reviewer: Yaniv Bronhaim
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: fixing repos for standard ci

[eedri]

Eyal Edri has posted comments on this change.

Change subject: fixing repos for standard ci
..


Patch Set 6: Verified-1

putting -verified since it will break until we'll have centos repos for ppc64le

-- 
To view, visit https://gerrit.ovirt.org/62123
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7edd684cf40f2122c4a3afe0091ac2eace1129ef
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eyal Edri 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Gal Ben Haim 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola 
Gerrit-Reviewer: Yaniv Bronhaim
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: fixing repos for standard ci

[eedri]

Eyal Edri has posted comments on this change.

Change subject: fixing repos for standard ci
..


Patch Set 5:

Since VDSM requires lago in .packages file we need python-libguestfs built, 
which is missing from ppc64le.

I've opened [1] to track it.

[1] https://bugs.centos.org/view.php?id=11304

-- 
To view, visit https://gerrit.ovirt.org/62123
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7edd684cf40f2122c4a3afe0091ac2eace1129ef
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eyal Edri 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Gal Ben Haim 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola 
Gerrit-Reviewer: Yaniv Bronhaim
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: Remove _udevrulesdir global macro

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Remove _udevrulesdir global macro
..


Patch Set 4: Verified+1

test worked with this patch.

-- 
To view, visit https://gerrit.ovirt.org/62196
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4c55297cdf08c5bc2484792a171880ee2fef25da
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Idan Shaby 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Piotr Kliczewski 
Gerrit-Reviewer: Tal Nisan 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: Remove _udevrulesdir global macro

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Remove _udevrulesdir global macro
..


Patch Set 4:

running:
http://jenkins.ovirt.org/job/test-repo_ovirt_experimental_master/746/console

-- 
To view, visit https://gerrit.ovirt.org/62196
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4c55297cdf08c5bc2484792a171880ee2fef25da
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Idan Shaby 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Piotr Kliczewski 
Gerrit-Reviewer: Tal Nisan 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: Remove _udevrulesdir global macro

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Remove _udevrulesdir global macro
..


Patch Set 4:

i published the new ioprocess to the repos, the test job should run shortly

-- 
To view, visit https://gerrit.ovirt.org/62196
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4c55297cdf08c5bc2484792a171880ee2fef25da
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Idan Shaby 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Piotr Kliczewski 
Gerrit-Reviewer: Tal Nisan 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: Remove _udevrulesdir global macro

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Remove _udevrulesdir global macro
..


Patch Set 4: -Verified

Allon, this is exactly what i'm trying to do,
run the experimental flow on this patch.. if you're positive it works, lets 
merge, if we can wait for 30 min, maybe we can get verification on the job...

-- 
To view, visit https://gerrit.ovirt.org/62196
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4c55297cdf08c5bc2484792a171880ee2fef25da
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Idan Shaby 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Piotr Kliczewski 
Gerrit-Reviewer: Tal Nisan 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: Remove _udevrulesdir global macro

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Remove _udevrulesdir global macro
..


Patch Set 4:

for some reason the ioprocess build artifacts didn't run from june, i run it 
once manually and it created newer rpms from august:
http://jenkins.ovirt.org/job/ioprocess_master_build-artifacts-el7-x86_64/

i'm running the publisher now to deploy it nightly and then we can retest this.

-- 
To view, visit https://gerrit.ovirt.org/62196
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4c55297cdf08c5bc2484792a171880ee2fef25da
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Idan Shaby 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Piotr Kliczewski 
Gerrit-Reviewer: Tal Nisan 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: Remove _udevrulesdir global macro

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Remove _udevrulesdir global macro
..


Patch Set 4: Verified-1

failing on missing ioprocess 0.17 from repo:
 Package rpm-build-4.11.3-17.el7.x86_64 already installed and latest version
DEBUG util.py:421:  Error: Package: 
python-ioprocess-0.17.0-1.201608101414.gitbd272f2.el7.centos.noarch 
(ovirt-snapshot)
DEBUG util.py:421: Requires: ioprocess = 
0.17.0-1.201608101414.gitbd272f2.el7.centos
DEBUG util.py:421: Available: ioprocess-0.15.1-1.el7.x86_64 
(centos-epel)
DEBUG util.py:421: ioprocess = 0.15.1-1.el7
DEBUG util.py:421: Available: 
ioprocess-0.17.0-1.201606160904.gitbd272f2.el7.centos.x86_64 (ovirt-snapshot)
DEBUG util.py:421: ioprocess = 
0.17.0-1.201606160904.gitbd272f2.el7.centos
DEBUG util.py:421:   You could try using --skip-broken to work around the 
problem
DEBUG util.py:421:   You could try running: rpm -Va --nofiles --nodigest

-- 
To view, visit https://gerrit.ovirt.org/62196
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4c55297cdf08c5bc2484792a171880ee2fef25da
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Idan Shaby 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Piotr Kliczewski 
Gerrit-Reviewer: Tal Nisan 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: changing repos to be multi arch and support ppc64le

[eedri]

Eyal Edri has posted comments on this change.

Change subject: changing repos to be multi arch and support ppc64le
..


Patch Set 1:

(2 comments)

https://gerrit.ovirt.org/#/c/62123/1/automation/check-patch.repos.el7
File automation/check-patch.repos.el7:

PS1, Line 4: candidate
@danken, @fromani - this repo doesn't exists for ppc64le.
do we need it? should we keep the x86_64.


PS1, Line 8: http://debuginfo.centos.org/7/$basearch/
@sandro - we have an open ticket for centos for this right?


-- 
To view, visit https://gerrit.ovirt.org/62123
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7edd684cf40f2122c4a3afe0091ac2eace1129ef
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eyal Edri 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Gal Ben Haim 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola 
Gerrit-Reviewer: Yaniv Bronhaim
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: changing repos to be multi arch and support ppc64le

[eedri]

Eyal Edri has posted comments on this change.

Change subject: changing repos to be multi arch and support ppc64le
..


Patch Set 1: Verified-1

some repos are still missing ppc64le

-- 
To view, visit https://gerrit.ovirt.org/62123
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7edd684cf40f2122c4a3afe0091ac2eace1129ef
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eyal Edri 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: changing repos to be multi arch and support ppc64le

[eedri]

Eyal Edri has uploaded a new change for review.

Change subject: changing repos to be multi arch and support ppc64le
..

changing repos to be multi arch and support ppc64le

Change-Id: I7edd684cf40f2122c4a3afe0091ac2eace1129ef
Signed-off-by: Eyal Edri 
---
M automation/check-patch.repos.el7
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/23/62123/1

diff --git a/automation/check-patch.repos.el7 b/automation/check-patch.repos.el7
index d136062..c0488df 100644
--- a/automation/check-patch.repos.el7
+++ b/automation/check-patch.repos.el7
@@ -1,8 +1,8 @@
 ovirt-snapshot,http://resources.ovirt.org/pub/ovirt-master-snapshot/rpm/$distro
 
ovirt-snapshot-static,http://resources.ovirt.org/pub/ovirt-master-snapshot-static/rpm/$distro
-centos-glusterfs,http://mirror.centos.org/centos/7/storage/x86_64/gluster-3.7/
-centos-ovirt40-candidate,http://cbs.centos.org/repos/virt7-ovirt-40-candidate/x86_64/os/
+centos-glusterfs,http://cbs.centos.org/repos/storage7-gluster-37-release/$basearch/
+centos-ovirt40-candidate,http://cbs.centos.org/repos/virt7-ovirt-40-candidate/$basearch/os/
 ovirt-ci-tools,http://resources.ovirt.org/repos/ci-tools/$distro
 lago,http://resources.ovirt.org/repos/lago/stable/0.0/rpm/$distro
-epel,http://download.fedoraproject.org/pub/epel/7/x86_64
+epel,http://download.fedoraproject.org/pub/epel/7/$basearch/
 centos-debuginfo,http://debuginfo.centos.org/7/$basearch/


-- 
To view, visit https://gerrit.ovirt.org/62123
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7edd684cf40f2122c4a3afe0091ac2eace1129ef
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eyal Edri 
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[ovirt-4.0]: automation: use 4.0 repos

[eedri]

Eyal Edri has posted comments on this change.

Change subject: automation: use 4.0 repos
..


Patch Set 1: Code-Review+2

-- 
To view, visit https://gerrit.ovirt.org/61969
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5d6f39009d5fa24de00c56eeb8297d9691af3b2a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Sandro Bonazzola 
Gerrit-Reviewer: Anton Marchukov 
Gerrit-Reviewer: Barak Korren 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Gil Shinar 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola 
Gerrit-Reviewer: Shlomo Ben David 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[ovirt-4.0.0]: build: Do not require openvswitch on downstream

[eedri]

Eyal Edri has posted comments on this change.

Change subject: build: Do not require openvswitch on downstream
..


Patch Set 1: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/59918
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I96c4eb23f6712f55a9835d8c67085b02d3ba2235
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0.0
Gerrit-Owner: Dan Kenigsberg 
Gerrit-Reviewer: Anton Marchukov 
Gerrit-Reviewer: Barak Korren 
Gerrit-Reviewer: Edward Haas 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Gil Shinar 
Gerrit-Reviewer: Petr Horáček 
Gerrit-Reviewer: Shlomo Ben David 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: Fix README for scratchpad VDSM hook

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Fix README for scratchpad VDSM hook
..


Patch Set 2: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/58582
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id65c2d54e28066b6bce3b17d1b61df0a11bc5912
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Kaul 
Gerrit-Reviewer: Anton Marchukov 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Kaul 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: automation: add python3-netaddr and libvirt-python3

[eedri]

Eyal Edri has posted comments on this change.

Change subject: automation: add python3-netaddr and libvirt-python3
..


Patch Set 1: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/56143
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0baebe2e113149d9b9ae6370b7fe02592d1ba42b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: ci:Fixed execution on el* chroots

[eedri]

Eyal Edri has posted comments on this change.

Change subject: ci:Fixed execution on el* chroots
..


Patch Set 1: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/56139
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I88823d4c94883ef94989197296860913a9c36322
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: David Caro 
Gerrit-Reviewer: David Caro 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: Yaniv Kaul 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: tests: remove model_tests.py form py3 tests

[eedri]

Eyal Edri has posted comments on this change.

Change subject: tests: remove model_tests.py form py3 tests
..


Patch Set 2: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/56119
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I44fc7e80fc01efe154c678aa1fc967314a090a33
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[ovirt-3.6]: spec: build rhev with hooks

[eedri]

Eyal Edri has posted comments on this change.

Change subject: spec: build rhev with hooks
..


Patch Set 2:

I think its worth opening a bug on it to track the change.

-- 
To view, visit https://gerrit.ovirt.org/55322
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I02deb713806ba3650ecc4dbe623a5fe2da10fa94
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[ovirt-3.6]: spec: build rhev with hooks

[eedri]

Eyal Edri has posted comments on this change.

Change subject: spec: build rhev with hooks
..


Patch Set 2:

Rerun-Hooks: patchset-created.bz.0.has_bug_url

-- 
To view, visit https://gerrit.ovirt.org/55322
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I02deb713806ba3650ecc4dbe623a5fe2da10fa94
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: virt: Use Drive.diskType instead of networkDev and blockDev

[eedri]

Eyal Edri has posted comments on this change.

Change subject: virt: Use Drive.diskType instead of networkDev and blockDev
..


Patch Set 6:

one last test

-- 
To view, visit https://gerrit.ovirt.org/40472
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id68bc74b3d788dc82fc61bf8c3de5a52164d0989
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Ala Hino 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Daniel Erez 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenkins CI RO
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: virt: Use Drive.diskType instead of networkDev and blockDev

[eedri]

Eyal Edri has posted comments on this change.

Change subject: virt: Use Drive.diskType instead of networkDev and blockDev
..


Patch Set 6:

sorry, this is really the last one

-- 
To view, visit https://gerrit.ovirt.org/40472
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id68bc74b3d788dc82fc61bf8c3de5a52164d0989
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Ala Hino 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Daniel Erez 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenkins CI RO
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: virt: Use Drive.diskType instead of networkDev and blockDev

[eedri]

Eyal Edri has posted comments on this change.

Change subject: virt: Use Drive.diskType instead of networkDev and blockDev
..


Patch Set 6:

test, please ignore

-- 
To view, visit https://gerrit.ovirt.org/40472
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id68bc74b3d788dc82fc61bf8c3de5a52164d0989
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Ala Hino 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Daniel Erez 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenkins CI RO
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[ovirt-3.5]: hotunplug: support python 2.6

[eedri]

Eyal Edri has posted comments on this change.

Change subject: hotunplug: support python 2.6
..


Patch Set 2:

Rerunning only bug_url to skip the 3.6 branch verification.
since its only 3.5 bug.

Rerun-Hooks: patchset-created.bz.0.has_bug_url

-- 
To view, visit https://gerrit.ovirt.org/48916
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia89620ff3aab14dc09373c44c8567b74e2103a60
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Amit Aviram 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: [WIP] Automation scripts for running vdsm functional tests job

[eedri]

Eyal Edri has abandoned this change.

Change subject: [WIP] Automation scripts for running vdsm functional tests job
..


Abandoned

already implemented. see danken comment.

-- 
To view, visit https://gerrit.ovirt.org/41928
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: abandon
Gerrit-Change-Id: I24b446d3ae38c944b5a28d56ed0965d0f2edc206
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: David Caro 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: David Caro 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: Yeela Kaplan 
Gerrit-Reviewer: automat...@ovirt.org
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[ovirt-3.5]: vdsm-tool: Do not panic with sanlock empty groups

[eedri]

Eyal Edri has posted comments on this change.

Change subject: vdsm-tool: Do not panic with sanlock empty groups
..


Patch Set 2:

if we want this also in 3.6.0, this has to be backported to 3.6.0 branch as 
well (master also?)

so: master/3.6/3.6.0

-- 
To view, visit https://gerrit.ovirt.org/47105
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I47f080beb748353970dec0753c9a0a7b1dd09bc6
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Eyal Edri 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Germano Veit Michel 
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: Cleaning libvirt requirements which are not required anymore

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Cleaning libvirt requirements which are not required anymore
..


Patch Set 1: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/43851
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idf656d3f026b32d2a5a30efe9ee2a57ac8f38b46
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Sagi Shnaidman sshna...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: Fixed scratchpad hook's image permissions and virtio device ...

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Fixed scratchpad hook's image permissions and virtio device name
..


Patch Set 2: Code-Review+1

Danken,
this patch has a potential of dramatically improving the ci performance, and 
its getting blocked for too long,

What do we need to do to push it forward asap?

-- 
To view, visit https://gerrit.ovirt.org/42573
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4e0e2b09eddeb020ac380bfb9b5d9be73ce5377b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Anton Marchukov amarc...@redhat.com
Gerrit-Reviewer: Amador Pahim apa...@redhat.com
Gerrit-Reviewer: Anton Marchukov amarc...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Doron Fediuck dfedi...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi shav...@redhat.com
Gerrit-Reviewer: Wenyi Gao we...@linux.vnet.ibm.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: Fixed scratchpad hook's image permissions and virtio device ...

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Fixed scratchpad hook's image permissions and virtio device name
..


Patch Set 2:

@anton, after this will be merged,
we should consider to back-port it to 3.5.4, as we're using 3.5 in the phx lab.

-- 
To view, visit https://gerrit.ovirt.org/42573
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4e0e2b09eddeb020ac380bfb9b5d9be73ce5377b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Anton Marchukov amarc...@redhat.com
Gerrit-Reviewer: Amador Pahim apa...@redhat.com
Gerrit-Reviewer: Anton Marchukov amarc...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Doron Fediuck dfedi...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi shav...@redhat.com
Gerrit-Reviewer: Wenyi Gao we...@linux.vnet.ibm.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: spec: Make vdsm arch specific package

[eedri]

Eyal Edri has posted comments on this change.

Change subject: spec: Make vdsm arch specific package
..


Patch Set 1:

(1 comment)

https://gerrit.ovirt.org/#/c/42490/1/vdsm.spec.in
File vdsm.spec.in:

Line 315: noarch
not worth using global %arch instead of specifying noarch repeatedly?


-- 
To view, visit https://gerrit.ovirt.org/42490
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iecbc71ccded31e8b80b14dbb03fd738694ceb37c
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek mskri...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: spec: Move vdsm package contents to vdsm-deamon

[eedri]

Eyal Edri has posted comments on this change.

Change subject: spec: Move vdsm package contents to vdsm-deamon
..


Patch Set 1:

how does this reflect on the rpms creation?

-- 
To view, visit https://gerrit.ovirt.org/42491
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I58b0b0e7461f9ac10a189dd8ca6715d02183ad21
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Michal Skrivanek mskri...@redhat.com
Gerrit-Reviewer: Nelly Credi ncr...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: remove safelease code and require the package instead

[eedri]

Eyal Edri has posted comments on this change.

Change subject: remove safelease code and require the package instead
..


Patch Set 4: Code-Review+1 -Verified

guys,thank you for making sure this patch doesn't break ci
and followed on getting +1 from jenkins on this.

let's keep up the communication channel and make sure this is the way we 
introduce new dependnecies going forward.

removing -1 due to succsesfull ci jobs.

-- 
To view, visit https://gerrit.ovirt.org/40724
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0c476fcab4e51edfb920842502b3cece8b2c6ca5
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: Removing safelease code from vdsm and requires the package i...

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Removing safelease code from vdsm and requires the package 
instead
..


Patch Set 1: Code-Review-1 Verified-1

missing saferelease rpm, please ensure it's avaliable via one of the existing 
repos or request infra help to add it from a 3rd party repo.

-- 
To view, visit https://gerrit.ovirt.org/40724
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0c476fcab4e51edfb920842502b3cece8b2c6ca5
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: spec: Update libvrit-daemon and libvirt-python requirements

[eedri]

Eyal Edri has posted comments on this change.

Change subject: spec: Update libvrit-daemon and libvirt-python requirements
..


Patch Set 2:

i have to ask, why are you ignoring the jenkins failures?
if these are false positive, then report to us and we'll check, but otherwise 
what's the point in running ci?

-- 
To view, visit https://gerrit.ovirt.org/39551
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I571cc86e0f478b4169f398a6d54c96fc8bb4da30
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Adam Litke ali...@redhat.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Francesco Romani from...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: spec: revert build dependency on non-existing blivet

[eedri]

Eyal Edri has posted comments on this change.

Change subject: spec: revert build dependency on non-existing blivet
..


Patch Set 1: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/39132
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7fc8f3c802e711a844b8aed0b927111cf0d593f4
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Max Kovgan m...@redhat.com
Gerrit-Reviewer: Ramesh N rnach...@redhat.com
Gerrit-Reviewer: Sahina Bose sab...@redhat.com
Gerrit-Reviewer: Sandro Bonazzola sbona...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: Arrange pthreading requirement to 0.1.3-2 which includes loc...

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Arrange pthreading requirement to 0.1.3-2 which includes 
locked() impl
..


Patch Set 1: Code-Review-1

please make sure new pthreading is available on repos before merging.
it currently fails vdsm install.

-- 
To view, visit http://gerrit.ovirt.org/29988
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I229efcb8291125b60e7a505d23d644d7e3efa480
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Douglas Schilling Landgraf dougsl...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: vdsm-upgrade: adds wrapper to ovirt-node-upgrade

[eedri]

Eyal Edri has posted comments on this change.

Change subject: vdsm-upgrade: adds wrapper to ovirt-node-upgrade
..


Patch Set 17:

fabian - i don't see any ovirt-node rpm job in jenkins, so not sure if there is 
nightly repo ready for this.
when do you plan to release a new version to be included in stable?

-- 
To view, visit http://gerrit.ovirt.org/28244
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7b997d70a440545497246d1a19d9671b054a56a5
Gerrit-PatchSet: 17
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf dougsl...@redhat.com
Gerrit-Reviewer: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Douglas Schilling Landgraf dougsl...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Fabian Deutsch fabi...@fedoraproject.org
Gerrit-Reviewer: Joey Boggs jbo...@redhat.com
Gerrit-Reviewer: Sandro Bonazzola sbona...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: pep8: make pep8-1.5 happy with E265

[eedri]

Eyal Edri has posted comments on this change.

Change subject: pep8: make pep8-1.5 happy with E265
..


Patch Set 3:

still fails on other issues:
http://pastebin.com/uMaXEfaT

-- 
To view, visit http://gerrit.ovirt.org/26248
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I91c49e1c0f36590b8052462b9e066c110eacd67b
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Dima Kuznetsov dkuzn...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Ohad Basan oba...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: mooli tayer mta...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: Changing python-cpopen requirement

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Changing python-cpopen requirement
..


Patch Set 1: Code-Review+1

-- 
To view, visit http://gerrit.ovirt.org/22936
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I543a51c9ed2a9612a4dde7a46810963f9ae279db
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: Changing python-cpopen requirement

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Changing python-cpopen requirement
..


Patch Set 1: Verified+1 Code-Review-1

works on the jenkins job: 
http://jenkins.ovirt.org/job/vdsm_master_install_rpm_sanity_gerrit/label=fedora19/198/console

-- 
To view, visit http://gerrit.ovirt.org/22936
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I543a51c9ed2a9612a4dde7a46810963f9ae279db
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: Changing python-cpopen requirement

[eedri]

Eyal Edri has posted comments on this change.

Change subject: Changing python-cpopen requirement
..


Patch Set 1:

of course pending pushing that pkg to fedora + el6 repos.

-- 
To view, visit http://gerrit.ovirt.org/22936
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I543a51c9ed2a9612a4dde7a46810963f9ae279db
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: fix the premature selinux dependency fix

[eedri]

Eyal Edri has posted comments on this change.

Change subject: fix the premature selinux dependency fix
..


Patch Set 1:

seems to work, but now fails on another pkg:
--- Package vdsm-xmlrpc.noarch 0:4.13.0-133.git7e51f7b.el6 will be installed
--- Package vdsm-yajsonrpc.noarch 0:4.13.0-133.git7e51f7b.el6 will be installed
-- Finished Dependency Resolution
Error: Package: vdsm-4.13.0-133.git7e51f7b.el6.x86_64 
(/vdsm-4.13.0-133.git7e51f7b.el6.x86_64)
   Requires: iscsi-initiator-utils = 6.2.0.873-3
   Installed: iscsi-initiator-utils-6.2.0.873-2.el6.x86_64 
(@anaconda-CentOS-201303020151.x86_64/6.4)
   iscsi-initiator-utils = 6.2.0.873-2.el6

-- 
To view, visit http://gerrit.ovirt.org/21284
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I627deb840d2351b48c19eb7d17f26f9ba9f54566
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: bugfix: UpdateVmDevice QoS

[eedri]

Eyal Edri has posted comments on this change.

Change subject: bugfix: UpdateVmDevice QoS
..


Patch Set 6: Code-Review+1

(1 comment)

but keep in mind, this will fail on centos 6.4 of course.


File vdsm.spec.in
Line 117: Requires: libvirt-daemon-driver-nwfilter
Line 118: Requires: libvirt-daemon-driver-qemu
Line 119: %else
Line 120: %if 0%{?rhel}
Line 121: Requires: libvirt = 0.10.2-29.el6
latest rhel repo has this:
[root@cinteg33 ~]# yum info libvirt
Loaded plugins: product-id, security, subscription-manager
This system is not registered to Red Hat Subscription Management. You can use 
subscription-manager to register.
Installed Packages
Name: libvirt
Arch: x86_64
Version : 0.10.2
Release : 29.el6
Size: 5.6 M
Repo: installed
From repo   : rhel
Summary : Library providing a simple virtualization API
URL : http://libvirt.org/
License : LGPLv2+
Description : Libvirt is a C toolkit to interact with the virtualization 
capabilities
: of recent versions of Linux (and other OSes). The main package 
includes
: the libvirtd server exporting the virtualization support.
Line 122: %else
Line 123: Requires: libvirt = 1.0.2-1
Line 124: %endif
Line 125: %endif


-- 
To view, visit http://gerrit.ovirt.org/19545
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I43a383b2a9cf96366927beebf63f1344027169fb
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Šebek pse...@redhat.com
Gerrit-Reviewer: Antoni Segura Puimedon asegu...@redhat.com
Gerrit-Reviewer: Assaf Muller amul...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Petr Šebek pse...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[ovirt-3.3]: fix premature selinux dependency

[eedri]

Eyal Edri has posted comments on this change.

Change subject: fix premature selinux dependency
..


Patch Set 1: Verified-1

http://jenkins.ovirt.org/job/vdsm_install_rpm_sanity_gerrit/label=centos64/201/consoleFull

centos 6.4 current version is lower than req: 
Requires: selinux-policy-targeted = 3.7.19-195.el6.13
Installed: selinux-policy-targeted-3.7.19-195.el6_4.12.noarch (@updates)

-- 
To view, visit http://gerrit.ovirt.org/20313
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea625d7c39035055b246357030372430b4bddf68
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Ohad Basan oba...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: fix premature selinux dependency

[eedri]

Eyal Edri has posted comments on this change.

Change subject: fix premature selinux dependency
..


Patch Set 3: Code-Review+1

-- 
To view, visit http://gerrit.ovirt.org/19807
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea625d7c39035055b246357030372430b4bddf68
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Ayal Baron aba...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Ohad Basan oba...@redhat.com
Gerrit-Reviewer: Toshio くらとみ a.bad...@gmail.com
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: vdsm-tool: fix qemu-sanlock configuration

[eedri]

Eyal Edri has posted comments on this change.

Change subject: vdsm-tool: fix qemu-sanlock configuration
..


Patch Set 1: Verified+1 Code-Review+1

-- 
To view, visit http://gerrit.ovirt.org/19094
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d2a6b05fb8fbf0ef2e9599fd1a740a7091680d4
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Ayal Baron aba...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: build: use pythondir for yajsonrpc

[eedri]

Eyal Edri has posted comments on this change.

Change subject: build: use pythondir for yajsonrpc
..


Patch Set 1: Looks good to me, but someone else must approve

-- 
To view, visit http://gerrit.ovirt.org/17131
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c98abc9f52f948d4807d4ee6eda6efba6c42bcd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Kiril Nesenko knese...@redhat.com
Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: build: do not distribute precompiled cpopen.so

[eedri]

Eyal Edri has posted comments on this change.

Change subject: build: do not distribute precompiled cpopen.so
..


Patch Set 1: Verified

-- 
To view, visit http://gerrit.ovirt.org/17189
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I45058a8d2c9c1824f812d16772ad98cfbcc92692
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Kiril Nesenko knese...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: build: use pythondir for yajsonrpc

[eedri]

Eyal Edri has posted comments on this change.

Change subject: build: use pythondir for yajsonrpc
..


Patch Set 1: Verified

-- 
To view, visit http://gerrit.ovirt.org/17131
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c98abc9f52f948d4807d4ee6eda6efba6c42bcd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Kiril Nesenko knese...@redhat.com
Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: fix testBuildCmdLine not to be Fedora specific.

[eedri]

Eyal Edri has posted comments on this change.

Change subject: fix testBuildCmdLine not to be Fedora specific.
..


Patch Set 3: Looks good to me, but someone else must approve

-- 
To view, visit http://gerrit.ovirt.org/15507
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Icf2fff6d0de0c97b83f800a9264753f4478b6094
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon asegu...@redhat.com
Gerrit-Reviewer: Antoni Segura Puimedon asegu...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Giuseppe Vallarelli gvall...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: BZ#840407 - Create a fake template when moving to backup SD.

[eedri]

Eyal Edri has posted comments on this change.

Change subject: BZ#840407 - Create a fake template when moving to backup SD.
..


Patch Set 1:

any update on pushing this patch?

--
To view, visit http://gerrit.ovirt.org/6519
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibbeef2480e03cc075b80880485739139e496d0b6
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Ayal Baron aba...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Daniel Paikov pai...@gmail.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Haim Ateya hat...@redhat.com
Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Moran Goldboim mgold...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: The method readMultipathConf should return a list

[eedri]

Eyal Edri has posted comments on this change.

Change subject: The method readMultipathConf should return a list
..


Patch Set 1: Verified

not shutting down in loop anymore,

though i still can't connect with vdsClient remotly to the host running vdsm 
(gets timeout), not sure if it's relevant to this bug.

--
To view, visit http://gerrit.ovirt.org/3863
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Icf1654e752698d1dbb41c09a8d409784570a9fcd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: bootstrap: installation now requires libvirt when defining m...

[eedri]

Eyal Edri has posted comments on this change.

Change subject: bootstrap: installation now requires libvirt when defining mgmt 
network
..


Patch Set 1: Verified

--
To view, visit http://gerrit.ovirt.org/3269
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2f366c8f88c22f2aa266087c57c04d019da1e70f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Douglas Schilling Landgraf dougsl...@redhat.com
Gerrit-Reviewer: Eyal Edri ee...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/vdsm-patches