Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: replace manifest with cluster definition/lock #742

Merged
merged 3 commits into from
Jun 23, 2022

Conversation

dB2510
Copy link
Contributor

@dB2510 dB2510 commented Jun 22, 2022

Replaces manifest with cluster definition/lock everywhere in the codebase.

category: docs
ticket: none

@codecov
Copy link

codecov bot commented Jun 22, 2022

Codecov Report

Merging #742 (d2c9b15) into main (2bc98cb) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #742   +/-   ##
=======================================
  Coverage   54.41%   54.41%           
=======================================
  Files         107      107           
  Lines       10408    10408           
=======================================
+ Hits         5663     5664    +1     
  Misses       3921     3921           
+ Partials      824      823    -1     
Impacted Files Coverage Δ
cmd/createcluster.go 51.11% <0.00%> (ø)
core/qbft/qbft.go 81.54% <0.00%> (-0.43%) ⬇️
app/app.go 58.50% <0.00%> (+0.75%) ⬆️

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 2bc98cb...d2c9b15. Read the comment docs.

@dB2510 dB2510 requested a review from xenowits June 23, 2022 13:43
cmd/createcluster.go Outdated Show resolved Hide resolved
@@ -104,12 +103,12 @@ There are a number of aspects to this trust surface that can be mitigated with a

### Using DKG without the launchpad

Charon clients can do a DKG with a manifest file that does not contain operator signatures if you pass a `--no-verify` flag to `charon dkg`. This can be used for testing purposes when strict signature verification is not of the utmost importance.
Charon clients can do a DKG with a cluster-definition file that does not contain operator signatures if you pass a `--no-verify` flag to `charon dkg`. This can be used for testing purposes when strict signature verification is not of the utmost importance.
Copy link
Contributor

Choose a reason for hiding this comment

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

--no-verify flag doesn't exist now, though an issue is open. i suggest we can keep it

Copy link
Contributor

Choose a reason for hiding this comment

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

docs/dkg.md Outdated Show resolved Hide resolved
dB2510 and others added 2 commits June 23, 2022 19:49
Co-authored-by: Abhishek Kumar <[email protected]>
Co-authored-by: Abhishek Kumar <[email protected]>
@dB2510 dB2510 added the merge when ready Indicates bulldozer bot may merge when all checks pass label Jun 23, 2022
@obol-bulldozer obol-bulldozer bot merged commit 043b2df into main Jun 23, 2022
@obol-bulldozer obol-bulldozer bot deleted the dhruv/removemanifest branch June 23, 2022 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge when ready Indicates bulldozer bot may merge when all checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants