Skip to content

Commit

Permalink
eks/app-mesh: skip namespace delete error for now
Browse files Browse the repository at this point in the history
Signed-off-by: Gyuho Lee <[email protected]>
  • Loading branch information
gyuho committed Jun 14, 2020
1 parent b659a35 commit 0a1370c
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion eks/app-mesh/app-mesh.go
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,9 @@ func (ts *tester) Delete() error {
k8s_client.DefaultNamespaceDeletionInterval,
k8s_client.DefaultNamespaceDeletionTimeout,
); err != nil {
errs = append(errs, fmt.Sprintf("failed to delete AppMesh namespace (%v)", err))
ts.cfg.Logger.Warn("failed to delete AppMesh namespace", zap.Error(err))
// TODO
// errs = append(errs, fmt.Sprintf("failed to delete AppMesh namespace (%v)", err))
}

if err := ts.deletePolicy(); err != nil {
Expand Down

0 comments on commit 0a1370c

Please sign in to comment.