ci: restore v0.60 sample environment initialization - #2663
Open
willdavsmith wants to merge 2 commits into
Open
Conversation
Signed-off-by: willdavsmith <willdavsmith@gmail.com>
Signed-off-by: willdavsmith <willdavsmith@gmail.com>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Restore explicit Radius environment initialization in the v0.60 sample-test workflow and correct release-channel selection for pull requests.
Reason for change
Fixes #2661.
Radius v0.60.2 installs only the control plane. The workflow still passed the removed installation
--previewflag for demo and assumed installation created the default resource group/environment for all samples.Adapt the initialization fix from #2657 to v0.60: create the default resource group and workspace, create the appropriate preview or legacy environment, and select it before deployment. Preserve the preview default recipe pack and existing legacy recipe registrations.
Release PRs also incorrectly compared their bare base branch against a pull-ref pattern, leaving the CLI version as
edge. Derive the release channel from the actual PR base branch instead.The separate issue-reporting permission failure is tracked in #2662 and is not changed here.
How to test
Test Samples (K3D)on the PR branch withversion: 0.60.2.RADIUS_VERSION=0.60, notedge.Completed validation: workflow run 34253711907 passed all three matrix jobs on commit
35b56643d2ef53641c2c26caccacf48c6eaa3f5fusing Radius 0.60.2. Local YAML parsing, modified shell syntax, eight event/version-selection cases, and diff checks also passed. PR-event and post-merge scheduled execution remain to be confirmed.File change summary
.github/workflows/test.yaml