Skip to content
This repository has been archived by the owner on Mar 21, 2022. It is now read-only.

make ImageInfo.RootFS nullable #862

Merged
merged 1 commit into from
Aug 9, 2017
Merged

Conversation

mattnworb
Copy link
Member

RootFS was added to the ImageInfo / inspectImage response in version
1.23 of the Docker Remote API (for Docker version 1.11). Earlier
versions of the API do not have this field present in ImageInfo.

We caught this because we run tests for spotify/helios on CircleCI which
uses Docker 1.10.

`RootFS` was added to the ImageInfo / inspectImage response in version
1.23 of the Docker Remote API (for Docker version 1.11). Earlier
versions of the API do not have this field present in ImageInfo.

We caught this because we run tests for spotify/helios on CircleCI which
uses Docker 1.10.
@mattnworb mattnworb force-pushed the mattbrown/nullable-rootfs branch from 277fea8 to c0e4a61 Compare August 8, 2017 18:22
@mattnworb mattnworb requested review from caipre and davidxia August 8, 2017 18:22
@codecov-io
Copy link

codecov-io commented Aug 8, 2017

Codecov Report

Merging #862 into master will increase coverage by 0.54%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #862      +/-   ##
============================================
+ Coverage     65.92%   66.46%   +0.54%     
- Complexity      737      745       +8     
============================================
  Files           166      166              
  Lines          3140     3140              
  Branches        357      357              
============================================
+ Hits           2070     2087      +17     
+ Misses          914      898      -16     
+ Partials        156      155       -1

@mattnworb mattnworb merged commit 317df60 into master Aug 9, 2017
@mattnworb mattnworb deleted the mattbrown/nullable-rootfs branch August 9, 2017 14:46
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