-
Notifications
You must be signed in to change notification settings - Fork 32
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
Cluster tools #1124
Cluster tools #1124
Conversation
Signed-off-by: Andre Kurait <[email protected]>
2ea3720
to
f296eb8
Compare
Signed-off-by: Andre Kurait <[email protected]>
Signed-off-by: Andre Kurait <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1124 +/- ##
============================================
+ Coverage 80.51% 80.63% +0.12%
- Complexity 2871 2910 +39
============================================
Files 393 399 +6
Lines 14554 14829 +275
Branches 1000 1007 +7
============================================
+ Hits 11718 11958 +240
- Misses 2229 2260 +31
- Partials 607 611 +4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
...ture/dockerSolution/src/main/docker/migrationConsole/cluster_tools/src/cluster_tools/main.py
Outdated
Show resolved
Hide resolved
...ture/dockerSolution/src/main/docker/migrationConsole/cluster_tools/src/cluster_tools/main.py
Show resolved
Hide resolved
...ture/dockerSolution/src/main/docker/migrationConsole/cluster_tools/src/cluster_tools/main.py
Outdated
Show resolved
Hide resolved
...ure/dockerSolution/src/main/docker/migrationConsole/cluster_tools/src/cluster_tools/utils.py
Outdated
Show resolved
Hide resolved
...ure/dockerSolution/src/main/docker/migrationConsole/cluster_tools/src/cluster_tools/utils.py
Show resolved
Hide resolved
...ure/dockerSolution/src/main/docker/migrationConsole/cluster_tools/src/tools/change_shards.py
Outdated
Show resolved
Hide resolved
parser.add_argument("primary_shards", type=int, help="Number of primary shards for the index") | ||
|
||
|
||
def create_index(env: Environment, index_name: str, primary_shards: int) -> str: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm curious if it will be sufficiently useful for people to be able to create an index and have only one setting specified. My guess is that most of the time there are mappings, other settings, etc. With that said, I don't think it's an issue to include it and we can add more flexibility later.
Signed-off-by: Andre Kurait <[email protected]>
Signed-off-by: Andre Kurait <[email protected]>
Signed-off-by: Andre Kurait <[email protected]>
7bfc76c
to
be19bc3
Compare
Description
Creates library cluster_tools for an easy to extend library of tools around opensearch business logic for migrations.
Ships with tools: create_index, change_shards, enable/disable_compatibility_mode
Was getting some codecov failures, added retry akin to security repo
Issues Resolved
https://opensearch.atlassian.net/browse/MIGRATIONS-2201
Is this a backport? If so, please add backport PR # and/or commits #
Testing
Unit testing and some manual testing
Check List
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.