-
Notifications
You must be signed in to change notification settings - Fork 526
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Clean up processes on start, but wait on shutdown #7185
Conversation
- Set the DCP log variables - Disable windows
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
@karolz-ms is there any way to associate this With the logs that come out of dcp? I have all of the dcp logs being extracted now but its hard to correlate. |
Also I'm seeing this repeated in the dcp logs: {"level":"debug","ts":"2025-01-23T05:37:11.087Z","logger":"dcpctrl.ContainerOrchestrator","msg":"Running Docker command","ContainerRuntime":"","Command":"/usr/bin/docker network rm --force 76dddaa083d5b207e8187fc1f9af0974b27a842c1fa9b78e0c999a4c833c671b"}
{"level":"debug","ts":"2025-01-23T05:37:11.090Z","logger":"dcpctrl.os-executor","msg":"starting waiting for process to exit","pid":28066}
{"level":"debug","ts":"2025-01-23T05:37:11.139Z","logger":"dcpctrl.os-executor","msg":"process wait ended","pid":28066,"Error":"exit status 125"} |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
That path to the |
Trying to narrow down what might be causing #7098 (comment). Just saw a flaky test #7184 on a PR that had 2 containers hanging around and 6 networks:
It's unclear if dcp would have cleaned up some of these because the test infrastructure kills it ungracefully. Instead, clean up on the start of the test and after the test runs, we will wait for 60 seconds for dcp to quit, if it didn't then fail.