Seagate / cortx-k8s

CORTX Kubernetes Orchestration Repository
https://github.com/Seagate/cortx
Apache License 2.0
6 stars 47 forks source link

CORTX-32842: Support multiple cortx-control pods #339

Closed walterlopatka closed 2 years ago

walterlopatka commented 2 years ago

Description

This change provides support for multiple cortx-control pods in a cluster. This is supported in helm value: control.replicaCount. There is no support in solution.yaml for this at this time.

Type of change

Applicable issues

CORTX image version requirements

N/A

How was this tested?

  1. Deploy with 1 pod, 3 pods, 10 pods
  2. Verify that all cortx-control pods are serving data
  3. Verify that all cortx-control pods are in sync (all accept write requests, all return same GET responses)
  4. Run basic k8s deploy regression test.

Additional information

There may be issues with running multiple cortx-control pods in terms of the implementation of those containers. I have done basic functional test, but I leave it to the team responsible for cortx-control implementation to do due diligence in developer test.

Checklist

If this change requires newer CORTX or third party image versions:

If this change addresses a CORTX Jira issue:

walterlopatka commented 2 years ago

I have completed testing. I think I have addressed all the comments and this should be ready to go. @keithpine , thanks for the thorough review.