Commit 6e9b51c5 authored by Karol Latecki's avatar Karol Latecki Committed by Tomasz Zawadzki
Browse files

autorun_post: exit with error on lcov and genhtml errors 



These were raising exceptions on errors but the
return code was still zero, masking issues in builds
which were actually failed.

Change-Id: Id441c9b9f913a0cf86943349d38ee6eefb4cd5e1
Signed-off-by: default avatarKarol Latecki <karol.latecki@nutanix.com>
Reviewed-on: https://review.spdk.io/c/spdk/spdk/+/25677


Reviewed-by: default avatarMichal Berger <michal.berger@nutanix.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz@tzawadzki.com>
Tested-by: default avatarTomasz Zawadzki <tomasz@tzawadzki.com>
Reviewed-by: default avatarKonrad Sztyber <ksztyber@nvidia.com>
parent 1c7892a7

autorun_post.py

+2 −1
Original line number Diff line number Diff line
@@ -83,13 +83,14 @@ def generateCoverageReport(output_dir, repo_dir): 
    except subprocess.CalledProcessError as e:

        print("lcov failed")

        print(e)

        return

        sys.exit(1)



    try:

        subprocess.check_call(genhtml)

    except subprocess.CalledProcessError as e:

        print("genhtml failed")

        print(e)

        sys.exit(1)

    for f in covfiles:

        os.remove(f)