Added jackson dependency to server #5366

ryanbogan · 2022-11-23T19:31:53Z

Signed-off-by: Ryan Bogan rbogan@amazon.com

Description

In order for extensions to be read, the jackson dependency must be included in the server build.gradle file. This PR adds the license, notice, and SHA files necessary to accomplish this.
Note: Although the jackson files for plugins and modules are being deleted with this PR, they still have access under server and no functionality will be changed for them.

Check List

New functionality includes testing.
- All tests pass
New functionality has been documented.
- New functionality has javadoc added
Commits are signed per the DCO using --signoff
Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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.

Signed-off-by: Ryan Bogan <rbogan@amazon.com>

github-actions · 2022-11-23T19:47:09Z

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/7294/
CommitID: 5de4cb1
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

github-actions · 2022-11-23T19:53:22Z

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/7295/
CommitID: 181da45
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

github-actions · 2022-11-25T19:43:50Z

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/7348/
CommitID: a814b11
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

reta · 2022-11-25T20:14:34Z

@ryanbogan this is interesting, it seems like :libs:opensearch-x-content brings these dependencies to the server already?

+--- project :libs:opensearch-x-content                  
|    +--- project :libs:opensearch-core                     
|    +--- org.yaml:snakeyaml:1.32                               
|    +--- com.fasterxml.jackson.core:jackson-core:2.14.1    
|    +--- com.fasterxml.jackson.dataformat:jackson-dataformat-smile:2.14.1
|    +--- com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:2.14.1
|    \--- com.fasterxml.jackson.dataformat:jackson-dataformat-cbor:2.14.1

Signed-off-by: Ryan Bogan <rbogan@amazon.com>

github-actions · 2022-11-25T21:44:39Z

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/7356/
CommitID: 0f6a154
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

ryanbogan · 2022-11-25T21:45:37Z

@ryanbogan this is interesting, it seems like :libs:opensearch-x-content brings these dependencies to the server already?

+--- project :libs:opensearch-x-content                  
|    +--- project :libs:opensearch-core                     
|    +--- org.yaml:snakeyaml:1.32                               
|    +--- com.fasterxml.jackson.core:jackson-core:2.14.1    
|    +--- com.fasterxml.jackson.dataformat:jackson-dataformat-smile:2.14.1
|    +--- com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:2.14.1
|    \--- com.fasterxml.jackson.dataformat:jackson-dataformat-cbor:2.14.1

Without the dependency in the server build.gradle file, import errors are thrown on compilation for every object in the jackson library.

github-actions · 2022-11-25T22:00:56Z

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/7357/
CommitID: 0f6a154
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

dblock · 2022-12-06T00:15:44Z

Assuming you can get this to 🟢 I don't see any reasons not to merge this.

ryanbogan · 2022-12-06T00:44:23Z

@dblock The problem is that I'm not sure why it's failing when the git diff is identical to changes that are passing here: #5347. Do you think it's possible that it would fail if no files are using the jackson dependency in server yet?

dblock · 2022-12-06T18:22:10Z

Can you reproduce the failure locally?


> Task :plugins:discovery-ec2:qa:amazon-ec2:yamlRestTestContainerCredentials FAILED

=== Standard output of node `node{:plugins:discovery-ec2:qa:amazon-ec2:yamlRestTestContainerCredentials-0}` ===

»    ↓ errors and warnings from /var/jenkins/workspace/gradle-check/search/plugins/discovery-ec2/qa/amazon-ec2/build/testclusters/yamlRestTestContainerCredentials-0/logs/opensearch.stdout.log ↓
» ERROR][o.o.b.OpenSearchUncaughtExceptionHandler] [yamlRestTestContainerCredentials-0] fatal error in thread [main], exiting
»  java.lang.ExceptionInInitializerError: null
»  	at com.amazonaws.util.VersionInfoUtils.userAgent(VersionInfoUtils.java:142) ~[?:?]
»  	at com.amazonaws.util.VersionInfoUtils.initializeUserAgent(VersionInfoUtils.java:137) ~[?:?]
»  	at com.amazonaws.util.VersionInfoUtils.getUserAgent(VersionInfoUtils.java:100) ~[?:?]
»  	at com.amazonaws.ClientConfiguration.<clinit>(ClientConfiguration.java:80) ~[?:?]

