You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2020/12/16 19:22:15 UTC
[GitHub] [ozone] bharatviswa504 opened a new pull request #1719: HDDS-4597. OM Terminates when adding acls to S3v volume.
bharatviswa504 opened a new pull request #1719:
URL: https://github.com/apache/ozone/pull/1719
## What changes were proposed in this pull request?
When OM starts it creates default volume "s3v" where S3 buckets are created with all required permissions.
When volume is created it is set the updateID with maxTransactionID. This causes a problem, when user tries to perform setAcl/AddAcl on volume, we update the updateID during that we have below check.
Now let's say, a transaction index 100 is trying to set ACL, it will fail in the preCondition check.
```
if (isRatisEnabled) {
Preconditions.checkArgument(updateId >= this.updateID, String.format(
"Trying to set updateID to %d which is not greater than the " +
"current value of %d for %s", updateId, this.updateID,
getObjectInfo()));
}
```
## What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4597
## How was this patch tested?
Added test.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] bharatviswa504 commented on pull request #1719: HDDS-4597. OM Terminates when adding acls to S3v volume.
Posted by GitBox <gi...@apache.org>.
bharatviswa504 commented on pull request #1719:
URL: https://github.com/apache/ozone/pull/1719#issuecomment-749663569
Thank You @vivekratnavel and @jojochuang for the review.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] bharatviswa504 merged pull request #1719: HDDS-4597. OM Terminates when adding acls to S3v volume.
Posted by GitBox <gi...@apache.org>.
bharatviswa504 merged pull request #1719:
URL: https://github.com/apache/ozone/pull/1719
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org