public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
* Latest kvm autotest patch queue
@ 2009-06-22 13:24 Lucas Meneghel Rodrigues
  2009-06-23  4:12 ` sudhir kumar
  0 siblings, 1 reply; 3+ messages in thread
From: Lucas Meneghel Rodrigues @ 2009-06-22 13:24 UTC (permalink / raw)
  To: KVM mailing list, Autotest mailing list; +Cc: Michael Goldish

KVM Autotest patch queue

--------------------------------------------------------------------------------
Michael Goldish (mgoldish@redhat.com)

1) KVM test: optionally convert PPM files to PNG format after test - Reviewed

2) Step file tests: introducing a new feature and some small changes - Reviewed

3) Introducing kvm_subprocess - Reviewed

--------------------------------------------------------------------------------
Jason Wang (jasowang@redhat.com)

1) TAP network support in kvm-autotest

Michael Goldish has a similar patch to this one, so we need to decide which one
is going to be used. Michael, when possible, please send your patch to support
TAP network on the kvm test.

--------------------------------------------------------------------------------
David Huff (dhuff@redhat.com)

1) Unattended installs - Needs review

Reviewed. Changes made to the unattended script, would like to create a python
module to handle the unattended install and study the possibility of doing things
with cobbler.

2) Move kvm functional tests to a 'test' directory

http://kerneltrap.org/mailarchive/linux-kvm/2009/5/26/5812453

I have reworked this patch and made my own version. I need to split it into
smaller patches though.

Needs review

--------------------------------------------------------------------------------
Yogi (anantyog@linux.vnet.ibm.com)

1) Support for remote migration

http://kerneltrap.org/mailarchive/linux-kvm/2009/4/30/5607344/thread

Rebased the patch. However, the patch review and discussion shows that we want
to implement remote migration as a server side test instead of a client side
one. The rebase is just in case we need this in the future.

Probably not going to apply, look at a different way to implement the test.

--------------------------------------------------------------------------------
Alexey Eromenko (aeromenk@redhat.com)

1) New test module: iperf

http://kerneltrap.org/mailarchive/linux-kvm/2009/5/31/5840973/thread

Rebase made, made some comments to it.
 * The idea of shipping binaries doesn't look very appealing to me
 * Autotest already has an iperf test, might be worth a look for linux guests

Pending some issues during review.

--------------------------------------------------------------------------------



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Latest kvm autotest patch queue
  2009-06-22 13:24 Latest kvm autotest patch queue Lucas Meneghel Rodrigues
@ 2009-06-23  4:12 ` sudhir kumar
  2009-06-25 13:02   ` jason wang
  0 siblings, 1 reply; 3+ messages in thread
From: sudhir kumar @ 2009-06-23  4:12 UTC (permalink / raw)
  To: Lucas Meneghel Rodrigues
  Cc: KVM mailing list, Autotest mailing list, Michael Goldish

On Mon, Jun 22, 2009 at 6:54 PM, Lucas Meneghel Rodrigues<lmr@redhat.com> wrote:
> KVM Autotest patch queue
>
> --------------------------------------------------------------------------------
> Michael Goldish (mgoldish@redhat.com)
>
> 1) KVM test: optionally convert PPM files to PNG format after test - Reviewed
>
> 2) Step file tests: introducing a new feature and some small changes - Reviewed
>
> 3) Introducing kvm_subprocess - Reviewed
>
> --------------------------------------------------------------------------------
> Jason Wang (jasowang@redhat.com)
>
> 1) TAP network support in kvm-autotest
>
> Michael Goldish has a similar patch to this one, so we need to decide which one
I remember Michael's patch using nmap to get the ip of the guest. I
tested the patch and found nmap taking too long to do a lookup and get
the guest ip. I will suggets two things here.
1. Modify nmap options to make nmap lookup really faster.
2. Provide an option to use a dictionary based(mac address to ip
mapping) lookup in config file and return mapped ip for a mac address.

I can send a patch for option 2 on top of Michael's patch. In the
simplest form it will generate a dictionary from a macaddress vs IP
address list(provided by the dhcp server in your lab/environment. you
can put this list in a file and use it). You need to specify macaddr
parameter in config file from this list. Looking the mac address the
parser will return the corresponding ip address from the dictionary
generated.

Please let me know if 2 sounds good. Also I think I missed to look at
Jason's patch?
> is going to be used. Michael, when possible, please send your patch to support
> TAP network on the kvm test.
>
> --------------------------------------------------------------------------------
> David Huff (dhuff@redhat.com)
>
> 1) Unattended installs - Needs review
>
> Reviewed. Changes made to the unattended script, would like to create a python
> module to handle the unattended install and study the possibility of doing things
> with cobbler.
>
> 2) Move kvm functional tests to a 'test' directory
>
> http://kerneltrap.org/mailarchive/linux-kvm/2009/5/26/5812453
>
> I have reworked this patch and made my own version. I need to split it into
> smaller patches though.
>
> Needs review
>
> --------------------------------------------------------------------------------
> Yogi (anantyog@linux.vnet.ibm.com)
>
> 1) Support for remote migration
>
> http://kerneltrap.org/mailarchive/linux-kvm/2009/4/30/5607344/thread
>
> Rebased the patch. However, the patch review and discussion shows that we want
> to implement remote migration as a server side test instead of a client side
> one. The rebase is just in case we need this in the future.
>
> Probably not going to apply, look at a different way to implement the test.
>
> --------------------------------------------------------------------------------
> Alexey Eromenko (aeromenk@redhat.com)
>
> 1) New test module: iperf
>
> http://kerneltrap.org/mailarchive/linux-kvm/2009/5/31/5840973/thread
>
> Rebase made, made some comments to it.
>  * The idea of shipping binaries doesn't look very appealing to me
>  * Autotest already has an iperf test, might be worth a look for linux guests
>
> Pending some issues during review.
>
> --------------------------------------------------------------------------------
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>



-- 
Sudhir Kumar

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Latest kvm autotest patch queue
  2009-06-23  4:12 ` sudhir kumar
