opsgenie: skip unimported members (#66)

When a team has a member which references a user that isn't imported, we shouldn't hard fail.

Additionally, stop OpsGenie SDK from logging to console unless an error is encountered, since most of the warnings are irrelevant to end-user.

Author: wilsonehusin

pager/opsgenie.go

@@ -28,12 +28,25 @@ type Opsgenie struct {
 }
 
 func NewOpsgenie(apiKey string) *Opsgenie {
-	conf := &client.Config{ApiKey: apiKey}
+	conf := &client.Config{
+		ApiKey: apiKey,
+
+		// This corresponds to logrus.ErrorLevel but avoids importing logrus,
+		// since we don't use it but the SDK imports it.
+		LogLevel: 2,
+	}
 	return NewOpsgenieWithConfig(conf)
 }
 
 func NewOpsgenieWithURL(apiKey, url string) *Opsgenie {
-	conf := &client.Config{ApiKey: apiKey, OpsGenieAPIURL: client.ApiUrl(url)}
+	conf := &client.Config{
+		ApiKey:         apiKey,
+		OpsGenieAPIURL: client.ApiUrl(url),
+
+		// This corresponds to logrus.ErrorLevel but avoids importing logrus,
+		// since we don't use it but the SDK imports it.
+		LogLevel: 2,
+	}
 	return NewOpsgenieWithConfig(conf)
 }
 
@@ -150,7 +163,11 @@ func (o *Opsgenie) LoadTeamMembers(ctx context.Context) error {
 				TeamID: t.ID,
 				UserID: m.User.ID,
 			}); err != nil {
-				return fmt.Errorf("saving team member to db: %w", err)
+				if sqlErr, ok := store.AsSQLError(err); ok && sqlErr.IsForeignKeyConstraint() {
+					console.Warnf("User %q (%s) isn't imported. Skipping...\n", m.User.Username, m.User.ID)
+					return nil
+				}
+				return fmt.Errorf("saving user %q (%s) as member of %q (%s) to db: %w", m.User.Username, m.User.ID, t.Name, t.ID, err)
 			}
 		}
 	}

pager/testdata/TestOpsgenie/apiserver/v2-teams-b7acbc33-9853-4150-8a4b-10156d9408c8-identifiertype-id.json

@@ -10,6 +10,13 @@
           "username": "[email protected]"
         },
         "role": "admin"
+      },
+      {
+        "user": {
+          "id": "e5b92115-bfe7-43eb-8c2a-e467f2e5ddc4",
+          "username": "[email protected]"
+        },
+        "role": "admin"
       }
     ],
     "links": {
@@ -19,4 +26,4 @@
   },
   "took": 0.029,
   "requestId": "5a53826f-7864-4bf2-ada3-2979784d1e98"
-}
+}

pager/testdata/TestOpsgenie/apiserver/v2-teams.json

@@ -12,4 +12,4 @@
   ],
   "took": 0.009,
   "requestId": "0fd19d50-632a-4f13-a357-b26d0065adc5"
-}
+}