FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Unit all docker client config to same private.key by laodouya · Pull Request #247 · CovenantSQL/CovenantSQL · GitHub

Unit all docker client config to same private.key - #247

Merged
xq262144 merged 2 commits into
developfrom
feature/unit_client_config
Feb 2, 2019
Merged

Unit all docker client config to same private.key#247
xq262144 merged 2 commits into
developfrom
feature/unit_client_config

Conversation

laodouya commented Feb 1, 2019

Copy link
Copy Markdown
Member

For docker-compose env:

  1. Delete unnecessary client config.
  2. Use same private key for all client role.
  3. Generate their own NodeID for each client role by same key.

laodouya requested review from auxten and xq262144 February 1, 2019 09:54

codecov Bot commented Feb 1, 2019

Copy link
Copy Markdown

Codecov Report

Merging #247 into develop will decrease coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #247      +/-   ##
===========================================
- Coverage    75.94%   75.88%   -0.07%     
===========================================
  Files          159      159              
  Lines        12198    12198              
===========================================
- Hits          9264     9256       -8     
- Misses        2235     2243       +8     
  Partials       699      699

1 similar comment

codecov Bot commented Feb 1, 2019

Copy link
Copy Markdown

Codecov Report

Merging #247 into develop will decrease coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #247      +/-   ##
===========================================
- Coverage    75.94%   75.88%   -0.07%     
===========================================
  Files          159      159              
  Lines        12198    12198              
===========================================
- Hits          9264     9256       -8     
- Misses        2235     2243       +8     
  Partials       699      699

auxten left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

xq262144 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Approve

xq262144 merged commit b283716 into develop Feb 2, 2019
laodouya deleted the feature/unit_client_config branch February 2, 2019 03:45
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL