This repository was archived by the owner on Jan 30, 2020. It is now read-only.
fleetd: Detecting the existing machine-id #1288
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| for _, ms := range machines { | ||
| if ms.ID == h.mach.State().ID { | ||
| if ms.PublicIP != h.mach.State().PublicIP { | ||
| err = errors.New("Machine ID already exist, it seems that there are machines with same machine-id in your cluster.") |
There was a problem hiding this comment.
Yes, has grammar issue, already updated.
| @@ -162,6 +162,7 @@ func (s *Server) Run() { | |||
| if err == nil { | |||
| break | |||
| } | |||
| log.Errorf("Error: %v", err) | |||
There was a problem hiding this comment.
Add some context to this error message. Perhaps "Failed machine heartbeat: %v"
Now support detecting the existing machine-id on startup. Fixes coreos#1241 coreos#615
|
@bcwaldon ,totally agree with your comments, updated as per your suggestion. Currently the result is like: |
|
@bcwaldon , the solution with my latest code has one problem. If the fleetd is killed by sigkill which can not be catched, the fleetd's registration will always be failing. |
|
Gently ping @jonboulle , I thought over again about the modification. I think the solution is fine. If the fleetd is killed by sigkill which can not be catched, the fleetd's registration will fail until the TTL is timeout. So my code is correct I think. |
|
This seems pretty reasonable, but since it's quite an important change, can we get a functional test for it? |
dongsupark
pushed a commit
to endocode/fleet
that referenced
this pull request
Apr 19, 2016
A new test TestDetectMachineId checks if a etcd registration fails when a duplicated entry for /etc/machine-id gets registered to different machines. Note that it's expected to fail in this case. Goal of the test is to cover the improvement patch by @wuqixuan ("fleetd: Detecting the existing machine-id"). See also coreos#1288, coreos#1241, coreos#615. Suggested-by: Olaf Buddenhagen <olaf@endocode.com> Cc: wuqixuan <wuqixuan@huawei.com>
dongsupark
pushed a commit
to endocode/fleet
that referenced
this pull request
Apr 20, 2016
A new test TestDetectMachineId checks if a etcd registration fails when a duplicated entry for /etc/machine-id gets registered to different machines. Note that it's expected to fail in this case. Goal of the test is to cover the improvement patch by @wuqixuan ("fleetd: Detecting the existing machine-id"). See also coreos#1288, coreos#1241, coreos#615. Suggested-by: Olaf Buddenhagen <olaf@endocode.com> Cc: wuqixuan <wuqixuan@huawei.com>
dongsupark
pushed a commit
to endocode/fleet
that referenced
this pull request
Apr 21, 2016
A new test TestDetectMachineId checks if a etcd registration fails when a duplicated entry for /etc/machine-id gets registered to different machines. Note that it's expected to fail in this case. Goal of the test is to cover the improvement patch by @wuqixuan ("fleetd: Detecting the existing machine-id"). See also coreos#1288, coreos#1241, coreos#615. Suggested-by: Olaf Buddenhagen <olaf@endocode.com> Cc: wuqixuan <wuqixuan@huawei.com>
dongsupark
pushed a commit
to endocode/fleet
that referenced
this pull request
Apr 21, 2016
A new test TestDetectMachineId checks if a etcd registration fails when a duplicated entry for /etc/machine-id gets registered to different machines. Note that it's expected to fail in this case. Goal of the test is to cover the improvement patch by @wuqixuan ("fleetd: Detecting the existing machine-id"). See also coreos#1288, coreos#1241, coreos#615. Suggested-by: Olaf Buddenhagen <olaf@endocode.com> Cc: wuqixuan <wuqixuan@huawei.com> Cc: Djalal Harouni <djalal@endocode.com>
dongsupark
pushed a commit
to endocode/fleet
that referenced
this pull request
Apr 21, 2016
A new test TestDetectMachineId checks if a etcd registration fails when a duplicated entry for /etc/machine-id gets registered to different machines. Note that it's expected to fail in this case. Goal of the test is to cover the improvement patch by @wuqixuan ("fleetd: Detecting the existing machine-id"). See also coreos#1288, coreos#1241, coreos#615. Suggested-by: Olaf Buddenhagen <olaf@endocode.com> Cc: wuqixuan <wuqixuan@huawei.com> Cc: Djalal Harouni <djalal@endocode.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Now support detecting the existing machine-id on startup.
Fixes #1241 #615