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] Change filenames for IndexMetadata and Manifest [#10557] #10700

Merged

Conversation

dhwanilpatel
Copy link
Contributor

@dhwanilpatel dhwanilpatel commented Oct 18, 2023

Description

This PR includes backporting of below PR:

Changes done during backporting:

  • Resolved conflicts of change log.

Related Issues

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)
  • Public documentation issue/PR created

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.

…0557)

* Change filenames for IndexMetadata and Manifest

Signed-off-by: Dhwanil Patel <[email protected]>
@dhwanilpatel dhwanilpatel changed the title [Backport] [Remote cluster state] Upload of global metadata and file name change for IndexMetadata/Manifest file [Backport] Change filenames for IndexMetadata and Manifest [#10557] Oct 18, 2023
@dhwanilpatel dhwanilpatel force-pushed the backport_file_name_changes branch from c74102f to 81133b8 Compare October 18, 2023 14:15
@github-actions
Copy link
Contributor

github-actions bot commented Oct 18, 2023

Compatibility status:

Checks if related components are compatible with change 4944ab6

Incompatible components

Skipped components

Compatible components

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

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

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

@codecov
Copy link

codecov bot commented Oct 18, 2023

Codecov Report

Merging #10700 (4944ab6) into 2.x (e611185) will decrease coverage by 0.01%.
Report is 7 commits behind head on 2.x.
The diff coverage is 72.43%.

@@             Coverage Diff              @@
##                2.x   #10700      +/-   ##
============================================
- Coverage     70.93%   70.92%   -0.01%     
+ Complexity    58575    58556      -19     
============================================
  Files          4830     4830              
  Lines        276453   276580     +127     
  Branches      40588    40610      +22     
============================================
+ Hits         196091   196172      +81     
- Misses        63676    63709      +33     
- Partials      16686    16699      +13     
Files Coverage Δ
...rg/opensearch/telemetry/tracing/DefaultTracer.java 100.00% <100.00%> (ø)
.../opensearch/telemetry/tracing/noop/NoopTracer.java 87.50% <100.00%> (+1.78%) ⬆️
...ch/repositories/gcs/GoogleCloudStorageService.java 82.43% <100.00%> (ø)
...search/telemetry/metrics/OTelMetricsTelemetry.java 100.00% <100.00%> (ø)
...rg/opensearch/telemetry/tracing/OTelTelemetry.java 100.00% <100.00%> (ø)
...search/telemetry/tracing/OTelTracingTelemetry.java 95.83% <100.00%> (+0.37%) ⬆️
...rg/opensearch/common/settings/ClusterSettings.java 92.85% <100.00%> (ø)
...va/org/opensearch/telemetry/TelemetrySettings.java 90.90% <100.00%> (+3.40%) ⬆️
...rg/opensearch/telemetry/tracing/WrappedTracer.java 78.57% <100.00%> (+1.64%) ⬆️
...lemetry/tracing/channels/TraceableHttpChannel.java 54.54% <100.00%> (ø)
... and 10 more

... and 446 files with indirect coverage changes

@shwetathareja shwetathareja changed the title [Backport] Change filenames for IndexMetadata and Manifest [#10557] [Backport 2.x] Change filenames for IndexMetadata and Manifest [#10557] Oct 18, 2023
@shwetathareja shwetathareja merged commit 89a655e into opensearch-project:2.x Oct 18, 2023
18 checks passed
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