[ 
https://issues.apache.org/jira/browse/HDFS-13148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16392091#comment-16392091
 ] 

Hanisha Koneru commented on HDFS-13148:
---------------------------------------

Hi [~shahrs87], thanks for the review.

If we make {{TestEncryptionZonesWithKMSandFederation}} extend 
{{TestEncryptionZonesWithKMS}} then the former would run all the test cases in 
the later with its initial {{setup}}. This would fail all the tests in 
{{TestEncryptionZonesWithKMS}} run against the setup of 
{{TestEncryptionZonesWithKMSandFederation}}. To get over this, we would have to 
modify all the test cases in {{TestEncryptionZonesWithKMS}} and 
{{TestEncryptionZones}} to work with the federated configuration setup. For 
example, instead of using variable {{dfsAdmin}} in 
{{TestEncryptionZonesWithKMS}}, we would have to change it to \{{dsfAdmin[0]}} 
to match the federated setup.

I think it would just complicate all the three Tests by doing this. We could 
instead have a \{{TestEncryptionZonesBaseTest}} class and make all the other 
Tests extend this class. Or we could just keep the non-federated Tests and 
federated Tests separate (as is in patch v03). Please let me know your thoughts.

> Unit test for EZ with KMS and Federation
> ----------------------------------------
>
>                 Key: HDFS-13148
>                 URL: https://issues.apache.org/jira/browse/HDFS-13148
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: test
>            Reporter: Hanisha Koneru
>            Assignee: Hanisha Koneru
>            Priority: Major
>         Attachments: HDFS-13148.001.patch, HDFS-13148.002.patch, 
> HDFS-13148.003.patch
>
>
> It would be good to have some unit tests for testing KMS and EZ on a 
> federated cluster. We can start with basic EZ operations. For example, create 
> EZs on two namespaces with different keys using one KMS.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to