operator-framework / operator-controller

A new and improved management framework for extending Kubernetes with Operators
https://operator-framework.github.io/operator-controller/
Apache License 2.0
55 stars 53 forks source link

🌱 Use SA from spec when installing bundles #907

Closed skattoju closed 3 months ago

skattoju commented 4 months ago

Description

Adds a ServiceAccountName field to the spec and sets up the action client getter to provide an action client that uses a token based on the service account provided in the spec. Related to #737

Reviewer Checklist

netlify[bot] commented 4 months ago

Deploy Preview for olmv1 ready!

Name Link
Latest commit 9b1e2e54b722e6c32c66e3214f6a5533c8e9eee4
Latest deploy log https://app.netlify.com/sites/olmv1/deploys/6678cffae477b4000824e2e6
Deploy Preview https://deploy-preview-907--olmv1.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

varshaprasad96 commented 3 months ago

/ok-to-test

(not sure if this label is accepted)

openshift-merge-robot commented 3 months ago

PR needs rebase.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.
everettraven commented 3 months ago

@skattoju I'm going to close this draft PR since we are going to split the implementation into individual PRs.

Thank you a ton for all the effort you put into this PoC that helped drive the discussion and design process!