[tmpnet] Separate start of prometheus and promtail collectors

[maru-ava]

• [tmpnet] s/Network.ChainConfigs/Network.PrimaryChainConfigs/ #3854
• [tmpnet] Provide genesis, subnet and chain config via content flags #3857
• [tmpnet] Ensure Node has a reference to Network #3870
• [tmpnet] Switch back to using maps for subnet config #3877
• [tmpnet] Refactor runtime configuration in preparation for kube #3867
• [tmpnet] Unify start network flag usage between e2e and tmpnetctl #3871
• [tmpnet] Avoid serializing the node data directory #3881
• [tmpnet] Rename NodeProcess to ProcessRuntime #3890
• [tmpnet] Switch FlagsMap from map[string]any to map[string]string #3884
• [tmpnet] Ensure tmpnet methods always have a logger #3893
• [tmpnet] Ensure all node runtime methods accept a context #3894
• [tmpnet] Move WaitForHealthy from a function to a tmpnet.Node method #3896
• [tmpnet] Delegate writing of the flag file to the runtime #3897
• [tmpnet] Move monitoring label handling to node #3898
• [tmpnet] Enable runtime-specific restart behavior #3882
• [tmpnet] Define reusable flags for configuring kubernetes client access #3945
• >>>>>> [tmpnet] Separate start of prometheus and promtail collectors #3947 <<<<<<
• [tmpnet] Enable deployment to kube #3615
• [tmpnet] Enable monitoring of nodes running in kube #3794

Why this should be merged

Previously local collectors were always started together. For kube, though, it makes sense to support only starting prometheus (e.g. to collect metrics from the load generation framework) since logs and metrics for nodes will be collected by agents within the kube cluster.

How this was tested

CI

Need to be documented in RELEASES.md?

N/A

[qdm12]

Nice catch 👍
Except for retro-compatibility, do we want to continue supporting TMPNET_START_COLLECTORS? Should we just support log+metrics collection flags to have code a bit simpler? But it's also fine as is, so feel free to ignore.

[maru-ava]

Nice catch 👍 Except for retro-compatibility, do we want to continue supporting TMPNET_START_COLLECTORS? Should we just support log+metrics collection flags to have code a bit simpler? But it's also fine as is, so feel free to ignore.

From the department of 'be careful what you wish for'... The latest commit removes start/stop collectors in favor of starting/stopping them individually.

[maru-ava]

Rebased