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

FEAT-Tenant Support for AVI #9

Open
CarlosOVillanueva opened this issue Sep 18, 2020 · 0 comments
Open

FEAT-Tenant Support for AVI #9

CarlosOVillanueva opened this issue Sep 18, 2020 · 0 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@CarlosOVillanueva
Copy link
Contributor

Currently, when the SDK fork connects to AVI, it explicitly passes the AVI Admin tenant to the connection string. We need to add an override that replaces Admin with whatever tenant the client desires.

CONCERNS

  • Tenants are meant to provide security, and since lbAPI relies on the product_code field, it makes sense to match tenants to product code. This works for Ticketmaster's specific use case, but not for all. Suggestions on how to approach this would be appreciated.

WANT

  • Field added to the virtualserver.Data for tenant
  • Want to be able to override the default 'admin' tenant target with the user defined value
  • Tie product_code into it to control access to the tenant
  • Tests to validate that the tenant exists, etc.
@CarlosOVillanueva CarlosOVillanueva added enhancement New feature or request help wanted Extra attention is needed labels Sep 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant