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

Commit 2bd53ff

Browse files
authored
fix: emr service role (#244)
* fix: permissions to create EMR service role in new accounts * fix: remove .venv from git * fix: remove unneccessary integ tests
1 parent 3307475 commit 2bd53ff

File tree

4 files changed

+185
-95
lines changed

4 files changed

+185
-95
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ package-lock.json
33
*__pycache__*
44
.pytest_cache
55
.env
6+
.venv
67
venv/
78
*.egg-info
89

core/package.json

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

core/src/integ.default.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import { EmrEksCluster } from '.';
77

88
const mockApp = new App();
99
const stack = new Stack(mockApp, 'stack');
10-
const cluster = new EmrEksCluster(stack, 'testCluster', { eksAdminRoleArn: 'arn:aws:iam::668876353122:role/gromav' });
10+
const cluster = new EmrEksCluster(stack, 'testCluster', { eksAdminRoleArn: 'arn:aws:iam::xxxxxxxxxx:role/xxxxxxxx' });
1111

1212

1313
const virtualCluster = cluster.addEmrVirtualCluster({
@@ -16,6 +16,6 @@ const virtualCluster = cluster.addEmrVirtualCluster({
1616
createNamespace: true,
1717
});
1818

19-
const execRole = Role.fromRoleArn(stack, 'execRole', 'arn:aws:iam::668876353122:role/gromav');
19+
const execRole = Role.fromRoleArn(stack, 'execRole', 'arn:aws:iam::xxxxxxxxxx:role/xxxxxxxx');
2020

21-
cluster.addManagedEndpoint(stack, 'ME', virtualCluster.attrId, execRole, 'arn:aws:acm:us-east-1:668876353122:certificate/aba0c2c8-c470-43a5-a3c0-07189ee96af0');
21+
cluster.addManagedEndpoint(stack, 'ME', virtualCluster.attrId, execRole, 'arn:aws:acm:us-east-1:xxxxxxxxxx:certificate/xxxxxxxxxxxxxxxxxxxxxxxx');

0 commit comments

Comments
 (0)