[
https://issues.apache.org/jira/browse/YETUS-1013?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17206162#comment-17206162
]
Duo Zhang commented on YETUS-1013:
----------------------------------
Thank you [~aw]. I've alreayd opened a PR, PTAL.
So we'd better make a new release soon? This is a big problem as we will eat a
lot of spaces on the build node...
> The --sentinel option in test-patch.sh should also set ROBOT to true
> --------------------------------------------------------------------
>
> Key: YETUS-1013
> URL: https://issues.apache.org/jira/browse/YETUS-1013
> Project: Yetus
> Issue Type: Bug
> Components: Precommit
> Reporter: Duo Zhang
> Assignee: Duo Zhang
> Priority: Critical
> Fix For: 0.13.0
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> For now it only set SENTINEL to true.
> And in core.d/docker.sh, in the docker_container_maintenance method, we have
> this at the beginning:
> {code}
> if [[ "${ROBOT}" = false ]]; then
> return
> fi
> {code}
> I think this will make us skip the docker cleanup step.
> In docker-cleanup.sh, we do have set ROBOT to true when parsing --sentinel
> but since in test-patch.sh, we have called delete_parameter so I'm afraid
> there is no way for docker-cleanup.sh to parse the parameter again?
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
