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

nfs: add pool setting CR option #9040

Merged
merged 1 commit into from Oct 29, 2021
Merged

nfs: add pool setting CR option #9040

merged 1 commit into from Oct 29, 2021

Conversation

leseb
Copy link
Member

@leseb leseb commented Oct 26, 2021

Description of your changes:

Ths NFS spec now supports the CephBlockPool spec which means that it can
take advantage of all the known settings like compression, size, failure
domain etc.

Closes: #9034
Signed-off-by: Sébastien Han seb@redhat.com

Which issue is resolved by this Pull Request:
Resolves #9034

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
  • Skip Tests for Docs: Add the flag for skipping the build if this is only a documentation change. See here for the flag.
  • Skip Unrelated Tests: Add a flag to run tests for a specific storage provider. See test options.
  • Reviewed the developer guide on Submitting a Pull Request
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.
  • Pending release notes updated with breaking and/or notable changes, if necessary.
  • Upgrade from previous release is tested and upgrade user guide is updated, if necessary.
  • Code generation (make codegen) has been run to update object specifications, if necessary.

cluster/charts/rook-ceph/templates/resources.yaml Outdated Show resolved Hide resolved
cluster/examples/kubernetes/ceph/nfs-test.yaml Outdated Show resolved Hide resolved
cluster/examples/kubernetes/ceph/nfs.yaml Outdated Show resolved Hide resolved
@leseb leseb force-pushed the fix-9034 branch 4 times, most recently from a0c78d4 to d43cfdd Compare October 27, 2021 11:53
@leseb leseb requested a review from travisn October 27, 2021 11:59
cluster/examples/kubernetes/ceph/nfs.yaml Outdated Show resolved Hide resolved
Documentation/ceph-nfs-crd.md Outdated Show resolved Hide resolved
pkg/apis/ceph.rook.io/v1/types.go Show resolved Hide resolved
@leseb
Copy link
Member Author

leseb commented Oct 28, 2021

@Mergifyio rebase

@mergify
Copy link

mergify bot commented Oct 28, 2021

rebase

✅ Branch has been successfully rebased

cluster/examples/kubernetes/ceph/nfs.yaml Outdated Show resolved Hide resolved
pkg/apis/ceph.rook.io/v1/types.go Show resolved Hide resolved
Ths NFS spec now supports the CephBlockPool spec which means that it can
take advantage of all the known settings like compression, size, failure
domain etc.

Closes: rook#9034
Signed-off-by: Sébastien Han <seb@redhat.com>
@leseb leseb merged commit c3d4ead into rook:master Oct 29, 2021
@leseb leseb deleted the fix-9034 branch October 29, 2021 13:42
mergify bot added a commit that referenced this pull request Nov 3, 2021
nfs: add pool setting CR option (backport #9040)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nfs: GaneshaRADOSSpec should use CephBlockPool spec
2 participants