Add support for encrypted async blob read#10131
Add support for encrypted async blob read#10131andrross merged 2 commits intoopensearch-project:mainfrom
Conversation
Compatibility status:Checks if related components are compatible with change 5551aaf Incompatible componentsIncompatible components: [https://github.com/opensearch-project/k-nn.git] Skipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git] |
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Codecov Report
@@ Coverage Diff @@
## main #10131 +/- ##
============================================
- Coverage 71.22% 71.12% -0.11%
+ Complexity 58245 58189 -56
============================================
Files 4828 4828
Lines 274340 274367 +27
Branches 39985 39986 +1
============================================
- Hits 195401 195141 -260
- Misses 62571 62857 +286
- Partials 16368 16369 +1
|
andrross
left a comment
There was a problem hiding this comment.
Do we have any way to actually test this? The mock-based tests seem to mostly be implementing the behavior you expect from the real system and therefore susceptible to things being wrong when the actual integration happens.
...er/src/main/java/org/opensearch/common/blobstore/AsyncMultiStreamEncryptedBlobContainer.java
Outdated
Show resolved
Hide resolved
...er/src/main/java/org/opensearch/common/blobstore/AsyncMultiStreamEncryptedBlobContainer.java
Outdated
Show resolved
Hide resolved
@vikasvb90 Any thoughts here for testing cypto logic, given that we do not have any specific implementation? |
Signed-off-by: Kunal Kotwani <kkotwani@amazon.com>
Signed-off-by: Kunal Kotwani <kkotwani@amazon.com>
421d3c5 to
5551aaf
Compare
Gradle Check (Jenkins) Run Completed with:
|
|
The backport to To backport manually, run these commands in your terminal: # Navigate to the root of your repository
cd $(git rev-parse --show-toplevel)
# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add ../.worktrees/OpenSearch/backport-2.x 2.x
# Navigate to the new working tree
pushd ../.worktrees/OpenSearch/backport-2.x
# Create a new branch
git switch --create backport/backport-10131-to-2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 c4c4ad84d995f75f8749a0f99aa8a1ecc3b71760
# Push it to GitHub
git push --set-upstream origin backport/backport-10131-to-2.x
# Go back to the original working tree
popd
# Delete the working tree
git worktree remove ../.worktrees/OpenSearch/backport-2.xThen, create a pull request where the |
* Add support for encrypted async blob read Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> * Add async blob read support for encrypted containers Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> --------- Signed-off-by: Kunal Kotwani <kkotwani@amazon.com>
* Add support for encrypted async blob read Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> * Add async blob read support for encrypted containers Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> --------- Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> Signed-off-by: Ivan Brusic <ivan.brusic@flocksafety.com>
* Add support for encrypted async blob read Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> * Add async blob read support for encrypted containers Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> --------- Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> (cherry picked from commit c4c4ad8)
* Add support for encrypted async blob read Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> * Add async blob read support for encrypted containers Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> --------- Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> (cherry picked from commit c4c4ad8)
* Add support for encrypted async blob read Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> * Add async blob read support for encrypted containers Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> --------- Signed-off-by: Kunal Kotwani <kkotwani@amazon.com>
* Add support for encrypted async blob read Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> * Add async blob read support for encrypted containers Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> --------- Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> Signed-off-by: Shivansh Arora <hishiv@amazon.com>
Description
Related Issues
Resolves #10105
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.