grafana

Commit Graph

Author	SHA1	Message	Date
Eric Leijonmarck	ddabef9895	RBAC: Add actionsets struct and write path (#86108 ) * Add actionsets struct and failing test * update from review * review comments * review comments update * refactor: create interface * actionset service * fix tests * move from wireoss to wire * Apply suggestions from code review remove unnecessary comments Co-authored-by: Ieva <ieva.vasiljeva@grafana.com> * nil for the actionsetservice * Revert "nil for the actionsetservice" This reverts commit `e3d3cc8171`. --------- Co-authored-by: Ieva <ieva.vasiljeva@grafana.com>	1 year ago
Santiago	529f55cfe8	Alerting: Remove isDefault field from receivers (Alertmanager configuration) (#86605 ) Alerting: Remove isDefault field from receivers in the Alertmanager configuration	1 year ago
Santiago	309a7e7684	Alerting: Implement SaveAndApplyDefaultConfig in the remote Alertmanager struct (#85005 ) * Alerting: Implement SaveAndApplyDefaultConfig in the remote Alertmanager struct * send the hash of the encrypted configuration * tests, default config hash in AM struct * add missing default config to test * restore build directory * go work file... * fix broken test * remove unnecessary conversion to []byte * go work again... * make things work again with latest main branch changes * update error messages in tests for decrypting config	1 year ago
Santiago	a2ce8fefed	Alerting: Use a struct when sending a Grafana AM configuration to the remote Alertmanager (#86451 ) * Alerting: Use a struct when sending a Grafana AM configuration to the remote Alertmanager * remove '-distroless' from mimir image name	1 year ago
Steve Simpson	5f7612834e	Alerting: Refactoring in api_prometheus.go to allow code reuse. (#86575 ) Preparing these functions to be used by some other part of the codebase, which does not have a `contextmodel.ReqContext`, only the normal request structure (`url.Values`, etc). This is slightly messy because of how Grafana allows url parameters to be in the URL or in the request body, so we need to make sure to invoke the form parsing logic in `ReqContext`.	1 year ago
Alex Khomenko	44e1bce55a	Feature toggles: Remove dashboardEmbed toggle (#86587 )	1 year ago
Steve Simpson	73873f5a8a	Alerting: Optimize rule status gathering APIs when a limit is applied. (#86568 ) * Alerting: Optimize rule status gathering APIs when a limit is applied. The frontend very commonly calls the `/rules` API with `limit_alerts=16`. When there are a very large number of alert instances present, this API is quite slow to respond, and profiling suggests that a big part of the problem is sorting the alerts by importance, in order to select the first 16. This changes the application of the limit to use a more efficient heap-based top-k algorithm. This maintains a slice of only the highest ranked items whilst iterating the full set of alert instances, which substantially reduces the number of comparisons needed. This is particularly effective, as the `AlertsByImportance` comparison is quite complex. I've included a benchmark to compare the new TopK function to the existing Sort/limit strategy. It shows that for small limits, the new approach is much faster, especially at high numbers of alerts, e.g. 100K alerts / limit 16: 1.91s vs 0.02s (-99%) For situations where there is no effective limit, sorting is marginally faster, therefore in the API implementation, if there is either a) no limit or b) no effective limit, then we just sort the alerts as before. There is also a space overhead using a heap which would matter for large limits. * Remove commented test cases * Make linter happy	1 year ago
Ryan McKinley	5a8384a245	QueryService: Add feature toggles to better support testing (#86493 )	1 year ago
Will Browne	8a5c0cfdc0	Plugins: Pass cancellable context during API server creation (#86545 )	1 year ago
Sofia Papagiannaki	aa825f5dee	Chore: Fix Swagger/OpenAPI instructions (#86541 ) Update README.md	1 year ago
Matthew Jacobson	a20197229e	Alerting: Prevent simplified routing zero duration GroupInterval and RepeatInterval (#86561 ) Prevent zero duration GroupInterval and RepeatInterval	1 year ago
Matthew Jacobson	71445002b7	Alerting: Fix simplified routing group by override (#86552 ) * Alerting: Fix simplified routing custom group by override Custom group by overrides for simplified routing were missing required fields GroupBy and GroupByAll normally set during upstream Route validation. This fix ensures those missing fields are applied to the generated routes. * Inline GroupBy and GroupByAll initialization instead of normalize after	1 year ago
ismail simsek	28a683cf28	InfluxDB: Remove influxdbSqlSupport feature toggle (#86518 ) Remove influxdbSqlSupport feature toggle	1 year ago
Vanilla	817f787947	Cli: Check missing plugin parameter of plugin update command (#86410 ) Co-authored-by: Marcus Efraimsson <marcus.efraimsson@gmail.com>	1 year ago
Andres Martinez Gotor	eac02a61e1	Return plugin error when requesting settings (#86052 )	1 year ago
Mihai Doarna	57848bbe23	Auth: encrypt/decrypt SAML secrets in SSO settings service (#85253 ) encrypt/decrypt saml secrets in sso settings service	1 year ago
Andres Martinez Gotor	0f4a47b180	Plugin load errors: Add more well-known errors (#85960 )	1 year ago
Andreas Christou	420067a7e1	MSSQL: Add `SQL_VARIANT` converter and update test (#85823 ) Add SQL_VARIANT converter and update test	1 year ago
Michael Mandrus	df4c8c3cbc	CloudMigrations: Move business logic out of api layer (#86406 ) * move run migration to the cloudmigrationimpl layer * add migration run list logic down a layer * remove useless comments * pull cms calls into their own service	1 year ago
Kyle Brandt	46efe41e33	Prometheus: (Instrumentation) Add rawExpr (pre-interpolation) to traces (#86449 )	1 year ago
Will Browne	dceb83ab42	Plugins: Expose backendplugin for client proto interface (#86207 ) * first pass * tidy * add err check * remove comment * simplify * remove unused stuff * add backend() * use SDK commit * add codeowners for go.mod * fix go.mod * bump SDK * go work sync	1 year ago
Leonor Oliveira	9baf96dbd2	Storage: dualwriter update implementation (#85844 ) * Add update methods for the dual writer * improve errors * [WIP] add tests for the update method * Move example package to its own package so it can be used by the rest package. Finish tests * Add codeowners * Use Pod as a dummy resource * :int * Lint * [REVIEW] rename var * [REVIEW] don't rely on legacy storage at all in mode4 * Update pkg/apiserver/rest/dualwriter_mode2.go Co-authored-by: Arati R. <33031346+suntala@users.noreply.github.com> * [REVIEW] improve comment * Update pkg/apiserver/rest/dualwriter_mode1.go Co-authored-by: Dan Cech <dcech@grafana.com> * [REVIEW] improve mode3 * Lint * Move test files * Update pkg/apiserver/rest/dualwriter_mode2.go Co-authored-by: Arati R. <33031346+suntala@users.noreply.github.com> * Lint * Update pkg/apiserver/rest/dualwriter_mode4_test.go Co-authored-by: Arati R. <33031346+suntala@users.noreply.github.com> * Fix error * Lint * Update pkg/apiserver/rest/dualwriter_mode2.go Co-authored-by: Arati R. <33031346+suntala@users.noreply.github.com> * Don't set the flag to true as updatedObj creates an object in case it's not found * Lint * Lint * Add tests on update * Lint --------- Co-authored-by: Arati R. <33031346+suntala@users.noreply.github.com> Co-authored-by: Dan Cech <dcech@grafana.com>	1 year ago
Aaron Godin	d409d8e860	IAM - Fix error messages for resource permissions endpoints (#85773 ) * IAM: fix many error messages in access-related code to provide more information * Remove debug statement * Refactor resourcepermissions package to use errutil * Replace a few more errors with errutil and wrap errors found in users and teams services * Apply diff of openAPI spec	1 year ago
Karl Persson	1a6777cb93	User: use update function for password updates (#86419 ) * Update password through Update function instead * Remove duplicated to lower * Refactor password code	1 year ago
Andre Pereira	93c24403b3	Explore: Move app to under explore > traces (#86436 ) Move app to under explore > traces	1 year ago
Kristin Laemmert	03b795844c	SQLStore: Improve recursive CTE support detection (#86397 ) sqlstore: improve recursive CTE support detection Vitess returns a not supported error, not a parse error Co-authored-by: Derek Perkins <derek@nozzle.io>	1 year ago
Leonor Oliveira	dcfaf7e434	Storage: Add dualwriter test mocks (#86243 ) * WIP: add mocks for dual writer * Test dualwriter mode1 * Re-add non implementation errors * Use testify assert * Write tests for mode2. * Lint * Remove comment * Update pkg/apiserver/rest/dualwriter_mode2_test.go Co-authored-by: Arati R. <33031346+suntala@users.noreply.github.com> * [REVIEW improve readability and call fn legacy fn * [REVIEW] rename mocks and setupFns * [REVIEW add missing test case * [REVIEW] add test case with getting object from storage and not legacy * Add deleteCollection * Test deleteCollection * Rename file --------- Co-authored-by: Arati R. <33031346+suntala@users.noreply.github.com>	1 year ago
Yuri Tseretyan	e9d6135e33	SSE: Threshold expression to use simple functions (#86062 ) * replace math expression with predicates	1 year ago
Eric Leijonmarck	9c1ef8b16e	Auth: Remove caseinsensitive check on update user (#86286 ) * Removal: case insensitive check on update * refactor and removal of test for duplicate user * refactor to still shadow user variable	1 year ago
Matthew Jacobson	533bed6d94	Alerting: Fix simplified routes '...' groupBy creating invalid routes (#86006 ) * 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	1 year ago
Arati R	79eab65f94	Storage: Add mode-specific DeleteCollection implementations (#86065 ) * Add DeleteCollection methods to modes 1 and 4 * Add DeleteCollection implementations for modes 2 and 3	1 year ago
Nick Richmond	d3fee607e2	Expressions: sort numeric metrics behind feature toggle (#85911 ) * feat: sort numeric metrics behind feature toggle * chore: upgrade `dataplane/sdata` to latest tag * chore: `go work sync`	1 year ago
Ieva	036f826b87	AuthZ: Further protect admin endpoints (#86285 ) * only users with Grafana Admin role can grant/revoke Grafana Admin role * check permissions to user amdin endpoints globally * allow checking global permissions for service accounts * use a middleware for checking whether the caller is Grafana Admin	1 year ago
Karl Persson	0f06120b56	User: Clean up update functions (#86341 ) * User: remove unused function * User: Remove UpdatePermissions and support IsGrafanaAdmin flag in Update function instead * User: Remove Disable function and use Update instead	1 year ago
owensmallwood	8c8885ef23	Storage Api: Adds traces (#85391 ) - adds traces and improved logging to the unified storage server - add a configurable logger to the gRPC server service	1 year ago
Kyle Brandt	a12669951b	Prometheus: Add BE support for Adhoc Filters (#85969 ) --------- Co-authored-by: ismail simsek <ismailsimsek09@gmail.com>	1 year ago
Karl Persson	8520892923	User: Fix GetByID (#86282 ) * Auth: Remove unused lookup param * Remove case sensitive lookup for GetByID	1 year ago
Gábor Farkas	2379498228	mysql additional decoupling steps (#86168 ) * mysql additional decoupling steps * updated snapshot test	1 year ago
Gábor Farkas	9015bfa3f4	mssql: decouple sqleng (#86130 ) * mssql: decouple sqleng * fix codeowners-file	1 year ago
Ryan McKinley	ba510d1a7d	Playlists: Enable kubernetesPlaylists by default in OSS (#86259 )	1 year ago
Brendan O'Handley	f85470d652	Prometheus: Use the frontend package in Prometheus and remove feature toggle (#86080 ) * add history links for monaco completion provider folder * add history links for monaco query field folder * add history links for components folder * add history links for configuration folder * add history links for dashboard json folder * add history links for gcopypaste folder * add history link for variableMigration * add history link for querybuilder/components/metrics-modal folder * add history link for querybuilder/components/promqail folder * add history links for querybuilder/components folder * add history links for querybuilder/hooks folder * add history links for querybuilder/shared folder * add history links for querybuilder folder * add history links for querycache folder * add history links for src folder * use frontend package and custom auth in module.ts * remove files and fix import issues * remove usePrometheusFrontendPackage * remove extra files * update betterer * remove extra files after rebase * fix betterer for rebase * fix e2e flakiness	1 year ago
Charandas	2aedd9dacf	K8s: add ID token to requests outbound to new query endpoint (#86214 )	1 year ago
Alexander Weaver	5b1498f98f	Alerting: Return a 400 and errutil error when trying to delete a contact point that is referenced by a policy (#85481 ) Return a 400 and errutil error when trying to delete a contact point that is referenced by a policy	1 year ago
Gábor Farkas	7ce7584151	postgres: decouple sqleng (#86055 )	1 year ago
Gábor Farkas	b02317d583	mysql: decouple sqleng (#86057 )	1 year ago
Dave Henderson	44adfea049	Server: Fix 'server' subcommand double-registration (#86083 ) fix 'server' subcommand double-registration Signed-off-by: Dave Henderson <dave.henderson@grafana.com>	1 year ago
linoman	51da96d94e	Auth: Add `IsClientEnabled` and `IsEnabled` for the `authn.Service` and `authn.Client` interfaces (#86034 ) * Add `Service. IsClientEnabled` and `Client.IsEnabled` functions * Implement `IsEnabled` function for authn clients * Implement `IsClientEnabled` function for authn services	1 year ago
Leonor Oliveira	d17af78e79	Storage: dualwriter delete implementation (#86000 ) * Add delete methods * Remove duplicated const * Add tests * Lint * Lint * Remove duplicated test file * Update pkg/apiserver/rest/dualwriter_mode2.go Co-authored-by: Arati R. <33031346+suntala@users.noreply.github.com> * Update pkg/apiserver/rest/dualwriter.go Co-authored-by: Arati R. <33031346+suntala@users.noreply.github.com> * Update pkg/apiserver/rest/dualwriter_mode2.go Co-authored-by: Arati R. <33031346+suntala@users.noreply.github.com> * Add missing dependency * Return if object deletion goes wrong * Add a more complete log --------- Co-authored-by: Arati R. <33031346+suntala@users.noreply.github.com>	1 year ago
Giuseppe Guerra	eec9d3dbc4	Plugins: Removed feature toggle pluginsDynamicAngularDetectionPatterns (#85956 ) * Plugins: Removed feature toggle pluginsDynamicAngularDetectionPatterns * re-generate feature toggles	1 year ago
Yuri Tseretyan	12605bfed2	Alerting: Update fixed roles to include silences permissions (#85826 ) * update fixed roles to include silences * add silence actions to managed permissions * update documentation	1 year ago

1 2 3 4 5 ...

13022 Commits (efa5e582cc8717793b7257ae993a439824862fd2)