-
Notifications
You must be signed in to change notification settings - Fork 206
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 tenant #79
Multi tenant #79
Conversation
Move config to environment variables only too
return internalServerError("Database error deleting instance").WithInternalError(err) | ||
} | ||
|
||
// TODO do we delete everything associated with an instance too? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
file a ticket
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's do hard remove for now. If we want to change it to soft delete later, we can do that all at once.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if globalConfig.OperatorToken == "" { | ||
logrus.Fatal("Operator token secret is required") | ||
} | ||
if globalConfig.DB.Namespace != "" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that isn't required right? The namespace corresponds to the name of the tables, which can be empty. We can use a single set of tables regardless of tenancy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you misread something? This is the code:
if globalConfig.DB.Namespace != "" {
models.Namespace = globalConfig.DB.Namespace
}
LGTM |
Fixes #54
- Summary
Add multi-tenant support. Changes config to environment variables only.
- Test plan
Existing tests pass. A few tests/assertions added.
- Description for the changelog
Add multi-tenant support.