Skip to content
This repository was archived by the owner on Jul 16, 2024. It is now read-only.

Commit c5424c2

Browse files
authored
fix: fix in emr eks critical pod def (#578)
* add memory unit to critical executor
1 parent f67969f commit c5424c2

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

core/src/emr-eks-platform/resources/k8s/emr-eks-config/critical.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
"spark.dynamicAllocation.maxExecutors": "40",
1212
"spark.kubernetes.allocation.batch.size": "8",
1313
"spark.kubernetes.executor.request.cores": "7.7",
14-
"spark.executor.memory": "20800",
14+
"spark.executor.memory": "20800m",
1515
"spark.kubernetes.driver.request.cores": "2.5",
1616
"spark.driver.memory": "6800m",
1717
"spark.dynamicAllocation.executorAllocationRatio": "1",

core/test/unit/cdk-nag/nag-docker-build.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ NagSuppressions.addResourceSuppressionsByPath(
173173
'docker-build/dockerbuild-ara-docker-assets/Resource',
174174
[{ id: 'AwsSolutions-S1', reason: 'Log disabled the bucket hold asset needed for build no data' }],
175175
);
176-
176+
177177
NagSuppressions.addResourceSuppressionsByPath(
178178
stack,
179179
'docker-build/test/CustomResourceProvider/framework-onEvent/ServiceRole/Resource',

0 commit comments

Comments
 (0)