Skip to content

Commit 6d834c0

Browse files
authored
chore(eks-v2): remove awsAuth (#32961)
### Reason for this change For `eks-v2` module, it uses `API` as authentication mode which means access entries will be used to replace aws-auth ConfigMap. `awsAuth` construct is not used anymore. ### Description of changes - `authenticationMode` is removed from L2 because the only authentication mode allowed is `API`. The value is hardcode when creating the L1 resource. - Remove `awsAuth` and its usage in the new module ### Description of how you validated changes unit tests ### Checklist - [ ] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent 5410e10 commit 6d834c0

13 files changed

+18
-1004
lines changed

packages/@aws-cdk/aws-eks-v2-alpha/lib/alb-controller.ts

+1-5
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import * as fs from 'fs';
22
import * as path from 'path';
33
import { Construct } from 'constructs';
4-
import { Cluster, AuthenticationMode } from './cluster';
4+
import { Cluster } from './cluster';
55
import { HelmChart } from './helm-chart';
66
import { ServiceAccount } from './service-account';
77
import * as iam from 'aws-cdk-lib/aws-iam';
@@ -359,10 +359,6 @@ export class AlbController extends Construct {
359359
// the controller relies on permissions deployed using these resources.
360360
chart.node.addDependency(serviceAccount);
361361
chart.node.addDependency(props.cluster.openIdConnectProvider);
362-
if (props.cluster.authenticationMode != AuthenticationMode.API) {
363-
// ensure the dependency only when ConfigMap is supported
364-
chart.node.addDependency(props.cluster.awsAuth);
365-
}
366362
}
367363

368364
private rewritePolicyResources(resources: string | string[] | undefined): string | string[] | undefined {

packages/@aws-cdk/aws-eks-v2-alpha/lib/aws-auth-mapping.ts

-18
This file was deleted.

packages/@aws-cdk/aws-eks-v2-alpha/lib/aws-auth.ts

-156
This file was deleted.

0 commit comments

Comments
 (0)