Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

Move user code background #1461

Merged
merged 3 commits into from
Oct 10, 2018
Merged

Move user code background #1461

merged 3 commits into from
Oct 10, 2018

Conversation

mzmssg
Copy link
Member

@mzmssg mzmssg commented Oct 8, 2018

Make our monitor process as docker main process, to avoid user command trick

A know issue:
For killing job sceanario, yarn script will exit before docker script, so the yarn log aggregation may collect incomplete dockerContainerDebug.log

@coveralls
Copy link

coveralls commented Oct 8, 2018

Coverage Status

Coverage increased (+0.02%) to 51.632% when pulling 86629d4 on zimiao/refactor_docker_script into b8ec568 on master.

@mzmssg mzmssg merged commit 5be7ffe into master Oct 10, 2018
@xudifsd xudifsd deleted the zimiao/refactor_docker_script branch May 24, 2019 02:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants