Skip to content

Commit

Permalink
chore: cleanup of unnecessary connection-tester (#3208)
Browse files Browse the repository at this point in the history
Co-authored-by: Francesco Casula <fracasula@users.noreply.github.com>
  • Loading branch information
saurav-malani and fracasula committed Apr 14, 2023
1 parent 1211e51 commit 4204f7a
Show file tree
Hide file tree
Showing 5 changed files with 0 additions and 245 deletions.
9 changes: 0 additions & 9 deletions router/batchrouter/batchrouter.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ import (

"github.com/rudderlabs/rudder-server/router/batchrouter/asyncdestinationmanager"
"github.com/rudderlabs/rudder-server/router/rterror"
destinationConnectionTester "github.com/rudderlabs/rudder-server/services/destination-connection-tester"
"github.com/rudderlabs/rudder-server/services/multitenant"
"github.com/rudderlabs/rudder-server/services/rsources"
"github.com/rudderlabs/rudder-server/services/transientsource"
Expand Down Expand Up @@ -203,14 +202,6 @@ func (brt *HandleT) backendConfigSubscriber() {
}
brt.encounteredMergeRuleMapLock.Unlock()
}

if val, ok := destination.Config["testConnection"].(bool); ok && val && misc.Contains(objectStorageDestinations, destination.DestinationDefinition.Name) {
destination := destination
rruntime.Go(func() {
testResponse := destinationConnectionTester.TestBatchDestinationConnection(destination)
destinationConnectionTester.UploadDestinationConnectionTesterResponse(testResponse, destination.ID)
})
}
}
}
}
Expand Down
2 changes: 0 additions & 2 deletions runner/runner.go
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ import (
"github.com/rudderlabs/rudder-server/services/controlplane"
"github.com/rudderlabs/rudder-server/services/db"
"github.com/rudderlabs/rudder-server/services/dedup"
destinationconnectiontester "github.com/rudderlabs/rudder-server/services/destination-connection-tester"
"github.com/rudderlabs/rudder-server/services/diagnostics"
"github.com/rudderlabs/rudder-server/services/multitenant"
"github.com/rudderlabs/rudder-server/services/oauth"
Expand Down Expand Up @@ -336,7 +335,6 @@ func runAllInit() {
pgnotifier.Init()
jobsdb.Init()
jobsdb.Init2()
destinationconnectiontester.Init()
warehouse.Init()
warehouse.Init2()
warehouse.Init3()
Expand Down

This file was deleted.

This file was deleted.

This file was deleted.

0 comments on commit 4204f7a

Please sign in to comment.