Skip to content

ZEN-14: Zenith deployer#1

Open
miax-gevu wants to merge 11 commits into
mainfrom
canton-ansible-deploy
Open

ZEN-14: Zenith deployer#1
miax-gevu wants to merge 11 commits into
mainfrom
canton-ansible-deploy

Conversation

@miax-gevu

Copy link
Copy Markdown

No description provided.

Comment on lines +117 to +118
(partyV(validator2), unitV),
(partyV(validator3), unitV)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
(partyV(validator2), unitV),
(partyV(validator3), unitV)

Only validator1 is a proposer

Comment on lines +212 to +214
(partyV(validator1), unitV),
(partyV(validator2), unitV),
(partyV(validator3), unitV)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
(partyV(validator1), unitV),
(partyV(validator2), unitV),
(partyV(validator3), unitV)

This is not needed because all of the validators can observe the block through zeo party.
Also because rules contract doesn't have these parties in his "observers" field, this value will be kept only in the genesis block. Immediately in block N1 this will be erased.

Comment on lines +292 to +293
createBlockProposal(participant2, validator2, "Validator2")
createBlockProposal(participant3, validator3, "Validator3")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
createBlockProposal(participant2, validator2, "Validator2")
createBlockProposal(participant3, validator3, "Validator3")

Only validator1 must be registered as a proposer, so other validators don't need to create proposal contracts. Their proposals will never be accepted by the rules contract.

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