prometheus

Commit Graph

Author	SHA1	Message	Date
Karsten Weiss	d79d573f71	Fix spelling mistakes found by codespell (#4065 ) Signed-off-by: Karsten Weiss <knweiss@gmail.com>	7 years ago
Chris Aniszczyk	adb81f943a	Reference non-goals in contributing guidelines (#4118 ) https://github.com/prometheus/docs/issues/149 Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>	7 years ago
Sneha Inguva	c1a851074b	promtool: add query instant and query range commands (#4085 ) * promtool: add QueryInstant and QueryRange cmds * promtool: add more query functions * promtool: finished query Instant * promtool: add range query * promtool: add query command and address arguments * vendor client and api	7 years ago
Brian Brazil	fbe66819c5	Update ALERTS docs for 2.0 staleness changes. (#4116 ) Signed-off-by: Brian Brazil <brian.brazil@robustperception.io>	7 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>	7 years ago
Björn Rabenstein	91e470d733	Merge pull request #4096 from simonpasquier/fix-scrape-races-2.2 Fix scrape races (release-2.2 branch)	7 years ago
Ben Kochie	4a4e8a7d3b	Fix spelling in Makefile.common. (#4105 ) Signed-off-by: Ben Kochie <superq@gmail.com>	7 years ago
Ben Kochie	76f6fe8f86	Merge pull request #4102 from krasi-georgiev/makefile run the style target to fail if the code is not properly formatted	7 years ago
Krasi Georgiev	98c51d241b	run the style target to fail if the code is not properly formated Signed-off-by: Krasi Georgiev <kgeorgie@redhat.com>	7 years ago
Krasi Georgiev	0b0c9f4b6b	unused target didn't trigger an error for unused packages (#4101 ) Signed-off-by: Krasi Georgiev <kgeorgie@redhat.com>	7 years ago
Krasi Georgiev	416db814e8	use package shorthand selection that excludes vendored. (#4100 ) Signed-off-by: Krasi Georgiev <kgeorgie@redhat.com>	7 years ago
Krasi Georgiev	3f2b2c50dd	use the Makefile.common (#3978 ) split common targets in a Makefile.common to reuse it across projects Signed-off-by: Krasi Georgiev <krasi.root@gmail.com>	7 years ago
Simon Pasquier	2cbba4e948	scrape: fix data races This commit avoids passing the full scrape configuration down to the scrape loop to fix data races when the scrape configuration is being reloaded. Signed-off-by: Simon Pasquier <spasquie@redhat.com>	7 years ago
Simon Pasquier	8b89ab0173	scrape: add test detecting data races Signed-off-by: Simon Pasquier <spasquie@redhat.com>	7 years ago
Rohit Gupta	30c3e02864	Fixes #4090 . Marathon service discovery for 5XX http response (#4091 ) Signed-off-by: rohit01 <hello@rohit.io>	7 years ago
Krasi Georgiev	d13db89548	Merge pull request #4073 from krasi-georgiev/remove-unused-vendored remove unused vendored packages	7 years ago
David King	6286c10df0	Fix OOM when a large K is used in topk queries (#4087 ) This attempts to close #3973. Handles cases where the length of the input vector to an aggregate topk / bottomk function is less than the K paramater. The change updates Prometheus to allocate a result vector the same length as the input vector in these cases. Previously Prometheus would out-of-memory panic for large K values. This change makes that unlikely unless the size of the input vector is equally large. Signed-off-by: David King <dave@davbo.org>	7 years ago
Björn Rabenstein	e7584ee345	Merge pull request #4072 from prometheus/beorn7/forward-merge Merge 2.2 bugfixes into master	7 years ago
Krasi Georgiev	7951f6a0f6	Merge pull request #4075 from prometheus/issue-use-case request a use case for proposals	7 years ago
Krasi Georgiev	1467d01147	request a use case for proposals Signed-off-by: Krasi Georgiev <krasi-georgiev@users.noreply.github.com>	7 years ago
Krasi Georgiev	7679bc169d	remove unused vendored packages Signed-off-by: Krasi Georgiev <krasi.root@gmail.com>	7 years ago
beorn7	94ff07b81d	Merge branch 'release-2.2' Signed-off-by: beorn7 <beorn@soundcloud.com>	7 years ago
Björn Rabenstein	f8dcf9b272	Merge pull request #4066 from krasi-georgiev/race-DiscoveredLabels add mutex for DiscoveredLabels	7 years ago
Krasi Georgiev	dc29dd1c6f	add mutex for DiscoveredLabels Signed-off-by: Krasi Georgiev <krasi.root@gmail.com>	7 years ago
Björn Rabenstein	e65fc8591a	Merge pull request #4064 from prometheus/beorn7/vendoring Update vendoring of prometheus/common/route to include data race fix	7 years ago
beorn7	bd44e7fe98	Update vendoring of prometheus/common/route to include data race fix See https://github.com/prometheus/common/pull/125 Signed-off-by: beorn7 <beorn@soundcloud.com>	7 years ago
Krasi Georgiev	ddd46de6f4	Races/3994 (#4005 ) Fix race by properly locking access to scrape pools. Use separate mutex for information needed by UI so that UI isn't blocked when targets are being updated.	7 years ago
Mario Trangoni	464e747f1e	fix some comments typos (#4059 )	7 years ago
Sneha Inguva	cbfb207cca	vendor: correctly update golang client (#4056 )	7 years ago
Tony Lee	7cd56f56df	add queue_time slice to query_duration_seconds (#4050 )	7 years ago
Julius Volz	fe10b36b30	Fix curl example for deleting series (#4046 )	7 years ago
sev3ryn	cc917aee7f	fix of endless loop while doing Consul service discovery. (#4044 ) Reloading Prometheus configs doesn't make loop end. It produced a goroutine leak	7 years ago
Philippe Laflamme	2aba238f31	Use common HTTPClientConfig for marathon_sd configuration (#4009 ) This adds support for basic authentication which closes #3090 The support for specifying the client timeout was removed as discussed in https://github.com/prometheus/common/pull/123. Marathon was the only sd mechanism doing this and configuring the timeout is done through `Context`. DC/OS uses a custom `Authorization` header for authenticating. This adds 2 new configuration properties to reflect this. Existing configuration files that use the bearer token will no longer work. More work is required to make this backwards compatible.	7 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.	7 years ago
Krasi Georgiev	406233e937	Merge pull request #4034 from si74/main_comments main: actor functionality comments	7 years ago
Sneha Inguva	7be846754a	main: actor functionality comments	7 years ago
albatross0	0245fd55bf	Add a machine type label to GCE SD (#4032 )	7 years ago
Kristiyan Nikolov	be85ba3842	discovery/ec2: Support filtering instances in discovery (#4011 )	7 years ago
Bryan Boreham	93494d8b7e	Add an OpenTracing span for each rule (#4027 ) * Add an OpenTracing span for each rule So that tags and child spans can be traced back to the rule that they refer to.	7 years ago
Björn Rabenstein	6cf725c56d	Merge pull request #4031 from codesome/fix-bug-from-4025 Fix bug from 4025	7 years ago
Ganesh Vernekar	b44ce11d1b	Added test to check pathPrefix	7 years ago
Ganesh Vernekar	cd2820e165	Fix pathPrefix bug from PR-4025	7 years ago
Solomon Van	68e394a56e	notifier: update use testutil for testing (#3695 )	7 years ago
Elif T. Kuş	daebf68ea2	Rewrote tests for relabel and template (#3754 ) * relabel: use testutil for testing * template: use testutil for testing	7 years ago
Björn Rabenstein	61accb51ac	Merge pull request #4025 from codesome/route-prefix Fixed pathPrefix for web pages	7 years ago
Ganesh Vernekar	f30b37e00b	Fixed pathPrefix for web pages	7 years ago
Fabian Reinartz	184b6e3767	Merge pull request #3968 from zjwzte/fix-magic-number Fix magic number.	7 years ago
Krasi Georgiev	dfd6709a44	update common package (#4015 )	7 years ago
Krasi Georgiev	5fec98d0a7	simplify server error handling (#4006 )	7 years ago
Corentin Chary	60dafd425c	consul: improve consul service discovery (#3814 ) * consul: improve consul service discovery Related to #3711 - Add the ability to filter by tag and node-meta in an efficient way (`/catalog/services` allow filtering by node-meta, and returns a `map[string]string` or `service`->`tags`). Tags and nore-meta are also used in `/catalog/service` requests. - Do not require a call to the catalog if services are specified by name. This is important because on large cluster `/catalog/services` changes all the time. - Add `allow_stale` configuration option to do stale reads. Non-stale reads can be costly, even more when you are doing them to a remote datacenter with 10k+ targets over WAN (which is common for federation). - Add `refresh_interval` to minimize the strain on the catalog and on the service endpoint. This is needed because of that kind of behavior from consul: https://github.com/hashicorp/consul/issues/3712 and because a catalog on a large cluster would basically change all the time. No need to discover targets in 1sec if we scrape them every minute. - Added plenty of unit tests. Benchmarks ---------- ```yaml scrape_configs: - job_name: prometheus scrape_interval: 60s static_configs: - targets: ["127.0.0.1:9090"] - job_name: "observability-by-tag" scrape_interval: "60s" metrics_path: "/metrics" consul_sd_configs: - server: consul.service.par.consul.prod.crto.in:8500 tag: marathon-user-observability # Used in After refresh_interval: 30s # Used in After+delay relabel_configs: - source_labels: [__meta_consul_tags] regex: ^(.,)?marathon-user-observability(,.)?$ action: keep - job_name: "observability-by-name" scrape_interval: "60s" metrics_path: "/metrics" consul_sd_configs: - server: consul.service.par.consul.prod.crto.in:8500 services: - observability-cerebro - observability-portal-web - job_name: "fake-fake-fake" scrape_interval: "15s" metrics_path: "/metrics" consul_sd_configs: - server: consul.service.par.consul.prod.crto.in:8500 services: - fake-fake-fake ``` Note: tested with ~1200 services, ~5000 nodes. \| Resource \| Empty \| Before \| After \| After + delay \| \| -------- \|:-----:\|:------:\|:-----:\|:-------------:\| \|/service-discovery size\|5K\|85MiB\|27k\|27k\|27k\| \|`go_memstats_heap_objects`\|100k\|1M\|120k\|110k\| \|`go_memstats_heap_alloc_bytes`\|24MB\|150MB\|28MB\|27MB\| \|`rate(go_memstats_alloc_bytes_total[5m])`\|0.2MB/s\|28MB/s\|2MB/s\|0.3MB/s\| \|`rate(process_cpu_seconds_total[5m])`\|0.1%\|15%\|2%\|0.01%\| \|`process_open_fds`\|16\|1236\|22\|22\| \|`rate(prometheus_sd_consul_rpc_duration_seconds_count{call="services"}[5m])`\|~0\|1\|1\|0.03\| \|`rate(prometheus_sd_consul_rpc_duration_seconds_count{call="service"}[5m])`\|0.1\|80\|0.5\|0.5\| \|`prometheus_target_sync_length_seconds{quantile="0.9",scrape_job="observability-by-tag"}`\|N/A\|200ms\|0.2ms\|0.2ms\| \|Network bandwidth\|~10kbps\|~2.8Mbps\|~1.6Mbps\|~10kbps\| Filtering by tag using relabel_configs uses 100kiB and 23kiB/s per service per job and quite a lot of CPU. Also sends and additional 1Mbps of traffic to consul. Being a little bit smarter about this reduces the overhead quite a lot. Limiting the number of `/catalog/services` queries per second almost removes the overhead of service discovery. * consul: tweak `refresh_interval` behavior `refresh_interval` now does what is advertised in the documentation, there won't be more that one update per `refresh_interval`. It now defaults to 30s (which was also the current waitTime in the consul query). This also make sure we don't wait another 30s if we already waited 29s in the blocking call by substracting the number of elapsed seconds. Hopefully this will do what people expect it does and will be safer for existing consul infrastructures.	7 years ago

1 2 3 4 5 ...

4922 Commits (d79d573f715383512dcebf19e68ccaf3996e9fd3) All Branches Search

4922 Commits (d79d573f715383512dcebf19e68ccaf3996e9fd3)

All Branches