Skip to content
This repository was archived by the owner on Nov 27, 2023. It is now read-only.

Conversation

@ndeloof
Copy link
Collaborator

@ndeloof ndeloof commented Jun 24, 2021

What I did
Introduced convergence struct to hold observed state as a set of containers grouped by service.
This prevent race condition accessing containers for distinct services
Applying convergence will handle containers on a per-service basis, then update the project model to translate cross-services dependencies into containers references.

Related issue
close #1813

(not mandatory) A picture of a cute animal, if possible in relation with what you did
image

@ndeloof ndeloof force-pushed the convergence branch 2 times, most recently from 106e20f to c5e338f Compare June 24, 2021 15:53
@github-actions github-actions bot added the local Local context (moby) label Jun 24, 2021
@ndeloof ndeloof force-pushed the convergence branch 5 times, most recently from 1f961c6 to c450de1 Compare June 25, 2021 09:25
@ndeloof ndeloof marked this pull request as ready for review June 25, 2021 11:36
@ndeloof ndeloof requested review from lorenrh and ulyssessouza June 25, 2021 11:40
@ndeloof ndeloof merged commit d20c3b0 into docker-archive:main Jun 28, 2021
@ndeloof ndeloof deleted the convergence branch June 28, 2021 06:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

local Local context (moby)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[TechDebt] Race condition

1 participant