Skip to content

Commit

Permalink
Merge pull request dapr#5339 from microsvs/optimize/tests1012
Browse files Browse the repository at this point in the history
bugfix(appmanager): pod name be cleared
  • Loading branch information
daixiang0 authored Oct 13, 2022
2 parents 857538f + 30317e7 commit 81fc87d
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions tests/platforms/kubernetes/appmanager.go
Original file line number Diff line number Diff line change
Expand Up @@ -311,12 +311,11 @@ func (m *AppManager) WaitUntilDeploymentState(isState func(*appsv1.Deployment, e
podStatus := map[string][]apiv1.ContainerStatus{}
if err == nil {
for i, pod := range podList.Items {
podStatus[pod.Name] = pod.Status.ContainerStatuses
// Reset Spec and ObjectMeta which could contain sensitive info like credentials
pod.Spec.Reset()
pod.ObjectMeta.Reset()
podList.Items[i] = pod

podStatus[pod.Name] = pod.Status.ContainerStatuses
}
j, _ := json.Marshal(podList)
log.Printf("deployment %s relate pods: %s", m.app.AppName, string(j))
Expand Down

0 comments on commit 81fc87d

Please sign in to comment.