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.

Sign up for GitHub

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

Jump to bottom

Error in NFS section #56

Closed
macvaz opened this issue Sep 10, 2022 · 5 comments · Fixed by #58
Closed

Error in NFS section #56

macvaz opened this issue Sep 10, 2022 · 5 comments · Fixed by #58

Comments

@macvaz
Copy link

macvaz commented Sep 10, 2022 •
edited

Bug Info

Describe the bug
I can use Samba section but NFS section is not working at all.

The project is amazing. This extension for managing NAS options from cockpit is great.

Screenshots
image
image

Client Side

Desktop (please complete the following information):

  • OS: Linux
  • Browser [e.g. chrome, safari]: Chrome
  • Version [e.g. 22]
@joshuaboud
Copy link
Member

Hi, what Linux distribution is your server running?

@joshuaboud
Copy link
Member

I was able to recreate this issue. Pretty sure it's from the directory /etc/exports.d not existing. Implementing a fix now.

joshuaboud added a commit that referenced this issue Sep 12, 2022
@joshuaboud
Handle missing /etc/exports.d path
921b5fd 
- add missing optional chain while loading exports from file
- create parent directories before saving exports file
closes #56
@joshuaboud joshuaboud mentioned this issue Sep 12, 2022
Merged
@macvaz macvaz mentioned this issue Sep 12, 2022
Closed
@macvaz
Copy link
Author

macvaz commented Sep 12, 2022

I was able to recreate this issue. Pretty sure it's from the directory /etc/exports.d not existing. Implementing a fix now.

Yes. The /etc/exports.d is missing. Should I create it?

I'm running Ubuntu 22.04 server.

Thanks for the response.

@joshuaboud
Copy link
Member

You can update to the latest version by following the generic installation steps in the readme (now with paths updated for v3.2.8), and the plugin should automatically create the path for you if it isn't present.

@macvaz
Copy link
Author

macvaz commented Sep 12, 2022

I didn't see your message. I created the folder manually. I have tested created a NFS drive and I now looks much better.

image

I will test for sure the new installer. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

File Sharing v3.2.8 45Drives/cockpit-file-sharing
2 participants
@macvaz @joshuaboud