What you would like to be added?
Delete e2e/utils/ and move its contents into the domain packages
Scope:
- Inline or move utils/polling.go → k8s/polling.go
- Inline or move utils/node.go → k8s/nodes.go
- Inline or move utils/grove_resources.go → grove/
- Inline or move utils/grove_config_k8s.go → grove/config.go
- Inline or move utils/topology.go, utils/kai_topology.go → grove/topology.go
- Inline or move utils/conversions.go → k8s/conversions.go
- Move utils/logger.go → standalone e2e/log/ package
- Update all imports across setup/, tests/, diagnostics/, grove/
- Delete e2e/utils/ directory
Why is this needed?
Once domain packages (k8s/, grove/, diagnostics/) exist but their methods still delegate to utils.* functions. This means two layers for every operation: the manager method and the utils function it wraps. The utils/ package has no clear domain — it's a dumping ground with node helpers, polling, Grove resource operations, conversions, and logging all in one place. Removing it eliminates the indirection and puts every function in the package that owns its domain.
What you would like to be added?
Delete e2e/utils/ and move its contents into the domain packages
Scope:
Why is this needed?
Once domain packages (k8s/, grove/, diagnostics/) exist but their methods still delegate to utils.* functions. This means two layers for every operation: the manager method and the utils function it wraps. The utils/ package has no clear domain — it's a dumping ground with node helpers, polling, Grove resource operations, conversions, and logging all in one place. Removing it eliminates the indirection and puts every function in the package that owns its domain.