public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Jiri Zupka <jzupka@redhat.com>
To: ehabkost@redhat.com, pbonzini@redhat.com, lmr@redhat.com
Cc: kvm-autotest@redhat.com, virt-test-devel@redhat.com, kvm@vger.kernel.org
Subject: virt: New version Cartesian config
Date: Fri, 3 May 2013 11:54:42 -0400 (EDT)	[thread overview]
Message-ID: <1303043389.9050536.1367596482456.JavaMail.root@redhat.com> (raw)
In-Reply-To: <1306414001.2530700.1366166867718.JavaMail.root@redhat.com>

Hi,
  the new version of cart config is on github https://github.com/autotest/virt-test/pull/335
Please send me your comment and I'll try to change code to good shape if there is a problem.
I'll choose one big patch because there is almost no possibility split parser to parts.
Unittest is in some patch too. Because it should be in one patch with tested code.

In future I'll change comments to sphinx syntax.

Regards,
  Jiří Župka

----- Original Message -----
> Jiří, okay, got it and thanks.
> 
> --
> Regards,
> Alex
> 
> 
> ----- Original Message -----
> From: "Jiri Zupka" <jzupka@redhat.com>
> To: "Alex Jia" <ajia@redhat.com>
> Cc: virt-test-devel@redhat.com, kvm@vger.kernel.org, kvm-autotest@redhat.com,
> lmr@redhat.com, ldoktor@redhat.com, ehabkost@redhat.com, pbonzini@redhat.com
> Sent: Tuesday, April 16, 2013 10:16:57 PM
> Subject: Re: [Virt-test-devel] [virt-test][PATCH 4/7] virt: Adds named
> variants to Cartesian config.
> 
> Hi Alex,
>   thanks again for review. I recognize now what you mean. I thought that
> you another thread of mails. I was try it again with
> https://github.com/autotest/virt-test/pull/255 and demo example works.
> 
> If you are interest in this feature. Check new version which I'll send in
> future days. There will be some changes in syntax and will be added lexer
> for better filtering.
> 
> regards,
>   Jiří Župka
> 
> 
> 
> ----- Original Message -----
> > Hi Alex,
> >   I hope you use "new" version of cart config in github
> >   https://github.com/autotest/virt-test/pull/255.
> > This was older RFC version of vart config. And I'm preparing new version
> > based on communication with Eduardo and Pablo.
> > 
> > If you don't please loot at documentation
> > https://github.com/autotest/virt-test/wiki/VirtTestDocumentation#wiki-id26
> > This documentation says how it works now.
> > 
> > regards
> >   Jiří Župka
> > 
> > ----- Original Message -----
> > > On 03/30/2013 01:14 AM, Jiří Župka wrote:
> > > > variants name=tests:
> > > >    - wait:
> > > >         run = "wait"
> > > >         variants:
> > > >           - long:
> > > >              time = short_time
> > > >           - short: long
> > > >              time = logn_time
> > > >    - test2:
> > > >         run = "test1"
> > > >
> > > > variants name=virt_system:
> > > >    - linux:
> > > >    - windows_XP:
> > > >
> > > > variants name=host_os:
> > > >    - linux:
> > > >         image = linux
> > > >    - windows_XP:
> > > >         image = windows
> > > >
> > > > tests>wait.short:
> > > >      shutdown = destroy
> > > >
> > > > only host_os>linux
> > > Jiří , I pasted above above example into demo.cfg and ran it via
> > > cartesian parser then I got the error "__main__.ParserError: 'variants'
> > > is not allowed inside a conditional block
> > > (libvirt/tests/cfg/demo.cfg:4)", any wrong with me? thanks.
> > > 
> > > 
> > 
> 
> 

_______________________________________________
Virt-test-devel mailing list
Virt-test-devel@redhat.com
https://www.redhat.com/mailman/listinfo/virt-test-devel

  reply	other threads:[~2013-05-03 15:54 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-29 17:14 [virt-test] virt: Cartesian config update v3 Jiří Župka
2013-03-29 17:14 ` [virt-test][PATCH 1/7] virt: Make Cartesian config more verbose Jiří Župka
2013-03-29 17:14 ` [virt-test][PATCH 2/7] virt: Update cartesian config unittest Jiří Župka
2013-03-29 17:14 ` [virt-test][PATCH 3/7] virt: Adds formal definition for Cartesian config Jiří Župka
2013-03-29 17:14 ` [virt-test][PATCH 4/7] virt: Adds named variants to " Jiří Župka
2013-04-01 14:30   ` Eduardo Habkost
2013-04-04 16:14     ` Jiri Zupka
2013-04-16  6:50   ` Alex Jia
2013-04-16 12:03     ` Jiri Zupka
2013-04-16 14:16       ` Jiri Zupka
2013-04-17  2:47         ` Alex Jia
2013-05-03 15:54           ` Jiri Zupka [this message]
2013-05-16  9:35             ` virt: New version " Jiri Zupka
2013-03-29 17:14 ` [virt-test][PATCH 5/7] virt: Update Cartesian config unittest named variants Jiří Župka
2013-04-01 14:21   ` Eduardo Habkost
2013-03-29 17:14 ` [virt-test][PATCH 6/7] virt: Adds possibility filter defaults variant from variants Jiří Župka
2013-04-01 14:40   ` Eduardo Habkost
2013-03-29 17:14 ` [virt-test][PATCH 7/7] virt: Update Cartesian config unittest default variants Jiří Župka

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1303043389.9050536.1367596482456.JavaMail.root@redhat.com \
    --to=jzupka@redhat.com \
    --cc=ehabkost@redhat.com \
    --cc=kvm-autotest@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=lmr@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=virt-test-devel@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox