Skip to content
This repository was archived by the owner on Jan 11, 2023. It is now read-only.

timeout docker info command #2577

Merged
merged 1 commit into from
Apr 2, 2018
Merged

timeout docker info command #2577

merged 1 commit into from
Apr 2, 2018

Conversation

jackfrancis
Copy link
Member

What this PR does / why we need it: docker info can get stuck, so I enforce timeout upon thee

If applicable:

  • documentation
  • unit tests
  • tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

NONE

@ghost ghost assigned jackfrancis Apr 2, 2018
@ghost ghost added the in progress label Apr 2, 2018
CecileRobertMichon
CecileRobertMichon approved these changes Apr 2, 2018
View reviewed changes
Copy link
Contributor

@CecileRobertMichon CecileRobertMichon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@jackfrancis jackfrancis merged commit e4a174c into Azure:master Apr 2, 2018
@ghost ghost removed the in progress label Apr 2, 2018
@jackfrancis jackfrancis deleted the timeout-docker-info branch April 2, 2018 23:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@CecileRobertMichon CecileRobertMichon CecileRobertMichon approved these changes

Assignees

@jackfrancis jackfrancis

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jackfrancis @CecileRobertMichon