Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multi-tenancy as a new capability. #143

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

sushaanttb
Copy link
Contributor

@sushaanttb sushaanttb commented Jan 20, 2025

This is a PR for having multi-tenancy as a new capability in the GenAI Backend scenario(Scenario 3).

Following issue describes the context in more detail -
#142

As part of demonstration of multi-tenancy capability,
We are creating 2 example multi-tenancy products with reference to existing OpenAI API , we will apply a new quota policy at the product level, such that if the number of requests to APIM via that product's subscription exceed as per the defined "calls" attribute's value, then the product policy will accordingly block the subsequent requests from that subscription until the quota is refreshed based on the defined "renewal-period" and "counter-key" attribute, everything else in the existing setup stays the same.

For example-
Following 2 new products will now be created:
image

with their own respective quota policies.
image

both linked to the existing OpenAI API:

image

& their corresponding subscriptions:

image

Finally, now as part of our scripts deployment output we will get 2 additional subscriptions for our multi-tenant products for the user to test the same.

image

Summary of changeset in this PR:

  • Added Bicep & Terraform scripts for the creation of two example multi-tenant products with their policies.
  • New ReadMe.md for the new capability
  • Updated ReadMe.md of Scenario 3 to link to the multi-tenancy capability

Both Bicep and Terraform scripts have been tested and seem to be working fine without any impact.

Copy link
Contributor

@prasann prasann left a comment

Choose a reason for hiding this comment

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

will test it once from my side by running the scripts. please give me a day or 2.

@sushaanttb sushaanttb requested a review from prasann January 20, 2025 10:19
@prasann
Copy link
Contributor

prasann commented Jan 22, 2025

will test it once from my side by running the scripts. please give me a day or 2.

done testing. working as expected.

@aarthiem aarthiem requested a review from RussSmi January 23, 2025 15:22
Copy link
Contributor

@RussSmi RussSmi left a comment

Choose a reason for hiding this comment

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

Tested the Terraform version

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.

4 participants