Skip to content
This repository was archived by the owner on Aug 2, 2022. It is now read-only.

Conversation

@dbbaughe
Copy link
Contributor

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@codecov
Copy link

codecov bot commented Nov 12, 2020

Codecov Report

Merging #322 (c0502cb) into master (981614b) will increase coverage by 0.25%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #322      +/-   ##
============================================
+ Coverage     73.80%   74.05%   +0.25%     
- Complexity     1166     1169       +3     
============================================
  Files           173      173              
  Lines          5894     5894              
  Branches        898      898              
============================================
+ Hits           4350     4365      +15     
+ Misses         1078     1064      -14     
+ Partials        466      465       -1     
Impacted Files Coverage Δ Complexity Δ
...nt/indexstatemanagement/model/destination/Slack.kt 40.90% <0.00%> (-13.64%) 2.00% <0.00%> (-2.00%)
...anagement/indexstatemanagement/model/Transition.kt 63.07% <0.00%> (-4.62%) 5.00% <0.00%> (ø%)
...icsearch/indexmanagement/IndexManagementIndices.kt 62.06% <0.00%> (-3.45%) 10.00% <0.00%> (ø%)
...rch/indexmanagement/rollup/model/RollupMetadata.kt 27.97% <0.00%> (+1.78%) 6.00% <0.00%> (+3.00%)
...statemanagement/model/destination/CustomWebhook.kt 94.20% <0.00%> (+28.98%) 14.00% <0.00%> (+2.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 981614b...c0502cb. Read the comment docs.

@thalurur
Copy link
Contributor

Just out of curiosity what is the security for? node to node communication is encrypted?

@dbbaughe
Copy link
Contributor Author

@thalurur
Yes, requires all traffic to be https from client and adds support for users (and everything else security plugin adds).

@dbbaughe dbbaughe merged commit f173dbe into opendistro-for-elasticsearch:master Nov 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants