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

fix failed IT cases for AD #320

Merged
merged 1 commit into from
Dec 9, 2020

Conversation

ylwu-amzn
Copy link
Contributor

Issue #, if available:

Description of changes:
Fix failed IT cases for AD monitor.

Test with security enabled
./gradlew integTest -Dtests.rest.cluster=localhost:9200 -Dtests.cluster=localhost:9200 -Dtests.clustername=es-integrationtest -Dhttps=true -Duser=admin -Dpassword=admin -Dsecurity=true

Test with security disabled
./gradlew integTest -Dtests.rest.cluster=localhost:9200 -Dtests.cluster=localhost:9200 -Dtests.clustername=es-integrationtest

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 Dec 9, 2020

Codecov Report

Merging #320 (3c2d877) into master (37d537d) will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #320      +/-   ##
============================================
+ Coverage     79.32%   79.33%   +0.01%     
+ Complexity      203      202       -1     
============================================
  Files           151      151              
  Lines          5324     5324              
  Branches        701      701              
============================================
+ Hits           4223     4224       +1     
+ Misses          727      725       -2     
- Partials        374      375       +1     
Impacted Files Coverage Δ Complexity Δ
...ing/destination/client/DestinationEmailClient.java 72.50% <0.00%> (-5.00%) 7.00% <0.00%> (-1.00%)
...distroforelasticsearch/alerting/core/JobSweeper.kt 72.19% <0.00%> (+0.53%) 35.00% <0.00%> (ø%)
...roforelasticsearch/alerting/alerts/AlertIndices.kt 67.17% <0.00%> (+1.52%) 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 37d537d...3c2d877. Read the comment docs.

@lezzago lezzago merged commit 77ffae2 into opendistro-for-elasticsearch:master Dec 9, 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