-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
[stable29] multi-instance object store #56691
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
base: stable29
Are you sure you want to change the base?
Conversation
641de93 to
c98c9cb
Compare
Signed-off-by: Robin Appelman <[email protected]>
Signed-off-by: Robin Appelman <[email protected]>
Signed-off-by: Robin Appelman <[email protected]>
Signed-off-by: Robin Appelman <[email protected]>
Signed-off-by: Robin Appelman <[email protected]>
Signed-off-by: Robin Appelman <[email protected]>
Signed-off-by: Robin Appelman <[email protected]>
Signed-off-by: Robin Appelman <[email protected]>
c98c9cb to
655c14e
Compare
| } | ||
| $this->config->setSystemValue('objectstore', $configs); | ||
| } else { | ||
| return 0; |
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.
Maybe display the piece of config to add here ?
| $config = $target; | ||
| } | ||
| } | ||
| $this->config->setSystemValue('objectstore', $configs); |
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.
When it duplicates a configuration it use the same bucket and later fails with:
Each object store configuration must use distinct bucket names
🚧