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.

Sign up for GitHub

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

Jump to bottom

config: rework zone configs structure for multi-tenancy #49854

Closed
Tracked by #52617
nvanbenschoten opened this issue Jun 3, 2020 · 2 comments
Closed
Tracked by #52617

config: rework zone configs structure for multi-tenancy #49854

nvanbenschoten opened this issue Jun 3, 2020 · 2 comments
Labels
A-multitenancy Related to multi-tenancy C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-multiregion X-anchored-telemetry The issue number is anchored by telemetry references.

Comments

@nvanbenschoten
Copy link
Member

#49784 collapsed all zone configs for secondary tenants down to a single pseudo "tenants" config. This is a short term solution to get zone configs and their interaction with SQL working for now. In the long-term, we'll need a new way for tenants to configure their data placement / retention policies. This will likely require us to rethink the zone config data model.
@nvanbenschoten nvanbenschoten added C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) A-multitenancy Related to multi-tenancy labels Jun 3, 2020
@asubiotto asubiotto mentioned this issue Aug 11, 2020
Closed
10 tasks
@knz knz added the X-anchored-telemetry The issue number is anchored by telemetry references. label Sep 11, 2020
This was referenced Sep 11, 2020
Closed
Merged
craig bot pushed a commit that referenced this issue Sep 14, 2020
@knz
Merge #54256
40f03d9 
54256: sql: make multi-tenancy errors report to telemetry r=asubiotto a=knz

Fixes #48164. 

Issues referenced, I also added the X-anchored-telemetry label to them on github (we keep those issues open on github until their reference in the code is removed):

#54254 
#54255 
#54250 
#54251 
#54252 
#49854 
#48274
#47150
#47971
#47970
#47900 
#47925 


Co-authored-by: Raphael 'kena' Poss <[email protected]>
@vy-ton
Copy link
Contributor

vy-ton commented Feb 4, 2021

cc @ajstorm @awoods187 for multi-region triage

@irfansharif irfansharif mentioned this issue May 31, 2021
Open
@irfansharif
Copy link
Contributor

Closing in favor of #67679.

@msbutler msbutler mentioned this issue Sep 22, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-multitenancy Related to multi-tenancy C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-multiregion X-anchored-telemetry The issue number is anchored by telemetry references.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@jlinder @knz @nvanbenschoten @irfansharif @vy-ton