[tmpnet] Switch FlagsMap from map[string]any to map[string]string

[maru-ava]

PR Chain: tmpnet+kube

This PR chain enables tmpnet to deploy temporary networks to Kubernetes. Early PRs refactor tmpnet to support the addition in #3615 of a new tmpnet node runtime for kube.

• [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] Enable deployment to kube #3615
• [tmpnet] Enable monitoring of local kind cluster #3794

Why this should be merged

map[string]any complicates comparison of values converted to strings (e.g. when node configuration is provided as env vars to a kube pod) to their original golang form. Switching FlagsMap to use map[string]string forces the decision of how to serialize on the developer to ensure that all subsequent usage will be consistent.

This change also updates subnet, chain and monitor configuration to stop using FlagsMap since their use-cases still require map[string]any.

How this was tested

CI

Need to be documented in RELEASES.md?

N/A

TODO

• Merge [tmpnet] Rename NodeProcess to ProcessRuntime #3890

[Copilot]

Pull Request Overview

This PR changes the FlagsMap type from map[string]any to map[string]string to enforce consistent string serialization for configuration values. Key changes include replacing FlagsMap with ConfigMap in subnet, process runtime, node, and network components, updating the related helper functions and tests, and revising the handling of boolean and duration values.

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
tests/fixture/tmpnet/subnet.go	Replaces FlagsMap with ConfigMap for subnet configuration
tests/fixture/tmpnet/process_runtime.go	Updates monitoring configuration to use ConfigMap and uses maps.Copy
tests/fixture/tmpnet/node.go	Modifies flag retrieval by removing type conversion helper methods
tests/fixture/tmpnet/network.go	Updates network configuration and sybil protection flag handling
tests/fixture/tmpnet/flagsmap.go	Changes FlagsMap from map[string]interface{} to map[string]string
tests/fixture/tmpnet/defaults.go	Converts boolean and duration flag values to string representations
tests/fixture/tmpnet/check_monitoring.go	Updates label retrieval to use the new ConfigMap style
tests/fixture/subnet/xsvm.go	Revises subnet configuration to use ConfigMap
tests/e2e/p/staking_rewards.go & others	Adjusts test assertions to compare against string representations
tests/antithesis/compose.go	Modifies compose project construction to remove redundant type checks

Comments suppressed due to low confidence (2)

tests/fixture/tmpnet/process_runtime.go:290

• The use of maps.Copy on line 291 relies on Go 1.21+ features; please confirm that the project’s minimum Go version supports this function to avoid runtime issues.

promtailLabels := map[string]string{}

tests/antithesis/compose.go:134

• [nitpick] Since the function signature was changed to remove error returns, please ensure that all failure cases are properly handled upstream, or add a clarifying comment if any errors are now impossible.

func newComposeProject(network *tmpnet.Network, nodeImageName string, workloadImageName string) *types.Project {

[qdm12]

Nice!