-
Notifications
You must be signed in to change notification settings - Fork 38
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
cli: set-eacl update #2490
cli: set-eacl update #2490
Conversation
You can add multiple commits, each solving a particular part of the problem. |
d4979b3
to
dffe16c
Compare
Codecov Report
@@ Coverage Diff @@
## master #2490 +/- ##
=======================================
Coverage 29.29% 29.29%
=======================================
Files 399 399
Lines 30346 30346
=======================================
Hits 8891 8891
Misses 20719 20719
Partials 736 736 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
b37ff3b
to
3f01823
Compare
9c437c9
to
a2add5f
Compare
974067a
to
883dde9
Compare
The eACL table can only be set by the container owner or his authorized representative. Similar to `container delete`, it makes sense to add an ownership check to `container set-eacl`. Precede the ACL extensibility check with an owner check: if the `--no-precheck` flag is set, then the caller must be the owner of the container or provide a session token signed by them. Refs #2436. Signed-off-by: Ekaterina Pavlova <[email protected]>
Deprecate no-precheck flag which is functionally the same as force(skips verification). Closes #2436. Signed-off-by: Ekaterina Pavlova <[email protected]>
883dde9
to
cbac4d3
Compare
Remove the "no-precheck" flag, which is equivalent to "force" (skips verification) and has been deprecated since #2490. Closes: #2496. Signed-off-by: Ekaterina Pavlova <[email protected]>
Remove the "no-precheck" flag, which is equivalent to "force" (skips verification) and has been deprecated since #2490. Closes: #2496. Signed-off-by: Ekaterina Pavlova <[email protected]>
Remove the "no-precheck" flag, which is equivalent to "force" (skips verification) and has been deprecated since #2490. Closes: #2496. Signed-off-by: Ekaterina Pavlova <[email protected]>
Remove the "no-precheck" flag, which is equivalent to "force" (skips verification) and has been deprecated since #2490. Closes: #2496. Signed-off-by: Ekaterina Pavlova <[email protected]>
force flag added, no-precheck deprecated, ownership check added
Refs #2436.