Closed SataQiu closed 4 years ago
This looks great to me pending @neolit123's changes.
The fake lock is a little bit weird, would it be possible to always return true? a simpler implementation would be better, but if this is necessary for now then let's leave it.
This looks great to me pending @neolit123's changes.
The fake lock is a little bit weird, would it be possible to always return true? a simpler implementation would be better, but if this is necessary for now then let's leave it.
Thanks @chuckha I don't think the fake lock should always return true. Because only when the lock is unlocked can the initializing machine obtain the lock.
/approve
thanks for the update this looks really good!
/assign @neolit123 for lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: chuckha, SataQiu
The full list of commands accepted by this bot can be found here.
The pull request process is described here
ping @neolit123
Hi @ncdc Could you have a look at this? This PR has been around for a long time
/lgtm
What this PR does / why we need it: Add a test to ensure that exactly one control plane machine initializes if there are multiple control plane machines defined
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged): Ref #214