adoroszlai opened a new pull request #3039:
URL: https://github.com/apache/ozone/pull/3039


   ## What changes were proposed in this pull request?
   
   Follow-up for HDDS-6206: avoid logging exception as error when they are 
known cases (e.g. invalid bucket name, checking existence (`head`) of 
non-existing bucket, permission denied, etc.).
   
   https://issues.apache.org/jira/browse/HDDS-6247
   
   ## How was this patch tested?
   
   Ran S3 acceptance tests before and after the change.  Verified S3 Gateway 
logs no longer have verbose stack trace for these requests.  We can still see 
request problems by setting log level to DEBUG, e.g.:
   
   ```
   s3g_1       | 2022-02-03 12:54:57,459 [qtp690052870-16] DEBUG 
exception.S3ErrorTable: <?xml version="1.0" encoding="UTF-8"?>
   s3g_1       | <Error>
   s3g_1       |   <Code>InvalidBucketName</Code>
   s3g_1       |   <Message>The specified bucket is not valid.</Message>
   s3g_1       |   <Resource>a_b_c</Resource>
   s3g_1       |   <RequestId/>
   s3g_1       | </Error>
   ...
   s3g_1       | 2022-02-03 12:59:56,517 [qtp690052870-20] DEBUG 
exception.S3ErrorTable: <?xml version="1.0" encoding="UTF-8"?>
   s3g_1       | <Error>
   s3g_1       |   <Code>NoSuchBucket</Code>
   s3g_1       |   <Message>The specified bucket does not exist</Message>
   s3g_1       |   <Resource>dont-exist</Resource>
   s3g_1       |   <RequestId/>
   s3g_1       | </Error>
   ```
   
   https://github.com/adoroszlai/hadoop-ozone/actions/runs/1787777620


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org
For additional commands, e-mail: issues-h...@ozone.apache.org

Reply via email to