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

Persistent Volume support for Operator Deployment Pod #2828

Open
vivekr-splunk opened this issue Aug 3, 2022 · 3 comments
Open

Persistent Volume support for Operator Deployment Pod #2828

vivekr-splunk opened this issue Aug 3, 2022 · 3 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@vivekr-splunk
Copy link

Feature Request

Is your feature request related to a problem? Please describe.
Hello, We are working on splunk operator. version 2.0.0 of splunk operator requires persistent volume to be mounted to splunk operator deployment. but bundle creation using olm do not allow persistent volume creation. How can I support in persistent volume when we use bundle deployment.

Describe the solution you'd like
support for persistent volume in bundle for operator pod deployment

@vivekr-splunk vivekr-splunk added the kind/feature Categorizes issue or PR as related to a new feature. label Aug 3, 2022
@dmesser
Copy link
Contributor

dmesser commented Aug 3, 2022

Hi @vivekr-splunk

https://github.com/operator-framework/operator-lifecycle-manager/blob/master/doc/design/subscription-config.md#volumemounts exists to augment a deployed operators with mounting an existing volume, but it doesn't seem that's quite what you are after.

Can you comment on the use case where a Kubernetes controllers needs a persistent volume mount?

@squakez
Copy link

squakez commented Mar 6, 2023

I have the same need. The volumeMounts works if the PVC was previously created. The requirements would be to create the PVC on the fly, right before creating the Deployment, for instance by specifying the StorageClass (or fallback to use the one marked as default if none is provided). Trying to define how the yaml could look like with the presence of a volumes parameter:

spec:
  apiservicedefinitions: {}
  customresourcedefinitions:
    ...
  install:
    spec:
      clusterPermissions:
     ...
      volumes:
      - camel-k-pvc:
          accessModes:
              - ReadWriteOnce
              resources:
                requests:
                  storage: 20Gi
              storageClassName: standard
      deployments:
      - name: camel-k-operator
        spec:
          ...
              serviceAccountName: camel-k-operator
              volumes:
              - name: camel-k-pvc
                persistentVolumeClaim:
                  claimName: camel-k-pvc

Without this feature, we are in the situation to ask the user to create such a PVC before running the installation procedure, which is a bit against the idea of letting everything managed by the subscription.

@squakez
Copy link

squakez commented Mar 6, 2023

@vivekr-splunk given you opened this sometime ago, have you managed to find some workaround by any chance?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

No branches or pull requests

3 participants