Make sure to not remove backends on stale notices #1370

Merged
merged 2 commits into from Aug 3, 2016

Projects

None yet

4 participants

@mrjana
Contributor
mrjana commented Aug 2, 2016

Sometimes you may get stale backend removal notices from gossip due to
some lingering state. If a stale backend notice is received and it is
already processed in this node ignore it rather than processing it.

Also fixing:
Ensure drivertable entries in joininfo is uptodate

The CopyTo function for joininfo is not copying the driver table entries
which then is missing when the endpoint is re-read for the store cache.

Signed-off-by: Jana Radhakrishnan mrjana@docker.com

mrjana added some commits Aug 2, 2016
@mrjana mrjana Ensure drivertable entries in joininfo is uptodate
The CopyTo function for joininfo is not copying the driver table entries
which then is missing when the endpoint is re-read for the store cache.

Signed-off-by: Jana Radhakrishnan <mrjana@docker.com>
344aa2f
@mrjana mrjana Make sure to not remove backends on stale notices
Sometimes you may get stale backend removal notices from gossip due to
some lingering state. If a stale backend notice is received and it is
already processed in this node ignore it rather than processing it.

Signed-off-by: Jana Radhakrishnan <mrjana@docker.com>
c191ea7
@sanimej
Contributor
sanimej commented Aug 3, 2016

LGTM

Please add a reference to the issues fixed by this.

@aboch
Contributor
aboch commented Aug 3, 2016

LGTM

@sanimej sanimej merged commit 969f451 into docker:master Aug 3, 2016

2 checks passed

ci/circleci Your tests passed on CircleCI!
Details
docker/dco-signed All commits signed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment