Skip to content

Commit

Permalink
Add profiling flags (based on kubectl)
Browse files Browse the repository at this point in the history
Signed-off-by: Praveen Rewar <[email protected]>
  • Loading branch information
praveenrewar committed Nov 15, 2023
1 parent b240f25 commit e43adb7
Show file tree
Hide file tree
Showing 2 changed files with 101 additions and 0 deletions.
7 changes: 7 additions & 0 deletions pkg/kapp/cmd/kapp.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ type KappOptions struct {
KubeAPIFlags cmdcore.KubeAPIFlags
KubeconfigFlags cmdcore.KubeconfigFlags
WarningFlags WarningFlags
ProfilingFlags ProfilingFlags
}

func NewKappOptions(ui *ui.ConfUI, configFactory cmdcore.ConfigFactory,
Expand Down Expand Up @@ -63,6 +64,10 @@ func NewKappCmd(o *KappOptions, flagsFactory cmdcore.FlagsFactory) *cobra.Comman
DisableAutoGenTag: true,
Version: version.Version,

PersistentPostRunE: func(*cobra.Command, []string) error {
return o.ProfilingFlags.flushProfiling()
},

// TODO bash completion
}

Expand All @@ -80,6 +85,7 @@ func NewKappCmd(o *KappOptions, flagsFactory cmdcore.FlagsFactory) *cobra.Comman
o.KubeAPIFlags.Set(cmd, flagsFactory)
o.KubeconfigFlags.Set(cmd, flagsFactory)
o.WarningFlags.Set(cmd, flagsFactory)
o.ProfilingFlags.Set(cmd, flagsFactory)

o.configFactory.ConfigurePathResolver(o.KubeconfigFlags.Path.Value)
o.configFactory.ConfigureContextResolver(o.KubeconfigFlags.Context.Value)
Expand Down Expand Up @@ -135,6 +141,7 @@ func NewKappCmd(o *KappOptions, flagsFactory cmdcore.FlagsFactory) *cobra.Comman
o.LoggerFlags.Configure(o.logger)
o.KubeAPIFlags.Configure(o.configFactory)
o.WarningFlags.Configure(o.depsFactory)
o.ProfilingFlags.initProfiling()
return nil
})

Expand Down
94 changes: 94 additions & 0 deletions pkg/kapp/cmd/profiling.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,94 @@
// Copyright 2023 VMware, Inc.
// SPDX-License-Identifier: Apache-2.0

package cmd

import (
"fmt"
"os"
"os/signal"
"runtime"
"runtime/pprof"

"github.com/spf13/cobra"
cmdcore "github.com/vmware-tanzu/carvel-kapp/pkg/kapp/cmd/core"
)

type ProfilingFlags struct {
profileName string
profileOutput string
}

func (p *ProfilingFlags) Set(cmd *cobra.Command, _ cmdcore.FlagsFactory) {
cmd.PersistentFlags().StringVar(&p.profileName, "profile", "none", "Name of profile to capture. One of (none|cpu|heap|goroutine|threadcreate|block|mutex)")
cmd.PersistentFlags().StringVar(&p.profileOutput, "profile-output", "profile.pprof", "Name of the file to write the profile to")
}

func (p *ProfilingFlags) initProfiling() error {
var (
f *os.File
err error
)
switch p.profileName {
case "none":
return nil
case "cpu":
f, err = os.Create(p.profileOutput)
if err != nil {
return err
}
err = pprof.StartCPUProfile(f)
if err != nil {
return err
}
// Block and mutex profiles need a call to Set{Block,Mutex}ProfileRate to
// output anything. We choose to sample all events.
case "block":
runtime.SetBlockProfileRate(1)
case "mutex":
runtime.SetMutexProfileFraction(1)
default:
// Check the profile name is valid.
if profile := pprof.Lookup(p.profileName); profile == nil {
return fmt.Errorf("unknown profile '%s'", p.profileName)
}
}

// If the command is interrupted before the end (ctrl-c), flush the
// profiling files
c := make(chan os.Signal, 1)
signal.Notify(c, os.Interrupt)
go func() {
<-c
f.Close()
p.flushProfiling()
os.Exit(0)
}()

return nil
}

func (p *ProfilingFlags) flushProfiling() error {
switch p.profileName {
case "none":
return nil
case "cpu":
pprof.StopCPUProfile()
case "heap":
runtime.GC()
fallthrough
default:
profile := pprof.Lookup(p.profileName)
if profile == nil {
return nil
}
f, err := os.Create(p.profileOutput)
if err != nil {
return err
}
defer f.Close()
profile.WriteTo(f, 0)
}

return nil
}

0 comments on commit e43adb7

Please sign in to comment.