-
Notifications
You must be signed in to change notification settings - Fork 4.5k
‼️ NOTICE: (aws-eks): Regression in installing Helm charts from assets #19179
Description
Please add your +1 👍 to let us know you have encountered this
Status: IN-PROGRESS
Overview:
Helm charts fail to install because the kubectl provider Lambda assumes 'repository' is non-null. This is not the case when a Helm chart is installed from an asset. Here's the newly introduced line that creates the regression: https://github.com/aws/aws-cdk/pull/18547/files#diff-8b2da7b95e6a38620688d9fd0f02cafb4374713386c3a50f9f9ee79369a77877R83 <- Error: 'NoneType' object has no attribute 'startswith'
Solution:
PR for the fix: #19180
Reproduction Steps
Install a Helm chart from an asset.
What did you expect to happen?
Helm chart to be installed successfully.
What actually happened?
Lambda threw an exception:
[ERROR] AttributeError: 'NoneType' object has no attribute 'startswith'
Traceback (most recent call last):
File "/var/task/index.py", line 17, in handler
return helm_handler(event, context)
File "/var/task/helm/init.py", line 83, in helm_handler
if repository.startswith('oci://'):
CDK CLI Version
2.12.0 (build c9786db)
Framework Version
No response
Node.js Version
v16.13.0
OS
Linux
Language
Typescript
Language Version
No response
Other information
No response