Skip to content

WIP: Check docker-machine instances using Docker API

Alessio Caiazza requested to merge docker-ping into master

What does this MR do?

Instead of using docker-machine config for checking docker availability this MR uses docker ping API

Why was this MR needed?

The problem is described in great details on #3424 (closed), the problem is that docker-machine config performs an API request on AWS DescribeInstances, if the account is rate limited than we will not be able to schedule a work on an already provisioned runner.

Are there points in the code the reviewer needs to double check?

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Tests
    • Added for this feature/bug
    • All builds are passing
  • Branch has no merge conflicts with master (if you do - rebase it please)

What are the relevant issue numbers?

Closes #3424 (closed)

Merge request reports