Skip to content

fix(expose): bind exposed services to localhost#26

Open
casey-brooks wants to merge 1 commit into
mainfrom
noa/issue-expose-loopback-connectivity
Open

fix(expose): bind exposed services to localhost#26
casey-brooks wants to merge 1 commit into
mainfrom
noa/issue-expose-loopback-connectivity

Conversation

@casey-brooks
Copy link
Copy Markdown
Contributor

Summary

  • Fix exposed OpenZiti services to use localhost for generated host.v1 configs instead of the numeric IPv4 loopback address.
  • Keep the exposure URL/intercept service unchanged (exposed-<id>.ziti:<port>).
  • Update the server unit expectation to cover the generated host address.

Closes #25

Investigation

  • agents-orchestrator PR #189 E2E run 26553883669 reaches expose add/list successfully and receives an active exposed-<id> URL, so the expose service, OpenZiti service, and policies are being created.
  • The E2E HTTP server is launched inside the agent workload container on 127.0.0.1:3000 and verified locally before expose add.
  • The timeout happens only when dialing through the generated OpenZiti service, which points at the bind-side host.v1 address.
  • The failing path is therefore in expose's generated OpenZiti host config rather than runners, gateway, or bootstrap networking.

Test & Lint Summary

  • buf generate buf.build/agynio/api --include-imports --path agynio/api/expose/v1 --path agynio/api/runner/v1 --path agynio/api/ziti_management/v1 --path agynio/api/runners/v1 --path agynio/api/notifications/v1 --path agynio/api/identity/v1 --path agynio/api/authorization/v1: passed
  • go test ./...: 2 packages passed, 0 failed, 0 skipped
  • go vet ./...: passed with no errors
  • go build ./...: passed
  • helm dependency build charts/expose && helm lint charts/expose: lint passed with no errors
  • git diff --check: passed

@casey-brooks
Copy link
Copy Markdown
Contributor Author

Test & Lint Summary

  • buf generate buf.build/agynio/api --include-imports --path agynio/api/expose/v1 --path agynio/api/runner/v1 --path agynio/api/ziti_management/v1 --path agynio/api/runners/v1 --path agynio/api/notifications/v1 --path agynio/api/identity/v1 --path agynio/api/authorization/v1: passed
  • go test ./...: 2 packages passed, 0 failed, 0 skipped
  • go vet ./...: passed with no errors
  • go build ./...: passed
  • helm dependency build charts/expose && helm lint charts/expose: lint passed with no errors
  • git diff --check: passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Expose services time out when bound to IPv4 loopback

1 participant