Skip to content

docs: notes for matrix and required permissions#469

Merged
kzantow merged 1 commit into
anchore:mainfrom
kzantow-anchore:docs/matrix-and-permissions
May 28, 2024
Merged

docs: notes for matrix and required permissions#469
kzantow merged 1 commit into
anchore:mainfrom
kzantow-anchore:docs/matrix-and-permissions

Conversation

@kzantow

@kzantow kzantow commented May 23, 2024

Copy link
Copy Markdown
Contributor

A recent update to the GitHub actions toolkit had an impact on a couple aspects of workflow artifact usage, this PR updates the docs with information that should get anyone unblocked who has issues with these changes.

See:

Comment thread README.md
This action needs the following permissions:

```
contents: write # for sbom-action artifact uploads

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm a bit confused by these YAML comments. It might help if there was a list of use case to required permissions. I suspect there are some permissions always needed, and some only needed for certain use cases.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm open to suggestions how to make this more clear. There are 2 main things this does: create an SBOM and upload it as a workflow artifact (the first permission) and attach release assets (the second permission), a user only needs to set the permission for the thing they're doing, which the comments indicate.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

How does actions: read grant permissions to attach things to the release assets? Does it not need some write permissions for that?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated -- is this more clear, @willmurphyscode ?

Signed-off-by: Keith Zantow <kzantow@gmail.com>
@kzantow
kzantow force-pushed the docs/matrix-and-permissions branch from 94b949f to 2e84037 Compare May 28, 2024 16:32
@kzantow
kzantow enabled auto-merge (squash) May 28, 2024 16:36
@kzantow
kzantow merged commit 2283abe into anchore:main May 28, 2024
@kzantow
kzantow deleted the docs/matrix-and-permissions branch May 28, 2024 21:10
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