ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Colm O hEigeartaigh <cohei...@apache.org>
Subject Re: Review Request 46829: RANGER-961 - Remove KMS xml-security-impl dependency
Date Fri, 27 May 2016 09:17:26 GMT


> On May 27, 2016, 3:20 a.m., Ankita Sinha wrote:
> > kms/src/main/java/org/apache/hadoop/crypto/key/RangerMasterKey.java, line 150
> > <https://reviews.apache.org/r/46829/diff/1/?file=1365719#file1365719line150>
> >
> >     While testing with this patch changes : Encryption key operations are failing.
> >     
> >     
> >     $ hadoop key create key1
> >     key1 has not been created. java.io.IOException: Can't store key key1@0
> >     java.io.IOException: Can't store key key1@0
> >     at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
> >     at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
> >     at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
> >     at java.lang.reflect.Constructor.newInstance(Constructor.java:422)
> >     at org.apache.hadoop.util.HttpExceptionUtils.validateResponse(HttpExceptionUtils.java:157)
> >     at org.apache.hadoop.crypto.key.kms.KMSClientProvider.call(KMSClientProvider.java:552)
> >     at org.apache.hadoop.crypto.key.kms.KMSClientProvider.call(KMSClientProvider.java:510)
> >     at org.apache.hadoop.crypto.key.kms.KMSClientProvider.createKeyInternal(KMSClientProvider.java:683)
> >     at org.apache.hadoop.crypto.key.kms.KMSClientProvider.createKey(KMSClientProvider.java:691)
> >     at org.apache.hadoop.crypto.key.KeyShell$CreateCommand.execute(KeyShell.java:483)
> >     at org.apache.hadoop.crypto.key.KeyShell.run(KeyShell.java:79)
> >     at org.apache.hadoop.util.ToolRunner.run(ToolRunner.java:76)
> >     at org.apache.hadoop.crypto.key.KeyShell.main(KeyShell.java:515)

Could you let me know how to reproduce this error + I'll take a look?


- Colm


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46829/#review135143
-----------------------------------------------------------


On May 27, 2016, 9:07 a.m., Colm O hEigeartaigh wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46829/
> -----------------------------------------------------------
> 
> (Updated May 27, 2016, 9:07 a.m.)
> 
> 
> Review request for ranger.
> 
> 
> Bugs: RANGER-961
>     https://issues.apache.org/jira/browse/RANGER-961
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> The KMS module uses the xml-security-impl dependency just to get a Base64 encoder/decoder.
This is not necessary as one is already available in Commons Codec, and in fact is used in
other classes in this module.
> 
> 
> Diffs
> -----
> 
>   kms/pom.xml a9f6c6c 
>   kms/src/main/java/org/apache/hadoop/crypto/key/DB2HSMMKUtil.java ca69dc0 
>   kms/src/main/java/org/apache/hadoop/crypto/key/HSM2DBMKUtil.java 73a5830 
>   kms/src/main/java/org/apache/hadoop/crypto/key/RangerHSM.java 6ab91d9 
>   kms/src/main/java/org/apache/hadoop/crypto/key/RangerMasterKey.java d70ec4e 
> 
> Diff: https://reviews.apache.org/r/46829/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Colm O hEigeartaigh
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message