Skip to content

Commit

Permalink
Merge pull request #769 from nrajb/fix/container-insights
Browse files Browse the repository at this point in the history
Fix/container insights
  • Loading branch information
shapirov103 authored Jul 11, 2023
2 parents 5bd430e + dfc8331 commit 0020d18
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions lib/addons/container-insights/index.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import { ManagedPolicy } from "aws-cdk-lib/aws-iam";
import { Construct } from "constructs";
import merge from "ts-deepmerge";
import { assertEC2NodeGroup } from "../..";
import { ClusterInfo } from "../../spi";
import { HelmAddOn, HelmAddOnUserProps } from "../helm-addon";
import { ValuesSchema } from "./values";
Expand Down Expand Up @@ -32,13 +31,8 @@ export class ContainerInsightsAddOn extends HelmAddOn {
@conflictsWith("AdotCollectorAddOn")
deploy(clusterInfo: ClusterInfo): Promise<Construct> {
const cluster = clusterInfo.cluster;
const nodeGroups = assertEC2NodeGroup(clusterInfo, ContainerInsightsAddOn.name);
const policy = ManagedPolicy.fromAwsManagedPolicyName('CloudWatchAgentServerPolicy');

nodeGroups.forEach(nodeGroup => {
nodeGroup.role.addManagedPolicy(policy);
});

// Create an adot-collector service account.
const serviceAccountName = "adot-collector-sa";
let serviceAccountNamespace;
Expand Down

0 comments on commit 0020d18

Please sign in to comment.