Skip to content

Conversation

cwperks
Copy link
Member

@cwperks cwperks commented Feb 22, 2023

Description

Reverts #2452 in response to the reversion from core (opensearch-project/OpenSearch#6431).

This retains the change in 2.x, but reverts for 2.6.

Check List

  • New functionality includes testing
  • New functionality has been documented
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Craig Perkins <cwperx@amazon.com>
Copy link
Contributor

@stephen-crawford stephen-crawford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Craig.

@codecov-commenter
Copy link

codecov-commenter commented Feb 22, 2023

Codecov Report

❗ No coverage uploaded for pull request base (2.6@b2f2a6d). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 7cdda5c differs from pull request most recent head bdb3c9d. Consider uploading reports for the commit bdb3c9d to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@          Coverage Diff           @@
##             2.6    #2460   +/-   ##
======================================
  Coverage       ?   61.00%           
  Complexity     ?     3246           
======================================
  Files          ?      258           
  Lines          ?    18130           
  Branches       ?     3231           
======================================
  Hits           ?    11060           
  Misses         ?     5494           
  Partials       ?     1576           

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

@cwperks
Copy link
Member Author

cwperks commented Feb 22, 2023

Merging this now as all checks, but MacOS have succeeded. MacOS is not required and has been removed on main.

@cwperks cwperks merged commit 222455f into opensearch-project:2.6 Feb 22, 2023
wuychn pushed a commit to ochprince/security that referenced this pull request Mar 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants