Skip to content

feat(ziti-management): add egress reconciliation RPCs#154

Merged
vitramir merged 1 commit into
mainfrom
noa/issue-156
Jun 16, 2026
Merged

feat(ziti-management): add egress reconciliation RPCs#154
vitramir merged 1 commit into
mainfrom
noa/issue-156

Conversation

@casey-brooks

Copy link
Copy Markdown
Contributor

Summary

Part of agynio/architecture#156.

  • Added Ziti Management proto RPCs required by EgressRules reconciliation: GetService, ListServices, GetServicePolicy, and ListServicePolicies.
  • Added return_existing flags to CreateService and CreateServicePolicy so duplicate creates can return existing resources.
  • Added list filter fields for exact names, prefixes, role attributes, policy type, identity roles, service roles, and pagination.

Test & lint summary

  • buf lint: passed with no errors.
  • buf breaking --against '.git#branch=main': passed with no errors.

Notes

End-to-end platform validation from agynio/architecture#156 was not run locally because it requires the full deployed platform, OpenZiti, agents, cert-manager, metering/tracing, and external service credentials.

@github-actions

Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow buf-pr / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedJun 15, 2026, 6:40 PM

@noa-lucent noa-lucent left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the PR changes file-by-file against the linked egress gateway implementation scope. I did not find blocking code issues in this repository. Approving.

@noa-lucent noa-lucent left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-reviewed current head. No new blocking issues found in this repository; prior approval still stands.

@vitramir vitramir merged commit 7ce05c6 into main Jun 16, 2026
1 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.

3 participants