Skip to content

Conversation

@derrabauke
Copy link
Contributor

Check docs for necessary permissions:
https://github.com/adfinis/container-scanning-action

@derrabauke derrabauke self-assigned this Jan 26, 2026
Copy link
Member

@open-dynaMIX open-dynaMIX left a comment

Choose a reason for hiding this comment

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

I don't know much about the exact needed perms. But I know in mySAGW we're doing the same, but with contents: none.

@derrabauke derrabauke force-pushed the ci-fix-container-registry branch from cbcf0ca to d3f5344 Compare January 26, 2026 07:47
@derrabauke
Copy link
Contributor Author

I don't know much about the exact needed perms. But I know in mySAGW we're doing the same, but with contents: none.

I think you are right. -> updated to content: none

@c0rydoras
Copy link
Contributor

https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax#permissions

Work with the contents of the repository. For example, contents: read permits an action to list the commits, and contents: write allows the action to create a release.

@c0rydoras
Copy link
Contributor

I don't think it matters for public repositories

@derrabauke
Copy link
Contributor Author

https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax#permissions

Work with the contents of the repository. For example, contents: read permits an action to list the commits, and contents: write allows the action to create a release.

As we do all the release related stuff manually we don't need any content permission I 'd say.

@derrabauke derrabauke merged commit 9c0a6bd into projectcaluma:main Jan 26, 2026
12 checks passed
@derrabauke derrabauke deleted the ci-fix-container-registry branch January 26, 2026 07:58
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.

3 participants