hashgraph / hashgraph/solo-weaver
Epic 5: solo-weaver imports pkg/jobs from solo-operator
- Dominant language
- Go
- Stars
- 3
- Forks
- 0
- Avg merge
- 3d 5h
- Merged PRs (30d)
- 47
Description
## Overview
Add `github.com/hashgraph/solo-operator` as a Go module dependency in solo-weaver, importing only the `pkg/jobs` sub-package. Align the Helm chart version pin in `infrastructure-catalog.yaml` with the Go module version. Replace duplicated phase enum constants.
This is a prerequisite for **Epic 4** (daemon-side job execution).
## Dependency verification
The import must NOT pull in `sigs.k8s.io/controller-runtime` transitively — verify with:
```
go mod graph | grep controller-runtime
```
## Stories
_To be linked below once created_
## Acceptance Criteria
- [ ] `go.mod` in solo-weaver references `github.com/hashgraph/solo-operator` at the correct version
- [ ] `infrastructure-catalog.yaml` Helm chart pin matches Go module version
- [ ] `internal/consensus/phase.go` imports `jobs.PhaseName` constants instead of local strings
- [ ] No controller-runtime transitive pull confirmed by `go mod graph`
- [ ] CI passes with the new dependency
Contributor guide
Research direction
Start with go.mod, infrastructure-catalog.yaml, and internal/consensus/phase.go. Add the solo-operator dependency and inspect its pkg/jobs package, align the Helm pin and replace local phase strings with jobs.PhaseName constants. Run go mod graph | grep controller-runtime, then run the repository CI checks; done means all acceptance criteria pass without a controller-runtime dependency.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go, helm
- Domain
- build-system, infrastructure
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100