fix: separate vc (#526)

Co-authored-by: Gyanendra Mishra <[email protected]>

Author: barnabasbusa

README.md

@@ -268,6 +268,7 @@ participants:
   # - nimbus: statusim/nimbus-eth2:multiarch-latest
   # - prysm: gcr.io/prysmaticlabs/prysm/beacon-chain:latest
   # - lodestar: chainsafe/lodestar:next
+  # - grandine: ethpandaops/grandine:develop
   cl_image: ""
 
   # The log level string that this participant's CL client should log at

src/package_io/input_parser.star

@@ -384,10 +384,14 @@ def parse_network_params(input_args):
         if participant["use_separate_vc"] == None:
             # Default to false for CL clients that can run validator clients
             # in the same process.
-            if cl_type in (
-                constants.CL_TYPE.nimbus,
-                constants.CL_TYPE.teku,
-                constants.CL_TYPE.grandine,
+            if (
+                cl_type
+                in (
+                    constants.CL_TYPE.nimbus,
+                    constants.CL_TYPE.teku,
+                    constants.CL_TYPE.grandine,
+                )
+                and vc_type == ""
             ):
                 participant["use_separate_vc"] = False
             else:
@@ -398,12 +402,6 @@ def parse_network_params(input_args):
             vc_type = cl_type
             participant["vc_type"] = vc_type
 
-        if (
-            cl_type == constants.CL_TYPE.grandine
-            and vc_type != constants.CL_TYPE.grandine
-        ):
-            fail("grandine does not support running a different validator client")
-
         vc_image = participant["vc_image"]
         if vc_image == "":
             if cl_image == "":

src/participant_network.star

@@ -282,19 +282,8 @@ def launch_participant_network(
 
         vc_context = None
         snooper_beacon_context = None
-        if participant.snooper_enabled:
-            if (
-                participant.cl_type == constants.CL_TYPE.teku
-                or participant.cl_type == constants.CL_TYPE.nimbus
-                or participant.cl_type == constants.CL_TYPE.grandine
-            ) and participant.use_separate_vc != False:
-                plan.print(
-                    "Beacon snooper not supported for non split operation for {0}".format(
-                        participant.cl_type
-                    )
-                )
-                continue
 
+        if participant.snooper_enabled:
             snooper_service_name = "snooper-beacon-{0}-{1}-{2}".format(
                 index_str, cl_type, vc_type
             )