feat: add delete freight endpoint to api server

api/service/v1alpha1/service.proto

@@ -59,6 +59,7 @@ service KargoService {
   /* Freight APIs */
 
   rpc QueryFreight(QueryFreightRequest) returns (QueryFreightResponse);
+  rpc DeleteFreight(DeleteFreightRequest) returns (DeleteFreightResponse);
 
   /* Warehouse APIs */
 
@@ -424,6 +425,15 @@ message QueryFreightResponse {
   map<string, FreightList> groups = 1;
 }
 
+message DeleteFreightRequest {
+  string project = 1;
+  string name = 2;
+}
+
+message DeleteFreightResponse {
+  /* explicitly empty */
+}
+
 message FreightList {
   repeated git.luolix.top.akuity.kargo.pkg.api.v1alpha1.Freight freight = 1;
 }

charts/kargo/templates/api/cluster-role.yaml

@@ -41,6 +41,7 @@ rules:
       - get
       - list
       - watch
+      - delete
   - apiGroups:
       - kargo.akuity.io
     resources:

internal/api/delete_freight_v1alpha1.go

@@ -0,0 +1,40 @@
+package api
+
+import (
+	"context"
+
+	"connectrpc.com/connect"
+	"github.com/pkg/errors"
+	apierrors "k8s.io/apimachinery/pkg/api/errors"
+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+
+	kargoapi "github.com/akuity/kargo/api/v1alpha1"
+	svcv1alpha1 "github.com/akuity/kargo/pkg/api/service/v1alpha1"
+)
+
+func (s *server) DeleteFreight(
+	ctx context.Context,
+	req *connect.Request[svcv1alpha1.DeleteFreightRequest],
+) (*connect.Response[svcv1alpha1.DeleteFreightResponse], error) {
+	if err := validateProjectAndFreightName(req.Msg.GetProject(), req.Msg.GetName()); err != nil {
+		return nil, err
+	}
+	if err := s.validateProject(ctx, req.Msg.GetProject()); err != nil {
+		return nil, err
+	}
+	if err := s.client.Delete(ctx, &kargoapi.Freight{
+		ObjectMeta: metav1.ObjectMeta{
+			Namespace: req.Msg.GetProject(),
+			Name:      req.Msg.GetName(),
+		},
+	}); err != nil {
+		if apierrors.IsNotFound(err) {
+			return nil, connect.NewError(
+				connect.CodeNotFound,
+				errors.Errorf("freight %q not found", req.Msg.GetName()),
+			)
+		}
+		return nil, errors.Wrap(err, "delete freight")
+	}
+	return connect.NewResponse(&svcv1alpha1.DeleteFreightResponse{}), nil
+}

internal/api/validators.go

@@ -44,6 +44,16 @@
 	return nil
 }
 
+func validateProjectAndFreightName(project, name string) error {
+	if project == "" {
+		return connect.NewError(connect.CodeInvalidArgument, errors.New("project should not be empty"))
+	}
+	if name == "" {
+		return connect.NewError(connect.CodeInvalidArgument, errors.New("freight should not be empty"))
+	}
+	return nil
+}
+
 func validateGroupByOrderBy(group string, groupBy string, orderBy string) error {
 	if group != "" && groupBy == "" {
 		return connect.NewError(