Skip to content

Commit

Permalink
fix: fix kubectl result formatting for debug logs (GoogleContainerToo…
Browse files Browse the repository at this point in the history
  • Loading branch information
briandealwis authored and bskaplan committed Apr 26, 2022
1 parent 3a4cca1 commit 6812958
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/skaffold/deploy/kubectl/cli.go
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ func (c *CLI) Apply(ctx context.Context, out io.Writer, manifests manifest.Manif
// Only redeploy modified or new manifests
// TODO(dgageot): should we delete a manifest that was deployed and is not anymore?
updated := c.previousApply.Diff(manifests)
log.Entry(ctx).Debug(len(manifests), "manifests to deploy.", len(updated), "are updated or new")
log.Entry(ctx).Debugf("%d manifests to deploy. %d are updated or new", len(manifests), len(updated))
c.previousApply = manifests
if len(updated) == 0 {
return nil
Expand Down Expand Up @@ -173,7 +173,7 @@ func (c *CLI) WaitForDeletions(ctx context.Context, out io.Writer, manifests man
}

list := `"` + strings.Join(marked, `", "`) + `"`
log.Entry(ctx).Debug("Resources are marked for deletion:", list)
log.Entry(ctx).Debug("Resources are marked for deletion: ", list)
if list != previousList {
if len(marked) == 1 {
fmt.Fprintf(out, "%s is marked for deletion, waiting for completion\n", list)
Expand Down

0 comments on commit 6812958

Please sign in to comment.