github-actions · 2022-12-06T18:39:28Z

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/7678/
CommitID: 0f6a154
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

ryanbogan · 2022-12-06T19:34:31Z

Can you reproduce the failure locally?

Yes I can reproduce it locally.

github-actions · 2022-12-06T20:01:48Z

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/7687/
CommitID: 55107cf
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

Signed-off-by: Ryan Bogan <rbogan@amazon.com>

github-actions · 2022-12-07T22:32:27Z

Gradle Check (Jenkins) Run Completed with:

RESULT: SUCCESS ✅
URL: https://build.ci.opensearch.org/job/gradle-check/7759/
CommitID: 69f7276

codecov-commenter · 2022-12-07T22:35:38Z

Codecov Report

Merging #5366 (69f7276) into main (5500114) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #5366      +/-   ##
============================================
- Coverage     71.02%   71.01%   -0.02%     
- Complexity    58192    58211      +19     
============================================
  Files          4711     4711              
  Lines        277574   277574              
  Branches      40182    40182              
============================================
- Hits         197155   197127      -28     
- Misses        64294    64332      +38     
+ Partials      16125    16115      -10

Impacted Files	Coverage Δ
...adonly/AddIndexBlockClusterStateUpdateRequest.java	`0.00% <0.00%> (-75.00%)`	⬇️
...adcast/BroadcastShardOperationFailedException.java	`33.33% <0.00%> (-66.67%)`	⬇️
...readonly/TransportVerifyShardIndexBlockAction.java	`9.75% <0.00%> (-58.54%)`	⬇️
...n/admin/indices/readonly/AddIndexBlockRequest.java	`17.85% <0.00%> (-53.58%)`	⬇️
...ch/transport/ReceiveTimeoutTransportException.java	`50.00% <0.00%> (-50.00%)`	⬇️
...opensearch/persistent/PersistentTasksExecutor.java	`22.22% <0.00%> (-44.45%)`	⬇️
...indices/readonly/TransportAddIndexBlockAction.java	`20.68% <0.00%> (-41.38%)`	⬇️
.../admin/cluster/reroute/ClusterRerouteResponse.java	`60.00% <0.00%> (-40.00%)`	⬇️
...luster/routing/allocation/RoutingExplanations.java	`62.06% <0.00%> (-37.94%)`	⬇️
.../admin/indices/readonly/AddIndexBlockResponse.java	`0.00% <0.00%> (-37.50%)`	⬇️
... and 497 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

owaiskazi19 · 2022-12-07T23:03:28Z

server/src/main/resources/org/opensearch/bootstrap/security.policy

