Skip to content

Commit

Permalink
remove useless tomb instances
Browse files Browse the repository at this point in the history
  • Loading branch information
mcorbin committed Jan 9, 2025
1 parent 526843e commit 1063197
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 8 deletions.
3 changes: 0 additions & 3 deletions healthcheck/http.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,6 @@ import (
"go.opentelemetry.io/contrib/instrumentation/net/http/httptrace/otelhttptrace"
"go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp"
"go.uber.org/zap"

"gopkg.in/tomb.v2"
)

// HTTPHealthcheckConfiguration defines an HTTP healthcheck configuration
Expand Down Expand Up @@ -93,7 +91,6 @@ type HTTPHealthcheck struct {
URL string

Tick *time.Ticker
t tomb.Tomb
Client *http.Client
}

Expand Down
3 changes: 0 additions & 3 deletions healthcheck/tcp.go
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,6 @@ import (

"github.com/pkg/errors"
"go.uber.org/zap"

"gopkg.in/tomb.v2"
)

// TCPHealthcheckConfiguration defines a TCP healthcheck configuration
Expand Down Expand Up @@ -56,7 +54,6 @@ type TCPHealthcheck struct {
URL string

Tick *time.Ticker
t tomb.Tomb
}

// buildURL build the target URL for the TCP healthcheck, depending of its
Expand Down
2 changes: 0 additions & 2 deletions healthcheck/tls.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ import (
"github.com/appclacks/cabourotte/tls"
"github.com/pkg/errors"
"go.uber.org/zap"
"gopkg.in/tomb.v2"
)

// TLSHealthcheckConfiguration defines a TLS healthcheck configuration
Expand All @@ -38,7 +37,6 @@ type TLSHealthcheck struct {
TLSConfig *cryptotls.Config

Tick *time.Ticker
t tomb.Tomb
}

// Validate validates the healthcheck configuration
Expand Down

0 comments on commit 1063197

Please sign in to comment.