PMM-10147 PMM-4112 Use one runner for jobs and actions (#938)

* PMM-10147 Use one runner for jobs and actions

* PMM-10147 Fix tests

* PMM-10147 Fix tests

* PMM-10147 Fix tests

* PMM-10147 Fix test

* PMM-10147 Disable ireturn linter

* PMM-10147 Refactoring

* PMM-10147 Add missing done flag to actions result

* PMM-10147 Fix tests

* PMM-10147 Fix licence headers

* PMM-10147 Limit max amount of running jobs and actions

* PMM-10147 Cleanup

* PMM-10147 Fix comment

* PMM-10147 Fix error message

* PMM-10147 Send jobs/actions results in case if samaphore wasn't acquired

* PMM-10147 Fix comment

* PMM-19147 Update codeowners

.github/CODEOWNERS

@@ -2,8 +2,7 @@
 /build/                     @nikita-b @atymchuk 
 /admin/                     @michal-kralik
 /agent/agents/postgres/     @JiriCtvrtka
-/agent/actions/             @artemgavrilov
-/agent/jobs/                @artemgavrilov
+/agent/runner/              @artemgavrilov
 /api/                       @BupycHuk
 /docs/api/                  @atymchuk
 /managed/services/checks/   @idoqo

.golangci.yml

@@ -66,7 +66,6 @@ linters:
     - gochecknoinits    # we use init functions
     - gomnd             # we are using numbers in many cases
     - gomoddirectives   # we use replace directives
-    - gomodguard        # we are not using modules
     - ifshort           # a lot of false positives
     - interfacer        # deprecated
     - maligned          # deprecated

agent/agents/mongodb/internal/profiler/profiler_test.go

@@ -30,9 +30,9 @@ import (
 	"go.mongodb.org/mongo-driver/bson"
 	"go.mongodb.org/mongo-driver/mongo"
 
-	"github.com/percona/pmm/agent/actions"
 	"github.com/percona/pmm/agent/agents/mongodb/internal/profiler/aggregator"
 	"github.com/percona/pmm/agent/agents/mongodb/internal/report"
+	"github.com/percona/pmm/agent/runner/actions"
 	"github.com/percona/pmm/agent/utils/templates"
 	"github.com/percona/pmm/agent/utils/tests"
 	"github.com/percona/pmm/api/agentpb"
@@ -216,14 +216,15 @@ func testProfiler(t *testing.T, url string) {
 	// This test is here to ensure the query example the profiler captures is valid to be used in Explain.
 	t.Run("TestMongoDBExplain", func(t *testing.T) {
 		id := "abcd1234"
-		ctx := context.TODO()
 
 		params := &agentpb.StartActionRequest_MongoDBExplainParams{
 			Dsn:   tests.GetTestMongoDBDSN(t),
 			Query: findBucket.Common.Example,
 		}
 
-		ex := actions.NewMongoDBExplainAction(id, params, os.TempDir())
+		ex := actions.NewMongoDBExplainAction(id, 5*time.Second, params, os.TempDir())
+		ctx, cancel := context.WithTimeout(context.Background(), ex.Timeout())
+		defer cancel()
 		res, err := ex.Run(ctx)
 		assert.Nil(t, err)