@@ -100,6 +100,9 @@ grant {
  permission jdk.net.NetworkPermission "getOption.TCP_KEEPCOUNT";
  permission jdk.net.NetworkPermission "setOption.TCP_KEEPCOUNT";

+  permission java.lang.RuntimePermission "accessDeclaredMembers";
+  permission java.lang.reflect.ReflectPermission "suppressAccessChecks";
+


@dblock @saratvemulapalli @dbwiddis since we are removing jackson library from the native plugins and adding it in server. Need to add the above permissions. Looking for some advice if providing accessDeclaredMembers will bring any security concerns since we are enabling it for the server module? If not, then I guess we are good to go.

^ We have these lines in feature/extensions as well so if there are security concerns we should fix them there too.

Let's open an issue. I don't know what the impact of this change is.

@ryanbogan can you take care of it?

Already open: #5504

dblock · 2022-12-09T20:00:49Z

server/src/main/resources/org/opensearch/bootstrap/security.policy

@@ -100,6 +100,9 @@ grant {
  permission jdk.net.NetworkPermission "getOption.TCP_KEEPCOUNT";
  permission jdk.net.NetworkPermission "setOption.TCP_KEEPCOUNT";

+  permission java.lang.RuntimePermission "accessDeclaredMembers";
+  permission java.lang.reflect.ReflectPermission "suppressAccessChecks";
+


Let's open an issue. I don't know what the impact of this change is.

opensearch-trigger-bot · 2022-12-09T20:08:36Z

The backport to 2.x failed:

The process '/usr/bin/git' failed with exit code 128

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add ../.worktrees/backport-2.x 2.x
# Navigate to the new working tree
pushd ../.worktrees/backport-2.x
# Create a new branch
git switch --create backport/backport-5366-to-2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 c6663fd7f4e74b3ed1b6ad70e758b3e6bd21f816
# Push it to GitHub
git push --set-upstream origin backport/backport-5366-to-2.x
# Go back to the original working tree
popd
# Delete the working tree
git worktree remove ../.worktrees/backport-2.x

Then, create a pull request where the base branch is 2.x and the compare/head branch is backport/backport-5366-to-2.x.

dblock · 2022-12-09T20:34:50Z

@ryanbogan will need a manual backport to come to 2.x

…ature/identity (#5581) * Fix flaky ShardIndexingPressureConcurrentExecutionTests (#5439) Add conditional check on assertNull to fix flaky tests. Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com> * Fix bwc for cluster manager throttling settings (#5305) Signed-off-by: Dhwanil Patel <dhwanip@amazon.com> * Update ingest-attachment plugin dependencies: Apache Tika 3.6.0, Apache Mime4j 0.8.8, Apache Poi 5.2.3, Apache PdfBox 2.0.27 (#5448) Signed-off-by: Andriy Redko <andriy.redko@aiven.io> Signed-off-by: Andriy Redko <andriy.redko@aiven.io> * Enhance CheckpointState to support no-op replication (#5282) * CheckpointState enhanced to support no-op replication Signed-off-by: Ashish Singh <ssashish@amazon.com> Co-authored-by: Bukhtawar Khan<bukhtawa@amazon.com> * [BUG] org.opensearch.repositories.s3.RepositoryS3ClientYamlTestSuiteIT/test {yaml=repository_s3/20_repository_permanent_credentials/Snapshot and Restore with repository-s3 using permanent credentials} flaky: randomizing basePath (#5482) Signed-off-by: Andriy Redko <andriy.redko@aiven.io> Signed-off-by: Andriy Redko <andriy.redko@aiven.io> * [Bug] fix case sensitivity for wildcard queries (#5462) Fixes the wildcard query to not normalize the pattern when case_insensitive is set by the user. This is achieved by creating a new normalizedWildcardQuery method so that query_string queries (which do not support case sensitivity) can still normalize the pattern when the default analyzer is used; maintaining existing behavior. Signed-off-by: Nicholas Walter Knize <nknize@apache.org> * Support OpenSSL Provider with default Netty allocator (#5460) Signed-off-by: Andriy Redko <andriy.redko@aiven.io> Signed-off-by: Andriy Redko <andriy.redko@aiven.io> * Revert "build no-jdk distributions as part of release build (#4902)" (#5465) This reverts commit 8c9ca4e. It seems that this wasn't entirely the correct way and is currently blocking us from removing the `build.sh` from the `opensearch-build` repository (i.e. this `build.sh` here is not yet being used). See the discussion in opensearch-project/opensearch-build#2835 for further details. Signed-off-by: Ralph Ursprung <Ralph.Ursprung@avaloq.com> Signed-off-by: Ralph Ursprung <Ralph.Ursprung@avaloq.com> * Add max_shard_size parameter for Shrink API (fix supported version after backport) (#5503) Signed-off-by: Andriy Redko <andriy.redko@aiven.io> Signed-off-by: Andriy Redko <andriy.redko@aiven.io> * Sync CODEOWNERS with MAINTAINERS. (#5501) Signed-off-by: Daniel (dB.) Doubrovkine <dblock@amazon.com> Signed-off-by: Daniel (dB.) Doubrovkine <dblock@amazon.com> * Added jackson dependency to server (#5366) * Added jackson dependency to server Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Updated CHANGELOG Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Update build.gradle files Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Add RuntimePermission to fix errors Signed-off-by: Ryan Bogan <rbogan@amazon.com> Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Fix flaky test BulkIntegrationIT.testDeleteIndexWhileIndexing (#5491) Signed-off-by: Poojita Raj <poojiraj@amazon.com> Signed-off-by: Poojita Raj <poojiraj@amazon.com> * Add release notes for 2.4.1 (#5488) Signed-off-by: Xue Zhou <xuezhou@amazon.com> Signed-off-by: Xue Zhou <xuezhou@amazon.com> * Properly skip OnDemandBlockSnapshotIndexInputTests.testVariousBlockSize on Windows. (#5511) PR #5397 skipped this test in @before block but still frequently throws a TestCouldNotBeSkippedException. This is caused by the after block still executing and throwing an exception while cleaning the directory created at the path in @before. Moving the assumption to the individual test prevents this exception by ensuring the path exists. Signed-off-by: Marc Handalian <handalm@amazon.com> Signed-off-by: Marc Handalian <handalm@amazon.com> * Merge first batch of feature/extensions into main (#5347) * Merge first batch of feature/extensions into main Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Fixed CHANGELOG Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Fixed newline errors Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Renaming and CHANGELOG fixes Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Refactor extension loading into private method Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Removed skipValidation and added connectToExtensionNode method Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Remove unnecessary feature flag calls Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Renaming and exception handling Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Change latches to CompletableFuture Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Removed unnecessary validateSettingKey call Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Fix azure-core dependency Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Update SHAs Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Remove unintended dependency changes Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Removed dynamic settings regitration, removed info() method, and added NoopExtensionsManager Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Add javadoc Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Fixed spotless failure Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Removed NoopExtensionsManager Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Added functioning NoopExtensionsManager Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Added missing javadoc Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Remove forbiddenAPI Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Fix spotless Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Change logger.info to logger.error in handleException Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Fix ExtensionsManagerTests Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Removing unrelated change Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Update SHAs Signed-off-by: Ryan Bogan <rbogan@amazon.com> Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Bump commons-compress from 1.21 to 1.22 (#5520) Bumps commons-compress from 1.21 to 1.22. --- updated-dependencies: - dependency-name: org.apache.commons:commons-compress dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [Segment Replication] Trigger a round of replication for replica shards during peer recovery when segment replication is enabled (#5332) * Fix new added replica shards falling behind primary. Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com> * Trigger a round of replication during peer recovery when segment replication is enabled. Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com> * Remove unnecessary start replication overloaded method. Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com> * Add test for failure case and refactor some code. Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com> * Apply spotless check. Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com> * Addressing comments on the PR. Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com> * Remove unnecessary condition check. Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com> * Apply spotless check. Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com> * Add step listeners to resolve forcing round of segment replication. Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com> Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com> * Adding support to register settings dynamically (#5495) * Adding support to register settings dynamically Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Update CHANGELOG Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Removed unnecessary registerSetting methods Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Change setting registration order Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Add unregisterSettings method Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Remove unnecessary feature flag Signed-off-by: Ryan Bogan <rbogan@amazon.com> Signed-off-by: Ryan Bogan <rbogan@amazon.com> * Updated 1.3.7 release notes date (#5536) Signed-off-by: owaiskazi19 <owaiskazi19@gmail.com> Signed-off-by: owaiskazi19 <owaiskazi19@gmail.com> * Pre conditions check before updating weighted routing metadata (#4955) * Pre conditions check to allow weight updates for non decommissioned attribute Signed-off-by: Rishab Nahata <rnnahata@amazon.com> * Atomically update cluster state with decommission status and corresponding action (#5093) * Atomically update the cluster state with decommission status and its corresponding action in the same execute call Signed-off-by: Rishab Nahata <rnnahata@amazon.com> * Update Netty to 4.1.86.Final (#5529) Signed-off-by: Andriy Redko <andriy.redko@aiven.io> Signed-off-by: Andriy Redko <andriy.redko@aiven.io> * Update release date in 2.4.1 release notes (#5549) Signed-off-by: Suraj Singh <surajrider@gmail.com> Signed-off-by: Suraj Singh <surajrider@gmail.com> * Update 2.4.1 release notes (#5552) Signed-off-by: Andriy Redko <andriy.redko@aiven.io> Signed-off-by: Andriy Redko <andriy.redko@aiven.io> * Refactor fuzziness interface on query builders (#5433) * Refactor Object to Fuzziness type for all query builders Signed-off-by: noCharger <lingzhichu.clz@gmail.com> * Revise on bwc Signed-off-by: noCharger <lingzhichu.clz@gmail.com> * Update change log Signed-off-by: noCharger <lingzhichu.clz@gmail.com> Signed-off-by: noCharger <lingzhichu.clz@gmail.com> Co-authored-by: Daniel (dB.) Doubrovkine <dblock@amazon.com> * Upgrade lucene version (#5570) * Added bwc version 2.4.2 Signed-off-by: Daniel (dB.) Doubrovkine <dblock@amazon.com> * Added 2.4.2. Signed-off-by: Daniel (dB.) Doubrovkine <dblock@amazon.com> * Update Lucene snapshot to 9.5.0-snapshot-d5cef1c Signed-off-by: Suraj Singh <surajrider@gmail.com> * Update changelog entry Signed-off-by: Suraj Singh <surajrider@gmail.com> * Add 2.4.2 bwc version Signed-off-by: Suraj Singh <surajrider@gmail.com> * Internal changes post lucene upgrade Signed-off-by: Suraj Singh <surajrider@gmail.com> Signed-off-by: Daniel (dB.) Doubrovkine <dblock@amazon.com> Signed-off-by: Suraj Singh <surajrider@gmail.com> Co-authored-by: opensearch-ci-bot <opensearch-ci-bot@users.noreply.github.com> Co-authored-by: Daniel (dB.) Doubrovkine <dblock@amazon.com> * Add CI bundle pattern to distribution download (#5348) * Add CI bundle pattern for ivy repo Signed-off-by: Zelin Hao <zelinhao@amazon.com> * Gradle update Signed-off-by: Zelin Hao <zelinhao@amazon.com> * Extract path Signed-off-by: Zelin Hao <zelinhao@amazon.com> * Change with customDistributionDownloadType Signed-off-by: Zelin Hao <zelinhao@amazon.com> * Add default for exception handle Signed-off-by: Zelin Hao <zelinhao@amazon.com> * Add documentations Signed-off-by: Zelin Hao <zelinhao@amazon.com> Signed-off-by: Zelin Hao <zelinhao@amazon.com> * Bump protobuf-java from 3.21.9 to 3.21.11 in /plugins/repository-hdfs (#5519) * Bump protobuf-java from 3.21.9 to 3.21.11 in /plugins/repository-hdfs Bumps [protobuf-java](https://github.com/protocolbuffers/protobuf) from 3.21.9 to 3.21.11. - [Release notes](https://github.com/protocolbuffers/protobuf/releases) - [Changelog](https://github.com/protocolbuffers/protobuf/blob/main/generate_changelog.py) - [Commits](protocolbuffers/protobuf@v3.21.9...v3.21.11) --- updated-dependencies: - dependency-name: com.google.protobuf:protobuf-java dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * Updating SHAs Signed-off-by: dependabot[bot] <support@github.com> * Updated changelog Signed-off-by: Owais Kazi <owaiskazi19@gmail.com> Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Owais Kazi <owaiskazi19@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Owais Kazi <owaiskazi19@gmail.com> Co-authored-by: Suraj Singh <surajrider@gmail.com> Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com> Signed-off-by: Dhwanil Patel <dhwanip@amazon.com> Signed-off-by: Andriy Redko <andriy.redko@aiven.io> Signed-off-by: Ashish Singh <ssashish@amazon.com> Signed-off-by: Nicholas Walter Knize <nknize@apache.org> Signed-off-by: Ralph Ursprung <Ralph.Ursprung@avaloq.com> Signed-off-by: Daniel (dB.) Doubrovkine <dblock@amazon.com> Signed-off-by: Ryan Bogan <rbogan@amazon.com> Signed-off-by: Poojita Raj <poojiraj@amazon.com> Signed-off-by: Xue Zhou <xuezhou@amazon.com> Signed-off-by: Marc Handalian <handalm@amazon.com> Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: owaiskazi19 <owaiskazi19@gmail.com> Signed-off-by: Rishab Nahata <rnnahata@amazon.com> Signed-off-by: Suraj Singh <surajrider@gmail.com> Signed-off-by: noCharger <lingzhichu.clz@gmail.com> Signed-off-by: Zelin Hao <zelinhao@amazon.com> Signed-off-by: Owais Kazi <owaiskazi19@gmail.com> Co-authored-by: Rishikesh Pasham <62345295+Rishikesh1159@users.noreply.github.com> Co-authored-by: Dhwanil Patel <dhwanip@amazon.com> Co-authored-by: Andriy Redko <andriy.redko@aiven.io> Co-authored-by: Ashish <ssashish@amazon.com> Co-authored-by: Nick Knize <nknize@apache.org> Co-authored-by: Ralph Ursprung <39383228+rursprung@users.noreply.github.com> Co-authored-by: Daniel (dB.) Doubrovkine <dblock@amazon.com> Co-authored-by: Ryan Bogan <10944539+ryanbogan@users.noreply.github.com> Co-authored-by: Poojita Raj <poojiraj@amazon.com> Co-authored-by: Xue Zhou <85715413+xuezhou25@users.noreply.github.com> Co-authored-by: Marc Handalian <handalm@amazon.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Owais Kazi <owaiskazi19@gmail.com> Co-authored-by: Rishab Nahata <rnnahata@amazon.com> Co-authored-by: Suraj Singh <surajrider@gmail.com> Co-authored-by: Louis Chu <lingzhichu.clz@gmail.com> Co-authored-by: opensearch-ci-bot <opensearch-ci-bot@users.noreply.github.com> Co-authored-by: Zelin Hao <87548827+zelinh@users.noreply.github.com> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>

ryanbogan added 2 commits November 23, 2022 19:26

Added jackson dependency to server

5de4cb1

Signed-off-by: Ryan Bogan <rbogan@amazon.com>

Updated CHANGELOG

181da45

Signed-off-by: Ryan Bogan <rbogan@amazon.com>

Merge branch 'main' into add_jackson_to_server

a814b11

Update build.gradle files

0f6a154

Signed-off-by: Ryan Bogan <rbogan@amazon.com>

ryanbogan mentioned this pull request Dec 2, 2022

Merge first batch of feature/extensions into main #5347

Merged

6 tasks

Merge branch 'main' into add_jackson_to_server

55107cf

Add RuntimePermission to fix errors

69f7276

Signed-off-by: Ryan Bogan <rbogan@amazon.com>

ryanbogan marked this pull request as ready for review December 7, 2022 22:46

ryanbogan requested review from a team and reta as code owners December 7, 2022 22:46

owaiskazi19 reviewed Dec 7, 2022

View reviewed changes

saratvemulapalli approved these changes Dec 9, 2022

View reviewed changes

dblock approved these changes Dec 9, 2022

View reviewed changes

dblock merged commit c6663fd into opensearch-project:main Dec 9, 2022

dblock added the backport 2.x Backport to 2.x branch label Dec 9, 2022

This was referenced Dec 9, 2022

Evaluate the permission change for jackson-databind #5504

Open

Evaluate the permission change for jackson-databind #5505

Closed

ryanbogan mentioned this pull request Dec 11, 2022

[Backport 2.x] Added jackson dependency to server #5517

Merged

6 tasks

This was referenced Dec 12, 2022

[BUG] Plugin installation fails due to jar hell opensearch-project/security#2324

Closed

Remove jackson-databind and jackson-annotations dependency now coming from core opensearch-project/security#2325

Merged

RyanL1997 mentioned this pull request Dec 12, 2022

[BUG] Plugin installation fails due to jar hell #5526

Closed

ryanbogan deleted the add_jackson_to_server branch December 12, 2022 20:09

YANG-DB mentioned this pull request Dec 14, 2022

Remove jackson databind and jackson annotations dependency now coming from core #2325 opensearch-project/sql-jdbc#37

Closed

6 tasks

reta mentioned this pull request Dec 15, 2022

Bump to 3.0 opensearch-project/notifications#556

Merged

1 task

This was referenced Dec 28, 2022

Remove jackson-databind and jackson-annotations dependencies now coming from core opensearch-project/ml-commons#651

Closed

Remove jackson-databind and jackson-annotations dependencies now coming from core opensearch-project/ml-commons#652

Merged

ryanbogan mentioned this pull request Jan 9, 2023

Revert 'Added jackson dependency to server" and change extension reading #5768

Merged

6 tasks

dbwiddis mentioned this pull request Jan 14, 2023

[BUG] OpenSearch installation failure during discovery-ec2 plugin load #5862

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Added jackson dependency to server #5366

Added jackson dependency to server #5366

ryanbogan commented Nov 23, 2022

github-actions bot commented Nov 23, 2022

github-actions bot commented Nov 23, 2022

github-actions bot commented Nov 25, 2022

reta commented Nov 25, 2022

github-actions bot commented Nov 25, 2022

ryanbogan commented Nov 25, 2022

github-actions bot commented Nov 25, 2022

dblock commented Dec 6, 2022

ryanbogan commented Dec 6, 2022

dblock commented Dec 6, 2022

github-actions bot commented Dec 6, 2022

ryanbogan commented Dec 6, 2022

github-actions bot commented Dec 6, 2022

github-actions bot commented Dec 7, 2022

codecov-commenter commented Dec 7, 2022

owaiskazi19 Dec 7, 2022

ryanbogan Dec 7, 2022 •

edited

Loading

dblock Dec 9, 2022

owaiskazi19 Dec 14, 2022

ryanbogan Dec 14, 2022

dblock Dec 9, 2022

opensearch-trigger-bot bot commented Dec 9, 2022

dblock commented Dec 9, 2022

Added jackson dependency to server #5366

Added jackson dependency to server #5366

Conversation

ryanbogan commented Nov 23, 2022

Description

Check List

github-actions bot commented Nov 23, 2022

Gradle Check (Jenkins) Run Completed with:

github-actions bot commented Nov 23, 2022

Gradle Check (Jenkins) Run Completed with:

github-actions bot commented Nov 25, 2022

Gradle Check (Jenkins) Run Completed with:

reta commented Nov 25, 2022

github-actions bot commented Nov 25, 2022

Gradle Check (Jenkins) Run Completed with:

ryanbogan commented Nov 25, 2022

github-actions bot commented Nov 25, 2022

Gradle Check (Jenkins) Run Completed with:

dblock commented Dec 6, 2022

ryanbogan commented Dec 6, 2022

dblock commented Dec 6, 2022

github-actions bot commented Dec 6, 2022

Gradle Check (Jenkins) Run Completed with:

ryanbogan commented Dec 6, 2022

github-actions bot commented Dec 6, 2022

Gradle Check (Jenkins) Run Completed with:

github-actions bot commented Dec 7, 2022

Gradle Check (Jenkins) Run Completed with:

codecov-commenter commented Dec 7, 2022

Codecov Report

owaiskazi19 Dec 7, 2022

Choose a reason for hiding this comment

ryanbogan Dec 7, 2022 • edited Loading

Choose a reason for hiding this comment

dblock Dec 9, 2022

Choose a reason for hiding this comment

owaiskazi19 Dec 14, 2022

Choose a reason for hiding this comment

ryanbogan Dec 14, 2022

Choose a reason for hiding this comment

dblock Dec 9, 2022

Choose a reason for hiding this comment

opensearch-trigger-bot bot commented Dec 9, 2022

dblock commented Dec 9, 2022

ryanbogan Dec 7, 2022 •

edited

Loading