opensearch-project / opensearch-oci-object-storage

OpenSearch Repository Plugin For Oci Object Storage

Apache License 2.0

7 stars 19 forks source link

Fix issue #13: https://github.com/opensearch-project/opensearch-oci-o… #22

Closed plemouel1 closed 1 year ago

plemouel1 commented 1 year ago

…bject-storage/issues/13

Description

This commit addresses issue #13 See issue #13 for details regarding the encountered problem.

This commit also include sl4j-simple as a dependency so that we can get logging statements from the OCI SDK to show up in OpenSearch logs

Issues Resolved

13

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.

samuel-oci commented 1 year ago

Hi @plemouel1 , thank you for contribution, I understand the need to disable the security manager to get things working until we can get the policy file to behave the way we would like. Can you please make the DCO checks to work?

plemouel1 commented 1 year ago

DCO should be fixed now.

plemouel1 commented 1 year ago

@samuel-oci Looks like now there isn't anything more for me to do. Can you have a look? Thanks

saratvemulapalli commented 1 year ago

@plemouel1 I am sad that we are disabling JSM but if thats something which gets us unblocked Im ok with it. Could you open an issue/bug to restore JSM?

saratvemulapalli commented 1 year ago

Will wait for @samuel-oci's ack to merge.