Skip to content

[ca][17.06] Include a signing profile for cross-signing CA certs externally#2213

Closed
cyli wants to merge 1 commit into
moby:bump_v17.06.0from
cyli:simplify-external-ca-cross-signing-17.06
Closed

[ca][17.06] Include a signing profile for cross-signing CA certs externally#2213
cyli wants to merge 1 commit into
moby:bump_v17.06.0from
cyli:simplify-external-ca-cross-signing-17.06

Conversation

@cyli
Copy link
Copy Markdown
Contributor

@cyli cyli commented Jun 1, 2017

Backport of #2212 to 17.06 branch.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 1, 2017

Codecov Report

Merging #2213 into bump_v17.06.0 will increase coverage by 0.02%.
The diff coverage is 100%.

@@                Coverage Diff                @@
##           bump_v17.06.0    #2213      +/-   ##
=================================================
+ Coverage          60.11%   60.13%   +0.02%     
=================================================
  Files                119      119              
  Lines              19849    19850       +1     
=================================================
+ Hits               11932    11937       +5     
+ Misses              6562     6552      -10     
- Partials            1355     1361       +6

…quest to external CAs.

This way it will be easier for them to implement extra policies.

Signed-off-by: Ying Li <ying.li@docker.com>
@cyli cyli force-pushed the simplify-external-ca-cross-signing-17.06 branch from be2d919 to e9ae19e Compare June 2, 2017 03:06
@aaronlehmann
Copy link
Copy Markdown
Collaborator

LGTM

@cyli
Copy link
Copy Markdown
Contributor Author

cyli commented Jun 2, 2017

After discussion with @aluzzardi, we aren't going to make PRs against the 17.06 branch. If these can be cherry-picked cleanly, we're just going to merge the master PRs and cherry-pick and push to the 17.06 branch. So since #2212 has been merged, I'll just close this PR and do that. Thanks @aaronlehmann and @diogomonica for the reviews!

@cyli cyli closed this Jun 2, 2017
@cyli cyli deleted the simplify-external-ca-cross-signing-17.06 branch June 2, 2017 18:28
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.

2 participants