Skip to content

[release/2.6] Cherry-Pick commits for release 2.6.3#2558

Merged
stevvooe merged 3 commits into
distribution:release/2.6from
mrueg:release/2.6
Apr 6, 2018
Merged

[release/2.6] Cherry-Pick commits for release 2.6.3#2558
stevvooe merged 3 commits into
distribution:release/2.6from
mrueg:release/2.6

Conversation

@mrueg

@mrueg mrueg commented Mar 28, 2018

Copy link
Copy Markdown

@stevvooe as you mentioned here [1] (sorry it took so long) Can you consider these commits for a 2.6.3 release?

[1] #2383 (comment)

@codecov

codecov Bot commented Mar 28, 2018

Copy link
Copy Markdown

Codecov Report

Merging #2558 into release/2.6 will decrease coverage by 8.14%.
The diff coverage is 45.45%.

Impacted file tree graph

@@               Coverage Diff               @@
##           release/2.6    #2558      +/-   ##
===============================================
- Coverage        59.69%   51.54%   -8.15%     
===============================================
  Files              127      127              
  Lines            14333    11380    -2953     
===============================================
- Hits              8556     5866    -2690     
+ Misses            4884     4761     -123     
+ Partials           893      753     -140
Impacted Files Coverage Δ
notifications/endpoint.go 94.28% <ø> (+94.28%) ⬆️
registry/storage/tagstore.go 70.19% <45.45%> (-0.65%) ⬇️
registry/storage/driver/gcs/gcs.go 0.4% <0%> (-66.05%) ⬇️
registry/storage/driver/oss/oss.go 0.57% <0%> (-56.67%) ⬇️
registry/storage/driver/s3-aws/s3.go 4.17% <0%> (-55.42%) ⬇️
registry/storage/driver/s3-goamz/s3.go 0.5% <0%> (-51.7%) ⬇️
registry/client/transport/transport.go 69.69% <0%> (-7.81%) ⬇️
registry/client/errors.go 49.18% <0%> (-4.16%) ⬇️
uuid/uuid.go 67.34% <0%> (-3.87%) ⬇️
registry/api/errcode/register.go 28.12% <0%> (-2.65%) ⬇️
... and 113 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 803ff0d...376cbc0. Read the comment docs.

@stevvooe

stevvooe commented Apr 2, 2018

Copy link
Copy Markdown
Collaborator

@mrueg can you do the cherry picks with the -x option? That will ensure we can trace to the original commit.

Thanks for doing this!

stevvooe and others added 3 commits April 5, 2018 13:49
Signed-off-by: Stephen J Day <stephen.day@docker.com>
(cherry picked from commit 1ba5b3b)
Signed-off-by: Wenkai Yin <yinw@vmware.com>
(cherry picked from commit 005c6e0)
Remove EndpointConfig.Transport from the return value of the
registry.notifications.endpoints expvar.Func.  It results in an empty
value for that expvar variable under Go 1.7 because it is a non-nil
*http.Transport, which Go 1.7 can no longer encode as JSON.

Signed-off-by: Noah Treuhaft <noah.treuhaft@docker.com>
(cherry picked from commit 9a58c91)
@mrueg

mrueg commented Apr 5, 2018

Copy link
Copy Markdown
Author

@stevvooe cherry-picked with -x. Thanks for letting me know!

@stevvooe stevvooe changed the title Cherry-Pick commits for release 2.6.3 [release/2.6] Cherry-Pick commits for release 2.6.3 Apr 6, 2018
@stevvooe

stevvooe commented Apr 6, 2018

Copy link
Copy Markdown
Collaborator

LGTM

@stevvooe stevvooe merged commit 49a2291 into distribution:release/2.6 Apr 6, 2018
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