Skip to content

[release/1.3 backport] platforms: fill default arm variant when parse platform specifier#3945

Merged
dmcgowan merged 1 commit intocontainerd:release/1.3from
zhsj:bpo-3939
Jan 8, 2020
Merged

[release/1.3 backport] platforms: fill default arm variant when parse platform specifier#3945
dmcgowan merged 1 commit intocontainerd:release/1.3from
zhsj:bpo-3939

Conversation

@zhsj
Copy link
Copy Markdown
Contributor

@zhsj zhsj commented Jan 8, 2020

Backport #3939

arm has been supported, but something is missing, causes test failure

    --- FAIL: TestParseSelector/linux (0.00s)
        platforms_test.go:292: arm support not fully implemented: not implemented
    --- FAIL: TestParseSelector/macOS (0.00s)
        platforms_test.go:292: arm support not fully implemented: not implemented

Signed-off-by: Shengjing Zhu <[email protected]>
(cherry picked from commit 90cd777)
Signed-off-by: Shengjing Zhu <[email protected]>
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Jan 8, 2020

Build succeeded.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jan 8, 2020

Codecov Report

Merging #3945 into release/1.3 will increase coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@               Coverage Diff               @@
##           release/1.3    #3945      +/-   ##
===============================================
+ Coverage        42.07%   42.07%   +<.01%     
===============================================
  Files              129      129              
  Lines            14322    14321       -1     
===============================================
  Hits              6026     6026              
+ Misses            7395     7394       -1     
  Partials           901      901
Flag Coverage Δ
#linux 45.59% <0%> (ø) ⬆️
#windows 37.02% <0%> (ø) ⬆️
Impacted Files Coverage Δ
platforms/platforms.go 80.21% <0%> (+0.87%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b71555a...054ce58. Read the comment docs.

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

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

LGTM

@dmcgowan dmcgowan merged commit bc7c954 into containerd:release/1.3 Jan 8, 2020
@zhsj zhsj deleted the bpo-3939 branch January 14, 2020 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants