Skip to content

[ca] Separate CreateRootCA from saving root CA#2050

Merged
aaronlehmann merged 1 commit into
moby:masterfrom
cyli:split-create-root-ca-and-save
Mar 23, 2017
Merged

[ca] Separate CreateRootCA from saving root CA#2050
aaronlehmann merged 1 commit into
moby:masterfrom
cyli:split-create-root-ca-and-save

Conversation

@cyli
Copy link
Copy Markdown
Contributor

@cyli cyli commented Mar 23, 2017

This is a minor refactor that would be useful when autogenerating Root CAs as part of root rotation (if the user wants swarm to just generate a key and cert for them)

@cyli cyli changed the title Separate CreateRootCA from saving root CA [ca] Separate CreateRootCA from saving root CA Mar 23, 2017
…d also call SaveRootCA.

Signed-off-by: cyli <ying.li@docker.com>
@cyli cyli force-pushed the split-create-root-ca-and-save branch from 6be19c8 to e310156 Compare March 23, 2017 18:53
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 23, 2017

Codecov Report

Merging #2050 into master will decrease coverage by 0.04%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master    #2050      +/-   ##
==========================================
- Coverage   54.05%   54.01%   -0.05%     
==========================================
  Files         111      111              
  Lines       19332    19332              
==========================================
- Hits        10450    10442       -8     
+ Misses       7632     7630       -2     
- Partials     1250     1260      +10

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 bdeeb89...e310156. Read the comment docs.

@aaronlehmann
Copy link
Copy Markdown
Collaborator

LGTM

@aaronlehmann aaronlehmann merged commit f19ff43 into moby:master Mar 23, 2017
@cyli cyli deleted the split-create-root-ca-and-save branch March 23, 2017 21:26
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.

3 participants