Skip to content

Commit

Permalink
runtime: remove unused context from profiler
Browse files Browse the repository at this point in the history
  • Loading branch information
nadiamoe committed Sep 13, 2023
1 parent c8e9d6b commit 87377ca
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
3 changes: 1 addition & 2 deletions pkg/runtime/fake.go
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
package runtime

import (
"context"
"io"
"os"
"strings"
Expand Down Expand Up @@ -110,7 +109,7 @@ func NewFakeProfiler() *FakeProfiler {
}

// Start updates the FakeProfiler to registers it was started
func (p *FakeProfiler) Start(context.Context, profiler.Config) (io.Closer, error) {
func (p *FakeProfiler) Start(profiler.Config) (io.Closer, error) {
p.started = true
return p, nil
}
Expand Down
5 changes: 2 additions & 3 deletions pkg/runtime/profiler/profiler.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
package profiler

import (
"context"
"io"
)

Expand All @@ -18,7 +17,7 @@ type Config struct {
// Profiler defines the methods to control execution profiling
type Profiler interface {
// Start stars the collection of profiling information with the given configuration
Start(ctx context.Context, config Config) (io.Closer, error)
Start(config Config) (io.Closer, error)
}

// Probe defines the interface for controlling a profiling probe
Expand All @@ -37,7 +36,7 @@ func NewProfiler() Profiler {
}

// Start stars the collection of profiling information with the given configuration
func (p *profiler) Start(_ context.Context, config Config) (io.Closer, error) {
func (p *profiler) Start(config Config) (io.Closer, error) {
probes, err := buildProbes(config)
if err != nil {
return nil, err
Expand Down

0 comments on commit 87377ca

Please sign in to comment.