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
next prev parent 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