Skip to content

build full compose model from resources, then filter by services#9250

Merged
ndeloof merged 1 commit intodocker:v2from
ndeloof:stop_9241
Mar 9, 2022
Merged

build full compose model from resources, then filter by services#9250
ndeloof merged 1 commit intodocker:v2from
ndeloof:stop_9241

Conversation

@ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Mar 9, 2022

What I did
introduced actualState (this reduces code duplication and risk to have same bug in other places) to list resources, rebuild compose project model, then filter by requested services

fix docker compose stop SERVICE with reference to other dependencies
bonus: docker compose rm doesn't need a compose file anymore

includes e2e test so we get rid of this bug and don't need yet another patch release🤞

Related issue
closes #9241
closes #9249

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

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
Copy link
Contributor

@glours glours left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ndeloof ndeloof force-pushed the stop_9241 branch 3 times, most recently from 0032d95 to d9c6a0d Compare March 9, 2022 08:43
@ndeloof ndeloof force-pushed the stop_9241 branch 5 times, most recently from 07a67f7 to 1ce67e8 Compare March 9, 2022 09:45
@ndeloof ndeloof merged commit 158b5ff into docker:v2 Mar 9, 2022
@ndeloof ndeloof deleted the stop_9241 branch December 20, 2023 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants