You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2020/04/03 08:53:07 UTC
[GitHub] [incubator-nuttx] liuguo09 opened a new pull request #704: CI:
remove the additional pull docker container step
liuguo09 opened a new pull request #704: CI: remove the additional pull docker container step
URL: https://github.com/apache/incubator-nuttx/pull/704
There is chance pull docker container failure in build jobs,
remove the additional pull docker container step to make it
more stable. Meanwhile, it also save some time for build job.
Signed-off-by: liuhaitao <li...@xiaomi.com>
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] yamt commented on issue #704: CI: remove the
additional pull docker container step
Posted by GitBox <gi...@apache.org>.
yamt commented on issue #704: CI: remove the additional pull docker container step
URL: https://github.com/apache/incubator-nuttx/pull/704#issuecomment-608410091
i don't think we need to keep it in testing. it's better to keep all repo in sync.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] yamt commented on issue #704: CI: remove the
additional pull docker container step
Posted by GitBox <gi...@apache.org>.
yamt commented on issue #704: CI: remove the additional pull docker container step
URL: https://github.com/apache/incubator-nuttx/pull/704#issuecomment-611319114
have any of you noticed any negative effect with this?
otherwise i want to apply this to other repos as well.
i haven't noticed any problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] yamt commented on issue #704: CI: remove the
additional pull docker container step
Posted by GitBox <gi...@apache.org>.
yamt commented on issue #704: CI: remove the additional pull docker container step
URL: https://github.com/apache/incubator-nuttx/pull/704#issuecomment-608368420
are you talking about this symptom?
Run ./testing/.github/actions/ci-container
/usr/bin/docker run --name dockerpkggithubcomapacheincubatornuttxtestingnuttxcilinux_ffbfc5 --label d00b33 --workdir /github/workspace --rm -e DOCKER_BUILDKIT -e DOCKER_CONFIG -e HOME -e GITHUB_JOB -e GITHUB_REF -e GITHUB_SHA -e GITHUB_REPOSITORY -e GITHUB_RUN_ID -e GITHUB_RUN_NUMBER -e GITHUB_ACTOR -e GITHUB_WORKFLOW -e GITHUB_HEAD_REF -e GITHUB_BASE_REF -e GITHUB_EVENT_NAME -e GITHUB_WORKSPACE -e GITHUB_ACTION -e GITHUB_EVENT_PATH -e RUNNER_OS -e RUNNER_TOOL_CACHE -e RUNNER_TEMP -e RUNNER_WORKSPACE -e ACTIONS_RUNTIME_URL -e ACTIONS_RUNTIME_TOKEN -e ACTIONS_CACHE_URL -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/incubator-nuttx-testing/incubator-nuttx-testing":"/github/workspace" docker.pkg.github.com/apache/incubator-nuttx-testing/nuttx-ci-linux "-c" ""
Unable to find image 'docker.pkg.github.com/apache/incubator-nuttx-testing/nuttx-ci-linux:latest' locally
/usr/bin/docker: Error response from daemon: Get https://docker.pkg.github.com/v2/: net/http: request canceled (Client.Timeout exceeded while awaiting headers).
See '/usr/bin/docker run --help'.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] yamt commented on issue #704: CI: remove the
additional pull docker container step
Posted by GitBox <gi...@apache.org>.
yamt commented on issue #704: CI: remove the additional pull docker container step
URL: https://github.com/apache/incubator-nuttx/pull/704#issuecomment-608424689
as the log has timestamps on each lines, you can see how long it took to pull images.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] btashton commented on issue #704: CI: remove the
additional pull docker container step
Posted by GitBox <gi...@apache.org>.
btashton commented on issue #704: CI: remove the additional pull docker container step
URL: https://github.com/apache/incubator-nuttx/pull/704#issuecomment-611320599
Let's make it consistent. It is a NOP in the end.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #704: CI:
remove the additional pull docker container step
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #704: CI: remove the additional pull docker container step
URL: https://github.com/apache/incubator-nuttx/pull/704
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] xiaoxiang781216 commented on issue #704: CI:
remove the additional pull docker container step
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on issue #704: CI: remove the additional pull docker container step
URL: https://github.com/apache/incubator-nuttx/pull/704#issuecomment-608377095
> are you talking about this symptom?
>
> Run ./testing/.github/actions/ci-container
> /usr/bin/docker run --name dockerpkggithubcomapacheincubatornuttxtestingnuttxcilinux_ffbfc5 --label d00b33 --workdir /github/workspace --rm -e DOCKER_BUILDKIT -e DOCKER_CONFIG -e HOME -e GITHUB_JOB -e GITHUB_REF -e GITHUB_SHA -e GITHUB_REPOSITORY -e GITHUB_RUN_ID -e GITHUB_RUN_NUMBER -e GITHUB_ACTOR -e GITHUB_WORKFLOW -e GITHUB_HEAD_REF -e GITHUB_BASE_REF -e GITHUB_EVENT_NAME -e GITHUB_WORKSPACE -e GITHUB_ACTION -e GITHUB_EVENT_PATH -e RUNNER_OS -e RUNNER_TOOL_CACHE -e RUNNER_TEMP -e RUNNER_WORKSPACE -e ACTIONS_RUNTIME_URL -e ACTIONS_RUNTIME_TOKEN -e ACTIONS_CACHE_URL -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/incubator-nuttx-testing/incubator-nuttx-testing":"/github/workspace" docker.pkg.github.com/apache/incubator-nuttx-testing/nuttx-ci-linux "-c" ""
> Unable to find image 'docker.pkg.github.com/apache/incubator-nuttx-testing/nuttx-ci-linux:latest' locally
> /usr/bin/docker: Error response from daemon: Get https://docker.pkg.github.com/v2/: net/http: request canceled (Client.Timeout exceeded while awaiting headers).
> See '/usr/bin/docker run --help'.
Yes, saving one pull redue the failure posibility. We can keep apache-nuttx-testing to measure the performance.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] xiaoxiang781216 commented on issue #704: CI:
remove the additional pull docker container step
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on issue #704: CI: remove the additional pull docker container step
URL: https://github.com/apache/incubator-nuttx/pull/704#issuecomment-608416995
If @btashton don't have concern, I am fine to remove it from testing too.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services