From e506a323b65a2dfc42fd6134c69b6e298ff3fb43 Mon Sep 17 00:00:00 2001 From: Young Bu Park Date: Tue, 23 Jan 2024 11:04:39 -0800 Subject: [PATCH] Avoid docker debian image ratelimitting issue in long running test (#7067) # Description `Test_Run_Logger` uses `debian` image for testing. This can encounter the rate limit problem during long-running test. This change uses magpie image to prevent this failure. ## Type of change - This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Radius (issue link optional). Signed-off-by: Young Bu Park --- test/functional/shared/cli/cli_test.go | 2 ++ .../shared/cli/testdata/corerp-kubernetes-cli-run.bicep | 5 ++++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/test/functional/shared/cli/cli_test.go b/test/functional/shared/cli/cli_test.go index ae38a5db3f..b832e7b7fa 100644 --- a/test/functional/shared/cli/cli_test.go +++ b/test/functional/shared/cli/cli_test.go @@ -294,6 +294,8 @@ func Test_Run_Logger(t *testing.T) { filepath.Join(cwd, template), "--application", applicationName, + "--parameters", + functional.GetMagpieImage(), } // 'rad run' streams logs until canceled by the user. This is why we can't 'just' run the command in diff --git a/test/functional/shared/cli/testdata/corerp-kubernetes-cli-run.bicep b/test/functional/shared/cli/testdata/corerp-kubernetes-cli-run.bicep index 9e3a1531ce..9a5c4f4396 100644 --- a/test/functional/shared/cli/testdata/corerp-kubernetes-cli-run.bicep +++ b/test/functional/shared/cli/testdata/corerp-kubernetes-cli-run.bicep @@ -2,13 +2,16 @@ import radius as radius param application string +@description('Specifies the image to be deployed.') +param magpieimage string + resource container 'Applications.Core/containers@2023-10-01-preview' = { name: 'k8s-cli-run-logger' location: 'global' properties: { application: application container: { - image: 'debian' + image: magpieimage command: ['/bin/sh'] // The test looks for this specific output, keep in sync with the CLI run test!