-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Enable config for NAS gateway mode #7948
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7948 +/- ##
=========================================
+ Coverage 44.78% 45.2% +0.42%
=========================================
Files 302 314 +12
Lines 38947 50416 +11469
=========================================
+ Hits 17442 22791 +5349
- Misses 19399 25515 +6116
- Partials 2106 2110 +4
Continue to review full report at Codecov.
|
7883574
to
ce54d68
Compare
Starting with minio#7751 we don't store config in etcd anymore, allow NAS to honor config on disk.
0ec3211
to
a3e8033
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if user edits the backend config json instead of using mc admin config get/set [with the incoming PR) and leaves the config in an inconsistent state, it will neither get reloaded nor reported. Suggest logging if error in validating config.
If user edits backend he is on his own @poornas - config.json is considered like metadata maintained by MinIO we don't need to be paranoid about it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Mint Automation
7948-54e0624/mint-large-bucket.sh.log:
7948-54e0624/mint-dist-xl.sh.log:
|
Description
Enable config for NAS gateway mode
Motivation and Context
Starting with #7751 we don't store config
in etcd anymore, allow NAS to honor config
on disk.
How to test this PR?
Just use
minio gateway nas ~/test
and usemc admin config set
commands.Types of changes
Checklist:
commit-id
orPR #
here)