From b7778a10453159aae8a555fa6fe77b45865ffcbf Mon Sep 17 00:00:00 2001
From: Karl Rister <krister@redhat.com>
Date: Mon, 17 Jun 2024 12:55:45 -0500
Subject: [PATCH 1/3] remove extraneous file write

---
 rickshaw-run | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/rickshaw-run b/rickshaw-run
index c4d541ce..305463b1 100755
--- a/rickshaw-run
+++ b/rickshaw-run
@@ -579,10 +579,6 @@ sub workshop_build_image {
         printf "skip_update was not defined, so setting to false\n";
         $skip_update = "false";
     }
-    if (put_json_file($cs_conf_file, \%cs_conf) > 0) {
-        printf "workshop_build_image(): put_json_file(): %s: add failed\n", $cs_conf_file;
-        exit 1;
-    }
     my $proj;
     if (defined $run{'reg-proj'}) {
         $proj = $run{'reg-host'} . "/" . $run{'reg-proj'};

From 1b8e2702028b7d7545efbe4aa45464e2e0461ff9 Mon Sep 17 00:00:00 2001
From: Karl Rister <krister@redhat.com>
Date: Mon, 17 Jun 2024 12:57:24 -0500
Subject: [PATCH 2/3] correct comment typo

---
 rickshaw-run | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/rickshaw-run b/rickshaw-run
index 305463b1..b9ac4ebe 100755
--- a/rickshaw-run
+++ b/rickshaw-run
@@ -910,7 +910,7 @@ sub source_container_image {
     build_reqs(\@requirements, $userenv, $benchmark);
 
     # First build a workshop-cmd args containing: (userenv_arg, req_args, tag), starting with the base userenv only (has no req_args),
-    # then each additional item in the list is a userenv arg mathcing the md5sum of the previous image, plus one more requirement.
+    # then each additional item in the list is a userenv arg matching the md5sum of the previous image, plus one more requirement.
     # Keep adding to this list until @requirements is empty.
     my @workshop_args;
     my $userenv_arg;

From f71bacfac45cfbde9c443c491087cf02cfd465c0 Mon Sep 17 00:00:00 2001
From: Karl Rister <krister@redhat.com>
Date: Mon, 17 Jun 2024 12:57:53 -0500
Subject: [PATCH 3/3] modify the image builder to not include the quay image
 expiration label in the tag hash calculation

---
 rickshaw-run | 43 ++++++++++++++++++++++++++-----------------
 1 file changed, 26 insertions(+), 17 deletions(-)

diff --git a/rickshaw-run b/rickshaw-run
index b9ac4ebe..69ae812f 100755
--- a/rickshaw-run
+++ b/rickshaw-run
@@ -931,7 +931,32 @@ sub source_container_image {
             $req_arg = shift(@requirements);
             $skip_update = "true";
         }
+
+        # keep separate cs_conf_file contents before and after md5
+        # calculation so that the quay.io image expiration value is
+        # not factored into the image hash
+        %cs_conf = (
+                'workshop' => {
+                    'schema' => {
+                        'version' => '2020.04.30'
+                    }
+                },
+                'config' => {
+                    'entrypoint' => [ "/bin/sh", "-c", "/usr/local/bin/bootstrap" ],
+                    'envs' => [ 'TOOLBOX_HOME=/opt/toolbox' ]
+                }
+            );
+        if (put_json_file($cs_conf_file, \%cs_conf) > 0) {
+            printf "put_json_file(): initial %s: failed\n", $cs_conf_file;
+            exit 1;
+        }
         my $tag = calc_image_md5($workshop_base_cmd, $userenv_arg, $req_arg, $container_arch);
+        $cs_conf{'config'}{'labels'} = [ 'quay.expires-after=' . $quay_image_expiration ];
+        if (put_json_file($cs_conf_file, \%cs_conf) > 0) {
+            printf "put_json_file(): update %s: failed\n", $cs_conf_file;
+            exit 1;
+        }
+
         my %args = ( 'userenv' => $userenv_arg, 'reqs' => $req_arg, 'tag' => $tag, 'skip-update' => $skip_update );
         push (@workshop_args, \%args);
 
@@ -2830,25 +2855,9 @@ if ($run{'reg-auth'} eq "") {
     printf "Disabling registry authorization due to empty 'reg-auth' variable\n";
     $skip_registry_auth = 1;
 }
-$cs_conf_file = $config_dir . "/cs-conf.json";
-%cs_conf = (
-                'workshop' => {
-                    'schema' => {
-                        'version' => '2020.04.30'
-                    }
-                },
-                'config' => {
-                    'entrypoint' => [ "/bin/sh", "-c", "/usr/local/bin/bootstrap" ],
-                    'labels' => [ 'quay.expires-after=' . $quay_image_expiration ],
-                    'envs' => [ 'TOOLBOX_HOME=/opt/toolbox' ]
-                }
-            );
-if (put_json_file($cs_conf_file, \%cs_conf) > 0) {
-    printf "put_json_file(): %s: failed\n", $cs_conf_file;
-    exit 1;
-}
 build_test_order();
 prepare_bench_tool_engines();
+$cs_conf_file = $config_dir . "/cs-conf.json";
 print "Preparing userenvs:\n";
 debug_log (sprintf "image_ids (before):\n" . Dumper \%image_ids);
 foreach my $bench_or_tool (sort (keys %image_ids)) {