Skip to content

Fix #1079: Set setVMMaxMapCount default to true for production readiness #1084

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix #1079: Set setVMMaxMapCount default to true for production readiness #1084

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
75aa0a1
Fix #1079: Set setVMMaxMapCount default to true for production readiness
synhershko Sep 5, 2025
55dd023
update design doc
josedev-union Oct 8, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion charts/opensearch-cluster/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ A Helm chart for OpenSearch Cluster
| cluster.general.securityContext | object | `{}` | Opensearch securityContext |
| cluster.general.serviceAccount | string | `""` | Opensearch serviceAccount name. If Service Account doesn't exist it could be created by setting `serviceAccount.create` and `serviceAccount.name` |
| cluster.general.serviceName | string | `""` | Opensearch service name |
| cluster.general.setVMMaxMapCount | bool | `true` | Enable setVMMaxMapCount. OpenSearch requires the Linux kernel vm.max_map_count option to be set to at least 262144 |
| cluster.general.setVMMaxMapCount | bool | `true` | Enable setVMMaxMapCount. OpenSearch requires the Linux kernel vm.max_map_count option to be set to at least 262144. Defaults to true for production readiness |
| cluster.general.snapshotRepositories | list | `[]` | Opensearch snapshot repositories configuration |
| cluster.general.vendor | string | `"Opensearch"` | |
| cluster.general.version | string | `"2.3.0"` | Opensearch version |
Expand Down
1 change: 1 addition & 0 deletions charts/opensearch-operator/files/opensearch.opster.io_opensearchclusters.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4501,6 +4501,7 @@ spec:
serviceName:
type: string
setVMMaxMapCount:
default: true
type: boolean
snapshotRepositories:
items:
Expand Down
2 changes: 1 addition & 1 deletion docs/designs/crd.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ GeneralConfig defines global Opensearch cluster configuration
<td><b>SetVMMaxMapCount</b></td>
<td>bool</td>
<td>will add VMmaxMapCount</td>
<td>false</td>
<td>true</td>
<td></td>
</tr><tr>
<td><b>additionalConfig</b></td>
Expand Down
9 changes: 5 additions & 4 deletions opensearch-operator/api/v1/opensearch_types.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,10 +46,11 @@ type GeneralConfig struct {
//+kubebuilder:default=9200
HttpPort int32 `json:"httpPort,omitempty"`
//+kubebuilder:validation:Enum=Opensearch;Op;OP;os;opensearch
Vendor string `json:"vendor,omitempty"`
Version string `json:"version,omitempty"`
ServiceAccount string `json:"serviceAccount,omitempty"`
ServiceName string `json:"serviceName"`
Vendor string `json:"vendor,omitempty"`
Version string `json:"version,omitempty"`
ServiceAccount string `json:"serviceAccount,omitempty"`
ServiceName string `json:"serviceName"`
//+kubebuilder:default=true
SetVMMaxMapCount bool `json:"setVMMaxMapCount,omitempty"`
DefaultRepo *string `json:"defaultRepo,omitempty"`
// Extra items to add to the opensearch.yml
Expand Down
1 change: 1 addition & 0 deletions opensearch-operator/config/crd/bases/opensearch.opster.io_opensearchclusters.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4501,6 +4501,7 @@ spec:
serviceName:
type: string
setVMMaxMapCount:
default: true
type: boolean
snapshotRepositories:
items:
Expand Down