mirror of https://github.com/grafana/grafana.git
533bed6d94
* Alerting: Fix simplified routes '...' groupBy creating invalid routes There were a few ways to go about this fix: 1. Modifying our copy of upstream validation to allow this 2. Modify our notification settings validation to prevent this 3. Normalize group by on save 4. Normalized group by on generate Option 4. was chosen as the others have a mix of the following cons: - Generated routes risk being incompatible with upstream/remote AM - Awkward FE UX when using '...' - Rule definition changing after save and potential pitfalls with TF With option 4. generated routes stay compatible with external/remote AMs, FE doesn't need to change as we allow mixed '...' and custom label groupBys, and settings we save to db are the same ones requested. In addition, it has the slight benefit of allowing us to hide the internal implementation details of `alertname, grafana_folder` from the user in the future, since we don't need to send them with every FE or TF request. * Safer use of DefaultNotificationSettingsGroupBy * Fix missed API tests |
||
|---|---|---|
| .. | ||
| test-data | ||
| api_admin_configuration_test.go | ||
| api_alertmanager_configuration_test.go | ||
| api_alertmanager_test.go | ||
| api_available_channel_test.go | ||
| api_backtesting_data.json | ||
| api_backtesting_test.go | ||
| api_notification_channel_test.go | ||
| api_notifications_time_interval_test.go | ||
| api_prometheus_test.go | ||
| api_provisioning_test.go | ||
| api_ruler_test.go | ||
| api_testing_test.go | ||
| testing.go | ||