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

Extra mkfs options #296

Open
mhkarimi1383 opened this issue Apr 18, 2024 · 3 comments · May be fixed by #335
Open

Extra mkfs options #296

mhkarimi1383 opened this issue Apr 18, 2024 · 3 comments · May be fixed by #335
Labels
enhancement New feature or request feature good first issue Good for newcomers
Milestone

Comments

@mhkarimi1383
Copy link

mhkarimi1383 commented Apr 18, 2024

Describe the problem/challenge you have

I want to use lvm-localpv withing MinIO and some other applications with high number of Inode usage and sometimes I need to inject extra params into mkfs command

Describe the solution you'd like

Adding mkfsOptions just like mountOptions

Anything else you would like to add:
I wanted to add this feature as a PR but I don't know where to change the code to add this (I mean connecting parameters struct into formatAndMountSensitive function

(line #358 of pkg/lvm/mount.go)

Environment:

  • LVM Driver version
  • Kubernetes version (use kubectl version): 1.28
  • Kubernetes installer & version: Kubespray
  • Cloud provider or hardware configuration: Self Hosted
  • OS (e.g. from /etc/os-release): debian bookworm
@avishnu
Copy link
Member

avishnu commented Sep 19, 2024

Scoping this for v4.3. Any takers from the community will be appreciated.

@avishnu avishnu added the good first issue Good for newcomers label Sep 19, 2024
@mhkarimi1383
Copy link
Author

@avishnu
I will do it

@mhkarimi1383 mhkarimi1383 linked a pull request Sep 19, 2024 that will close this issue
7 tasks
@mhkarimi1383
Copy link
Author

Added in #335

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature good first issue Good for newcomers
Projects
Status: Todo
Development

Successfully merging a pull request may close this issue.

3 participants