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 btrfs support #200

Merged
merged 1 commit into from
May 3, 2021
Merged

Add btrfs support #200

merged 1 commit into from
May 3, 2021

Conversation

LKaemmerling
Copy link
Member

Replaces #186

Signed-off-by: Lukas Kämmerling lukas.kaemmerling@hetzner-cloud.de

Replaces #186

Signed-off-by: Lukas Kämmerling <lukas.kaemmerling@hetzner-cloud.de>
@LKaemmerling LKaemmerling mentioned this pull request Apr 21, 2021
@fnkr
Copy link

fnkr commented May 2, 2021

I tried this today and it works great. Resizing works too.

@fnkr fnkr mentioned this pull request May 2, 2021
@LKaemmerling LKaemmerling requested a review from fhofherr May 3, 2021 04:44
@fhofherr fhofherr merged commit 2ea4803 into master May 3, 2021
@fhofherr fhofherr deleted the add-btrfs-support branch May 3, 2021 10:54
@felixscheinost
Copy link
Contributor

felixscheinost commented Jun 16, 2023

For some reason this isn't working for me. How are you supposed to use this functionality?

I created a new StorageClass with csi.storage.k8s.io/fstype: btrfs. When I try to create a pod that uses that storage class I get an error: MountVolume.SetUp failed for volume "pvc-eebbbe87-469b-4aba-a17c-9b080160051c" : rpc error: code = Internal desc = failed to publish volume: unsupported disk format btrfs

Looking for unsupported disk format I found FormatDisk in volumes/mount.go, where btrfs isn't handled yet. Is this an error? Do the disks need to be preformatted? Do I need to set some other parameter?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants