Attachment 16809 Details for Bug 14841 – patch backported to 4.15 (only)

[patch] patch backported to 4.15 (only)

4-15-0001-autobuild-allow-AUTOBUILD_FAIL_IMMEDIATELY-0-say-fro.patch (text/plain), 1.67 KB, created by Andrew Bartlett on 2021-09-22 00:38:05 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andrew Bartlett

Created: 2021-09-22 00:38:05 UTC

Size: 1.67 KB

patch

obsolete

>From 3193ac33139336afada18b2836dbb49b4a9530ef Mon Sep 17 00:00:00 2001
>From: Andrew Bartlett <abartlet@samba.org>
>Date: Fri, 17 Sep 2021 16:43:00 +1200
>Subject: [PATCH] autobuild: allow AUTOBUILD_FAIL_IMMEDIATELY=0 (say from a
> gitlab variable)
>
>This allows making a push to do a full test ignoring errors without
>needing "HACK!!!" commits on top.
>
>Use like this:
>
>git push -o ci.variable='AUTOBUILD_FAIL_IMMEDIATELY=0'
>
>RN: Samba CI runs can now continue past the first error if AUTOBUILD_FAIL_IMMEDIATELY=0 is set
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=14841
>
>Signed-off-by: Andrew Bartlett <abartlet@samba.org>
>Reviewed-by: Michael Adam <obnox@samba.org
>Reviewed-by: Noel Power <npower@samba.org>
>(cherry picked from commit b81f6f3d71487085bb355392ce7f8eff2db5bb4d)
>---
> script/autobuild.py | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
>diff --git a/script/autobuild.py b/script/autobuild.py
>index 97731215282..ca9ffb93f68 100755
>--- a/script/autobuild.py
>+++ b/script/autobuild.py
>@@ -166,7 +166,6 @@ def format_option(name, value=None):
> 
> def make_test(
>         cmd='make testonly',
>-        FAIL_IMMEDIATELY=1,
>         INJECT_SELFTEST_PREFIX=1,
>         TESTS='',
>         include_envs=None,
>@@ -182,7 +181,13 @@ def make_test(
>         TESTS = (TESTS + ' ' + ' '.join(test_options)).strip()
> 
>     _options = []
>-    if FAIL_IMMEDIATELY:
>+
>+    # Allow getting a full CI with
>+    # git push -o ci.variable='AUTOBUILD_FAIL_IMMEDIATELY=0'
>+
>+    FAIL_IMMEDIATELY = os.getenv("AUTOBUILD_FAIL_IMMEDIATELY", "1")
>+
>+    if int(FAIL_IMMEDIATELY):
>         _options.append('FAIL_IMMEDIATELY=1')
>     if TESTS:
>         _options.append("TESTS='{}'".format(TESTS))
>-- 
>2.25.1
>

Flags:

npower: review+

Actions: View

Attachments on bug 14841: 16809 | 16810