Re: [PR] HADOOP-18850 Enable dual-layer server-side encryption with AWS KMS keys [hadoop]

2023-11-01 Thread via GitHub
steveloughran merged PR #6140: URL: https://github.com/apache/hadoop/pull/6140 -- 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:

Re: [PR] HADOOP-18850 Enable dual-layer server-side encryption with AWS KMS keys [hadoop]

2023-10-27 Thread via GitHub
virajjasani commented on PR #6140: URL: https://github.com/apache/hadoop/pull/6140#issuecomment-1783254839 > `us-west-2`: > > Bucket with DSSE encryption enabled. > > `mvn clean verify -Dparallel-tests -DtestsThreadCount=8 -Dscale -Dprefetch` Re-tested with latest

Re: [PR] HADOOP-18850 Enable dual-layer server-side encryption with AWS KMS keys [hadoop]

2023-10-26 Thread via GitHub
hadoop-yetus commented on PR #6140: URL: https://github.com/apache/hadoop/pull/6140#issuecomment-1780942130 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18850 Enable dual-layer server-side encryption with AWS KMS keys [hadoop]

2023-10-26 Thread via GitHub
hadoop-yetus commented on PR #6140: URL: https://github.com/apache/hadoop/pull/6140#issuecomment-1780941631 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18850 Enable dual-layer server-side encryption with AWS KMS keys [hadoop]

2023-10-26 Thread via GitHub
virajjasani commented on code in PR #6140: URL: https://github.com/apache/hadoop/pull/6140#discussion_r1372703138 ## hadoop-tools/hadoop-aws/src/site/markdown/tools/hadoop-aws/encryption.md: ## @@ -316,6 +320,79 @@ metadata. Since only one encryption key can be provided at a

Re: [PR] HADOOP-18850 Enable dual-layer server-side encryption with AWS KMS keys [hadoop]

2023-10-25 Thread via GitHub
steveloughran commented on code in PR #6140: URL: https://github.com/apache/hadoop/pull/6140#discussion_r1372139869 ## hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/ITestS3AEncryptionDSSEKMSUserDefinedKey.java: ## @@ -39,10 +42,12 @@ protected Configuration

Re: [PR] HADOOP-18850 Enable dual-layer server-side encryption with AWS KMS keys [hadoop]

2023-10-21 Thread via GitHub
hadoop-yetus commented on PR #6140: URL: https://github.com/apache/hadoop/pull/6140#issuecomment-1773879202 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18850 Enable dual-layer server-side encryption with AWS KMS keys [hadoop]

2023-10-12 Thread via GitHub
hadoop-yetus commented on PR #6140: URL: https://github.com/apache/hadoop/pull/6140#issuecomment-1760698771 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18850 Enable dual-layer server-side encryption with AWS KMS keys [hadoop]

2023-10-03 Thread via GitHub
virajjasani commented on PR #6140: URL: https://github.com/apache/hadoop/pull/6140#issuecomment-1745420870 > `us-west-2`: > > Bucket with DSSE encryption enabled. > > `mvn clean verify -Dparallel-tests -DtestsThreadCount=8 -Dscale -Dprefetch` Re-tested with latest

Re: [PR] HADOOP-18850 Enable dual-layer server-side encryption with AWS KMS keys [hadoop]

2023-10-03 Thread via GitHub
hadoop-yetus commented on PR #6140: URL: https://github.com/apache/hadoop/pull/6140#issuecomment-1744613950 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] HADOOP-18850 Enable dual-layer server-side encryption with AWS KMS keys [hadoop]

2023-10-02 Thread via GitHub
steveloughran commented on code in PR #6140: URL: https://github.com/apache/hadoop/pull/6140#discussion_r1342962593 ## hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/EncryptionTestUtils.java: ## @@ -94,6 +96,13 @@ public static void assertEncrypted(S3AFileSystem