MinaFoundation / mina

Not the official repo. See https://github.com/MinaProtocol/mina
https://github.com/MinaProtocol/mina
Apache License 2.0
1 stars 0 forks source link

Check if initialise_infra function requires any config changes #61

Open ejMina226 opened 1 year ago

ejMina226 commented 1 year ago

Check if initialise_infra function requires any config changes

ejMina226 commented 1 year ago

We can't remove this as it's in theintf.ml file and is called in the test_executive itself. To minimise changes to the cloudengine we should keep the function as is below. In the GCP case, the function looks like it's just testing all the pods were deployed with nodes in them. Given that we've abstracted away the infrastructure from the test I assume this will now be done in the network runner (e.g. minimina or AWS), so we don't need anything other than a unit returning function, but we will need some process like this in the network runner that basically checks the network is ready to start the nodes.