Commit 2b8717b
authored
Add support for addon updates (#48)
Signed-off-by: smcavallo <[email protected]>
Issue #, if available:
Description of changes:
Issue affects `public.ecr.aws/aws-controllers-k8s/eks-controller:v0.1.4`
Currently unable to update Addon with the below exception.
```
- lastTransitionTime: "2022-08-12T14:48:28Z"
message: Unable to determine if desired resource state matches latest observed
state
reason: |
InvalidParameter: 1 validation error(s) found.
- missing required field, UpdateAddonInput.AddonName.
```
the Name is a requirement due to the shape -> https://github.com/aws/aws-sdk-go/blob/ec3763d8d5abacbf47c0b4296fb5d27821297508/models/apis/eks/2017-11-01/api-2.json#L2128-L2133
This same configuration exists for `CreateAddon | DescribeAddon | DeleteAddon`
With the change in place the Addon upgrades as expected.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.1 parent f38b5d3 commit 2b8717b
File tree
5 files changed
+11
-2
lines changed- apis/v1alpha1
- pkg/resource
- addon
- nodegroup
- templates/hooks/nodegroup
5 files changed
+11
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
24 | 27 | | |
25 | 28 | | |
26 | 29 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
24 | 27 | | |
25 | 28 | | |
26 | 29 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
| |||
0 commit comments