Various fixes for long-running tests

[willdavsmith]

Description

• Updating Kubernetes version in AKS cluster creation step
• Changed functionality to delete all namespaces except specified: [Long Running Tests] Remove Hardcoded List of Test Namespaces in Cleanup Step #7966
• Updated comment for long-running-test inputs to make it more clear

Type of change

• This pull request fixes a bug in Radius and has an approved issue (issue link required).
• This pull request adds or changes features of Radius and has an approved issue (issue link required).
• 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).

Fixes: #issue_number

Contributor checklist

Please verify that the PR meets the following requirements, where applicable:

• An overview of proposed schema changes is included in a linked GitHub issue.
  • Yes
  • Not applicable
• A design document PR is created in the design-notes repository, if new APIs are being introduced.
  • Yes
  • Not applicable
• The design document has been reviewed and approved by Radius maintainers/approvers.
  • Yes
  • Not applicable
• A PR for the samples repository is created, if existing samples are affected by the changes in this PR.
  • Yes
  • Not applicable
• A PR for the documentation repository is created, if the changes in this PR affect the documentation or any user facing updates are made.
  • Yes
  • Not applicable
• A PR for the recipes repository is created, if existing recipes are affected by the changes in this PR.
  • Yes
  • Not applicable

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.89%. Comparing base (bbb1d24) to head (066a530).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8680   +/-   ##
=======================================
  Coverage   59.88%   59.89%           
=======================================
  Files         596      596           
  Lines       40512    40512           
=======================================
+ Hits        24261    24264    +3     
+ Misses      14423    14422    -1     
+ Partials     1828     1826    -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[radius-functional-tests]

Radius functional test overview

🔍 Go to test action run

Name	Value
Repository	willdavsmith/radius
Commit ref	5d176de
Unique ID	funca7508d9228
Image tag	pr-funca7508d9228

Click here to see the list of tools in the current test run

• gotestsum 1.12.0
• KinD: v0.20.0
• Dapr:
• Azure KeyVault CSI driver: 1.4.2
• Azure Workload identity webhook: 1.3.0
• Bicep recipe location ghcr.io/radius-project/dev/test/testrecipes/test-bicep-recipes/<name>:pr-funca7508d9228
• Terraform recipe location http://tf-module-server.radius-test-tf-module-server.svc.cluster.local/<name>.zip (in cluster)
• applications-rp test image location: ghcr.io/radius-project/dev/applications-rp:pr-funca7508d9228
• dynamic-rp test image location: ghcr.io/radius-project/dev/dynamic-rp:pr-funca7508d9228
• controller test image location: ghcr.io/radius-project/dev/controller:pr-funca7508d9228
• ucp test image location: ghcr.io/radius-project/dev/ucpd:pr-funca7508d9228
• deployment-engine test image location: ghcr.io/radius-project/deployment-engine:latest

Test Status

⌛ Building Radius and pushing container images for functional tests...
✅ Container images build succeeded
⌛ Publishing Bicep Recipes for functional tests...
✅ Recipe publishing succeeded
⌛ Starting corerp-cloud functional tests...
⌛ Starting ucp-cloud functional tests...
✅ corerp-cloud functional tests succeeded
✅ ucp-cloud functional tests succeeded

[kachawla]

Have we spent time investigating it before marking it skipped? If we are skipping the test, can you make sure that a task to unskip it is created and assigned as well?

[willdavsmith]

IMO this system is too complicated. Instead I think it's better to specify the namespaces we want to keep and delete everything else. Looks like there's already been some discussion about this: #7966

[willdavsmith]

I was unable to run this bicep deployment locally because of the outdated kubernetes version. This should make sure that next time we create a long-running test cluster we won't hit this issue.