grafana

Commit Graph

Author	SHA1	Message	Date
Kristin Laemmert	05709ce411	chore: remove sqlstore & mockstore dependencies from (most) packages (#57087 ) * chore: add alias for InitTestDB and Session Adds an alias for the sqlstore InitTestDB and Session, and updates tests using these to reduce dependencies on the sqlstore.Store. * next pass of removing sqlstore imports * last little bit * remove mockstore where possible	3 years ago
Sofia Papagiannaki	8b77ee2734	SQLStore: Ensure that sessions are always closed (#55864 ) * SQLStore: Ensure that sessions are always closed Delete `NewSession()` in favour of `WithDbSession()` * Add WithDbSessionForceNewSession to the interface * Apply suggestions from code review	3 years ago
Emil Tullstedt	b287047052	Chore: Upgrade Go to 1.19.1 (#54902 ) * WIP * Set public_suffix to a pre Ruby 2.6 version * we don't need to install python * Stretch->Buster * Bump versions in lib.star * Manually update linter Sort of messy, but the .mod-file need to contain all dependencies that use 1.16+ features, otherwise they're assumed to be compiled with -lang=go1.16 and cannot access generics et al. Bingo doesn't seem to understand that, but it's possible to manually update things to get Bingo happy. * undo reformatting * Various lint improvements * More from the linter * goimports -w ./pkg/ * Disable gocritic * Add/modify linter exceptions * lint + flatten nested list Go 1.19 doesn't support nested lists, and there wasn't an obvious workaround. https://go.dev/doc/comment#lists	3 years ago
Kat Yang	3c3039f5b3	Chore: Remove Wrap (#50048 ) * Chore: Remove Wrap and Wrapf * Fix: Add error check	3 years ago
ying-jeanne	a8eef45a44	Logger migration from log15 to gokit/log (#41636 ) * migrate log15 to gokit/log * fix console log * update some unittest * fix all unittest * fix the build * Update pkg/infra/log/log.go Co-authored-by: Yuriy Tseretyan <tceretian@gmail.com> * general type vector * correct the level key Co-authored-by: Yuriy Tseretyan <tceretian@gmail.com>	3 years ago
idafurjes	56c3875bb9	Chore: Remove context.TODO (#43458 ) * Remove context.TODO() from services * Fix live test	3 years ago
idafurjes	b8852ef6a3	Chore: Remove context.TODO() (#43409 ) * Remove context.TODO() from services * Fix live test * Remove context.TODO	3 years ago
Todd Treece	1781c8ec7d	Chore: Add go-redis v8 dependency (#39442 ) * adds redis v8 client dependency * remove go-redis v5 dependency	4 years ago
Serge Zaitsev	643c7fa0cb	Chore: update all +build statements (#38782 )	4 years ago
Arve Knudsen	78596a6756	Migrate to Wire for dependency injection (#32289 ) Fixes #30144 Co-authored-by: dsotirakis <sotirakis.dim@gmail.com> Co-authored-by: Marcus Efraimsson <marcus.efraimsson@gmail.com> Co-authored-by: Ida Furjesova <ida.furjesova@grafana.com> Co-authored-by: Jack Westbrook <jack.westbrook@gmail.com> Co-authored-by: Will Browne <wbrowne@users.noreply.github.com> Co-authored-by: Leon Sorokin <leeoniya@gmail.com> Co-authored-by: Andrej Ocenas <mr.ocenas@gmail.com> Co-authored-by: spinillos <selenepinillos@gmail.com> Co-authored-by: Karl Persson <kalle.persson@grafana.com> Co-authored-by: Leonard Gram <leo@xlson.com>	4 years ago
Carl Bergquist	a10fa5cad3	Instrumentation: Start tracing database requests (#34572 ) Signed-off-by: bergquist <carl.bergquist@gmail.com>	4 years ago
Serge Zaitsev	da13f88862	Redact sensitive values before logging them (#33829 ) * use a common way to redact sensitive values before logging them * fix panic on missing testCase.err, simplify require checks * fix a silly typo * combine readConfig and buildConnectionString methods, as they are closely related	4 years ago
Arve Knudsen	12661e8a9d	Move middleware context handler logic to service (#29605 ) * middleware: Move context handler to own service Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com> Co-authored-by: Emil Tullsted <sakjur@users.noreply.github.com> Co-authored-by: Will Browne <wbrowne@users.noreply.github.com>	5 years ago
Arve Knudsen	8d5b0084f1	Middleware: Simplifications (#29491 ) * Middleware: Simplify Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com> * middleware: Rename auth_proxy directory to authproxy Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>	5 years ago
Arve Knudsen	b5379c5335	Chore: Fix SQL related Go variable naming (#28887 ) * Chore: Fix variable naming Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>	5 years ago
Arve Knudsen	a5d9196a53	Chore/fix lint issues (#27704 ) * Chore: Fix linting issues Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>	5 years ago
Arve Knudsen	41d432b5ae	Chore: Enable whitespace linter (#25903 ) Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>	5 years ago
Mario Trangoni	5116420e9a	Fix misspell issues (#23905 ) * Fix misspell issues See, $ golangci-lint run --timeout 10m --disable-all -E misspell ./... Signed-off-by: Mario Trangoni <mjtrangoni@gmail.com> * Fix codespell issues See, $ codespell -S './.git' -L 'uint,thru,pres,unknwon,serie,referer,uptodate,durationm' Signed-off-by: Mario Trangoni <mjtrangoni@gmail.com> ci please? * non-empty commit - ci? * Trigger build Co-authored-by: bergquist <carl.bergquist@gmail.com> Co-authored-by: Kyle Brandt <kyle@grafana.com>	5 years ago
Arve Knudsen	7349a6b96c	pkg/infra: Check errors (#19705 ) * pkg/infra: Check errors Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com> * pkg/infra: Handle errors Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com> * Update pkg/infra/usagestats/usage_stats.go Co-Authored-By: Marcus Efraimsson <marcus.efraimsson@gmail.com>	6 years ago
Kyle Brandt	f689b60426	remotecache: support SSL with redis (#18511 ) * update go-redis lib from v2 -> v5 * add ssl option to the redis connection string fixes #18498	6 years ago
Markus Blaschke	31547597d3	remote_cache: Fix redis connstr parsing (#18204 ) * Fix redis connstr parsing * Don’t log the password	6 years ago
Kyle Brandt	0adbb001db	RemoteCache: redis connection string parsing test (#17702 )	6 years ago
Sofia Papagiannaki	7b70e7db2d	AuthProxy: Optimistic lock pattern for remote cache Set (#17485 ) * Implementation of optimistic lock pattern Try to insert the remote cache key and handle integrity error * Remove transaction Integrity error inside a transaction results in deadlock * Remove check for existing remote cache key Is no longer needed since integrity constrain violations are handled * Add check for integrity constrain violation Do not update the row if the insert statement fails for other than an integrity constrain violation * Handle failing inserts because of deadlocks If the insert statement fails because of a deadlock try to update the row * Add utility function for returning SQL error code Useful for debugging * Add logging for failing expired cache key deletion Do not shallow it completely * Revert "Add utility function for returning SQL error code" This reverts commit 8e0b82c79633e7d8bc350823cbbab2ac7a58c0a5. * Better log for failing deletion of expired cache key * Add some comments * Remove check for existing cache key Attempt to insert the key without checking if it's already there and handle the error situations * Do not propagate deadlocks created during update Most probably somebody else is trying to insert/update the key at the same time so it is safe enough to ignore it	6 years ago
Kyle Brandt	c09fe3c3b4	remote_cache: Fix redis (#17483 ) * wip: fix remote cache for redis connstr parsing and non-negative expires for #17377 TODO: finish parse, check zero case, find out why negative duration in the first place * finish parse. Still TODO, find out negative value, and decide if would be better to make database specific entries in the .ini file * update ini files * remove accidental uncomment in defaults.ini * auth_proxy: expiration non-negative so expiration is not in the past * fix test, revert neg in redis * review: use errutil	6 years ago
Mario Trangoni	87760d4fde	Codestyle: Fix govet issues (#17178 ) ref #10381 Signed-off-by: Mario Trangoni <mjtrangoni@gmail.com>	6 years ago
Carl Bergquist	aed3d0d3ad	Remotecache: Avoid race condition in Set causing error on insert. (#17082 ) * remotecache: avoid race condition in set since set called the database twice without transactions another operation could insert a value before the first operation completed. which would raise an error on insert since the data have been inserted by the other request. closes #17079	6 years ago
zhulongcheng	2fff8f77dc	move log package to /infra (#17023 ) ref #14679 Signed-off-by: zhulongcheng <zhulongcheng.me@gmail.com>	6 years ago
Carl Bergquist	3f136e0da9	tech: replace bmizerany/assert with stretchr/testify (#16625 ) bmizerany is old and unsupported. so we are replacing it with stretchr which is an drop in replacement and something we want to use more in Grafana.	6 years ago
Carl Bergquist	490515aec6	build: partially replace gometalinter with golangci-lint (#16610 ) we still use gometalinter for goconst since it doesn't report errors for duplicated in test files	6 years ago
Oleg Gaidarenko	67cbc7d4cf	Chore: use remote cache instead of session storage (#16114 ) Replaces session storage in auth_proxy middleware with remote cache Fixes #15161	6 years ago
Oleg Gaidarenko	c5bc723a6e	Correct table names of sql storage for remotecache	6 years ago
bergquist	6d42d43b22	use constants for cache type	6 years ago
bergquist	0a86a1d7b6	updates old distcache names	6 years ago
bergquist	c001cfe1d9	dont allow inifinite expiration	6 years ago
bergquist	5186273731	return error if cache type is invalid	6 years ago
bergquist	7aeab0a235	use `Get` instead of `Find`	6 years ago
bergquist	b2967fbb37	avoid exposing cache client directly	6 years ago
bergquist	7e7427637c	renames distcache -> remotecache	6 years ago

38 Commits (c52d4e2a64806e8cbf47782854fdd1f795cf549d)