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

juicefs.umask configuration does not work #5336

Closed
kyungwan-nam opened this issue Dec 3, 2024 · 4 comments · Fixed by #5337
Closed

juicefs.umask configuration does not work #5336

kyungwan-nam opened this issue Dec 3, 2024 · 4 comments · Fixed by #5337
Assignees
Labels
kind/bug Something isn't working

Comments

@kyungwan-nam
Copy link
Contributor

What happened: juicefs.umask does not work

What you expected to happen:

I set juicefs.umask as 0022 in core-site.xml
https://juicefs.com/docs/community/hadoop_java_sdk#other-configurations
It should be 644 (rw-r--r--) when I upload a file to the JuiceFS using hdfs dfs -put. but, it doesn't work.
It works only if I set the fs.permissions.umask-mode

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?

Environment:

  • JuiceFS version (use juicefs --version) or Hadoop Java SDK version:
  • Cloud provider or hardware configuration running JuiceFS:
  • OS (e.g cat /etc/os-release):
  • Kernel (e.g. uname -a):
  • Object storage (cloud provider and region, or self maintained):
  • Metadata engine info (version, cloud provider managed or self maintained):
  • Network connectivity (JuiceFS to metadata engine, JuiceFS to object storage):
  • Others:
@kyungwan-nam kyungwan-nam added the kind/bug Something isn't working label Dec 3, 2024
@kyungwan-nam
Copy link
Contributor Author

fs.permissions.umask-mode should be set as juicefs.umask.
when a file is created, the permission will be decided by fs.permissions.umask-mode
https://github.com/apache/hadoop/blob/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FileSystem.java#L1233
PR: #5337

@kyungwan-nam
Copy link
Contributor Author

@tangyoupeng
Thank you for your comment
as your comment, respect fs.permissions.umask-mode only when creating a file using hdfs shell command.
so, I think fs.permissions.umask-mode have to replace with juicefs.umask.

@tangyoupeng
Copy link
Contributor

Thanks, i missed the pr. Already merged.

@kyungwan-nam
Copy link
Contributor Author

@tangyoupeng
Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants