Skip to content

Commit

Permalink
eks: fix wrong namespace for "kubectl describe job" call
Browse files Browse the repository at this point in the history
Signed-off-by: Gyuho Lee <leegyuho@amazon.com>
  • Loading branch information
gyuho committed Jul 13, 2020
1 parent b61285b commit 561cdfe
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions eks/configmaps/remote/configmaps.go
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,7 @@ func (ts *tester) Create() (err error) {
descArgs := []string{
ts.cfg.EKSConfig.KubectlPath,
"--kubeconfig=" + ts.cfg.EKSConfig.KubeConfigPath,
"--namespace=" + ts.cfg.EKSConfig.AddOnCSRsRemote.Namespace,
"--namespace=" + ts.cfg.EKSConfig.AddOnConfigmapsRemote.Namespace,
"describe",
"job",
configmapsJobName,
Expand Down Expand Up @@ -761,7 +761,7 @@ func (ts *tester) checkResults() (err error) {
if err == nil {
ts.cfg.Logger.Info("reading writes results raw",
zap.String("writes-dir", writesDirRaw),
zap.String("s3-dir", path.Dir(ts.cfg.EKSConfig.AddOnCSRsRemote.RequestsRawWritesJSONS3Key)),
zap.String("s3-dir", path.Dir(ts.cfg.EKSConfig.AddOnConfigmapsRemote.RequestsRawWritesJSONS3Key)),
)
cnt := 0
err = filepath.Walk(writesDirRaw, func(fpath string, info os.FileInfo, werr error) error {
Expand Down
2 changes: 1 addition & 1 deletion eks/secrets/remote/secrets.go
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,7 @@ func (ts *tester) Create() (err error) {
descArgs := []string{
ts.cfg.EKSConfig.KubectlPath,
"--kubeconfig=" + ts.cfg.EKSConfig.KubeConfigPath,
"--namespace=" + ts.cfg.EKSConfig.AddOnCSRsRemote.Namespace,
"--namespace=" + ts.cfg.EKSConfig.AddOnStresserRemote.Namespace,
"describe",
"job",
secretsJobName,
Expand Down
2 changes: 1 addition & 1 deletion eks/stresser/remote/stresser.go
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ func (ts *tester) Create() (err error) {
descArgs := []string{
ts.cfg.EKSConfig.KubectlPath,
"--kubeconfig=" + ts.cfg.EKSConfig.KubeConfigPath,
"--namespace=" + ts.cfg.EKSConfig.AddOnCSRsRemote.Namespace,
"--namespace=" + ts.cfg.EKSConfig.AddOnStresserRemote.Namespace,
"describe",
"job",
stresserJobName,
Expand Down

0 comments on commit 561cdfe

Please sign in to comment.