From 7948760ffb32589f3ada79abb7e657d621a9a2d5 Mon Sep 17 00:00:00 2001
From: Antoine Musso <hashar@free.fr>
Date: Thu, 16 Mar 2023 17:50:35 +0100
Subject: [PATCH] Remove ignored success-url job attribute

Since Zuul 4.7.0:

> The following attributes are now ignored:
>
> * The ``report-build-page`` tenant configuration setting.
> * The ``success-url`` job attribute.
> * The ``failure-url`` job attribute.

Remove `success-url` from all jobs.

Change-Id: I66a03cbf2939d95f31781e7e1ff332699a752f99
---
 zuul-tests.d/python-jobs.yaml | 1 -
 zuul.d/js-jobs.yaml           | 4 ----
 2 files changed, 5 deletions(-)

diff --git a/zuul-tests.d/python-jobs.yaml b/zuul-tests.d/python-jobs.yaml
index 04d22526a..4d7e470fc 100644
--- a/zuul-tests.d/python-jobs.yaml
+++ b/zuul-tests.d/python-jobs.yaml
@@ -348,7 +348,6 @@
     # parenting on tox-docs as we need builded docs first and also this is
     # the main use-case for this role
     parent: tox-docs
-    success-url: docs/
     files:
       - roles/fetch-sphinx-tarball/.*
       - roles/tox/library/.*
diff --git a/zuul.d/js-jobs.yaml b/zuul.d/js-jobs.yaml
index 16bdedc18..d5f8e434f 100644
--- a/zuul.d/js-jobs.yaml
+++ b/zuul.d/js-jobs.yaml
@@ -93,7 +93,6 @@
          :default: dist
 
          Directory, relative to zuul_work_dir, holding build content.
-    success-url: npm/html/
     post-run: playbooks/javascript/tarball.yaml
 
 - job:
@@ -191,7 +190,6 @@
 
          Path to operate in.
     post-run: playbooks/tox/docs-post.yaml
-    success-url: docs/
     vars:
       js_build_command: docs
 
@@ -285,7 +283,6 @@
          :default: dist
 
          Directory, relative to zuul_work_dir, holding build content.
-    success-url: npm/html/
 
 - job:
     name: build-javascript-content-tarball
@@ -404,6 +401,5 @@
 
          Path to operate in.
     post-run: playbooks/tox/docs-post.yaml
-    success-url: docs/
     vars:
       npm_command: docs