hashgraph / hashgraph/solo-weaver

Epic 5: solo-weaver imports pkg/jobs from solo-operator

Open
#860 0 comments 0 reactions 0 assignees View on GitHub
Epic Improvement
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.