-
Notifications
You must be signed in to change notification settings - Fork 85
Conversation
Signed-off-by: Lakshita Jain <lakshita.jain@seagate.com>
Thanks for your contribution in opening this pull request! Now you can be rewarded with a CORTX sticker by requesting cortx sticker |
Can one of the admins verify this patch? |
Signed-off-by: Lakshita Jain <lakshita.jain@seagate.com>
Signed-off-by: Lakshita Jain <lakshita.jain@seagate.com>
Signed-off-by: Lakshita Jain <lakshita.jain@seagate.com>
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.
LGTM
Signed-off-by: Lakshita Jain <lakshita.jain@seagate.com>
Signed-off-by: Lakshita Jain <lakshita.jain@seagate.com>
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.
please make sure , After deployment API is working and returning correct topology.
Paste the results in PR description.
@tanujashinde0405 : here's the output
|
Signed-off-by: Lakshita Jain <lakshita.jain@seagate.com>
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.
LGTM
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.
+1
Thanks for your contribution to CORTX! 🎉 |
Signed-off-by: Lakshita Jain lakshita.jain@seagate.com
Problem Statement
Design
Approach followed for generating query deployment topology:
Coding
Testing
Review Checklist
Before posting the PR please ensure:
KVstore/Confstore feature (changes/fixes) checklist
Confirm changes are made for:
Documentation