autobuild now building samba with cluster support! (Re: [SCM] Samba Shared Repository - branch master updated)

[Michael Adam]

Hi,

I wanted to point out that our autobuild will now protect us from
build regression of the clustered samba build with both the
autoconf and the waf build.

I achieved this by _copying_ the ctdb headers from current ctdb
master branch to the samba repo under ./ctdb/include/, a directory
that will never automatically be searched for headers, and
adding two new targets to autobuild. These targets build samba
with cluster support (using the newly added headers), and then
as a test only check whether "smbd -b" prints "CLUSTER_SUPPORT".

So this should not extend the overall autobuild time noticeably.

I am aware of the fact that copying the headers is not optimal
design, but retrieving them over the net via a ceckout or
download seemed too error prone to me. So this is what worked
for most easily for a start.


Let me share some ideas I have to drive this even further:

(1) I intend to extend this to build a full ctdb
and add a clustered setup to our selftest, so that we can
also do runtime tests with clustering in make test and in
autobuild:

I did already work on it a bit in late 2011 but got distracted.
Here is my branch of what was partially already working:

http://gitweb.samba.org/?p=obnox/samba/samba-obnox.git;a=shortlog;h=refs/heads/master-ctdbtest-wip

This will not apply to current master, but the ideas should
be applicable.

(2) In the long run I want to propose to move ctdb master into
the samba repo, so that samba and ctdb can easily be tested
togehter and both ctdb and samba commits can't get pushed
unless the clustered build and runtime tests survive.
The releases should of course stay independent, just as
tdb, talloc, tevent and friends are released independently.

Cheers - Michael


Michael Adam wrote:
> The branch, master has been updated
>        via  732f28b autobuild: run "samba3-ctdb" by default
>        via  80921de autobuild: run "samba-ctdb" by default to catch dbwrap_ctdb regressions
>        via  a0f6014 autobuild: add a target "samba3-ctdb" that builds samba3 (autoconf) with cluster support
>        via  18ce23b autobuild: add a target "samba-ctdb" that builds samba (waf) with cluster support
>        via  f9b722d autobuild: rename the "samba4-libs" target to "samba-libs"
>        via  a24d78d autobuild: rename the "samba4" target to "samba"
>        via  a518648 Add current headers from ctdb master.
>       from  cea3bdb wintest: enable dns forwarding for internal dns
> 
> http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master
> 
> 
> - Log -----------------------------------------------------------------
> commit 732f28b85e34cdba5cc6dd8df8e35b7132b3ac5c
> Author: Michael Adam <obnox at samba.org>
> Date:   Wed Jun 20 14:16:13 2012 +0200
> 
>     autobuild: run "samba3-ctdb" by default
>     
>     Autobuild-User(master): Michael Adam <obnox at samba.org>
>     Autobuild-Date(master): Fri Jun 22 01:46:13 CEST 2012 on sn-devel-104
> 
> commit 80921deae0e0ef282ed53ad8dcb04c72792ae132
> Author: Michael Adam <obnox at samba.org>
> Date:   Wed Jun 20 14:15:34 2012 +0200
> 
>     autobuild: run "samba-ctdb" by default to catch dbwrap_ctdb regressions
> 
> commit a0f601451785686ba45f3be5c7ca9b5cf3b86113
> Author: Michael Adam <obnox at samba.org>
> Date:   Wed Jun 20 14:09:09 2012 +0200
> 
>     autobuild: add a target "samba3-ctdb" that builds samba3 (autoconf) with cluster support
>     
>     This is achieved by using the ctdb header files copied in from current ctdb
>     master. Tests are not run. For this we would need to have a full ctdb built and
>     configured.
> 
> commit 18ce23b31d8232bc2be871982bb92e296f96ba7f
> Author: Michael Adam <obnox at samba.org>
> Date:   Wed Jun 20 14:05:50 2012 +0200
> 
>     autobuild: add a target "samba-ctdb" that builds samba (waf) with cluster support
>     
>     This is achieved by using the ctdb header files copied in from current ctdb
>     master. Tests are not run. For this we would need to have a full ctdb built and
>     configured.
> 
> commit f9b722d50e589c9f2a47f3892987902fda7bab4c
> Author: Michael Adam <obnox at samba.org>
> Date:   Wed Jun 20 13:23:04 2012 +0200
> 
>     autobuild: rename the "samba4-libs" target to "samba-libs"
> 
> commit a24d78d344b581f4efa1b78d334f4c6ada882a50
> Author: Michael Adam <obnox at samba.org>
> Date:   Wed Jun 20 13:22:18 2012 +0200
> 
>     autobuild: rename the "samba4" target to "samba"
> 
> commit a518648beaa96765769ee6d3d0c9a62c2ca19ab2
> Author: Michael Adam <obnox at samba.org>
> Date:   Wed Jun 20 12:01:32 2012 +0200
> 
>     Add current headers from ctdb master.
>     
>     (ctdb.h, ctdb_protocol.h, ctdb_private.h, ctdb_typesafe_cb.h, ctdb_client.h)
>     
>     This will allow us to check for build regressions against ctdb from autobuild.
> 
> -----------------------------------------------------------------------
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 206 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20120622/67c9ee54/attachment.pgp>