feat(server)!: add multitenancy support with per-tenant routing by ehsavoie · Pull Request #1084 · a2aproject/a2a-java · GitHub
Skip to content

feat(server)!: add multitenancy support with per-tenant routing - #1084

Merged
ehsavoie merged 1 commit into
a2aproject:mainfrom
ehsavoie:issue_1034_router
Aug 25, 2026
Merged

ehsavoie merged 1 commit into
a2aproject:mainfrom
ehsavoie:issue_1034_router

Conversation

@ehsavoie

Copy link
Copy Markdown
Collaborator
  • Add AgentExecutorRouter and AgentCardRouter SPI in server-common
  • Add @tenant CDI qualifier and CDI router implementations in extras/multitenancy
  • Wire tenant-aware resolution in DefaultRequestHandler and all transport handlers
  • Add tenant field to RequestContext for execute/cancel flows
  • Update GetExtendedAgentCardRequest to carry GetExtendedAgentCardParams

Fixes #1034 🦕

@ehsavoie
ehsavoie requested review from jmesnil and kabir and removed request for jmesnil August 21, 2026 09:42
@ehsavoie

Copy link
Copy Markdown
Collaborator Author

@kabir kabir left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Overall looks good, some minor stuff.

But should we put this into compat-0.3 too?

Comment thread server-common/src/main/java/org/a2aproject/sdk/server/util/CdiUtils.java Outdated
Comment thread server-common/src/main/java/org/a2aproject/sdk/server/util/CdiUtils.java Outdated
@ehsavoie
ehsavoie requested a review from kabir August 21, 2026 11:19
@kabir
kabir force-pushed the issue_1034_router branch from c175054 to ca6b7f8 Compare August 24, 2026 10:23

@kabir kabir left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@ehsavoie Do we need an extra/multi-tenancy file?

Comment thread docs/content/dev/extras.md Outdated
@ehsavoie
ehsavoie force-pushed the issue_1034_router branch 2 times, most recently from a368f41 to 4391cdf Compare August 24, 2026 15:53

@kabir kabir left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Missing extra/multi-tenancy.md

Comment thread docs/content/dev/extras.md
Comment thread docs/content/dev/multi-tenancy.md Outdated
@ehsavoie
ehsavoie force-pushed the issue_1034_router branch 2 times, most recently from 3474697 to edea0d1 Compare August 25, 2026 16:40
- Add AgentExecutorRouter and AgentCardRouter SPI in server-common
- Add @tenant CDI qualifier and CDI router implementations in extras/multitenancy
- Wire tenant-aware resolution in DefaultRequestHandler and all transport handlers
- Add tenant field to RequestContext for execute/cancel flows
- Update GetExtendedAgentCardRequest to carry GetExtendedAgentCardParams

Signed-off-by: Emmanuel Hugonnet <ehugonne@redhat.com>
@ehsavoie
ehsavoie merged commit c59bf76 into a2aproject:main Aug 25, 2026
15 checks passed
@ehsavoie
ehsavoie deleted the issue_1034_router branch August 26, 2026 16:05
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.

[Bug]: JSON-RPC server documents tenant-prefixed routing but registers no route that can match it

2 participants