Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2;
From:   Vignesh Raman <vignesh.raman@collabora.com>
To:     helen.koike@collabora.com, airlied@gmail.com, daniel@ffwll.ch
Cc:     david.heidelberg@collabora.com, sergi.blanch.torne@collabora.com,
        guilherme.gallo@collabora.com, daniels@collabora.com,
        gustavo.padovan@collabora.com, emma@anholt.net,
        robdclark@gmail.com, dri-devel@lists.freedesktop.org,
        linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org,
        virtualization@lists.linux-foundation.org,
        linux-arm-msm@vger.kernel.org
Subject: [PATCH v5 8/9] drm: ci: Add pipeline url to fails and flakes files
Date:   Thu, 19 Oct 2023 12:36:49 +0530
Message-Id: <20231019070650.61159-9-vignesh.raman@collabora.com>
In-Reply-To: <20231019070650.61159-1-vignesh.raman@collabora.com>
References: <20231019070650.61159-1-vignesh.raman@collabora.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: bulk

Add or update pipeline url in the fails and flakes files.
It helps to check the pipeline where the failure was seen
and check artifacts for failure cases.

Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
---

v5:
  - Added a new patch in the series to add pipeline url to fails and flakes files.

---
 drivers/gpu/drm/ci/xfails/update-xfails.py | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/gpu/drm/ci/xfails/update-xfails.py b/drivers/gpu/drm/ci/xfails/update-xfails.py
index c401004ffd45..651f804721f2 100755
--- a/drivers/gpu/drm/ci/xfails/update-xfails.py
+++ b/drivers/gpu/drm/ci/xfails/update-xfails.py
@@ -63,6 +63,10 @@ def remove_unit_test_if_present(lines, unit_test_name):
         return
     lines[:] = [line for line in lines if unit_test_name not in line]
 
+def add_pipeline_url(lines, pipelines_urls):
+    for url in pipelines_urls:
+        pipeline_url = "# " + url + "\n"
+        lines.insert(0, pipeline_url)
 
 def add_unit_test_if_not_present(lines, unit_test_name, file_name):
     # core_getversion is mandatory
@@ -185,9 +189,11 @@ def main(pipelines_urls, only_flakes):
         flakes_txt.sort()
 
         if fails_txt != fails_txt_original:
+            add_pipeline_url(fails_txt, pipelines_urls)
             save_file(fails_txt, fails_txt_path)
             print_diff(fails_txt_original, fails_txt, os.path.basename(fails_txt_path))
         if flakes_txt != flakes_txt_original:
+            add_pipeline_url(flakes_txt, pipelines_urls)
             save_file(flakes_txt, flakes_txt_path)
             print_diff(flakes_txt_original, flakes_txt, os.path.basename(flakes_txt_path))
 
-- 
2.40.1