Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 1.3] Update org.springframework:spring-core to 5.3.20 #1871

Merged
merged 1 commit into from
Jun 6, 2022

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 4c9e403 from #1850

@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team June 2, 2022 22:46
@codecov-commenter
Copy link

codecov-commenter commented Jun 2, 2022

Codecov Report

Merging #1871 (3fd34e4) into 1.3 (e7fa52e) will increase coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                1.3    #1871      +/-   ##
============================================
+ Coverage     64.64%   64.70%   +0.05%     
- Complexity     3219     3222       +3     
============================================
  Files           247      247              
  Lines         17363    17363              
  Branches       3086     3086              
============================================
+ Hits          11224    11234      +10     
+ Misses         4590     4583       -7     
+ Partials       1549     1546       -3     
Impacted Files Coverage Δ
...earch/security/ssl/util/SSLConnectionTestUtil.java 95.45% <0.00%> (+2.27%) ⬆️
...nsearch/security/dlic/rest/api/AuditApiAction.java 68.08% <0.00%> (+4.25%) ⬆️
...urity/ssl/transport/SecuritySSLNettyTransport.java 73.40% <0.00%> (+4.25%) ⬆️
...ecurity/configuration/StaticResourceException.java 25.00% <0.00%> (+25.00%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e7fa52e...3fd34e4. Read the comment docs.

Copy link
Member

@peternied peternied left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I need to make sure this will properly install in OpenSearch 1.3.3+ before merging

@peternied
Copy link
Member

Following the process from plugin-install this looks like its working correctly

curl -k https://admin:admin@localhost:9200 
{
  "name" : "smoketestnode",
  "cluster_name" : "opensearch",
  "cluster_uuid" : "pxH5B-paTpmAHfgRzjHUsQ",
  "version" : {
    "distribution" : "opensearch",
    "number" : "1.3.3",
    "build_type" : "tar",
    "build_hash" : "6d63fc707f6010c5980ec5cb51324856caf1f03d",
    "build_date" : "2022-06-06T18:22:57.556003Z",
    "build_snapshot" : false,
    "lucene_version" : "8.10.1",
    "minimum_wire_compatibility_version" : "6.8.0",
    "minimum_index_compatibility_version" : "6.0.0-beta1"
  },
  "tagline" : "The OpenSearch Project: https://opensearch.org/"
}

@peternied peternied merged commit ffaf81c into 1.3 Jun 6, 2022
@peternied peternied deleted the backport/backport-1850-to-1.3 branch June 6, 2022 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants