# Copyright Andrew Tridgell 2010
# released under GNU GPL v3 or later
-from subprocess import Popen, PIPE
-import os, signal, tarfile, sys, time
+from subprocess import call, check_call,Popen, PIPE
+import os, tarfile, sys, time
from optparse import OptionParser
-
+import smtplib
+from email.mime.text import MIMEText
samba_master = os.getenv('SAMBA_MASTER', 'git://git.samba.org/samba.git')
samba_master_ssh = os.getenv('SAMBA_MASTER_SSH', 'git+ssh://git.samba.org/data/git/samba.git')
cleanup_list = []
-os.putenv('CC', "ccache gcc")
+builddirs = {
+ "samba3" : "source3",
+ "samba4" : ".",
+ "ldb" : "lib/ldb",
+ "tdb" : "lib/tdb",
+ "talloc" : "lib/talloc",
+ "replace" : "lib/replace",
+ "tevent" : "lib/tevent",
+ "pidl" : "pidl",
+ "pass" : ".",
+ "fail" : ".",
+ "retry" : "."
+ }
+
+defaulttasks = [ "samba3", "samba4", "ldb", "tdb", "talloc", "replace", "tevent", "pidl" ]
tasks = {
- "source3" : [ "./autogen.sh",
- "./configure.developer ${PREFIX}",
- "make basics",
- "make -j 4 everything", # don't use too many processes
- "make install",
- "TDB_NO_FSYNC=1 make test" ],
-
- "source4" : [ "./autogen.sh",
- "./configure.developer ${PREFIX}",
- "make -j",
- "make install",
- "TDB_NO_FSYNC=1 make test" ],
-
- "source4/lib/ldb" : [ "./autogen-waf.sh",
- "./configure --enable-developer -C ${PREFIX}",
- "make -j",
- "make install",
- "make test" ],
-
- "lib/tdb" : [ "./autogen-waf.sh",
- "./configure --enable-developer -C ${PREFIX}",
- "make -j",
- "make install",
- "make test" ],
-
- "lib/talloc" : [ "./autogen-waf.sh",
- "./configure --enable-developer -C ${PREFIX}",
- "make -j",
- "make install",
- "make test" ],
-
- "lib/replace" : [ "./autogen-waf.sh",
- "./configure --enable-developer -C ${PREFIX}",
- "make -j",
- "make install",
- "make test" ],
-
- "lib/tevent" : [ "./autogen-waf.sh",
- "./configure --enable-developer -C ${PREFIX}",
- "make -j",
- "make install",
- "make test" ],
+ "samba3" : [ ("autogen", "./autogen.sh", "text/plain"),
+ ("configure", "./configure.developer ${PREFIX}", "text/plain"),
+ ("make basics", "make basics", "text/plain"),
+ # we split 'make -j 4', 'make bin/smbtorture4' and 'make -j 4 everything'
+ # because it makes it much easier to find errors.
+ ("make", "make -j 4", "text/plain"), # don't use too many processes
+ ("make bin/smbtorture4", "make -j 4 bin/smbtorture4", "text/plain"),
+ ("make everything", "make -j 4 everything", "text/plain"),
+ ("install", "make install", "text/plain"),
+ ("test", "TDB_NO_FSYNC=1 make test FAIL_IMMEDIATELY=1", "text/plain"),
+ ("check-clean-tree", "../script/clean-source-tree.sh", "text/plain"),
+ ("clean", "make clean", "text/plain") ],
+
+ # We have 'test' before 'install' because, 'test' should work without 'install'
+ "samba4" : [ ("configure", "./configure.developer ${PREFIX} --with-selftest-prefix=./bin/ab", "text/plain"),
+ ("make", "make -j", "text/plain"),
+ ("test", "TDB_NO_FSYNC=1 make test FAIL_IMMEDIATELY=1", "text/plain"),
+ ("install", "make install", "text/plain"),
+ ("check-clean-tree", "script/clean-source-tree.sh", "text/plain"),
+ ("clean", "make clean", "text/plain") ],
+
+ "ldb" : [ ("configure", "./configure --enable-developer -C ${PREFIX}", "text/plain"),
+ ("make", "make -j", "text/plain"),
+ ("install", "make install", "text/plain"),
+ ("test", "TDB_NO_FSYNC=1 make test", "text/plain"),
+ ("check-clean-tree", "../../script/clean-source-tree.sh", "text/plain"),
+ ("distcheck", "make distcheck", "text/plain"),
+ ("clean", "make clean", "text/plain") ],
+
+ # We don't use TDB_NO_FSYNC=1 here, because we want to test the transaction code
+ "tdb" : [ ("configure", "./configure --enable-developer -C ${PREFIX}", "text/plain"),
+ ("make", "make -j", "text/plain"),
+ ("install", "make install", "text/plain"),
+ ("test", "make test", "text/plain"),
+ ("check-clean-tree", "../../script/clean-source-tree.sh", "text/plain"),
+ ("distcheck", "make distcheck", "text/plain"),
+ ("clean", "make clean", "text/plain") ],
+
+ "talloc" : [ ("configure", "./configure --enable-developer -C ${PREFIX}", "text/plain"),
+ ("make", "make -j", "text/plain"),
+ ("install", "make install", "text/plain"),
+ ("test", "make test", "text/plain"),
+ ("check-clean-tree", "../../script/clean-source-tree.sh", "text/plain"),
+ ("distcheck", "make distcheck", "text/plain"),
+ ("clean", "make clean", "text/plain") ],
+
+ "replace" : [ ("configure", "./configure --enable-developer -C ${PREFIX}", "text/plain"),
+ ("make", "make -j", "text/plain"),
+ ("install", "make install", "text/plain"),
+ ("test", "make test", "text/plain"),
+ ("check-clean-tree", "../../script/clean-source-tree.sh", "text/plain"),
+ ("distcheck", "make distcheck", "text/plain"),
+ ("clean", "make clean", "text/plain") ],
+
+ "tevent" : [ ("configure", "./configure --enable-developer -C ${PREFIX}", "text/plain"),
+ ("make", "make -j", "text/plain"),
+ ("install", "make install", "text/plain"),
+ ("test", "make test", "text/plain"),
+ ("check-clean-tree", "../../script/clean-source-tree.sh", "text/plain"),
+ ("distcheck", "make distcheck", "text/plain"),
+ ("clean", "make clean", "text/plain") ],
+
+ "pidl" : [ ("configure", "perl Makefile.PL PREFIX=${PREFIX_DIR}", "text/plain"),
+ ("touch", "touch *.yp", "text/plain"),
+ ("make", "make", "text/plain"),
+ ("test", "make test", "text/plain"),
+ ("install", "make install", "text/plain"),
+ ("check-clean-tree", "../script/clean-source-tree.sh", "text/plain"),
+ ("clean", "make clean", "text/plain") ],
+
+ # these are useful for debugging autobuild
+ 'pass' : [ ("pass", 'echo passing && /bin/true', "text/plain") ],
+ 'fail' : [ ("fail", 'echo failing && /bin/false', "text/plain") ]
}
-def run_cmd(cmd, dir=".", show=None):
- cwd = os.getcwd()
- os.chdir(dir)
+retry_task = [ ( "retry",
+ '''set -e
+ git remote add -t master master %s
+ git fetch master
+ while :; do
+ sleep 60
+ git describe master/master > old_master.desc
+ git fetch master
+ git describe master/master > master.desc
+ diff old_master.desc master.desc
+ done
+ ''' % samba_master, "test/plain" ) ]
+
+def run_cmd(cmd, dir=".", show=None, output=False, checkfail=True):
if show is None:
show = options.verbose
if show:
print("Running: '%s' in '%s'" % (cmd, dir))
- ret = os.system(cmd)
- os.chdir(cwd)
- if ret != 0:
- raise Exception("FAILED %s: %d" % (cmd, ret))
+ if output:
+ return Popen([cmd], shell=True, stdout=PIPE, cwd=dir).communicate()[0]
+ elif checkfail:
+ return check_call(cmd, shell=True, cwd=dir)
+ else:
+ return call(cmd, shell=True, cwd=dir)
-class builder:
+
+class builder(object):
'''handle build of one directory'''
+
def __init__(self, name, sequence):
self.name = name
- self.dir = self.name
- if name == 'pass' or name == 'fail':
- self.dir = "."
+ self.dir = builddirs[name]
+
self.tag = self.name.replace('/', '_')
self.sequence = sequence
self.next = 0
- self.stdout_path = "%s/%s.stdout" % (testbase, self.tag)
- self.stderr_path = "%s/%s.stderr" % (testbase, self.tag)
- cleanup_list.append(self.stdout_path)
- cleanup_list.append(self.stderr_path)
+ self.stdout_path = "%s/%s.stdout" % (gitroot, self.tag)
+ self.stderr_path = "%s/%s.stderr" % (gitroot, self.tag)
+ if options.verbose:
+ print("stdout for %s in %s" % (self.name, self.stdout_path))
+ print("stderr for %s in %s" % (self.name, self.stderr_path))
run_cmd("rm -f %s %s" % (self.stdout_path, self.stderr_path))
self.stdout = open(self.stdout_path, 'w')
self.stderr = open(self.stderr_path, 'w')
cleanup_list.append(self.prefix)
os.makedirs(self.sdir)
run_cmd("rm -rf %s" % self.sdir)
- run_cmd("git clone --shared %s %s" % (gitroot, self.sdir))
+ run_cmd("git clone --shared %s %s" % (test_master, self.sdir), dir=test_master, show=True)
self.start_next()
def start_next(self):
print '%s: Completed OK' % self.name
self.done = True
return
- self.cmd = self.sequence[self.next].replace("${PREFIX}", "--prefix=%s" % self.prefix)
- print '%s: Running %s' % (self.name, self.cmd)
+ (self.stage, self.cmd, self.output_mime_type) = self.sequence[self.next]
+ self.cmd = self.cmd.replace("${PREFIX}", "--prefix=%s" % self.prefix)
+ self.cmd = self.cmd.replace("${PREFIX_DIR}", "%s" % self.prefix)
+# if self.output_mime_type == "text/x-subunit":
+# self.cmd += " | %s --immediate" % (os.path.join(os.path.dirname(__file__), "selftest/format-subunit"))
+ print '%s: [%s] Running %s' % (self.name, self.stage, self.cmd)
cwd = os.getcwd()
os.chdir("%s/%s" % (self.sdir, self.dir))
self.proc = Popen(self.cmd, shell=True,
self.next += 1
-class buildlist:
+class buildlist(object):
'''handle build of multiple directories'''
+
def __init__(self, tasklist, tasknames):
global tasks
self.tlist = []
self.tail_proc = None
- if tasknames == ['pass']:
- tasks = { 'pass' : [ '/bin/true' ]}
- if tasknames == ['fail']:
- tasks = { 'fail' : [ '/bin/false' ]}
+ self.retry = None
if tasknames == []:
- tasknames = tasklist
+ tasknames = defaulttasks
for n in tasknames:
b = builder(n, tasks[n])
self.tlist.append(b)
+ if options.retry:
+ self.retry = builder('retry', retry_task)
+ self.need_retry = False
def kill_kids(self):
+ if self.tail_proc is not None:
+ self.tail_proc.terminate()
+ self.tail_proc.wait()
+ self.tail_proc = None
+ if self.retry is not None:
+ self.retry.proc.terminate()
+ self.retry.proc.wait()
+ self.retry = None
for b in self.tlist:
if b.proc is not None:
+ run_cmd("killbysubdir %s > /dev/null 2>&1" % b.sdir, checkfail=False)
b.proc.terminate()
b.proc.wait()
b.proc = None
- if self.tail_proc is not None:
- self.tail_proc.terminate()
- self.tail_proc.wait()
- self.tail_proc = None
def wait_one(self):
while True:
continue
b.proc = None
return b
+ if options.retry:
+ ret = self.retry.proc.poll()
+ if ret is not None:
+ self.need_retry = True
+ self.retry = None
+ return None
if none_running:
return None
time.sleep(0.1)
def run(self):
while True:
b = self.wait_one()
+ if options.retry and self.need_retry:
+ self.kill_kids()
+ print("retry needed")
+ return (0, None, None, None, "retry")
if b is None:
break
if os.WIFSIGNALED(b.status) or os.WEXITSTATUS(b.status) != 0:
self.kill_kids()
- return (b.status, "%s: failed '%s' with status %d" % (b.name, b.cmd, b.status))
+ return (b.status, b.name, b.stage, b.tag, "%s: [%s] failed '%s' with status %d" % (b.name, b.stage, b.cmd, b.status))
b.start_next()
self.kill_kids()
- return (0, "All OK")
+ return (0, None, None, None, "All OK")
def tarlogs(self, fname):
tar = tarfile.open(fname, "w:gz")
for b in self.tlist:
tar.add(b.stdout_path, arcname="%s.stdout" % b.tag)
tar.add(b.stderr_path, arcname="%s.stderr" % b.tag)
+ if os.path.exists("autobuild.log"):
+ tar.add("autobuild.log")
tar.close()
def remove_logs(self):
def start_tail(self):
cwd = os.getcwd()
cmd = "tail -f *.stdout *.stderr"
- os.chdir(testbase)
+ os.chdir(gitroot)
self.tail_proc = Popen(cmd, shell=True)
os.chdir(cwd)
def find_git_root():
'''get to the top of the git repo'''
- cwd=os.getcwd()
- while os.getcwd() != '/':
+ p=os.getcwd()
+ while p != '/':
+ if os.path.isdir(os.path.join(p, ".git")):
+ return p
+ p = os.path.abspath(os.path.join(p, '..'))
+ return None
+
+
+def daemonize(logfile):
+ pid = os.fork()
+ if pid == 0: # Parent
+ os.setsid()
+ pid = os.fork()
+ if pid != 0: # Actual daemon
+ os._exit(0)
+ else: # Grandparent
+ os._exit(0)
+
+ import resource # Resource usage information.
+ maxfd = resource.getrlimit(resource.RLIMIT_NOFILE)[1]
+ if maxfd == resource.RLIM_INFINITY:
+ maxfd = 1024 # Rough guess at maximum number of open file descriptors.
+ for fd in range(0, maxfd):
try:
- os.stat(".git")
- ret = os.getcwd()
- os.chdir(cwd)
- return ret
- except:
- os.chdir("..")
+ os.close(fd)
+ except OSError:
pass
- os.chdir(cwd)
- return None
+ os.open(logfile, os.O_RDWR | os.O_CREAT)
+ os.dup2(0, 1)
+ os.dup2(0, 2)
+
+def write_pidfile(fname):
+ '''write a pid file, cleanup on exit'''
+ f = open(fname, mode='w')
+ f.write("%u\n" % os.getpid())
+ f.close()
+
def rebase_tree(url):
print("Rebasing on %s" % url)
+ run_cmd("git describe HEAD", show=True, dir=test_master)
run_cmd("git remote add -t master master %s" % url, show=True, dir=test_master)
run_cmd("git fetch master", show=True, dir=test_master)
if options.fix_whitespace:
run_cmd("git rebase --whitespace=fix master/master", show=True, dir=test_master)
else:
run_cmd("git rebase master/master", show=True, dir=test_master)
+ diff = run_cmd("git --no-pager diff HEAD master/master", dir=test_master, output=True)
+ if diff == '':
+ print("No differences between HEAD and master/master - exiting")
+ sys.exit(0)
+ run_cmd("git describe master/master", show=True, dir=test_master)
+ run_cmd("git describe HEAD", show=True, dir=test_master)
+ run_cmd("git --no-pager diff --stat HEAD master/master", show=True, dir=test_master)
def push_to(url):
print("Pushing to %s" % url)
if options.mark:
- run_cmd("EDITOR=script/commit_mark.sh git commit --amend -c HEAD", dir=test_master)
+ run_cmd("git config --replace-all core.editor script/commit_mark.sh", dir=test_master)
+ run_cmd("git commit --amend -c HEAD", dir=test_master)
+ # the notes method doesn't work yet, as metze hasn't allowed refs/notes/* in master
+ # run_cmd("EDITOR=script/commit_mark.sh git notes edit HEAD", dir=test_master)
run_cmd("git remote add -t master pushto %s" % url, show=True, dir=test_master)
run_cmd("git push pushto +HEAD:master", show=True, dir=test_master)
default=False, action="store_true")
parser.add_option("", "--fix-whitespace", help="fix whitespace on rebase",
default=False, action="store_true")
+parser.add_option("", "--retry", help="automatically retry if master changes",
+ default=False, action="store_true")
+parser.add_option("", "--email", help="send email to the given address on failure",
+ type='str', default=None)
+parser.add_option("", "--always-email", help="always send email, even on success",
+ action="store_true")
+parser.add_option("", "--daemon", help="daemonize after initial setup",
+ action="store_true")
+
+
+def email_failure(status, failed_task, failed_stage, failed_tag, errstr):
+ '''send an email to options.email about the failure'''
+ user = os.getenv("USER")
+ text = '''
+Dear Developer,
+
+Your autobuild failed when trying to test %s with the following error:
+ %s
+
+the autobuild has been abandoned. Please fix the error and resubmit.
+
+A summary of the autobuild process is here:
+
+ http://git.samba.org/%s/samba-autobuild/autobuild.log
+''' % (failed_task, errstr, user)
+
+ if failed_task != 'rebase':
+ text += '''
+You can see logs of the failed task here:
+
+ http://git.samba.org/%s/samba-autobuild/%s.stdout
+ http://git.samba.org/%s/samba-autobuild/%s.stderr
+
+or you can get full logs of all tasks in this job here:
+
+ http://git.samba.org/%s/samba-autobuild/logs.tar.gz
+
+The top commit for the tree that was built was:
+
+%s
+
+''' % (user, failed_tag, user, failed_tag, user, top_commit_msg)
+ msg = MIMEText(text)
+ msg['Subject'] = 'autobuild failure for task %s during %s' % (failed_task, failed_stage)
+ msg['From'] = 'autobuild@samba.org'
+ msg['To'] = options.email
+
+ s = smtplib.SMTP()
+ s.connect()
+ s.sendmail(msg['From'], [msg['To']], msg.as_string())
+ s.quit()
+
+def email_success():
+ '''send an email to options.email about a successful build'''
+ user = os.getenv("USER")
+ text = '''
+Dear Developer,
+
+Your autobuild has succeeded.
+
+'''
+
+ if options.keeplogs:
+ text += '''
+
+you can get full logs of all tasks in this job here:
+
+ http://git.samba.org/%s/samba-autobuild/logs.tar.gz
+
+''' % user
+
+ text += '''
+The top commit for the tree that was built was:
+
+%s
+''' % top_commit_msg
+
+ msg = MIMEText(text)
+ msg['Subject'] = 'autobuild success'
+ msg['From'] = 'autobuild@samba.org'
+ msg['To'] = options.email
+
+ s = smtplib.SMTP()
+ s.connect()
+ s.sendmail(msg['From'], [msg['To']], msg.as_string())
+ s.quit()
(options, args) = parser.parse_args()
-testbase = "%s/build.%u" % (options.testbase, os.getpid())
+if options.retry:
+ if not options.rebase_master and options.rebase is None:
+ raise Exception('You can only use --retry if you also rebase')
+
+testbase = "%s/b%u" % (options.testbase, os.getpid())
test_master = "%s/master" % testbase
gitroot = find_git_root()
if gitroot is None:
raise Exception("Failed to find git root")
+# get the top commit message, for emails
+top_commit_msg = run_cmd("git log -1", dir=gitroot, output=True)
+
try:
os.makedirs(testbase)
except Exception, reason:
raise Exception("Unable to create %s : %s" % (testbase, reason))
cleanup_list.append(testbase)
-try:
- run_cmd("rm -rf %s" % test_master)
- cleanup_list.append(test_master)
- run_cmd("git clone --shared %s %s" % (gitroot, test_master))
-except:
- cleanup()
- raise
+if options.daemon:
+ logfile = os.path.join(testbase, "log")
+ print "Forking into the background, writing progress to %s" % logfile
+ daemonize(logfile)
-try:
- if options.rebase is not None:
- rebase_tree(options.rebase)
- elif options.rebase_master:
- rebase_tree(samba_master)
- blist = buildlist(tasks, args)
- if options.tail:
- blist.start_tail()
- (status, errstr) = blist.run()
-except:
- cleanup()
- raise
+write_pidfile(gitroot + "/autobuild.pid")
+
+while True:
+ try:
+ run_cmd("rm -rf %s" % test_master)
+ cleanup_list.append(test_master)
+ run_cmd("git clone --shared %s %s" % (gitroot, test_master), show=True, dir=gitroot)
+ except:
+ cleanup()
+ raise
+
+ try:
+ try:
+ if options.rebase is not None:
+ rebase_tree(options.rebase)
+ elif options.rebase_master:
+ rebase_tree(samba_master)
+ except:
+ email_failure(-1, 'rebase', 'rebase', 'rebase', 'rebase on master failed')
+ sys.exit(1)
+ blist = buildlist(tasks, args)
+ if options.tail:
+ blist.start_tail()
+ (status, failed_task, failed_stage, failed_tag, errstr) = blist.run()
+ if status != 0 or errstr != "retry":
+ break
+ cleanup()
+ except:
+ cleanup()
+ raise
+
+cleanup_list.append(gitroot + "/autobuild.pid")
blist.kill_kids()
if options.tail:
if options.keeplogs:
blist.tarlogs("logs.tar.gz")
print("Logs in logs.tar.gz")
+ if options.always_email:
+ email_success()
blist.remove_logs()
cleanup()
print(errstr)
# something failed, gather a tar of the logs
blist.tarlogs("logs.tar.gz")
-blist.remove_logs()
+
+if options.email is not None:
+ email_failure(status, failed_task, failed_stage, failed_tag, errstr)
+
cleanup()
print(errstr)
print("Logs in logs.tar.gz")
-sys.exit(os.WEXITSTATUS(status))
+sys.exit(status)