search

Seagate / cortx-hare

CORTX Hare configures Motr object store, starts/stops Motr services, and notifies Motr of service and device faults.
https://github.com/Seagate/cortx
Apache License 2.0
13 stars 80 forks source link

CORTX-29741: replace obsolete conf-store keys #2054

Closed lakshita-jain closed 2 years ago

lakshita-jain commented 2 years ago

keys replaced old to new: cvg_count to num_cvg storage_set_count to num_storage_set Also changed: TMPL_CVG_COUNT TO TMPL_NUM_CVG TMPL_STORAGESET_COUNT TO TMPL_NUM_STORAGESET Signed-off-by: Lakshita Jain lakshita.jain@seagate.com

Signed-off-by: Lakshita Jain lakshita.jain@seagate.com

cortx-admin commented 2 years ago

Can one of the admins verify this patch?

lakshita-jain commented 2 years ago

3N deployment with this feature branch was successful https://eos-jenkins.colo.seagate.com/job/Provisioner/job/N-Node_CORTX_Deployment_on_K8s_Cluster/203/console

pavankrishnat commented 2 years ago

@mssawant started premerge test manually https://eos-jenkins.colo.seagate.com/job/Cortx-PR-Build/job/Component-Test/job/Hare-PreMerge-Test/1417/console

pavankrishnat commented 2 years ago

@mssawant, PreMerge Test passed with changes in branch https://github.com/lakshita-jain/cortx-hare/tree/CORTX-29741 commitID#404f229f0c5311f2062a9df8373affd8b4899bf4 Hare-PreMerge-Test#1417_console.log

mssawant commented 2 years ago

somehow pre-merge tests from cicd are not completing, Hare team (@pavankrishnat) tried triggering pre-merge tests explicitly which completed successfully as mentioned above. Going ahead with merging.