Re: [virt-tools-list] [virt-manager PATCH] test_urls: remove commented fedora21 alpha urls

2014-12-11 Thread Cole Robinson
On 12/10/2014 08:39 PM, Chen Hanxiao wrote: Discoverd by pep8 '# ' style check. introduced by 92b443df. Guess that you forgot to delete it. Signed-off-by: Chen Hanxiao chenhanx...@cn.fujitsu.com --- tests/test_urls.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/tests/test_urls.py

Re: [virt-tools-list] [virt-manager PATCH] clitest: replace DIR from current pwd by '/var'

2014-12-11 Thread Cole Robinson
On 12/10/2014 10:46 PM, Chen Hanxiao wrote: Introduced by f16dc4dd34f4f1aa326939cf79fcbdc204e2146f This will cause test case failure on different machines. Use '/var' to replace it. Signed-off-by: Chen Hanxiao chenhanx...@cn.fujitsu.com ---

Re: [virt-tools-list] [libvirt] [PATCH] virtinst: refresh pools status before fetch_pools

2014-12-11 Thread Michal Privoznik
On 11.12.2014 03:27, Chun Yan Liu wrote: On 12/5/2014 at 09:54 PM, in message 5481b907.4040...@redhat.com, Cole Robinson crobi...@redhat.com wrote: On 12/05/2014 03:40 AM, Chunyan Liu wrote: snip/ +# Refresh pools before poll_helper. For those +# 'active' but target path

Re: [virt-tools-list] [libvirt] [PATCH] virtinst: refresh pools status before fetch_pools

2014-12-11 Thread Daniel P. Berrange
On Thu, Dec 11, 2014 at 02:03:37PM +0100, Michal Privoznik wrote: On 11.12.2014 03:27, Chun Yan Liu wrote: On 12/5/2014 at 09:54 PM, in message 5481b907.4040...@redhat.com, Cole Robinson crobi...@redhat.com wrote: On 12/05/2014 03:40 AM, Chunyan Liu wrote: snip/ +# Refresh pools

Re: [virt-tools-list] [virt-manager PATCH] clitest: replace DIR from current pwd by '/var'

2014-12-11 Thread Chen, Hanxiao
-Original Message- From: Cole Robinson [mailto:crobi...@redhat.com] Sent: Thursday, December 11, 2014 8:55 PM To: Chen, Hanxiao/陈 晗霄; virt-tools-list@redhat.com Subject: Re: [virt-tools-list] [virt-manager PATCH] clitest: replace DIR from current pwd by '/var' On 12/10/2014

Re: [virt-tools-list] [virt-manager PATCH] test_urls: remove commented fedora21 alpha urls

2014-12-11 Thread Chen, Hanxiao
-Original Message- From: Cole Robinson [mailto:crobi...@redhat.com] Sent: Thursday, December 11, 2014 8:56 PM To: Chen, Hanxiao/陈 晗霄; virt-tools-list@redhat.com Subject: Re: [virt-tools-list] [virt-manager PATCH] test_urls: remove commented fedora21 alpha urls On 12/10/2014