test(e2e): exercise expose lifecycle

[casey-brooks]

Summary

• expand expose e2e test to cover add/list/duplicate/remove lifecycle using JSON output
• bump default agent init image to 0.4.4 for agn/expose tests

Testing

• buf generate buf.build/agynio/api --include-imports --path agynio/api/runner/v1 --path agynio/api/runners/v1 --path agynio/api/threads/v1 --path agynio/api/notifications/v1 --path agynio/api/metering/v1 --path agynio/api/agents/v1 --path agynio/api/secrets/v1 --path agynio/api/ziti_management/v1 --path agynio/api/identity/v1 --path agynio/api/llm/v1 --path agynio/api/users/v1 --path agynio/api/organizations/v1 --path agynio/api/tracing/v1
• GOMAXPROCS=2 go vet -p 1 ./...
• GOMAXPROCS=2 go test -p 1 ./...
• go build ./...

Ref #138

[casey-brooks]

Summary:

• Expanded expose e2e coverage to add/list/duplicate/remove with JSON parsing.
• Bumped AGN init image defaults to 0.4.4 for agn/expose tests.

Tests/Lint:

• buf generate buf.build/agynio/api --include-imports --path agynio/api/runner/v1 --path agynio/api/runners/v1 --path agynio/api/threads/v1 --path agynio/api/notifications/v1 --path agynio/api/metering/v1 --path agynio/api/agents/v1 --path agynio/api/secrets/v1 --path agynio/api/ziti_management/v1 --path agynio/api/identity/v1 --path agynio/api/llm/v1 --path agynio/api/users/v1 --path agynio/api/organizations/v1 --path agynio/api/tracing/v1
• GOMAXPROCS=2 go vet -p 1 ./...
• GOMAXPROCS=2 go test -p 1 ./...
• go build ./...

Test stats: passed 7, failed 0, skipped 5.
Lint status: no errors.

CI:

• CI (success): https://github.com/agynio/agents-orchestrator/actions/runs/24392199295
• E2E (failed): https://github.com/agynio/agents-orchestrator/actions/runs/24392199270
  • expose add returned "invalid_argument: workload_id is required"; likely needs newer gateway/expose chart versions (bootstrap #337).

[noa-lucent]

Clean, well-structured E2E test that covers the full expose lifecycle as specified in #138. Assertions are actionable (stdout/stderr included on failure), JSON output is properly parsed, and all five acceptance criteria are addressed.

Two minor suggestions left inline — consolidating the duplicate init image default and updating the test name to reflect the broader scope.

[noa-lucent]

[minor] This local exposeInitImage duplicates the package-level agnInitImage from main_test.go (same image, same tag 0.4.4). Having the default hardcoded in two places means they can drift silently when one is bumped but not the other — as happened before this PR (0.3.5 here vs 0.4.1 in main_test.go).

Consider reusing the package-level agnInitImage directly, unless there's an intentional reason to pin the expose test to a different image.

[noa-lucent]

[nit] The function name TestAgentExposeListExec was accurate when the test only verified expose list, but now it covers the full add → list → duplicate → remove → list lifecycle. Consider renaming to something like TestAgentExposeLifecycle to reflect the broader scope.

[rowan-stein]

Note: E2E failure (invalid_argument: workload_id is required) is expected until bootstrap defaults are bumped to deploy gateway/expose versions that derive workload context from identity metadata.

Bootstrap PR (ready to merge): agynio/bootstrap#337
Once that lands, we’ll rerun E2E here.

[casey-brooks]

Summary

• pass the workload record ID to ziti management while keeping runner instance IDs for start/stop
• update reconciler lifecycle tests to assert workload key usage alongside instance IDs

Testing

• GOMAXPROCS=2 go test -p 1 ./...
• GOMAXPROCS=2 go vet -p 1 ./...