CLOUDP-57517: List Alert configuration, C/OM (#42)

Author: gssbzn

internal/cli/atlas_alert_config.go

@@ -22,7 +22,7 @@ func AtlasAlertConfigsBuilder() *cobra.Command {
 	cmd := &cobra.Command{
 		Use:     "configs",
 		Aliases: []string{"config"},
-		Short:   "Manage Atlas alert configurations for your project.",
+		Short:   "Manage alerts configuration for your project.",
 		Long:    "The configs command provides access to your alerts configurations. You can create, edit, and delete alert configurations.",
 	}
 

internal/cli/atlas_alert_config_create.go

@@ -191,7 +191,7 @@ func AtlasAlertConfigCreateBuilder() *cobra.Command {
 	}
 	cmd := &cobra.Command{
 		Use:   "create",
-		Short: "Create an Atlas alert configuration for a project.",
+		Short: "Create an alert configuration for a project.",
 		Args:  cobra.NoArgs,
 		PreRunE: func(cmd *cobra.Command, args []string) error {
 			return opts.init()

internal/cli/atlas_alert_config_list.go

@@ -65,7 +65,7 @@ func AtlasAlertConfigListBuilder() *cobra.Command {
 	}
 	cmd := &cobra.Command{
 		Use:     "list",
-		Short:   "List Atlas alert configurations for a project.",
+		Short:   "List alert configurations for a project.",
 		Aliases: []string{"ls"},
 		Args:    cobra.NoArgs,
 		PreRunE: func(cmd *cobra.Command, args []string) error {

internal/cli/atlas_alerts.go

@@ -22,7 +22,7 @@ func AtlasAlertsBuilder() *cobra.Command {
 	cmd := &cobra.Command{
 		Use:     "alerts",
 		Aliases: []string{"alert"},
-		Short:   "Manage Atlas alerts for your project.",
+		Short:   "Manage alerts for your project.",
 	}
 
 	cmd.AddCommand(AtlasAlertConfigsBuilder())

internal/cli/cloud_manager.go

@@ -26,6 +26,7 @@ func CloudManagerBuilder() *cobra.Command {
 	}
 
 	cmd.AddCommand(CloudManagerClustersBuilder())
+	cmd.AddCommand(AtlasAlertsBuilder())
 
 	return cmd
 }

internal/cli/ops_manager.go

@@ -26,6 +26,7 @@ func OpsManagerBuilder() *cobra.Command {
 	}
 
 	cmd.AddCommand(CloudManagerClustersBuilder())
+	cmd.AddCommand(AtlasAlertsBuilder())
 	cmd.AddCommand(OpsManagerOwnerBuilder())
 
 	return cmd

internal/store/alert_configuration.go

@@ -19,6 +19,7 @@ import (
 	"fmt"
 
 	atlas "github.com/mongodb/go-client-mongodb-atlas/mongodbatlas"
+	om "github.com/mongodb/go-client-mongodb-ops-manager/opsmngr"
 	"github.com/mongodb/mongocli/internal/config"
 )
 
@@ -46,6 +47,9 @@ func (s *Store) AlertConfigurations(projectID string, opts *atlas.ListOptions) (
 	case config.CloudService:
 		result, _, err := s.client.(*atlas.Client).AlertConfigurations.List(context.Background(), projectID, opts)
 		return result, err
+	case config.OpsManagerService, config.CloudManagerService:
+		result, _, err := s.client.(*om.Client).AlertConfigurations.List(context.Background(), projectID, opts)
+		return result, err
 	default:
 		return nil, fmt.Errorf("unsupported service: %s", s.service)
 	}
@@ -55,10 +59,10 @@ func (s *Store) AlertConfigurations(projectID string, opts *atlas.ListOptions) (
 func (s *Store) CreateAlertConfiguration(alertConfig *atlas.AlertConfiguration) (*atlas.AlertConfiguration, error) {
 	switch s.service {
 	case config.CloudService:
-		// TODO: fix me https://github.com/mongodb/go-client-mongodb-atlas/pull/56
-		project := alertConfig.GroupID
-		alertConfig.GroupID = ""
-		result, _, err := s.client.(*atlas.Client).AlertConfigurations.Create(context.Background(), project, alertConfig)
+		result, _, err := s.client.(*atlas.Client).AlertConfigurations.Create(context.Background(), alertConfig.GroupID, alertConfig)
+		return result, err
+	case config.OpsManagerService, config.CloudManagerService:
+		result, _, err := s.client.(*om.Client).AlertConfigurations.Create(context.Background(), alertConfig.GroupID, alertConfig)
 		return result, err
 	default:
 		return nil, fmt.Errorf("unsupported service: %s", s.service)
@@ -71,6 +75,9 @@ func (s *Store) DeleteAlertConfiguration(projectID, id string) error {
 	case config.CloudService:
 		_, err := s.client.(*atlas.Client).AlertConfigurations.Delete(context.Background(), projectID, id)
 		return err
+	case config.OpsManagerService, config.CloudManagerService:
+		_, err := s.client.(*om.Client).AlertConfigurations.Delete(context.Background(), projectID, id)
+		return err
 	default:
 		return fmt.Errorf("unsupported service: %s", s.service)
 	}

internal/store/store.go

@@ -62,7 +62,7 @@ func New() (*Store, error) {
 	case config.CloudService:
 		s.setAtlasClient(client)
 	case config.CloudManagerService, config.OpsManagerService:
-		s.setCloudManagerClient(client)
+		s.setOpsManagerClient(client)
 	default:
 		return nil, errors.New("unsupported service")
 	}
@@ -86,7 +86,7 @@ func NewUnauthenticated() (*Store, error) {
 
 	switch s.service {
 	case config.OpsManagerService:
-		s.setCloudManagerClient(nil)
+		s.setOpsManagerClient(nil)
 	default:
 		return nil, errors.New("unsupported service")
 	}
@@ -104,7 +104,7 @@ func (s *Store) setAtlasClient(client *http.Client) {
 	s.client = atlasClient
 }
 
-func (s *Store) setCloudManagerClient(client *http.Client) {
+func (s *Store) setOpsManagerClient(client *http.Client) {
 	cmClient := om.NewClient(client)
 	if s.baseURL != nil {
 		cmClient.BaseURL = s.baseURL