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] Use BuildParams.isCi() instead of checking env var (#5368) #5375

Merged
merged 1 commit into from
Nov 25, 2022

Conversation

reta
Copy link
Collaborator

@reta reta commented Nov 24, 2022

Backport of #5368 to 1.3

@reta reta added backport PRs or issues specific to backporting features or enhancments skip-changelog labels Nov 24, 2022
@reta reta requested a review from andrross November 24, 2022 15:03
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

…t#5368)

Signed-off-by: Andrew Ross <[email protected]>

Signed-off-by: Andrew Ross <[email protected]>
Signed-off-by: Andriy Redko <[email protected]>
@reta reta force-pushed the backport/backport-5368-to-1.3 branch from f78f90f to aad43db Compare November 24, 2022 15:13
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov-commenter
Copy link

Codecov Report

Merging #5375 (aad43db) into 1.3 (f3e2d7f) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                1.3    #5375      +/-   ##
============================================
- Coverage     77.88%   77.85%   -0.03%     
+ Complexity    63452    63403      -49     
============================================
  Files          4454     4454              
  Lines        274943   274943              
  Branches      41166    41166              
============================================
- Hits         214132   214058      -74     
- Misses        43898    43964      +66     
- Partials      16913    16921       +8     
Impacted Files Coverage Δ
...regations/pipeline/SimulatedAnealingMinimizer.java 0.00% <0.00%> (-93.11%) ⬇️
...ava/org/opensearch/action/NoSuchNodeException.java 0.00% <0.00%> (-50.00%) ⬇️
...ch/transport/ReceiveTimeoutTransportException.java 50.00% <0.00%> (-50.00%) ⬇️
.../opensearch/client/indices/CloseIndexResponse.java 55.55% <0.00%> (-34.57%) ⬇️
...ion/admin/cluster/node/info/PluginsAndModules.java 53.12% <0.00%> (-34.38%) ⬇️
...opensearch/transport/TransportResponseHandler.java 75.00% <0.00%> (-25.00%) ⬇️
...min/cluster/tasks/PendingClusterTasksResponse.java 35.00% <0.00%> (-25.00%) ⬇️
...g/opensearch/geometry/utils/StandardValidator.java 30.30% <0.00%> (-24.25%) ⬇️
...rc/main/java/org/opensearch/monitor/os/OsInfo.java 64.70% <0.00%> (-19.61%) ⬇️
...n/java/org/opensearch/transport/TransportInfo.java 74.13% <0.00%> (-18.97%) ⬇️
... and 431 more

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

@reta reta merged commit 7828029 into opensearch-project:1.3 Nov 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport PRs or issues specific to backporting features or enhancments skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants