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

Fix integ tests run with security #284

Merged
merged 3 commits into from
Oct 25, 2020

Conversation

skkosuri-amzn
Copy link
Contributor

Issue #, if available:

Fix integ tests run with security
Description of changes:

Fix integ tests run with security

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 Oct 25, 2020

Codecov Report

Merging #284 into master will increase coverage by 0.15%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #284      +/-   ##
============================================
+ Coverage     79.35%   79.50%   +0.15%     
- Complexity      196      197       +1     
============================================
  Files           151      151              
  Lines          5211     5211              
  Branches        673      673              
============================================
+ Hits           4135     4143       +8     
+ Misses          719      713       -6     
+ Partials        357      355       -2     
Impacted Files Coverage Δ Complexity Δ
...endistroforelasticsearch/alerting/MonitorRunner.kt 78.40% <0.00%> (+0.40%) 0.00% <0.00%> (ø%)
...roforelasticsearch/alerting/alerts/AlertIndices.kt 67.71% <0.00%> (+1.57%) 0.00% <0.00%> (ø%)
...distroforelasticsearch/alerting/core/JobSweeper.kt 72.19% <0.00%> (+2.13%) 35.00% <0.00%> (+1.00%)
...ing/model/destination/DestinationContextFactory.kt 70.37% <0.00%> (+3.70%) 0.00% <0.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 8bda814...b5b9fe6. Read the comment docs.

@skkosuri-amzn skkosuri-amzn merged commit f443099 into opendistro-for-elasticsearch:master Oct 25, 2020
tlfeng pushed a commit that referenced this pull request Feb 6, 2021
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