Skip to content

Add checksum verification for SOPS in actions.yml#1257

Merged
stefanprodan merged 1 commit into
fluxcd:mainfrom
Nordix:fix/sops_action_update
Jun 30, 2026
Merged

Add checksum verification for SOPS in actions.yml#1257
stefanprodan merged 1 commit into
fluxcd:mainfrom
Nordix:fix/sops_action_update

Conversation

@ivonaest

Copy link
Copy Markdown

Issue for flux2: fluxcd/flux2#5958

Added checksum verification for SOPS based off of yq/action.yml

@ivonaest ivonaest requested a review from a team as a code owner June 30, 2026 10:24
@stefanprodan stefanprodan added the area/actions GitHub Actions related issues and pull requests label Jun 30, 2026
@stefanprodan

Copy link
Copy Markdown
Member

@ivonaest please do not use GitHub merge, use the CLI to rebase with upstream and force push, this PR should have a single commit. Thanks

@ivonaest ivonaest force-pushed the fix/sops_action_update branch from a3dde48 to 675f70f Compare June 30, 2026 11:02
@ivonaest

Copy link
Copy Markdown
Author

@ivonaest please do not use GitHub merge, use the CLI to rebase with upstream and force push, this PR should have a single commit. Thanks

Done now

@stefanprodan

Copy link
Copy Markdown
Member

Did you checked before pushing? 24 file changes!?

Signed-off-by: ivonaest <ivona.cvije@est.tech>
@ivonaest ivonaest force-pushed the fix/sops_action_update branch from 675f70f to c5da74e Compare June 30, 2026 12:19
@ivonaest

Copy link
Copy Markdown
Author

My mistake, only relevant code pushed now.

@stefanprodan stefanprodan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Thanks @ivonaest

@stefanprodan stefanprodan merged commit 61592ec into fluxcd:main Jun 30, 2026
14 checks passed
@ivonaest ivonaest deleted the fix/sops_action_update branch June 30, 2026 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/actions GitHub Actions related issues and pull requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants