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: Re: virt: New version Cartesian config
Date: Thu, 16 May 2013 05:35:49 -0400 (EDT)	[thread overview]
Message-ID: <764312958.2761418.1368696949647.JavaMail.root@redhat.com> (raw)
In-Reply-To: <1303043389.9050536.1367596482456.JavaMail.root@redhat.com>

Hi,
  I have sent email about new Cart config update few days ago but nobody did  respond to on it.
Could you look at new cart config? Mainly Eduardo.

Have a nice day,
  Jiří Župka

----- Original Message -----
> 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-16  9:35 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           ` virt: New version " Jiri Zupka
2013-05-16  9:35             ` Jiri Zupka [this message]
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=764312958.2761418.1368696949647.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