-
Notifications
You must be signed in to change notification settings - Fork 29
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
Add parameter to customize the number of manilaAPI processes #371
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fmount The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/test manila-operator-build-deploy-kuttl |
@@ -672,6 +672,11 @@ spec: | |||
caBundleSecretName: | |||
type: string | |||
type: object | |||
workers: | |||
default: 3 |
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.
@gouthampacha is this a good "default" ? Do we want to stay w/ 4
(the original hardcoded value), or do we want something else?
Restored 4
as default. Do we want a different value?
I am against adding a parameter for something you can use the custom config way to set it. we do not want to introduce tht params |
ok, wait this is different. this is for httpd customization |
form httpd tuning, check this keystone PR openstack-k8s-operators/keystone-operator#500 . we should do the same here |
14b46b2
to
55cc6a9
Compare
Signed-off-by: Francesco Pantano <[email protected]>
No description provided.