fix(eks): ConfigCommand and GetTokenCommand are output by default

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

@@ -1325,7 +1325,7 @@ export class Cluster extends ClusterBase {
       this.defineCoreDnsComputeType(CoreDnsComputeType.FARGATE);
     }
 
-    const outputConfigCommand = (props.outputConfigCommand ?? true) && props.mastersRole;
+    const outputConfigCommand = (props.outputConfigCommand ?? true);
     if (outputConfigCommand) {
       const postfix = commonCommandOptions.join(' ');
       new CfnOutput(this, 'ConfigCommand', { value: `${updateConfigCommandPrefix} ${postfix}` });

packages/@aws-cdk/aws-eks-v2-alpha/test/cluster.test.ts

@@ -990,7 +990,7 @@ describe('cluster', () => {
   });
 
   describe('outputs', () => {
-    test('no outputs are synthesized by default', () => {
+    test('ConfigCommand and GetTokenCommand are output by default', () => {
       // GIVEN
       const { app, stack } = testFixtureNoVpc();
 
@@ -1000,7 +1000,10 @@ describe('cluster', () => {
       // THEN
       const assembly = app.synth();
       const template = assembly.getStackByName(stack.stackName).template;
-      expect(template.Outputs).toBeUndefined(); // no outputs
+      expect(template.Outputs).toEqual({
+        ClusterConfigCommand43AAE40F: { Value: { 'Fn::Join': ['', ['aws eks update-kubeconfig --name ', { Ref: 'ClusterEB0386A7' }, ' --region us-east-1']] } },
+        ClusterGetTokenCommand06AE992E: { Value: { 'Fn::Join': ['', ['aws eks get-token --cluster-name ', { Ref: 'ClusterEB0386A7' }, ' --region us-east-1']] } },
+      });
     });
 
     describe('boostrap user-data', () => {

packages/aws-cdk-lib/aws-eks/lib/cluster.ts

@@ -1842,7 +1842,7 @@ export class Cluster extends ClusterBase {
         this.addNodegroupCapacity('DefaultCapacity', { instanceTypes: [instanceType], minSize: minCapacity }) : undefined;
     }
 
-    const outputConfigCommand = (props.outputConfigCommand ?? true) && props.mastersRole;
+    const outputConfigCommand = (props.outputConfigCommand ?? true);
     if (outputConfigCommand) {
       const postfix = commonCommandOptions.join(' ');
       new CfnOutput(this, 'ConfigCommand', { value: `${updateConfigCommandPrefix} ${postfix}` });

packages/aws-cdk-lib/aws-eks/test/cluster.test.ts

@@ -1350,7 +1350,7 @@ describe('cluster', () => {
   });
 
   describe('outputs', () => {
-    test('no outputs are synthesized by default', () => {
+    test('ConfigCommand and GetTokenCommand are output by default', () => {
       // GIVEN
       const { app, stack } = testFixtureNoVpc();
 
@@ -1360,7 +1360,10 @@ describe('cluster', () => {
       // THEN
       const assembly = app.synth();
       const template = assembly.getStackByName(stack.stackName).template;
-      expect(template.Outputs).toBeUndefined(); // no outputs
+      expect(template.Outputs).toEqual({
+        ClusterConfigCommand43AAE40F: { Value: { 'Fn::Join': ['', ['aws eks update-kubeconfig --name ', { Ref: 'Cluster9EE0221C' }, ' --region us-east-1']] } },
+        ClusterGetTokenCommand06AE992E: { Value: { 'Fn::Join': ['', ['aws eks get-token --cluster-name ', { Ref: 'Cluster9EE0221C' }, ' --region us-east-1']] } },
+      });
     });
 
     test('if masters role is defined, it should be included in the config command', () => {
@@ -1385,7 +1388,7 @@ describe('cluster', () => {
       });
     });
 
-    test('if `outputConfigCommand=false` will disabled the output', () => {
+    test('outputConfigCommand=false` will disabled the output of ConfigCommand and GetTokenCommand', () => {
       // GIVEN
       const { app, stack } = testFixtureNoVpc();