Skip to content

bug: Stack ingestion port-forward uses cluster name instead of Helm release name #49

@maximilien

Description

@maximilien

Description

weave stack ingest internal port-forward fails silently because StartMilvusPortForward() uses cluster name instead of Helm release name.

Steps to Reproduce

weave stack up --runtime kind
weave stack ingest Documents data/

Expected Behavior

Should port-forward to: my-rag-stack-weave-stack-milvus

Actual Behavior

Tries to port-forward to: weave-stack-milvus (doesn't exist)

Port-forward fails but error is suppressed, leading to connection issues.

Root Cause

src/pkg/stack/ingest.go:53 - StartMilvusPortForward() was using cluster name:

serviceName := fmt.Sprintf("%s-milvus", clusterInfo.Name) // Wrong!

Same issue as #45 but in the internal port-forward helper function.

Fix

Commit: 291a056

Load stack config and use Helm release name:

stackConfig, err := LoadStackConfig("")
serviceName := fmt.Sprintf("%s-weave-stack-milvus", stackConfig.Name)

Impact

HIGH - Ingestion port-forward fails, but masked by #44 (pipeline bug)

Related

Status

FIXED in commit 291a056

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions