Skip to content
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

feat(HA): adding support to have controller in HA #31

Merged
merged 2 commits into from
Jan 6, 2020

Conversation

pawanpraka1
Copy link
Contributor

@pawanpraka1 pawanpraka1 commented Dec 31, 2019

We can have more than one controller in the system, but only one will
be the master and others will be slave. Once master is down, one of the slave will
take over via lease mechanism and start provisioning/deprovisioning the volumes.

controller pod will have anti-affinity so that no two pods get scheduled on the same node. Also keeping the default replica to 1, if HA feature is required, we can change replica count to 2(or more).

Signed-off-by: Pawan [email protected]

@pawanpraka1 pawanpraka1 requested a review from kmova December 31, 2019 11:29
@codecov-io
Copy link

codecov-io commented Dec 31, 2019

Codecov Report

Merging #31 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #31   +/-   ##
=======================================
  Coverage   83.33%   83.33%           
=======================================
  Files           1        1           
  Lines          72       72           
=======================================
  Hits           60       60           
  Misses         12       12

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dfe4631...3ef10dd. Read the comment docs.

@pawanpraka1 pawanpraka1 changed the title [WIP] feat(HA): adding supportto have controller in HA [WIP] feat(HA): adding support to have controller in HA Dec 31, 2019
We can have more than one controller in the system, but only one will
be the master and others will be slave. Once master is down, one of the slave will
take over via lease mechanism and start provisioning/deprovisioning the volumes.

Signed-off-by: Pawan <[email protected]>
@pawanpraka1 pawanpraka1 added this to the v0.3.0 milestone Jan 3, 2020
@pawanpraka1 pawanpraka1 force-pushed the lease branch 3 times, most recently from 5445cc5 to 2d275f4 Compare January 6, 2020 07:36
so that no two pods get scheduled on the same node. Also keeping
the default replica to 1, if HA feature is required, we can change
replica count to 2(or more).

Signed-off-by: Pawan <[email protected]>
@pawanpraka1 pawanpraka1 changed the title [WIP] feat(HA): adding support to have controller in HA feat(HA): adding support to have controller in HA Jan 6, 2020
@kmova kmova merged commit 7094c48 into openebs:master Jan 6, 2020
@pawanpraka1 pawanpraka1 deleted the lease branch January 7, 2020 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants