From 170e17a67390c09871de3f05866ecac8ede8b23d Mon Sep 17 00:00:00 2001
From: Krzesimir Nowak <knowak@microsoft.com>
Date: Thu, 28 Sep 2023 08:02:25 +0200
Subject: [PATCH] ci-automation/image-changes: Second parameter is not optional

---
 ci-automation/image_changes.sh | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/ci-automation/image_changes.sh b/ci-automation/image_changes.sh
index 554bd1122a..7303a000bf 100644
--- a/ci-automation/image_changes.sh
+++ b/ci-automation/image_changes.sh
@@ -19,7 +19,7 @@
 # INPUT:
 #
 #   1. Architecture (ARCH) of the TARGET OS image ("arm64", "amd64").
-#   2. What to compare against, can be "last-release" or "last-nightly".
+#   2. What to compare against, must be either "last-release" or "last-nightly".
 #
 # OPTIONAL INPUT:
 #
@@ -33,9 +33,10 @@
 # variables are not spilled into the caller.
 function image_changes() (
     set -euo pipefail
+    local arch what
 
-    local arch=${1}; shift
-    local what=${1-last-release}; shift
+    arch=${1}; shift
+    what=${1}; shift
 
     local -a package_diff_env package_diff_params
     local -a size_changes_env size_changes_params