@ 2009-06-25 13:02   ` jason wang
  0 siblings, 0 replies; 3+ messages in thread
From: jason wang @ 2009-06-25 13:02 UTC (permalink / raw)
  To: sudhir kumar
  Cc: Lucas Meneghel Rodrigues, KVM mailing list, Autotest mailing list,
	Michael Goldish

sudhir kumar 写道:
> On Mon, Jun 22, 2009 at 6:54 PM, Lucas Meneghel Rodrigues<lmr@redhat.com> wrote:
>   
>> KVM Autotest patch queue
>>
>> --------------------------------------------------------------------------------
>> Michael Goldish (mgoldish@redhat.com)
>>
>> 1) KVM test: optionally convert PPM files to PNG format after test - Reviewed
>>
>> 2) Step file tests: introducing a new feature and some small changes - Reviewed
>>
>> 3) Introducing kvm_subprocess - Reviewed
>>
>> --------------------------------------------------------------------------------
>> Jason Wang (jasowang@redhat.com)
>>
>> 1) TAP network support in kvm-autotest
>>
>> Michael Goldish has a similar patch to this one, so we need to decide which one
>>     
> I remember Michael's patch using nmap to get the ip of the guest. I
> tested the patch and found nmap taking too long to do a lookup and get
> the guest ip. I will suggets two things here.
> 1. Modify nmap options to make nmap lookup really faster.
> 2. Provide an option to use a dictionary based(mac address to ip
> mapping) lookup in config file and return mapped ip for a mac address.
>
> I can send a patch for option 2 on top of Michael's patch. In the
> simplest form it will generate a dictionary from a macaddress vs IP
> address list(provided by the dhcp server in your lab/environment. you
> can put this list in a file and use it). You need to specify macaddr
> parameter in config file from this list. Looking the mac address the
> parser will return the corresponding ip address from the dictionary
> generated.
>
> Please let me know if 2 sounds good. Also I think I missed to look at
> Jason's patch?
>   
My patch could be seen at http://patchwork.kernel.org/patch/23524/. I 
think the mac:ip dictionary is not flexible enough especially in a 
network with public dhcp server. The scanning method could be used in 
either public or private bridge. But my patch still have some problems 
to solve and need advices:
1 Mac address generation: I could rework the patch and make it could 
accept user specified address range, but the user should guarantee its 
validity.
2 IP scanner: Is there any option of nmap could make the scanning faster 
than -sP? I also agree that nmap is not a elegant method, but it seems 
the sole choice ( or write a simple scanner through python?) . Another 
method is to snoop the dhcp session through tcpdump during the 
vm.create() but this method does not feed for all dhcp server.
3 I want the make network method to be an exclusive options and want to 
know if it is useful to have both user mode nics and bridge nics 
simultaneously.
4 Based on advices of Michael Goldish I would add an index to 
get_address() and get_port() to adapt the environments with multiple nics.

I would rework the patches based on the comments, and like to see the 
patch of michael.
Thanks.
**

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-06-25 13:01 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-06-22 13:24 Latest kvm autotest patch queue Lucas Meneghel Rodrigues
2009-06-23  4:12 ` sudhir kumar
2009-06-25 13:02   ` jason wang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox