Skip to content

Conversation

@gargmanik13
Copy link
Contributor

Description

  • Added IndexMetadataModel to metadata lib along with other required dependencies
  • Created ContextModel, SettingsModel and IndexMetadataModel.
  • Created MetadataWriteable interface for serialization of dependencies which can't or we don't want to move to metadata lib

Related Issues

Resolves #13197

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

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.

…Model

Signed-off-by: Manik Garg <gargmanik1317@gmail.com>
@github-actions github-actions bot added Cluster Manager enhancement Enhancement or improvement to existing feature or request Roadmap:Modular Architecture Project-wide roadmap label labels Jan 21, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 21, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@gargmanik13 gargmanik13 force-pushed the md_lib_stateless_cm_2 branch from a72504b to eb45ca7 Compare January 21, 2026 11:41
@github-actions
Copy link
Contributor

❌ Gradle check result for eb45ca7: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@gargmanik13 gargmanik13 force-pushed the md_lib_stateless_cm_2 branch from eb45ca7 to a932a83 Compare January 22, 2026 08:57
@github-actions
Copy link
Contributor

❌ Gradle check result for a932a83: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

…pendencies

Signed-off-by: Manik Garg <gargmanik1317@gmail.com>
@gargmanik13 gargmanik13 force-pushed the md_lib_stateless_cm_2 branch from a932a83 to 71188da Compare January 22, 2026 10:06
@github-actions
Copy link
Contributor

❌ Gradle check result for 71188da: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@gargmanik13
Copy link
Contributor Author

❌ Gradle check result for 71188da: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Test Result (3 failures / +3)
org.opensearch.common.settings.ConsistentSettingsServiceTests.testSingleStringSetting
org.opensearch.common.settings.ConsistentSettingsServiceTests.testSingleAffixSetting
org.opensearch.common.settings.ConsistentSettingsServiceTests.testStringAndAffixSettings

java.lang.RuntimeException: The "PBKDF2WithHmacSHA512" algorithm is required for consistent secure settings' hashes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Cluster Manager enhancement Enhancement or improvement to existing feature or request Roadmap:Modular Architecture Project-wide roadmap label

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

[RFC] Refactor Metadata management in Cluster Manager code as separate lib

1 participant