Skip to content
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

Add new required field to ClusterExtension spec for users to reference a ServiceAccount #971

Closed
Tracked by #737
everettraven opened this issue Jun 25, 2024 · 2 comments
Assignees

Comments

@everettraven
Copy link
Contributor

everettraven commented Jun 25, 2024

In order to begin using a ServiceAccount to install and manage content, a user needs to be able to specify a ServiceAccount for operator-controller to use when creating a ClusterExtension. Since OLMv1 will not have any permissions for installing and managing permissions by default this will need to be a required, immutable field.

An example of what a ClusterExtension resource could look like with this additional field:

apiVersion: olm.operatorframework.io/v1alpha1
kind: ClusterExtension
metadata:
  name: clusterextension-sample
spec:
  installNamespace: default
  packageName: argocd-operator
  version: 0.6.0
  serviceAccount:
    name: argocd-installer

Acceptance Criteria:

  • A new required, immutable field in the ClusterExtension spec for specifying a ServiceAccount that should be used to install and manage content
  • Updates to unit and e2e tests as necessary
@makon57
Copy link
Contributor

makon57 commented Jun 27, 2024

/assign

@everettraven
Copy link
Contributor Author

This is done as of #1011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

2 participants