All of lore.kernel.org
 help / color / mirror / Atom feed
From: Doug Goldstein <cardoe@cardoe.com>
To: Wei Liu <wei.liu2@citrix.com>
Cc: Keir Fraser <keir@xen.org>,
	Ian Campbell <ian.campbell@citrix.com>, Tim Deegan <tim@xen.org>,
	Ian Jackson <ian.jackson@eu.citrix.com>,
	xen-devel@lists.xen.org, Jan Beulich <jbeulich@suse.com>
Subject: Re: [PATCH] travis: add initial Travis CI script to do builds
Date: Fri, 5 Feb 2016 09:19:30 -0600	[thread overview]
Message-ID: <56B4BD82.8000207@cardoe.com> (raw)
In-Reply-To: <20160205150911.GB13808@citrix.com>


[-- Attachment #1.1: Type: text/plain, Size: 2395 bytes --]

On 2/5/16 9:09 AM, Wei Liu wrote:
> On Fri, Feb 05, 2016 at 08:48:49AM -0600, Doug Goldstein wrote:
>> This is just suppose to do a simple compile test on Travis CI. Currently
>> due to linux86 (bcc/bin86/dev86) not being whitelisted the tools cannot
>> be built.
>>
>> Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
>> ---
>>
>> So this will work great if we get a regular cronjob or a post-receive hook
>> to push to GitHub (https://github.com/xen-project/xen) since Travis CI provides
>> free integration with public repositories. So an example run of this is
>> available at https://travis-ci.org/cardoe/xen/builds/107246160 if this gets
>> merged I will setup a xen-project space under Travis for us. Currently its
>> under my namespace because this file only exists in my branches.
>>
>> The goal here is not to replace osstest by any means but to augment it by
>> providing some easy to do build tests on every revision and reporting back. It
>> should be possible in the future to potentially tie this into osstest to
>> allow this to build and if the tree for some reason failed to build not kick
>> off a test flight on osstest. Just some thoughts, I know I'm getting a bit
>> ahead of myself.
>>
> 
> I think adding travis CI is a good idea. It would be valuable if
> individual developers can run their series through travis before
> submitting. Thanks for your work.
> 
>> Other future tweaks I've got are some simple dom0 boot tests using qemu to
>> boot Xen and then boot a kernel and see that it came up. I've also got some
>> scripts to check the code base for code style consistency using clang-format.
>>
>> If you're curious to see the output of these tests come to #xentest on
>> Freenode.
>>
> 
> The purpose of #xentest is (or was) to coordinate test day activities,
> I would rather use a different channel.
> 
> However, there has been idea floating that test day should be organised
> in developer channels, and there is no test day activity at the moment,
> so I think it would fine for now to use it for travis.
> 
> A stupid question: does this mean if I fork xen.git on github, travis
> will pick up my fork and spam #xentest?
> 
> Wei.
> 

Hmm. Good point. It will. I'll drop this part of the patch. There might
be a way to properly filter this but I'll look into that as a follow on.

-- 
Doug Goldstein


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 959 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

  reply	other threads:[~2016-02-05 15:19 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-05 14:48 [PATCH] travis: add initial Travis CI script to do builds Doug Goldstein
2016-02-05 15:09 ` Wei Liu
2016-02-05 15:19   ` Doug Goldstein [this message]
2016-02-05 15:24     ` Wei Liu
2016-02-05 16:33 ` Ian Campbell
2016-02-05 16:41   ` Doug Goldstein

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=56B4BD82.8000207@cardoe.com \
    --to=cardoe@cardoe.com \
    --cc=ian.campbell@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=jbeulich@suse.com \
    --cc=keir@xen.org \
    --cc=tim@xen.org \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xen.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.