-
Notifications
You must be signed in to change notification settings - Fork 783
Description
What happened:
I was attempting to use a yaml file contained in the config directory to install the aws vpc cni. This line: https://github.com/aws/amazon-vpc-cni-k8s/blob/master/config/master/aws-k8s-cni-cn.yaml#L208 should be joined with the preceding line.
Errors like this could be prevented in the future by including a step in CI to have the files processed a tools like yq.
As a note, I am aware of the helm chart that is available. I prefer to use files like those included in the projects config directory and make the require modifications through kustomize.
This malformed line has been around for a few releases without anyone complaining so I would bet that most people are using the provided helm chart.
What you expected to happen:
For any yaml contained in the project to be well formed. Specifically, I would expect that config files included in the config directory are suitable to use as a base for installing.
How to reproduce it (as minimally and precisely as possible):
cat aws-k8s-cni-cn.yaml | yq
Observe the outputted error when trying to process the malformed yaml.
Anything else we need to know?:
Not relevant