Skip to content

Conversation

synhershko
Copy link
Collaborator

Add native NFS volume support to additionalVolumes configuration, enabling direct mounting of NFS shares for snapshot repositories and other use cases without external workarounds.

Changes:

  • Add NFS field to AdditionalVolume struct in CRD schema
  • Implement NFS volume mounting logic in CreateAdditionalVolumes function
  • Update CRD manifests with NFS volume validation (server and path required)
  • Add comprehensive example configuration showing NFS integration with snapshot repositories
  • Update Helm chart CRD files

This allows operators to configure NFS-based snapshot repositories directly in the OpenSearchCluster spec:

spec:
  general:
    additionalVolumes:
      - name: nfs-backups
        path: /mnt/backups/opensearch
        nfs:
          server: 192.168.1.233
          path: /export/backups/opensearch

Fixes
#1088

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Add native NFS volume support to additionalVolumes configuration,
enabling direct mounting of NFS shares for snapshot repositories
and other use cases without external workarounds.

Changes:
- Add NFS field to AdditionalVolume struct in CRD schema
- Implement NFS volume mounting logic in CreateAdditionalVolumes
function
- Update CRD manifests with NFS volume validation (server and path
required)
- Add comprehensive example configuration showing NFS integration with
snapshot repositories
- Update Helm chart CRD files

This allows operators to configure NFS-based snapshot repositories
directly in the OpenSearchCluster spec:

```yaml
spec:
  general:
    additionalVolumes:
      - name: nfs-backups
        path: /mnt/backups/opensearch
        nfs:
          server: 192.168.1.233
          path: /export/backups/opensearch
```

Fixes
opensearch-project#1088

Signed-off-by: Itamar Syn-Hershko <[email protected]>
@josedev-union
Copy link
Contributor

josedev-union commented Oct 6, 2025

Notes for myself

  • add unit test
  • add xValidation for making sure one additional volume type defined in one volume

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants