feat: add default alertmanagers to silence form

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## v0.113
+
+### Added
+
+- Added `silenceForm:defaultAlertmanagers` config option - #5086 (@david-caro).
+
 ## v0.112
 
 ### Fixed

cmd/karma/tests/testscript/059_log_full_config_env.txt

@@ -63,6 +63,7 @@ env RECEIVERS_STRIP='strip1 strip2'
 env SILENCES_EXPIRED=1s
 
 env SILENCEFORM_STRIP_LABELS='job severity region'
+env SILENCEFORM_DEFAULTALERTMANAGERS='am1'
 
 env UI_REFRESH=10s
 env UI_HIDEFILTERSWHENIDLE=false
@@ -232,6 +233,8 @@ level=info msg="    labels:"
 level=info msg="      - job"
 level=info msg="      - severity"
 level=info msg="      - region"
+level=info msg="  defaultAlertmanagers:"
+level=info msg="    - am1"
 level=info msg="ui:"
 level=info msg="  refresh: 10s"
 level=info msg="  hideFiltersWhenIdle: false"

cmd/karma/tests/testscript/060_log_full_config_file.txt

@@ -269,6 +269,7 @@ level=info msg="    labels:"
 level=info msg="      - job"
 level=info msg="      - severity"
 level=info msg="      - region"
+level=info msg="  defaultAlertmanagers: []"
 level=info msg="ui:"
 level=info msg="  refresh: 10s"
 level=info msg="  hideFiltersWhenIdle: true"

cmd/karma/tests/testscript/065_proxy-with-readonly.txt

@@ -120,6 +120,7 @@ level=info msg="      rules: []"
 level=info msg="silenceForm:"
 level=info msg="  strip:"
 level=info msg="    labels: []"
+level=info msg="  defaultAlertmanagers: []"
 level=info msg="ui:"
 level=info msg="  refresh: 30s"
 level=info msg="  hideFiltersWhenIdle: true"

cmd/karma/tests/testscript/066_proxy.txt

@@ -120,6 +120,7 @@ level=info msg="      rules: []"
 level=info msg="silenceForm:"
 level=info msg="  strip:"
 level=info msg="    labels: []"
+level=info msg="  defaultAlertmanagers: []"
 level=info msg="ui:"
 level=info msg="  refresh: 30s"
 level=info msg="  hideFiltersWhenIdle: true"

cmd/karma/tests/testscript/067_readonly.txt

@@ -120,6 +120,7 @@ level=info msg="      rules: []"
 level=info msg="silenceForm:"
 level=info msg="  strip:"
 level=info msg="    labels: []"
+level=info msg="  defaultAlertmanagers: []"
 level=info msg="ui:"
 level=info msg="  refresh: 30s"
 level=info msg="  hideFiltersWhenIdle: true"

cmd/karma/tests/testscript/070_upper_case_keys.txt

@@ -130,6 +130,7 @@ level=info msg="      rules: []"
 level=info msg="silenceForm:"
 level=info msg="  strip:"
 level=info msg="    labels: []"
+level=info msg="  defaultAlertmanagers: []"
 level=info msg="ui:"
 level=info msg="  refresh: 30s"
 level=info msg="  hideFiltersWhenIdle: true"

cmd/karma/tests/testscript/097_proxy_url_config.txt

@@ -120,6 +120,7 @@ level=info msg="      rules: []"
 level=info msg="silenceForm:"
 level=info msg="  strip:"
 level=info msg="    labels: []"
+level=info msg="  defaultAlertmanagers: []"
 level=info msg="ui:"
 level=info msg="  refresh: 30s"
 level=info msg="  hideFiltersWhenIdle: true"

cmd/karma/views.go

@@ -193,6 +193,7 @@ func alerts(w http.ResponseWriter, r *http.Request) {
 			Strip: models.SilenceFormStripSettings{
 				Labels: config.Config.SilenceForm.Strip.Labels,
 			},
+			DefaultAlertmanagers: config.Config.SilenceForm.DefaultAlertmanagers,
 		},
 		AlertAcknowledgement: models.AlertAcknowledgementSettings{
 			Enabled:         config.Config.AlertAcknowledgement.Enabled,

cmd/karma/views_test.go

@@ -1124,6 +1124,7 @@ func TestEmptySettings(t *testing.T) {
 			Strip: models.SilenceFormStripSettings{
 				Labels: []string{},
 			},
+			DefaultAlertmanagers: []string{},
 		},
 		AlertAcknowledgement: models.AlertAcknowledgementSettings{
 			Enabled:         false,

demo/karma.yaml

@@ -120,6 +120,8 @@ silenceForm:
       - job
       - severity
       - region
+  defaultAlertmanagers:
+    - single
 ui:
   refresh: 10s
   hideFiltersWhenIdle: true

docs/CONFIGURATION.md

@@ -1224,10 +1224,15 @@ Syntax:
 
 ```YAML
 silenceForm:
+  defaultAlertmanagers: list of strings
   strip:
     labels: list of strings
 ```
 
+- `defaultAlertmanagers` - list of Alertmanager names that will be used as
+  default when creating a new silence.
+  If selected alertmanager is part of a cluster then the whole cluster will
+  be used in the silence form.
 - `strip:labels` - list of labels to ignore when populating silence form from
   individual alerts or group of alerts. This allows to create silences matching
   only unique labels, like `instance` or `host`, ignoring any common labels like
@@ -1242,6 +1247,16 @@ silenceForm:
       - job
 ```
 
+Example where alertmanagers `prod1` and `prod2` will be the default ones when
+creating a new silence
+
+```YAML
+silenceForm:
+  defaultAlertmanagers:
+    - prod1
+    - prod2
+```
+
 ## UI defaults
 
 `ui` section allows configuring default values for UI settings controled via the

docs/example.yaml

@@ -60,6 +60,8 @@ silenceForm:
   strip:
     labels:
       - job
+  defaultAlertmanagers:
+    - local
 ui:
   refresh: 30s
   hideFiltersWhenIdle: true

internal/config/config.go

@@ -131,6 +131,7 @@ func SetupFlags(f *pflag.FlagSet) {
 
 	f.Duration("silences.expired", time.Minute*10, "Maximum age of expired silences to show on active alerts")
 	f.StringSlice("silenceform.strip.labels", []string{}, "List of labels to ignore when auto-filling silence form from alerts")
+	f.StringSlice("silenceform.defaultAlertmanagers", []string{}, "List of Alertmanager names to use as default when creating a new silence")
 
 	f.String("listen.address", "", "IP/Hostname to listen on")
 	f.Int("listen.port", 8080, "HTTP port to listen on")
@@ -236,6 +237,8 @@ func readEnvVariables(k *koanf.Koanf) {
 			return "labels.valueOnly_re"
 		case "SILENCEFORM_STRIP_LABELS":
 			return "silenceForm.strip.labels"
+		case "SILENCEFORM_DEFAULTALERTMANAGERS":
+			return "silenceForm.defaultAlertmanagers"
 		case "UI_HIDEFILTERSWHENIDLE":
 			return "ui.hideFiltersWhenIdle"
 		case "UI_COLORTITLEBAR":

internal/config/config_test.go

@@ -149,6 +149,7 @@ silences:
 silenceForm:
   strip:
     labels: []
+  defaultAlertmanagers: []
 ui:
   refresh: 30s
   hideFiltersWhenIdle: true
@@ -393,6 +394,7 @@ func TestDefaultConfig(t *testing.T) {
 	expectedConfig.Receivers.Keep = []string{}
 	expectedConfig.Receivers.Strip = []string{}
 	expectedConfig.SilenceForm.Strip.Labels = []string{}
+	expectedConfig.SilenceForm.DefaultAlertmanagers = []string{}
 
 	if diff := cmp.Diff(expectedConfig.Annotations, Config.Annotations); diff != "" {
 		t.Errorf("Wrong annotations config returned (-want +got):\n%s", diff)
@@ -406,6 +408,9 @@ func TestDefaultConfig(t *testing.T) {
 	if diff := cmp.Diff(expectedConfig.SilenceForm.Strip, Config.SilenceForm.Strip); diff != "" {
 		t.Errorf("Wrong silence form config returned (-want +got):\n%s", diff)
 	}
+	if diff := cmp.Diff(expectedConfig.SilenceForm.DefaultAlertmanagers, Config.SilenceForm.DefaultAlertmanagers); diff != "" {
+		t.Errorf("Wrong defaultAlertmanagers form config returned (-want +got):\n%s", diff)
+	}
 }
 
 func TestValidateConfigMissingFile(t *testing.T) {

internal/config/models.go

@@ -206,6 +206,7 @@ type configSchema struct {
 		Strip struct {
 			Labels []string
 		}
+		DefaultAlertmanagers []string `yaml:"defaultAlertmanagers" koanf:"defaultAlertmanagers"`
 	} `yaml:"silenceForm" koanf:"silenceForm"`
 	// nolint: maligned
 	UI struct {

internal/models/api.go

@@ -402,7 +402,8 @@ type SilenceFormStripSettings struct {
 }
 
 type SilenceFormSettings struct {
-	Strip SilenceFormStripSettings `json:"strip"`
+	Strip                SilenceFormStripSettings `json:"strip"`
+	DefaultAlertmanagers []string                 `json:"defaultAlertmanagers"`
 }
 
 type AlertAcknowledgementSettings struct {

ui/src/Components/SilenceModal/AlertManagerInput/index.test.tsx

@@ -206,4 +206,48 @@ describe("<AlertManagerInput />", () => {
       value: ["am2"],
     });
   });
+
+  it("uses default alertmanagers to select the cluster", () => {
+    const upstreams = generateUpstreams();
+    alertStore.settings.setValues({
+      ...alertStore.settings.values,
+      ...{
+        silenceForm: {
+          strip: {
+            labels: [],
+          },
+          defaultAlertmanagers: ["am2", "bob"],
+        },
+      },
+    });
+    MountedAlertManagerInput();
+    alertStore.data.setUpstreams(upstreams);
+    expect(silenceFormStore.data.alertmanagers).toHaveLength(1);
+    expect(silenceFormStore.data.alertmanagers).toContainEqual({
+      label: "Cluster: HA",
+      value: ["am1", "am2"],
+    });
+  });
+
+  it("uses default alertmanagers to select the instance", () => {
+    const upstreams = generateUpstreams();
+    alertStore.settings.setValues({
+      ...alertStore.settings.values,
+      ...{
+        silenceForm: {
+          strip: {
+            labels: [],
+          },
+          defaultAlertmanagers: ["am0", "am3", "bob"],
+        },
+      },
+    });
+    MountedAlertManagerInput();
+    alertStore.data.setUpstreams(upstreams);
+    expect(silenceFormStore.data.alertmanagers).toHaveLength(1);
+    expect(silenceFormStore.data.alertmanagers).toContainEqual({
+      label: "am3",
+      value: ["am3"],
+    });
+  });
 });

ui/src/Components/SilenceModal/AlertManagerInput/index.tsx

@@ -21,8 +21,19 @@ const AlertManagerInput: FC<{
 }> = observer(({ alertStore, silenceFormStore }) => {
   useEffect(() => {
     if (silenceFormStore.data.alertmanagers.length === 0) {
+      // get only the clusters that match the defaults, or all of them if there are no defaults
       silenceFormStore.data.setAlertmanagers(
-        AlertmanagerClustersToOption(alertStore.data.clustersWithoutReadOnly)
+        AlertmanagerClustersToOption(
+          alertStore.data.clustersWithoutReadOnly
+        ).filter((am) => {
+          return (
+            alertStore.settings.values.silenceForm.defaultAlertmanagers.some(
+              (amName) => am.value.includes(amName)
+            ) ||
+            alertStore.settings.values.silenceForm.defaultAlertmanagers
+              .length === 0
+          );
+        })
       );
     }
   }, []); // eslint-disable-line react-hooks/exhaustive-deps

ui/src/Models/APITypes.ts

@@ -200,6 +200,7 @@ export interface APISettingsT {
     strip: {
       labels: string[];
     };
+    defaultAlertmanagers: string[];
   };
   alertAcknowledgement: {
     enabled: boolean;

ui/src/Stores/AlertStore.ts

@@ -466,6 +466,7 @@ class AlertStore {
             strip: {
               labels: [] as string[],
             },
+            defaultAlertmanagers: [] as string[],
           },
           alertAcknowledgement: {
             enabled: false as boolean,

ui/src/__fixtures__/Fetch.ts

@@ -64,6 +64,7 @@ const EmptyAPIResponse = (): APIAlertsResponseT => ({
       strip: {
         labels: [],
       },
+      defaultAlertmanagers: [],
     },
     alertAcknowledgement: {
       enabled: false,