Dinesh Chitlangia created HDDS-1889:
---------------------------------------

             Summary: Add support for verifying multiline log entry
                 Key: HDDS-1889
                 URL: https://issues.apache.org/jira/browse/HDDS-1889
             Project: Hadoop Distributed Data Store
          Issue Type: Test
          Components: test
            Reporter: Dinesh Chitlangia


This jira aims to test the failure scenario where a multi-line stack trace will 
be added in the audit log. Currently, for test assumes that even in failure 
scenario we don't have multi-line log entry.
Example:

{code:java}
private static final AuditMessage READ_FAIL_MSG =
      new AuditMessage.Builder()
          .setUser("john")
          .atIp("192.168.0.1")
          .forOperation(DummyAction.READ_VOLUME.name())
          .withParams(PARAMS)
          .withResult(FAILURE)
          .withException(null).build();
{code}

Therefore in verifyLog() we only compare for first line of the log file with 
the expected message.
The test would fail if in future someone were to create a scenario with 
multi-line log entry.

1. Update READ_FAIL_MSG so that it has multiple lines of Exception stack trace.
This is what multi-line log entry could look like:
{code:java}
ERROR | OMAudit | user=dchitlangia | ip=127.0.0.1 | op=GET_ACL 
{volume=volume80100, bucket=bucket83878, key=null, aclType=CREATE, 
resourceType=volume, storeType=ozone} | ret=FAILURE
org.apache.hadoop.ozone.om.exceptions.OMException: User dchitlangia doesn't 
have CREATE permission to access volume
 at org.apache.hadoop.ozone.om.OzoneManager.checkAcls(OzoneManager.java:1809) 
~[classes/:?]
 at org.apache.hadoop.ozone.om.OzoneManager.checkAcls(OzoneManager.java:1769) 
~[classes/:?]
 at 
org.apache.hadoop.ozone.om.OzoneManager.createBucket(OzoneManager.java:2092) 
~[classes/:?]
 at 
org.apache.hadoop.ozone.protocolPB.OzoneManagerRequestHandler.createBucket(OzoneManagerRequestHandler.java:526)
 ~[classes/:?]
 at 
org.apache.hadoop.ozone.protocolPB.OzoneManagerRequestHandler.handle(OzoneManagerRequestHandler.java:185)
 ~[classes/:?]
 at 
org.apache.hadoop.ozone.protocolPB.OzoneManagerProtocolServerSideTranslatorPB.submitRequestDirectlyToOM(OzoneManagerProtocolServerSideTranslatorPB.java:192)
 ~[classes/:?]
 at 
org.apache.hadoop.ozone.protocolPB.OzoneManagerProtocolServerSideTranslatorPB.submitRequest(OzoneManagerProtocolServerSideTranslatorPB.java:110)
 ~[classes/:?]
 at 
org.apache.hadoop.ozone.protocol.proto.OzoneManagerProtocolProtos$OzoneManagerService$2.callBlockingMethod(OzoneManagerProtocolProtos.java)
 ~[classes/:?]
 at 
org.apache.hadoop.ipc.ProtobufRpcEngine$Server$ProtoBufRpcInvoker.call(ProtobufRpcEngine.java:524)
 ~[hadoop-common-3.2.0.jar:?]
 at org.apache.hadoop.ipc.RPC$Server.call(RPC.java:1025) 
~[hadoop-common-3.2.0.jar:?]
 at org.apache.hadoop.ipc.Server$RpcCall.run(Server.java:876) 
~[hadoop-common-3.2.0.jar:?]
 at org.apache.hadoop.ipc.Server$RpcCall.run(Server.java:822) 
~[hadoop-common-3.2.0.jar:?]
 at java.security.AccessController.doPrivileged(Native Method) ~[?:1.8.0_144]
 at javax.security.auth.Subject.doAs(Subject.java:422) ~[?:1.8.0_144]
 at 
org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1730)
 ~[hadoop-common-3.2.0.jar:?]
 at org.apache.hadoop.ipc.Server$Handler.run(Server.java:2682) 
~[hadoop-common-3.2.0.jar:?]

{code}

2. Update verifyLog method to accept variable number of arguments.
3. Update the assertion so that it compares beyond the first line when the 
expected in a multi-line log entry.
{code:java}
assertTrue(expected.equalsIgnoreCase(lines.get(0)));
{code}




--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-dev-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-dev-h...@hadoop.apache.org

Reply via email to