Skip to content

Update workflow reference and permissions in YAML#301

Merged
masap merged 1 commit into
mainfrom
aalves08-patch-1
Jul 6, 2026
Merged

Update workflow reference and permissions in YAML#301
masap merged 1 commit into
mainfrom
aalves08-patch-1

Conversation

@aalves08

@aalves08 aalves08 commented Jul 3, 2026

Copy link
Copy Markdown
Member

Updates were done on the reusable workflow https://github.com/rancher/dashboard/blob/master/.github/workflows/build-extension-charts.yml in terms of security hardening, reducing the set of permissions needed to the minimum required for the workflow to run.

With that change in mind, we've done testing on elemental-ui with this new version https://github.com/rancher/elemental-ui/blob/main/.github/workflows/build-extension-charts.yml#L15 + https://github.com/rancher/elemental-ui/actions/runs/28600678629 and we will need to update all the official extensions repositories to consume this new version/hash of the workflow

Updates were done on the reusable workflow https://github.com/rancher/dashboard/blob/master/.github/workflows/build-extension-charts.yml in terms of security hardening, reducing the set of permissions needed to the minimum required for the workflow to run.

With that change in mind, we've done testing on `elemental-ui` with this new version https://github.com/rancher/elemental-ui/blob/main/.github/workflows/build-extension-charts.yml#L15 + https://github.com/rancher/elemental-ui/actions/runs/28600678629 and we will need to update all the official extensions repositories to consume this new version/hash of the workflow
@aalves08 aalves08 requested a review from masap July 3, 2026 09:26
@aalves08 aalves08 self-assigned this Jul 3, 2026

@masap masap left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you. I'll test this after merge.

@masap masap merged commit 6dd536c into main Jul 6, 2026
1 check passed
@masap masap deleted the aalves08-patch-1 branch July 6, 2026 01:11
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