OpenLiberty / open-liberty-operator

Eclipse Public License 2.0
29 stars 38 forks source link

Serviceability storage in S3 Object Buckets #171

Open ron1 opened 4 years ago

ron1 commented 4 years ago

Feature Request

The only shared storage in many environments is Object Buckets. Currently the Open Liberty Operator serviceability feature requires shared file storage, i.e., RWX PVCs, which are not available in these environments.

Support serviceability storage in Object Buckets by allowing users to configure a Service Binding to one of the following S3 Object Storage Backing Service Operators available on OperatorHub.io: awss3operator, cos-bucket-operator, lib-bucket-provisioner, minio-operator, noobaa, or rook-ceph.

leochr commented 4 years ago

@ron1 The serviceability PVC created automatically by the operator (based on the serviceability.size parameter) specifies access modes ReadWriteMany (to allow multiple pods to write to the same storage) as well as ReadWriteOnce. Alternatively, a custom PVC can also be specified using the serviceability.volumeClaimName parameter. https://github.com/OpenLiberty/open-liberty-operator/blob/master/doc/user-guide.adoc#storage-for-serviceability

We'll look into using storage configured by Service Binding for serviceability. Thank you.

ron1 commented 4 years ago

@leochr The following phrase in the documentation "The single storage will be shared by all Pods of an OpenLibertyApplication instance" gave me the impression that an RWX PV was required. If the operator is smart enough to use a RWO PV for each pod in the Deployment when no RWX PV is available, then maybe this Feature Request should be replaced with a simple Documentation change to clarify the above phrase? In this case, the Feature Request is probably overkill. WDYT?