-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: list minimal permissions set required for Constellation setup #1442
Conversation
✅ Deploy Preview for constellation-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@msanft Nice, thanks for the work! Could you please validate that we use that exact permissions in the CI for our e2e testing? That way we'll quickly notice if anything changes (at least if we require more permissions). |
We should probably merge this with https://docs.edgeless.systems/constellation/getting-started/install#set-up-cloud-credentials and also extend the docs on which permissions are needed for cluster creation there. (as its already done on AWS) |
Co-authored-by: Nils Hanke <Nirusu@users.noreply.github.com>
Co-authored-by: Nils Hanke <Nirusu@users.noreply.github.com>
0f74675
to
2e5ca64
Compare
Co-authored-by: Thomas Tendyck <51411342+thomasten@users.noreply.github.com>
Co-authored-by: Thomas Tendyck <51411342+thomasten@users.noreply.github.com>
Co-authored-by: Thomas Tendyck <51411342+thomasten@users.noreply.github.com>
Co-authored-by: Thomas Tendyck <51411342+thomasten@users.noreply.github.com>
Any way to silence the review-🐶 here? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't forget to set Microsoft.Attestation
above for the required registered providers, too. You maybe can merge the footnotes for this, but not sure how it would look good given that they are in separate paragraphs.
Yeah I will need to look at this in the version built by Docusaurus. Just saved it so that I don't forget about it the next time I come back to this |
Co-authored-by: Thomas Tendyck <51411342+thomasten@users.noreply.github.com>
Co-authored-by: Nils Hanke <Nirusu@users.noreply.github.com>
Proposed change(s)
constellation iam
commands (iam create
&iam destroy
) for each CSP.Checklist