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
Launch Management Chain for E2E #1303
Conversation
…, no Docker, no remote
# Conflicts: # test/e2e/multi_chain_test.go
…rbs-network/orbs-network-go into feature/simple-multi-chain-e2e
…rbs-network/orbs-network-go into feature/simple-multi-chain-e2e
race detector falls (https://circleci.com/gh/orbs-network/orbs-network-go/18624?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link) on LH race condition, Issue fixed in orbs-network/lean-helix-go#70 |
@itamararjuan need your help with ci/e2e -
|
@@ -127,7 +127,7 @@ jobs: | |||
|
|||
deploy_testnet: | |||
docker: | |||
- image: circleci/node:11.2.0 | |||
- image: itamararjuan/golang_with_node_aws |
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.
@itamararjuan why are you using your own image?
config/system_factory_presets.go
Outdated
constantConsensusLeader primitives.NodeAddress, | ||
activeConsensusAlgo consensus.ConsensusAlgoType, | ||
) NodeConfig { | ||
func ForE2E(httpAddress string, virtualChainId primitives.VirtualChainId, gossipListenPort int, nodeAddress primitives.NodeAddress, nodePrivateKey primitives.EcdsaSecp256K1PrivateKey, gossipPeers map[string]GossipPeer, genesisValidatorNodes map[string]ValidatorNode, blockStorageDataDirPrefix string, processorArtifactPath string, ethereumEndpoint string, constantConsensusLeader primitives.NodeAddress, activeConsensusAlgo consensus.ConsensusAlgoType) NodeConfig { |
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.
please unfmt
@@ -51,7 +51,7 @@ services: | |||
ipv4_address: 192.168.199.12 | |||
|
|||
|
|||
orbs-network-node-1: | |||
orbs-app-chain-node-1: |
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 file has become long and repetitive; please see if we can simplify it (separate PR)
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.
opened #1318
test/e2e/harness.go
Outdated
"", | ||
primitives.NodeAddress{}, | ||
0). | ||
return config.ForE2E("", 0, 0, primitives.NodeAddress{}, primitives.EcdsaSecp256K1PrivateKey{}, nil, nil, "", "", "", primitives.NodeAddress{}, 0). |
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.
please unfmt
@ronnno merge on Sunday? |
launch a second virtual chain for e2e tests:
100000 + (prNumber * 2)
for mgmt chain and100000 + (prNumber * 2 + 1)
for app chainfor in-process e2e harness:
8090, 8090+1,...,8090+(n-1)
8090+n, 8090+(n+1),...,8090(2n-1)
test/e2e/_tmp/
contract compiler work dir) and virtual chain data dir will be appended by a vchain id component of the form:vcid_[vcid]