-
Couldn't load subscription status.
- Fork 656
Import cortex-jsonnet into mimir repo #506
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 250 commits
Commits
Show all changes
304 commits
Select commit
Hold shift + click to select a range
a2c2f28
Added mega_user class
pracucci 3363cea
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/94 …
pracucci 1815a63
Fine-tune blocks storage config
pracucci 3964f49
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/96 …
pracucci 41aa70d
Disable tests by default to fix README instructions
amckinley b87ec2f
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/97 …
tomwilkie bf55175
Run store-gateway without CPU limits
pracucci 443de56
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/103…
pracucci f4195ef
Merge branch 'master' into add-clarifying-comment
jtlisi e5820c5
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/52 …
jtlisi 04d14d0
Use v1 API for Deployment and StatefulSet resources
30d37b9
Version bump to v1.1.0
af82850
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/110…
gouthamve 32e2308
Actually include the ruler
42e05b4
Update config option name
6f5c7f1
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/112…
tomwilkie 56f59b2
Added ruler_enabled and alertmanager_enabled flags. (https://github.c…
pstibrany 82ab771
Added publish not ready addresses
joe-elliott 78e222b
Removed -experimental.tsdb.store-gateway-enabled flag
pracucci fbe8643
Added a discovery svc and pointed the querier service at itself
joe-elliott a2e2b22
lint
joe-elliott a02b59c
Added PodDisruptionBudget for store-gateway
pracucci deb23b3
Allow to configure the blocks replication factor
pracucci e36c2af
Switch store-gateway StatefulSets to Parallel Pod Management
pracucci 7c774cd
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/125…
pracucci a9ca8e8
Ruler should use metadata cache as well, if configured. (https://gith…
pstibrany b5b38c2
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/126…
pracucci 7546759
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/118…
joe-elliott e259402
Allow to customize ingester disk size and class
pracucci 1a03df7
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/136…
pracucci d3aede7
Version bump to 1.2.0
dea1138
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/141…
pracucci 366e99c
refactor: use jaeger-agent-mixin
Duologic 59b048a
Switch blocks storage ingesters to Parallel pod management policy and…
pracucci 3c77836
Fixed comment
pracucci 5a51212
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/143…
pracucci 66e79ea
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/142…
Duologic 0c1a3c7
Chunks blocks migration (https://github.com/grafana/cortex-jsonnet/pu…
pstibrany 451f89e
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/121…
pracucci 67fe1ff
refactor(cortex): use first class citizens
Duologic e9a1740
Update blocks storage CLI flags
pracucci 4eaccc5
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/154…
pracucci 3b4afb7
Do not apply blocks storage config to query-frontend, table-manager a…
pracucci f87e787
Cleaned up blocks storage config
pracucci 3201fc8
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/155…
pracucci 2aa49ec
Apply chunks-store config if primary or secondary store use chunks. (…
pstibrany 5150802
Enable table manager when using chunks storage as secondary storage e…
pstibrany 73347f3
fix(ksonnet): backwards compatibility with ksonnet
Duologic 875012d
add overrides config to tsdb store-gateway
sandeepsukhani 6080bc4
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/167…
sandeepsukhani 654a891
Add jsonnet for ingester StatefulSet with WAL (https://github.com/gra…
codesome e464366
Change max query length to 32 days
gouthamve 3c0e87a
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/173…
gouthamve f23f8f7
Fix ruler S3 config option (https://github.com/grafana/cortex-jsonnet…
amckinley 969ec8b
Add support for local ruler_client_type (https://github.com/grafana/c…
jpdstan 8f86fb8
Support Alertmanager HA
gotjosh ba90cd6
Setup the gossip port
gotjosh f838e68
s/isGossiping/isHa
gotjosh a028494
Bump to 3 replicas by default
gotjosh 9351824
Bump the cortex image, the latest stable is 1.3
gotjosh e0e6da2
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/147…
gotjosh a911d41
Fix typo in Alertmanager configuration
gotjosh 787acc8
Alertmanager configuration tweaks
gotjosh b6a56e8
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/179…
gotjosh 2fe4d94
Remove different name for a headless service
gotjosh ef4a0bd
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/180…
gotjosh b713d88
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/153…
jtlisi f2fa98c
Fix ruler s3 storage configuration
forestsword 560c224
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/171…
pracucci 796511d
Block storage support for s3
forestsword 7f71e61
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/181…
pracucci 869d2c7
Added Azure support to blocks storage
pracucci 680822f
Fixed linter
pracucci 0ad4b2a
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/182…
pracucci 70c3691
Removed the experimental prefix from blocks storage CLI flags
pracucci bc4f43d
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/184…
pracucci ddca9be
Lower default ingestion limits and create a new overrides user
gotjosh f494779
Address review feedback
gotjosh ebd04c9
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/185…
gotjosh 007ab92
Bump default series limit by 50%
gotjosh aeb7c21
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/186…
gotjosh a5fd8e8
Add flusher job for blocks.
pstibrany c96f81d
Fixed Azure account name/key config
pracucci 0bccf9e
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/190…
pracucci 2f87772
Rename changed flags for 1.4 release.
gouthamve 9936951
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/191…
gouthamve 8c7f703
Make sure only a single ruler rolls out at a time
gouthamve aa748cb
Cut 1.4.0
pracucci d8210db
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/195…
pracucci 49b9989
Add overrides exporter
simonswine c21db60
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/198…
simonswine 3a61f95
Refactor limits and overrides
simonswine a425699
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/200…
simonswine 0b7aa72
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/192…
gotjosh 4bdfc00
Removed support for ingester.statefulset_replicas
pracucci 5a290e2
Merge branch 'master' into flusher-job-blocks
pstibrany bdd7a8e
Merge branch 'master' into flusher-job-blocks
pracucci 73104cd
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/187…
pracucci 18d1a4b
Switch compactor statefulset to Parallel pod management policy
pracucci 026e5ab
Cut 1.5.0 release
pracucci 8ff9cbb
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/216…
pracucci a3208c6
Add ruler limits
gotjosh fdd325f
Add for the last user
gotjosh 45e900e
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/213…
gotjosh d72dae5
Enabled compactor sharding
pracucci 94f981a
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/219…
pracucci d752fa2
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/215…
pracucci 2c01208
Rollback PR 213
pracucci f0b6efd
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/220…
pracucci 5c3c71a
Re-introduce ruler limits
pracucci adb95d6
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/221…
gotjosh e7af91a
[fixup] ruler limits config key name
gotjosh 72b57a0
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/222…
gotjosh 51d8e00
Removed postings-compression-enabled
pracucci bd5f782
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/232…
pracucci 3019e23
Fine-tuned gRPC keepalive pings settings
pracucci dfa077e
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/233…
pracucci aeb5863
Fixed gRPC settings
pracucci 932beee
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/234…
pracucci 9a3d0ac
Release 1.6.0
pracucci 472ec1c
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/237…
pracucci dffba44
Add option to configure unregister ingesters on shutdown
pracucci 30f9d64
Fixed config
pracucci b2dd6d5
Improved comment
pracucci 061ed16
Updated doc
pracucci 680189a
Removed ifs
pracucci eb981b5
Updated comment
pracucci 829021a
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/236…
pracucci 003ca9e
Fixed syntax error
pracucci a3bf452
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/240…
pracucci 2e64bf6
Remove misleading comment (https://github.com/grafana/cortex-jsonnet/…
pracucci cca8c2e
Add option to customise the configmap name
gouthamve 4e019ad
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/244…
gouthamve b327bd6
Fix for real
pracucci 18a162a
Added bucket index flag, and enable bucket index by default. (https:/…
pstibrany 2049ec9
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/253…
pracucci 758d4f8
Cleanup blocks storage config
pracucci 6e91df2
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/257…
pracucci 6f12977
feat: allow for Alertmanager to configure multiple storage backends
jtlisi e961968
Update cortex/config.libsonnet
jtlisi 8268f43
Update cortex/alertmanager.libsonnet
jtlisi 06fcb95
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/258…
jtlisi b2de4ff
Release 1.7.0. (https://github.com/grafana/cortex-jsonnet/pull/260)
pstibrany 72aeada
cortex: config: Fix error message for alertmanager_client_type.
umamialex 00dee47
cortex: alertmanager: Remove space in dot notation.
umamialex 4acc87c
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/264…
umamialex e99bd2c
Up metadata connection limits
gouthamve b5b73a4
Add flag to enable streaming of chunks. (https://github.com/grafana/c…
pstibrany 892ecb2
Add recording rules to calculate Cortex scaling
tomwilkie beb8044
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/278…
tomwilkie 37d2c67
chore: update lib to use new API paths
jtlisi 530cc25
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/281…
jtlisi 1a7571b
Create 1.8.0 release. (https://github.com/grafana/cortex-jsonnet/pull…
pstibrany 505c4a3
Do not use deprecated Alertmanager cluster flags
pracucci 23e7aa6
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/285…
pracucci 642a4ff
fix: Update ksonnet-util vendor lock
jdbaldry 3a13fd9
Add function to customize compactor statefulset
pracucci e58fdfb
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/287…
pracucci 1ce76ab
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/286…
pracucci a1a1b82
Add querier_service_ignored_labels (https://github.com/grafana/cortex…
tsanghi a2c66cd
Introduce ingester instance limits to configuration, and add alerts. …
pstibrany 8384ea8
Add `query-scheduler.libsonnet` (https://github.com/grafana/cortex-js…
pstibrany 4283cd4
Replace use of querier.compress-http-responses removed in Cortex 1.9
56quarters bd08b7b
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/299…
pracucci 0a924bc
Enable index-header lazy loading in store-gateway
pracucci f5c771a
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/303…
pracucci f7b54f2
Do not use deprecated/removed flag -limits.per-user-override-config
pracucci 12fda45
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/304…
pracucci 3db36ef
Use new ruler storage config and enable API compression
pracucci e5cba39
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/306…
pracucci 594070b
Changed alertmanager config to use the new storage config
pracucci ef7d7f0
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/308…
pracucci 4257612
Cut release 1.9.0
gouthamve 70a50df
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/309…
gouthamve 93cadfa
Mount overrides configmap to alertmanager too
pracucci 6d491bf
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/315…
pracucci 48d5ef1
Upgrade memcached
pracucci 2e3505c
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/316…
pracucci f3d45f8
Increase default store-gateway memory request and limit
pracucci 137bff2
Fix
pracucci abac7d2
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/322…
pracucci d00e8b4
Set -server.grpc-max-*-msg-size-bytes for ruler and ingester. (https:…
stevesg 95b2e61
Fixed --alertmanager.cluster.peers
pracucci e601c12
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/329…
pracucci 9ee775c
Set empty alertmanager listen address with 1 replica
hamishforbes e1285bd
Add option to disable anti-affinity in newIngesterStatefulSet()
pracucci 7acc6b8
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/358…
pracucci fc6ce7c
Fix alertmanager config change introduced in https://github.com/grafa…
pracucci 8389bb0
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/359…
pracucci 9f114e7
Create another tier with 300K active series
gouthamve f330b19
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/364…
gouthamve c08f7f6
Improve config settings based on recent learnings
pracucci 2fa5bcb
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/369…
pracucci 9e78130
Added functions to create query-frontend and querier deployments
pracucci b751472
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/374…
pracucci 5b73ff7
Added function to create query-scheduler deployment
pracucci 4579b00
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/375…
pracucci 41cff4b
chore: upgrade to latest etcd-operator
jvrplmlmn f8d8e14
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/263…
jvrplmlmn 7553b4b
Alertmanager: Allow storage configuration to support Azure
gotjosh 608e9ea
remove new line
gotjosh 9a526e6
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/381…
gotjosh 7c5d370
Fix comment on medium_small_user config
colega 2e8b8e5
Remove wrong comment
colega a453364
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/383…
pracucci 60c5a8b
Add overrides to compactor
gouthamve 9dfcdd0
Split limits config into a variable we can reuse
gouthamve 2f255ac
Review feedback
gouthamve 66b972b
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/391…
gouthamve ce5f18b
Fix missing ruler limits
gouthamve 5e5f21d
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/392…
gouthamve 67f6503
Alertmanager: Add sharding configuration.
gotjosh f098a10
Fix `compactor_blocks_retention_period` type in `extra_small_user` (h…
kavirajk 00d8fde
Update rule limits to be inline with customer expectations
gouthamve 484f4a4
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/396…
gouthamve f1edc62
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/394…
stevesg a46a22c
Remove max_samples_per_query limit. (https://github.com/grafana/corte…
pstibrany f7cd61c
Removed chunks storage query sharding config support
pracucci 6739bd4
Add queryEngineConfig
pracucci 765a677
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/399…
pracucci 1b9c03d
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/398…
pracucci d8a64ea
tsdb: Add multi concurrency and max idle connections store gateway pa…
aknuds1 fa8552c
Update cortex/tsdb.libsonnet
aknuds1 163e6a7
Fix formatting
aknuds1 75ae394
tsdb: Use literal numbers instead of variables
aknuds1 ca0bdc6
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/414…
aknuds1 9129be2
cortex: Make ruler object storage support generic
aknuds1 e3ad382
Remove ruler-storage.gcs.bucket-name for Azure
aknuds1 4e69f2a
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/415…
aknuds1 4c565ce
cortex: Define Azure ruler args
aknuds1 db807aa
Parameterize
aknuds1 bdc3c89
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/416…
pracucci 13f3133
Further document ingester_stream_chunks_when_using_blocks parameter
aknuds1 7d46d66
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/417…
pracucci 944570c
Add options to disable anti-affinity
pracucci 98a694e
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/418…
pracucci e7759ad
Upstream some config improvements
pracucci 2bbb7de
Increased max connections for memcached chunks and index-queries too
pracucci a1c68bd
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/420…
pracucci 2b1e021
Ruler: Pass `-ruler-storage.s3.endpoint` to ruler when using S3.
stevesg ec2c846
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/421…
stevesg 3c98410
Allow to create custom store-gateway StatefulSets via newStoreGateway…
pracucci c592bfb
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/424…
pracucci bb2c109
Fix newStoreGatewayStatefulSet() to use input container
pracucci f686b05
Merge pull request https://github.com/grafana/cortex-jsonnet/pull/425…
pracucci 65398c0
Merge commit 'f686b0568542c833f9a31f032b8ef55aa504bf5f' into 20211118…
simonswine 5751c2e
Add CI check for jsonnet manifests
simonswine 2b6f26a
Remove additional git diff in check-mixin
simonswine 82a04e3
Imported cortex-jsonnet CHANGELOG entries from 1.9.0
pracucci 930ec05
Improved CHANGELOG header
pracucci File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.