PMM-11341 Status error instead common one.

percona · Nov 1, 2023 · 67a9bf5 · 67a9bf5
1 parent 6f99bb1
commit 67a9bf5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/managed/services/inventory/agents.go b/managed/services/inventory/agents.go
@@ -20,7 +20,6 @@ import (
 	"context"
 
 	"github.com/AlekSi/pointer"
-	"github.com/pkg/errors"
 	"google.golang.org/grpc/codes"
 	"google.golang.org/grpc/status"
 	"gopkg.in/reform.v1"
@@ -502,7 +501,7 @@ func (as *AgentsService) ChangeQANMySQLSlowlogAgent(ctx context.Context, req *in
 // AddPostgresExporter inserts postgres_exporter Agent with given parameters.
 func (as *AgentsService) AddPostgresExporter(ctx context.Context, req *inventorypb.AddPostgresExporterRequest) (*inventorypb.PostgresExporter, error) {
 	if req.AutoDiscoveryLimit < 0 {
-		return nil, errors.New("auto discovery limit cannot be lower than 0")
+		return nil, status.Errorf(codes.InvalidArgument, "auto_discovery_limit cannot be lower than 0")
 	}
 
 	var res *inventorypb.PostgresExporter

diff --git a/managed/services/management/postgresql.go b/managed/services/management/postgresql.go
@@ -19,7 +19,8 @@ import (
 	"context"
 
 	"github.com/AlekSi/pointer"
-	"github.com/pkg/errors"
+	"google.golang.org/grpc/codes"
+	"google.golang.org/grpc/status"
 	"gopkg.in/reform.v1"
 
 	"github.com/percona/pmm/api/inventorypb"
@@ -49,7 +50,7 @@ func NewPostgreSQLService(db *reform.DB, state agentsStateUpdater, cc connection
 // Add adds "PostgreSQL Service", "PostgreSQL Exporter Agent" and "QAN PostgreSQL PerfSchema Agent".
 func (s *PostgreSQLService) Add(ctx context.Context, req *managementpb.AddPostgreSQLRequest) (*managementpb.AddPostgreSQLResponse, error) {
 	if req.AutoDiscoveryLimit < 0 {
-		return nil, errors.New("auto discovery limit cannot be lower than 0")
+		return nil, status.Errorf(codes.InvalidArgument, "auto_discovery_limit cannot be lower than 0")
 	}
 
 	res := &managementpb.AddPostgreSQLResponse{}