Skip to content

ci: restore explicit Radius environment setup - #2657

Merged
DariuszPorowski merged 1 commit into
edgefrom
dp/able-chipmunk-salmon
Aug 28, 2026
Merged

ci: restore explicit Radius environment setup#2657
DariuszPorowski merged 1 commit into
edgefrom
dp/able-chipmunk-salmon

Conversation

@DariuszPorowski

@DariuszPorowski DariuszPorowski commented Aug 28, 2026

Copy link
Copy Markdown
Member

Summary

Restore explicit resource group, workspace, and environment initialization in the sample-test workflow.

Reason for change

The edge rad install kubernetes command now installs only the control plane. It no longer creates the default resource group or environment, and no longer accepts --preview, causing all sample-test matrix jobs to fail.

ref: radius-project/radius#12829

How to test

  • yamllint -d '{extends: default, rules: {line-length: disable, document-start: disable, truthy: {check-keys: false}}}' .github/workflows/test.yaml
  • actionlint .github/workflows/test.yaml reports only the workflow's existing shellcheck findings; no new findings were introduced.

File change summary

File Summary of change
.github/workflows/test.yaml Create and select the default group and environment explicitly; use preview only for commands that continue to support it.

Signed-off-by: Dariusz Porowski <3431813+DariuszPorowski@users.noreply.github.com>
@DariuszPorowski
DariuszPorowski requested review from a team as code owners August 28, 2026 16:10
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@DariuszPorowski DariuszPorowski self-assigned this Aug 28, 2026
@DariuszPorowski
DariuszPorowski merged commit 3617ee7 into edge Aug 28, 2026
7 checks passed
@DariuszPorowski
DariuszPorowski deleted the dp/able-chipmunk-salmon branch August 28, 2026 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants