63 Commits (4e4f0d4e41dd7b44cd91dbeea06762451ed2db7e)

Author	SHA1	Message	Date
Karsten Weiss	d79d573f71	Fix spelling mistakes found by codespell (#4065) ... Signed-off-by: Karsten Weiss <knweiss@gmail.com>	8 years ago
Adam Shannon	809881d7f5	support reading basic_auth password_file for HTTP basic auth (#4077) ... Issue: https://github.com/prometheus/prometheus/issues/4076 Signed-off-by: Adam Shannon <adamkshannon@gmail.com>	8 years ago
Manos Fokas	25f929b772	Yaml UnmarshalStrict implementation. (#4033) ... * Updated yaml vendor package. * remove checkOverflow duplicate in rulefmt * remove duplicated HTTPClientConfig.Validate() * Added yaml static check.	8 years ago
Solomon Van	68e394a56e	notifier: update use testutil for testing (#3695)	8 years ago
Conor Broderick	1fd20fc954	Add dropped alertmanagers to alertmanagers API (#3865)	8 years ago
Matt Bostock	f0fd701309	Clarify notify metric descriptions (#2551) ... The implementation of `sendAll` means that we observe latencies even for notifications that would be considered dropped due to errors when sending them. Similarly, we count alerts as 'sent' even if an error occurred when trying to send them (meaning they are potentially not sent at all). 336c7870ea/notifier/notifier.go (L340-L347)	8 years ago
Krasi Georgiev	7858745c04	rename structs for consistency	8 years ago
Krasi Georgiev	719c579f7b	refactor main execution reloadReady handling, update some comments	8 years ago
Krasi Georgiev	87a8fc24d5	fix the test	8 years ago
Krasi Georgiev	5260c650ec	use the config hash for the map lookup	8 years ago
Krasi Georgiev	8369826808	comment to rethink the map reference for the notifier discovery	8 years ago
Krasi Georgiev	2e058b1c6f	reload test ... Signed-off-by: Krasi Georgiev <krasi.root@gmail.com>	8 years ago
Krasi Georgiev	767faa44b6	fixed the tests ... Signed-off-by: Krasi Georgiev <krasi.root@gmail.com>	8 years ago
Krasi Georgiev	d12e6f29fc	discovery manager ApplyConfig now takes a direct ServiceDiscoveryConfig so that it can be used for the notify manager ... reimplement the service discovery for the notify manager Signed-off-by: Krasi Georgiev <krasi.root@gmail.com>	8 years ago
Shubheksha Jalan	0471e64ad1	Use shared types from the `common` repo (#3674) ... * refactor: use shared types from common repo, remove util/config * vendor: add common/config * fix nit	8 years ago
Shubheksha Jalan	ec94df49d4	Refactor SD configuration to remove `config` dependency (#3629) ... * refactor: move targetGroup struct and CheckOverflow() to their own package * refactor: move auth and security related structs to a utility package, fix import error in utility package * refactor: Azure SD, remove SD struct from config * refactor: DNS SD, remove SD struct from config into dns package * refactor: ec2 SD, move SD struct from config into the ec2 package * refactor: file SD, move SD struct from config to file discovery package * refactor: gce, move SD struct from config to gce discovery package * refactor: move HTTPClientConfig and URL into util/config, fix import error in httputil * refactor: consul, move SD struct from config into consul discovery package * refactor: marathon, move SD struct from config into marathon discovery package * refactor: triton, move SD struct from config to triton discovery package, fix test * refactor: zookeeper, move SD structs from config to zookeeper discovery package * refactor: openstack, remove SD struct from config, move into openstack discovery package * refactor: kubernetes, move SD struct from config into kubernetes discovery package * refactor: notifier, use targetgroup package instead of config * refactor: tests for file, marathon, triton SD - use targetgroup package instead of config.TargetGroup * refactor: retrieval, use targetgroup package instead of config.TargetGroup * refactor: storage, use config util package * refactor: discovery manager, use targetgroup package instead of config.TargetGroup * refactor: use HTTPClient and TLS config from configUtil instead of config * refactor: tests, use targetgroup package instead of config.TargetGroup * refactor: fix tagetgroup.Group pointers that were removed by mistake * refactor: openstack, kubernetes: drop prefixes * refactor: remove import aliases forced due to vscode bug * refactor: move main SD struct out of config into discovery/config * refactor: rename configUtil to config_util * refactor: rename yamlUtil to yaml_config * refactor: kubernetes, remove prefixes * refactor: move the TargetGroup package to discovery/ * refactor: fix order of imports	8 years ago
Krasi Georgiev	e405e2f1ea	refactored discovery	8 years ago
Luke Overend	9532c2c700	Pass ams to go routine when sending alerts (#3284) ... Currently when sending alerts via the go routine within `sendAll`, the value of `ams` is not passed to the routine, causing it to use the updated value of `ams`. Example config: ``` alerting: alertmanagers: - basic_auth: username: 'prometheus' password: 'test123' static_configs: - targets: - localhost:9094 - static_configs: - targets: - localhost:9095 ``` In this example alerts sent to `localhost:9094` fail with: ``` level=error ts=2017-10-12T10:03:53.456819948Z caller=notifier.go:445 component=notifier alertmanager=http://localhost:9094/api/v1/alerts count=1 msg="Error sending alert" err="bad response status 401 Unauthorized" ``` If you change the order to be: ``` alerting: alertmanagers: - static_configs: - targets: - localhost:9095 - basic_auth: username: 'prometheus' password: 'test123' static_configs: - targets: - localhost:9094 ``` It works as expected. This commit changes the behavour so `ams` is passed to the go routine so `n.sendOne` uses the appropriate `http.Client` details.	8 years ago
Julius Volz	099df0c5f0	Migrate "golang.org/x/net/context" -> "context" (#3333) ... In some places, where ctxhttp or gRPC are concerned, we still need to use the old contexts.	8 years ago
Marc Sluiter	6a633eece1	Added go-conntrack for monitoring http connections (#3241) ... Added metrics for in- and outgoing traffic with go-conntrack.	8 years ago
Fabian Reinartz	d21f149745	*: migrate to go-kit/log	8 years ago
Lynn Lin	1bf25dc1b2	fix issues reported by gofmt and spelling typo (#3127)	8 years ago
Chris Goller	42de0ae013	Use log.Logger interface for all discovery services	9 years ago
Frederic Branczyk	94e8b43aae	notifier: clone and not reuse LabelSet in AM discovery	9 years ago
Frederic Branczyk	0c96c4b157	notifier: expose metric for number of discovered alertmanagers	9 years ago
David Symonds	04ad889751	Preserve Alertmanager URLs as *url.URL. ... Render a nicer link in the web UI.	9 years ago
Goutham Veeramachaneni	f27ce34a13	Use Registerer to Register All Metrics ... * Made Metric a Gauge so that it can be registered.	9 years ago
Goutham Veeramachaneni	7ba0a9e81a	Add Comment About Initialising Counters	9 years ago
Goutham Veeramachaneni	0d0c9d5440	Move Registerer to Config Struct in Notifier	9 years ago
Julius Volz	815762a4ad	Move retrieval.NewHTTPClient -> httputil.NewClientFromConfig	9 years ago
Goutham Veeramachaneni	af222b63fc	Fix broken tests	9 years ago
Goutham Veeramachaneni	f35816613e	Refactored Notifier to use Registerer ... * Brought metrics back into Notifier Notifier still implements a Collector. Check if that is needed.	9 years ago
Goutham Veeramachaneni	41da5c4ef2	Better Metrics For Alerts ... * Closes prometheus/prometheus#2429 * Moved metrics to top of file for easier access * Initialised CounterVecs	9 years ago
Julius Volz	f152ac5e23	notifier: Allow swapping out HTTP Doer ... We need to be able to modify the HTTP POST in Weave Cortex to add multitenancy information to a notification. Since we only really need a special header in the end, the other option would be to just allow passing in headers to the notifier. But swapping out the whole Doer is more general and allows others to swap out the network-talky bits of the notifier for their own use. Doing this via contexts here wouldn't work well, due to the decoupled flow of data in the notifier. There was no existing interface containing the ctxhttp.Post() or ctxhttp.Do() methods, so I settled on just using Do() as a swappable function directly (and with a more minimal signature than Post).	9 years ago
Matt Bostock	4160892109	Correct notifications_dropped description ... The current description does not accurately describe when the metric is incremented. Aside from Alertmanger missing from the configuration, `prometheus_notifications_dropped_total` is incremented when errors occur while sending alert notifications to Alertmanager, or because the notifications queue is full, or because the number of notifications to be sent exceeds the queue capacity. I think calling these cases 'errors' in a generic sense is more useful than the current description.	9 years ago
Fabian Reinartz	8b4e4a9d2b	notifier: fully use labels.Labels	9 years ago
Fabian Reinartz	5817cb5bde	*: migrate from model.* to promql.* types	9 years ago
Fabian Reinartz	2ad56aabd4	notifier: extract alertmanager into interface	9 years ago
Fabian Reinartz	b1f28b48a3	Fix typo	9 years ago
Fabian Reinartz	3fb4d1191b	config: rename AlertingConfig, resolve file paths	9 years ago
Fabian Reinartz	d4deb8bbf2	web: show discovered Alertmanagers in UI	9 years ago
Fabian Reinartz	f210d96497	notifier: use dynamic service discovery	9 years ago
Dominik Schulz	182e17958a	Trivial spelling corrections and a small comment.	9 years ago
Frederic Branczyk	572423cbe5	notifier: attach external labels before relabelling ... By attaching external labels before relabelling, they are available during relabelling.	9 years ago
Tobias Schmidt	8f3b62bfe4	Simplify struct initialization	9 years ago
Frederic Branczyk	b655aa002f	introduce top level alerting config node	10 years ago
Frederic Branczyk	7714b9c781	move relabeling functionality to its own package ... also remove the returned error as it was always nil	10 years ago
Frederic Branczyk	679d225c8d	allow relabeling of alerts ... in case of dropping don't even enqueue them	10 years ago
Dmitry Vorobev	273e457da4	web: return status code and error message for config resource	10 years ago
beorn7	064b57858e	Consistently use the `Seconds()` method for conversion of durations ... This also fixes one remaining case of recording integral numbers of seconds only for a metric, i.e. this will probably fix #1796.	10 years ago