Change in vdsm[master]: fileSD: Fix image deletion on gluster domain

2013-12-19 Thread nsoffer
Nir Soffer has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 6: Verified+1

Verified on glusterfs and NFS, see Hunt comment.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-19 Thread danken
Dan Kenigsberg has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 6:

Thanks for fixing this, albeit in the price of an added reference to the image 
object. It's unfortunate that we have to produce an image in order to delete it.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-19 Thread danken
Dan Kenigsberg has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 7:

I believe this patch requires a backport to the ovirt-3.3 branch. Would you do 
this?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-19 Thread nsoffer
Nir Soffer has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 7:

Sure

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-19 Thread nsoffer
Nir Soffer has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 7:

Here: http://gerrit.ovirt.org/22563

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-18 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 5:

No Builds Executed 

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/6142/ : To avoid overloading the 
infrastructure, a whitelist for running gerrit triggered jobs has been set in 
place, if you feel like you should be in it, please contact infra at ovirt dot 
org.

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/5355/ : To avoid 
overloading the infrastructure, a whitelist for running gerrit triggered jobs 
has been set in place, if you feel like you should be in it, please contact 
infra at ovirt dot org.

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/6245/ : To avoid 
overloading the infrastructure, a whitelist for running gerrit triggered jobs 
has been set in place, if you feel like you should be in it, please contact 
infra at ovirt dot org.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-18 Thread nsoffer
Nir Soffer has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 5: Code-Review+1

Thanks for your nice patch!

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-18 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 5:

No Builds Executed 

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/6143/ : To avoid overloading the 
infrastructure, a whitelist for running gerrit triggered jobs has been set in 
place, if you feel like you should be in it, please contact infra at ovirt dot 
org.

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/5356/ : To avoid 
overloading the infrastructure, a whitelist for running gerrit triggered jobs 
has been set in place, if you feel like you should be in it, please contact 
infra at ovirt dot org.

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/6246/ : To avoid 
overloading the infrastructure, a whitelist for running gerrit triggered jobs 
has been set in place, if you feel like you should be in it, please contact 
infra at ovirt dot org.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-18 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 5:

No Builds Executed 

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/6143/ : To avoid overloading the 
infrastructure, a whitelist for running gerrit triggered jobs has been set in 
place, if you feel like you should be in it, please contact infra at ovirt dot 
org.

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/5356/ : To avoid 
overloading the infrastructure, a whitelist for running gerrit triggered jobs 
has been set in place, if you feel like you should be in it, please contact 
infra at ovirt dot org.

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/6250/ : To avoid 
overloading the infrastructure, a whitelist for running gerrit triggered jobs 
has been set in place, if you feel like you should be in it, please contact 
infra at ovirt dot org.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-18 Thread danken
Dan Kenigsberg has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 5:

(1 comment)


Commit Message
Line 6: 
Line 7: fileSD: Fix image deletion on gluster domain
Line 8: 
Line 9: getImagePath() does not work for glusterSD. Now we use the storage
Line 10: domain's domaindir to get the image's directory correctly.
Sorry, I do not follow what's been broken on the module-level getImagePath() 
when it comes to gluster. Do you know when it has happened? Could you explain 
it in this commit message? The mailing list reference below has multiple ideas 
about the proper solution.
Line 11: 
Line 12: Reference:
Line 13:   * http://lists.ovirt.org/pipermail/users/2013-December/018653.html
Line 14: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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[master]: fileSD: Fix image deletion on gluster domain

2013-12-18 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 6:

No Builds Executed 

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/6152/ : To avoid overloading the 
infrastructure, a whitelist for running gerrit triggered jobs has been set in 
place, if you feel like you should be in it, please contact infra at ovirt dot 
org.

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/5365/ : To avoid 
overloading the infrastructure, a whitelist for running gerrit triggered jobs 
has been set in place, if you feel like you should be in it, please contact 
infra at ovirt dot org.

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/6256/ : To avoid 
overloading the infrastructure, a whitelist for running gerrit triggered jobs 
has been set in place, if you feel like you should be in it, please contact 
infra at ovirt dot org.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-18 Thread nsoffer
Nir Soffer has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 6: Code-Review+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-18 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 6: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-18 Thread nsoffer
Nir Soffer has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 6:

Hunt, can you describe how you verified your change on glusterfs and nfs?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-18 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 6:

No Builds Executed 

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/6168/ : To avoid overloading the 
infrastructure, a whitelist for running gerrit triggered jobs has been set in 
place, if you feel like you should be in it, please contact infra at ovirt dot 
org.

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/5381/ : To avoid 
overloading the infrastructure, a whitelist for running gerrit triggered jobs 
has been set in place, if you feel like you should be in it, please contact 
infra at ovirt dot org.

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/6272/ : To avoid 
overloading the infrastructure, a whitelist for running gerrit triggered jobs 
has been set in place, if you feel like you should be in it, please contact 
infra at ovirt dot org.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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]: fileSD: Fix image deletion on gluster domain

2013-12-18 Thread mhuntxu
Hunt Xu has posted comments on this change.

Change subject: fileSD: Fix image deletion on gluster domain
..


Patch Set 6:

Hi, Nir. I verified that using an oVirt environment.

I created two data centers in ovirt-engine, one's storage type is GlusterFS and 
the other one is NFS. Then I created VMs in each data center, installed systems 
on the newly-created disks, ran the VMs for a while, then deleted the VMs with 
Remove Disk(s) checked. No error was reported by both ovirt-engine and vdsm. 
Then I manually mounted the storage to check whether the disks were actually 
removed and I got a positive result.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee6cb963a428f18b6e43f0585d461d0975c377a
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Allon Mureinik amure...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Eduardo ewars...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Hunt Xu mhun...@gmail.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Vered Volansky vvola...@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