Skip to content

Commit 3f87bb6

Browse files
Merge pull request #291 from splunk/revert-286-CSPL555
Revert "CSPL-555 Adding resource requests/limits for MC"
2 parents 466aace + 1dd695a commit 3f87bb6

File tree

1 file changed

+0
-17
lines changed

1 file changed

+0
-17
lines changed

pkg/splunk/enterprise/monitoringconsole.go

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ import (
2828
splutil "github.com/splunk/splunk-operator/pkg/splunk/util"
2929
appsv1 "k8s.io/api/apps/v1"
3030
corev1 "k8s.io/api/core/v1"
31-
"k8s.io/apimachinery/pkg/api/resource"
3231
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
3332
"k8s.io/apimachinery/pkg/types"
3433
)
@@ -206,22 +205,6 @@ func getMonitoringConsoleStatefulSet(client splcommon.ControllerClient, cr splco
206205
MountPath: "/opt/splunk/var",
207206
},
208207
},
209-
//Below requests/limits for MC are defined taking into account below EC2 validated architecture and its defined limits
210-
//1. https://www.splunk.com/pdfs/technical-briefs/deploying-splunk-enterprise-on-amazon-web-services-technical-brief.pdf
211-
//defines the validate architecture for License Master and Monitoring console i.e, c5.2xlarge
212-
//2. (c5.2xlarge) architecture req from https://aws.amazon.com/ec2/instance-types/c5/
213-
//defines that for c5.2xlarge architecture we need 8vCPU and 16Gi memory
214-
//since we only have MC here (as we have separate LM) so 4vCPU and 8Gi memory has been set as limit for MC pod
215-
Resources: corev1.ResourceRequirements{
216-
Requests: corev1.ResourceList{
217-
corev1.ResourceCPU: resource.MustParse("0.1"),
218-
corev1.ResourceMemory: resource.MustParse("512Mi"),
219-
},
220-
Limits: corev1.ResourceList{
221-
corev1.ResourceCPU: resource.MustParse("4"),
222-
corev1.ResourceMemory: resource.MustParse("8Gi"),
223-
},
224-
},
225208
},
226209
},
227210
Volumes: []corev1.Volume{

0 commit comments

Comments
 (0)