eksconfig: fix "AddOnAmiSoftLockupIssue454" env parsing (#199)

* eksconfig: fix "AddOnAmiSoftLockupIssue454" env parsing Signed-off-by: Gyuho Lee <leegyuho@amazon.com> * CHANGELOG: update Signed-off-by: Gyuho Lee <leegyuho@amazon.com>
aws · Jan 26, 2021 · 43735a9 · 43735a9
1 parent d483003
commit 43735a9
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 3 deletions.
diff --git a/CHANGELOG/CHANGELOG-1.5.md b/CHANGELOG/CHANGELOG-1.5.md
@@ -1,8 +1,22 @@
 
 
+<hr>
+
+
+## [v1.5.7](https://github.com/aws/aws-k8s-tester/releases/tag/v1.5.7) (2021-01-25)
+
+See [code changes](https://github.com/aws/aws-k8s-tester/compare/v1.5.6...v1.5.7).
+### `eksconfig`
+
+- Fix ["AddOnAmiSoftLockupIssue454" parsing](https://github.com/aws/aws-k8s-tester/pull/199).
+### Go
+
+- Compile with [*Go 1.16beta1*](https://golang.org/doc/devel/release.html#go1.16).
+
 
 <hr>
 
+
 ## [v1.5.6](https://github.com/aws/aws-k8s-tester/releases/tag/v1.5.6) (2021-01-25)
 
 See [code changes](https://github.com/aws/aws-k8s-tester/compare/v1.5.5...v1.5.6).

diff --git a/eksconfig/README.md b/eksconfig/README.md
@@ -266,6 +266,9 @@ AWS_K8S_TESTER_EKS_ADD_ON_CLUSTER_VERSION_UPGRADE_ENABLE=true \
 | AWS_K8S_TESTER_EKS_ADD_ON_CONFORMANCE_NAMESPACE                           | read-only "false" | *eksconfig.AddOnConformance.Namespace                       | string             |
 | AWS_K8S_TESTER_EKS_ADD_ON_CONFORMANCE_SONOBUOY_PATH                       | read-only "false" | *eksconfig.AddOnConformance.SonobuoyPath                    | string             |
 | AWS_K8S_TESTER_EKS_ADD_ON_CONFORMANCE_SONOBUOY_DOWNLOAD_URL               | read-only "false" | *eksconfig.AddOnConformance.SonobuoyDownloadURL             | string             |
+| AWS_K8S_TESTER_EKS_ADD_ON_CONFORMANCE_SONOBUOY_E2E_REPO_CONFIG            | read-only "false" | *eksconfig.AddOnConformance.SonobuoyE2eRepoConfig           | string             |
+| AWS_K8S_TESTER_EKS_ADD_ON_CONFORMANCE_SONOBUOY_IMAGE                      | read-only "false" | *eksconfig.AddOnConformance.SonobuoyImage                   | string             |
+| AWS_K8S_TESTER_EKS_ADD_ON_CONFORMANCE_SYSTEMD_LOGS_IMAGE                  | read-only "false" | *eksconfig.AddOnConformance.SystemdLogsImage                | string             |
 | AWS_K8S_TESTER_EKS_ADD_ON_CONFORMANCE_SONOBUOY_DELETE_TIMEOUT             | read-only "false" | *eksconfig.AddOnConformance.SonobuoyDeleteTimeout           | time.Duration      |
 | AWS_K8S_TESTER_EKS_ADD_ON_CONFORMANCE_SONOBUOY_DELETE_TIMEOUT_STRING      | read-only "true"  | *eksconfig.AddOnConformance.SonobuoyDeleteTimeoutString     | string             |
 | AWS_K8S_TESTER_EKS_ADD_ON_CONFORMANCE_SONOBUOY_RUN_TIMEOUT                | read-only "false" | *eksconfig.AddOnConformance.SonobuoyRunTimeout              | time.Duration      |

diff --git a/eksconfig/env.go b/eksconfig/env.go
@@ -563,9 +563,6 @@ func (cfg *Config) UpdateFromEnvs() (err error) {
 	}
 	if av, ok := vv.(*AddOnAmiSoftLockupIssue454); ok {
 		cfg.AddOnAmiSoftLockupIssue454 = av
-		if !cfg.AddOnAmiSoftLockupIssue454.Enable {
-			return fmt.Errorf("WTF %T", cfg)
-		}
 	} else {
 		return fmt.Errorf("expected *AddOnAmiSoftLockupIssue454, got %T", vv)
 	}