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.x] Remove custom-codecs plugin (#9944) #9948

Merged

Conversation

andrross
Copy link
Member

@andrross andrross commented Sep 8, 2023

Backports 5ad472f to 2.x

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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.

This plugin has been moved to its own [repository][1].

[1]: https://github.com/opensearch-project/custom-codecs

Signed-off-by: Andrew Ross <andrross@amazon.com>
(cherry picked from commit 5ad472f)
@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2023

Compatibility status:

Checks if related components are compatible with change 6a9e8db

Incompatible components

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/neural-search.git]

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2023

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.snapshots.DedicatedClusterSnapshotRestoreIT.testIndexDeletionDuringSnapshotCreationInQueue

@codecov
Copy link

codecov bot commented Sep 8, 2023

Codecov Report

Merging #9948 (6a9e8db) into 2.x (5505f09) will decrease coverage by 0.05%.
Report is 1 commits behind head on 2.x.
The diff coverage is 81.52%.

@@             Coverage Diff              @@
##                2.x    #9948      +/-   ##
============================================
- Coverage     70.77%   70.73%   -0.05%     
- Complexity    58164    58175      +11     
============================================
  Files          4817     4810       -7     
  Lines        275315   275125     -190     
  Branches      40456    40433      -23     
============================================
- Hits         194867   194606     -261     
- Misses        63724    63874     +150     
+ Partials      16724    16645      -79     
Files Changed Coverage Δ
.../opensearch/telemetry/tracing/noop/NoopTracer.java 72.72% <ø> (+18.18%) ⬆️
...g/opensearch/transport/nio/NioTransportPlugin.java 25.00% <ø> (ø)
...a/org/opensearch/common/network/NetworkModule.java 92.00% <ø> (ø)
...ain/java/org/opensearch/plugins/NetworkPlugin.java 100.00% <ø> (ø)
...racing/ThreadContextBasedTracerContextStorage.java 93.10% <0.00%> (+20.68%) ⬆️
...etry/tracing/listener/TraceableActionListener.java 58.82% <58.82%> (ø)
.../org/opensearch/telemetry/tracing/SpanBuilder.java 91.66% <66.66%> (+3.43%) ⬆️
...lemetry/tracing/channels/TraceableHttpChannel.java 70.58% <70.58%> (ø)
...lemetry/tracing/channels/TraceableRestChannel.java 84.21% <84.21%> (ø)
...rg/opensearch/telemetry/tracing/DefaultTracer.java 100.00% <100.00%> (+6.45%) ⬆️
... and 7 more

... and 468 files with indirect coverage changes

@andrross andrross merged commit 7cb6851 into opensearch-project:2.x Sep 8, 2023
36 checks passed
@andrross andrross deleted the backport/backport-9944-to-2.x branch September 8, 2023 22:18
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.

2 participants