Pahud Hsieh

Results 356 comments of Pahud Hsieh

Hi @leonmk-aws Thank you for the excellent feedback! You're absolutely correct about `Names.uniqueId()` being path-based, and I'm happy to clarify that our revised implementation actually **does not use `Names.uniqueId()`** for...

We probably need the official Helm Chart support in [aws/eks-chart](https://github.com/aws/eks-charts) for container insights before we can `addChart()` to install this in AWS CDK as a cluster plug-in. https://github.com/aws-samples/amazon-cloudwatch-container-insights/issues/17

@eduardomourar It might be possible as container insight has different deployment mode including daemonset, service and sidecar. https://github.com/aws-samples/amazon-cloudwatch-container-insights/tree/master/k8s-deployment-manifest-templates/deployment-mode Are you interested to dive in and probably a PR for this?...

@jbhalla-godaddy As far as I know, if we deploy EKS with cloudformation CfnCluster L1 resource, the admin role of the cluster will be the CFN deploying role. With the admin...

@gricey432 Yes we have the plan for the rewrite per the RFC https://github.com/aws/aws-cdk-rfcs/issues/605

https://github.com/aws/aws-cdk/blob/c7d6fb696c0d9a728ff0027c775fbf7750eec787/packages/%40aws-cdk/custom-resource-handlers/lib/aws-eks/cluster-resource-handler/fargate.ts#L9 https://github.com/aws/aws-cdk/blob/c7d6fb696c0d9a728ff0027c775fbf7750eec787/packages/%40aws-cdk/custom-resource-handlers/lib/aws-eks/cluster-resource-handler/fargate.ts#L39 Yes, thanks for pointing this out. We should get it fixed. Making it a p1 and we welcome PRs as well.

Hi @heathedavid, Thank you for the feature request! However, the functionality you're looking for already exists in the CDK. You can use [`StateMachine.fromStateMachineArn()`](https://github.com/aws/aws-cdk/blob/main/packages/aws-cdk-lib/aws-stepfunctions/lib/state-machine.ts#L179) to create an `IStateMachine` from an ARN...

Hi @alex-at-cascade, Thank you for this feature request! This appears to be a valuable addition to the CDK that would help teams better handle configuration drift scenarios. CloudFormation's drift-aware change...

I am not 100% clear about your use case. Are you able to share a minimal code to illustrate your use case? Generally when you define a [HelmChart](https://github.com/aws/aws-cdk/blob/1d1876c2caf741953cf42ffc59029ea875ebccb7/packages/aws-cdk-lib/aws-eks/lib/helm-chart.ts#L106), under the...

Hi @mbloch1986, Thank you for this feature request and for quickly following up with PR #36185! This is a valid enhancement to support VPCLink v2 (from aws-apigatewayv2) in REST API...