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

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/20

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 wrote: > >>On 12/05/2014 03:40 AM, Chunyan Liu wrote: > >>> > >>>+# Refresh pools

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 wrote: On 12/05/2014 03:40 AM, Chunyan Liu wrote: >>> +# Refresh pools before poll_helper. For those +# 'active' but target path not exist (or other

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 --- tests/cli-test-xml/compare/virt-install-many-devices.xml | 2 +- tests/clitest

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 --- tests/test_urls.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/tests/test_urls.py b/tests/test_urls.py index c