Skip to content

docs/spec/manifest-v2-2.md: fix ARM variant#3371

Merged
wy65701436 merged 1 commit intodistribution:mainfrom
AkihiroSuda:remove-wrong-arm
Mar 4, 2021
Merged

docs/spec/manifest-v2-2.md: fix ARM variant#3371
wy65701436 merged 1 commit intodistribution:mainfrom
AkihiroSuda:remove-wrong-arm

Conversation

@AkihiroSuda
Copy link
Copy Markdown
Contributor

The correct vairant string for ARM v6 is "v6", not "armv6l".

There is no known implementation that actually uses "armv6l".

See the discussion in opencontainers/image-spec#817 (comment)

The correct `vairant` string for ARM v6 is "v6", not "armv6l".

There is no known implementation that actually uses "armv6l".

See the discussion in opencontainers/image-spec PR 817

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #3371 (c432849) into main (742aab9) will decrease coverage by 2.38%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3371      +/-   ##
==========================================
- Coverage   58.22%   55.84%   -2.39%     
==========================================
  Files         105      102       -3     
  Lines        8362     7222    -1140     
==========================================
- Hits         4869     4033     -836     
+ Misses       2836     2552     -284     
+ Partials      657      637      -20     
Flag Coverage Δ
linux ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
context/context.go
registry/client/errors.go
registry/client/blob_writer.go
registry/api/v2/routes.go
context/version.go
registry/client/auth/api_version.go
registry/storage/catalog.go
health/checks/checks.go
registry/api/errcode/handler.go
registry/storage/blobwriter_resumable.go
... and 197 more

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 742aab9...c432849. Read the comment docs.

Copy link
Copy Markdown
Contributor

@mikebrow mikebrow 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
Collaborator

@wy65701436 wy65701436 left a comment

Choose a reason for hiding this comment

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

lgtm

@wy65701436 wy65701436 merged commit 89337b7 into distribution:main Mar 4, 2021
dylanrhysscott pushed a commit to digitalocean/docker-distribution that referenced this pull request Jan 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants