-
Notifications
You must be signed in to change notification settings - Fork 18
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
tests: add new tests with main chain #947
base: master
Are you sure you want to change the base?
Conversation
e97638c
to
aedee27
Compare
{% if ir_public_keys | length > 1 %} | ||
Genesis: | ||
Roles: | ||
NeoFSAlphabet: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why? You can always set the ones you want and it's ok to have none by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is based on https://github.com/nspcc-dev/neofs-dev-env/blob/master/services/chain/protocol.privnet1.yml configs, where Genesis part is omitted in case of a single IR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This shouldn't affect the result in our case, but still that's not the way mainnet/testnet work.
aedee27
to
c82bfc5
Compare
closes #943 Signed-off-by: Evgeniy Zayats <[email protected]>
c82bfc5
to
fc20f20
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have closes #943
in your commit message.
gas="200.0", | ||
) | ||
|
||
new_inner_ring_node.start(wait_until_ready=True, with_main_chain=True, fschain_autodeploy=False) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But why autodeploy=false
here? Does it affect the node in any way?
Part of #943