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 2.4] Fix test to use a file from another temp directory #5163

Merged
merged 1 commit into from
Nov 9, 2022

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 9970617 from #5158.

The test used an absolute path which is incorrect as it made a platform dependent assumption. This change fixes that.

Signed-off-by: Rabi Panda <adnapibar@gmail.com>

Signed-off-by: Rabi Panda <adnapibar@gmail.com>
(cherry picked from commit 9970617)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot opensearch-trigger-bot bot requested review from a team and reta as code owners November 8, 2022 23:14
@codecov-commenter
Copy link

codecov-commenter commented Nov 8, 2022

Codecov Report

Merging #5163 (fae5b38) into 2.4 (9a85769) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                2.4    #5163      +/-   ##
============================================
- Coverage     70.70%   70.61%   -0.09%     
+ Complexity    58364    58312      -52     
============================================
  Files          4700     4700              
  Lines        279108   279109       +1     
  Branches      40748    40748              
============================================
- Hits         197335   197106     -229     
- Misses        65312    65539     +227     
- Partials      16461    16464       +3     
Impacted Files Coverage Δ
.../index/shard/IndexShardNotRecoveringException.java 0.00% <0.00%> (-100.00%) ⬇️
...adonly/AddIndexBlockClusterStateUpdateRequest.java 0.00% <0.00%> (-75.00%) ⬇️
...readonly/TransportVerifyShardIndexBlockAction.java 9.75% <0.00%> (-58.54%) ⬇️
...n/admin/indices/readonly/AddIndexBlockRequest.java 17.85% <0.00%> (-53.58%) ⬇️
...ava/org/opensearch/action/NoSuchNodeException.java 0.00% <0.00%> (-50.00%) ⬇️
...indices/readonly/TransportAddIndexBlockAction.java 20.68% <0.00%> (-41.38%) ⬇️
.../admin/cluster/reroute/ClusterRerouteResponse.java 60.00% <0.00%> (-40.00%) ⬇️
...earch/client/indices/GetIndexTemplatesRequest.java 46.15% <0.00%> (-38.47%) ⬇️
...luster/routing/allocation/RoutingExplanations.java 62.06% <0.00%> (-37.94%) ⬇️
...java/org/opensearch/threadpool/ThreadPoolInfo.java 56.25% <0.00%> (-37.50%) ⬇️
... and 488 more

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

@github-actions
Copy link
Contributor

github-actions bot commented Nov 9, 2022

Gradle Check (Jenkins) Run Completed with:

@mch2 mch2 merged commit ebd4d85 into 2.4 Nov 9, 2022
@mch2 mch2 deleted the backport/backport-5158-to-2.4 branch November 9, 2022 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants