apitech/loki - loki - Gitea: Git with a cup of tea

Commit Graph

Author	SHA1	Message	Date
Travis Patterson	641c9ee48a	Don't allow unbounded parallelism when downloading indices (#10634 ) Index Gateways can stall when certain queries try to download tons of indices because. `ErrGroup` needs an explicit limit set on it. This PR sets the limit provisionally until a more configurable solution can be introduced. --------- Co-authored-by: Dylan Guedes <dylan.guedes@grafana.com>	2 years ago
Šimon Lukašík	f184c5db51	docs: master tag hasn't been pushed for almost 2 years (#10633 )	2 years ago
Owen Diehl	02ae366451	tool for building bloom filters from log contents (#10594 ) This adds a tool which builds scalable bloom filters from the contents of logs themselves & records metrics. --------- Signed-off-by: Owen Diehl <ow.diehl@gmail.com>	2 years ago
Pablo	222ffc52d3	Add CloudFormation template for lambda + EventBridge setup (#10540 ) What this PR does / why we need it: https://github.com/grafana/loki/pull/10449 added support so that lambda-promtail can handle s3 events through AWS EventBridge. This PR is a follow up adding a CloudFormation template to deploy lambda-promtail, with the corresponding EventBridge rule to deliver s3 events to the lambda. Which issue(s) this PR fixes: Rel https://github.com/grafana/loki/issues/10209 --------- Co-authored-by: J Stickler <julie.stickler@grafana.com> Co-authored-by: Christian Haudum <christian.haudum@gmail.com>	2 years ago
Christian Haudum	3a8a5e54dd	Fix object store type for table manager (#10632 ) What this PR does / why we need it: This commit fixes a regression in the object store client creation introduced with #10518 Signed-off-by: Christian Haudum <christian.haudum@gmail.com>	2 years ago
Christian Haudum	769979c9f6	chore(storage): Refactor store factory (#10518 ) What this PR does / why we need it: This PR makes a clearer distinction between supported and deprecated index/chunk storage types. There are individual commits for the different factory functions: * Refactor table client factory * Refactor index client factory * Refactor chunk client factory * Split initStore function --------- Signed-off-by: Christian Haudum <christian.haudum@gmail.com>	2 years ago
Kevin Burke	9dad6a1350	clients,pkg: use http.NewRequestWithContext (#10616 ) What this PR does / why we need it: It saves one line, and the new function (vs. http.NewRequest) performs additional validation of the request. Which issue(s) this PR fixes: None Special notes for your reviewer: None Checklist - [X] Reviewed the [`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md) guide (required) Not necessary: - [ ] Documentation added - [ ] Tests updated - [ ] `CHANGELOG.md` updated - [ ] If the change is worth mentioning in the release notes, add `add-to-release-notes` label - [ ] Changes that require user attention or interaction to upgrade are documented in `docs/sources/setup/upgrade/_index.md` - [ ] For Helm chart changes bump the Helm chart version in `production/helm/loki/Chart.yaml` and update `production/helm/loki/CHANGELOG.md` and `production/helm/loki/README.md`. [Example PR](`d10549e3ec`) Co-authored-by: Michel Hollands <42814411+MichelHollands@users.noreply.github.com>	2 years ago
Kevin Burke	23d8c7c63c	clients/pkg/promtail/client: better error on nil metrics (#10611 ) Fixes #10610. Checklist - [X] Reviewed the [`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md) guide (required) Not necessary: - [ ] Documentation added - [ ] Tests updated - [ ] `CHANGELOG.md` updated - [ ] If the change is worth mentioning in the release notes, add `add-to-release-notes` label - [ ] Changes that require user attention or interaction to upgrade are documented in `docs/sources/setup/upgrade/_index.md` - [ ] For Helm chart changes bump the Helm chart version in `production/helm/loki/Chart.yaml` and update `production/helm/loki/CHANGELOG.md` and `production/helm/loki/README.md`. [Example PR](`d10549e3ec`)	2 years ago
Kevin Burke	1256692f95	clients/pkg/promtail/targets/gcplog: fix spelling error (#10615 ) What this PR does / why we need it: Fix spelling error Which issue(s) this PR fixes: none Checklist - [X] Reviewed the [`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md) guide (required) Not necessary: - [ ] Documentation added - [ ] Tests updated - [ ] `CHANGELOG.md` updated - [ ] If the change is worth mentioning in the release notes, add `add-to-release-notes` label - [ ] Changes that require user attention or interaction to upgrade are documented in `docs/sources/setup/upgrade/_index.md` - [ ] For Helm chart changes bump the Helm chart version in `production/helm/loki/Chart.yaml` and update `production/helm/loki/CHANGELOG.md` and `production/helm/loki/README.md`. [Example PR](`d10549e3ec`)	2 years ago
Robert Jacob	835d0452d4	operator: Update example configurations to use demo size (#10606 )	2 years ago
Periklis Tsirakidis	5415b11468	operator: Update Loki operand to v2.9.1 (#10600 )	2 years ago
Grot (@grafanabot)	ca43a6ba77	[CI/CD] Update yaml file `./production/helm/loki/Chart.yaml` (+1 other) (#10592 ) Here is a summary of the updates contained in this PR: * Update attribute `$.appVersion` in yaml file `./production/helm/loki/Chart.yaml` to the following value: `2.9.1` * Bump version of Helm Chart Add changelog entry to `./production/helm/loki/CHANGELOG.md` Re-generate docs Co-authored-by: Christian Haudum <christian.haudum@gmail.com>	2 years ago
Christian Haudum	f17e55030a	Reflect release 2.9.1 on main branch (#10597 ) What this PR does / why we need it: Reflect Loki 2.9.1 release on main branch to be up-to-date with latest released version. Signed-off-by: Christian Haudum <christian.haudum@gmail.com>	2 years ago
Raul Gonzales	a7b83bf9bb	chore(docs): Fix structured_metadata usage (#10526 ) What this PR does / why we need it: According to the code the pipeline name uses `_` not `-` for the name https://github.com/grafana/loki/blob/main/clients/pkg/logentry/stages/stage.go#L46C38-L46C38 Co-authored-by: J Stickler <julie.stickler@grafana.com>	2 years ago
Christian Haudum	e858779e6b	Fix bug in cache of the index object client (#10587 ) The bug could manifest in missing results when querying logs older than what is kept on ingesters. This commit changes the behaviour of the caching in the storage client client used by the indexshipper to download indexes from object store. The object store client maintains an internal, in-memory cache of tables (= per-day index directories) available on object storage to reduce the amount of expensive List operations. This cache gets updated in a regular interval (1m) by the table manager that uses the client. However, when the per-tenant setting `query_ready_index_num_days` is set the 0, which is the default (to disable index pre-fetching), the sync is only performed once on startup, but not subsequently. This leads to a stale list of table names in the cache and any new index directory added to the object storage is not added to the client cache. This commit changes the object client to use a read-through caching strategy for listing tables. If a table is not found locally in cache, it performs a lookup against object storage. In this way, the cache is updated even without the regular sync described above. Special notes for your reviewer: This is a port of #10585 which was first applied on the 2.9 release branch. --------- Signed-off-by: Christian Haudum <christian.haudum@gmail.com>	2 years ago
Dylan Guedes	79fcdb4f4c	Update references to 2.8.5 (#10590 ) What this PR does / why we need it: We're releasing a new 2.8.5 patch. This adds references to it to our docs.	2 years ago
J Stickler	bdfa240647	[Docs] Fix 404s/broken links in main branch (#10556 ) Cleaning up broken links reported against the main branch. --------- Co-authored-by: Jack Baldry <jack.baldry@grafana.com>	2 years ago
Robert Jacob	8133b34b23	operator: Update gateway arguments to enable namespace extraction (#10545 )	2 years ago
Periklis Tsirakidis	a5d56e2315	operator: Update dashboards for Loki v2.9.0 (#10558 )	2 years ago
Karsten Jeschkies	81fae24ce2	Return `StepResult` from `StepEvaluator.Next()`. (#10567 ) What this PR does / why we need it: This change makes the `StepEvaluator` more flexible for changes like #10417 that introduce a different step result. With the interface `StepResult` we can support other vector types in the future. Checklist - [ ] Reviewed the [`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md) guide (required) - [ ] Documentation added - [x] Tests updated - [ ] `CHANGELOG.md` updated - [ ] If the change is worth mentioning in the release notes, add `add-to-release-notes` label - [ ] Changes that require user attention or interaction to upgrade are documented in `docs/sources/setup/upgrade/_index.md` - [ ] For Helm chart changes bump the Helm chart version in `production/helm/loki/Chart.yaml` and update `production/helm/loki/CHANGELOG.md` and `production/helm/loki/README.md`. [Example PR](`d10549e3ec`)	2 years ago
Grot (@grafanabot)	e20a1be195	[CI/CD] Update yaml file `./production/helm/loki/values.yaml` (+1 other) (#10572 ) Here is a summary of the updates contained in this PR: * Update attribute `$.enterprise.version` in yaml file `./production/helm/loki/values.yaml` to the following value: `v1.8.1` * Bump version of Helm Chart Add changelog entry to `./production/helm/loki/CHANGELOG.md` Re-generate docs Co-authored-by: Dylan Guedes <djmgguedes@gmail.com> Co-authored-by: Michel Hollands <42814411+MichelHollands@users.noreply.github.com>	2 years ago
Dylan Guedes	4681386775	Update alpine versions (#10573 ) What this PR does / why we need it: Update alpine versions used by our images to `3.18.3` Which issue(s) this PR fixes: N/A Special notes for your reviewer: Checklist - [ ] Reviewed the [`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md) guide (required) - [ ] Documentation added - [ ] Tests updated - [ ] `CHANGELOG.md` updated - [ ] If the change is worth mentioning in the release notes, add `add-to-release-notes` label - [ ] Changes that require user attention or interaction to upgrade are documented in `docs/sources/setup/upgrade/_index.md` - [ ] For Helm chart changes bump the Helm chart version in `production/helm/loki/Chart.yaml` and update `production/helm/loki/CHANGELOG.md` and `production/helm/loki/README.md`. [Example PR](`d10549e3ec`)	2 years ago
J Stickler	f7bc8d39c0	[Docs] Add upgrade link to Manage page. (#10566 ) The new information architecture moved the Upgrade docs under "Setup." Based on feedback received via Slack, adding a link to the Upgrade docs to the Manage/Operations landing page to help users find the Upgrade docs. Special notes for your reviewer: Note that because how the page is structured (the list is a Hugo "section") this is a bit of a workaround. The link to the Upgrade docs is at the bottom, not in alphabetical order.	2 years ago
Paul Rogers	9f7f821194	Update version of loki referenced in docker files to 2.9.0 (#10564 ) This PR updates the version of Loki used by our sample Dockerfiles to the most recent version of 2.9.0.	2 years ago
HandyDaddy	216fb7569b	fix xss vulnerability (#10559 ) What this PR does / why we need it: XSS vulnerability of fmt.Sprintf. fmt.Sprintf would reflect part of the input in the output without escaping, which causes XSS.	2 years ago
Salva Corts	5f7bde74ac	Add limits (#10552 ) What this PR does / why we need it: This PR introduces two new limits for structured metadata: - `max_structured_metadata_size`: Defaults to 64 KB. Limits the maximum size of the structured metadata attached to a log line. - `max_structured_metadata_entries_count`: Default to 128. Limits the maximum number of structured metadata entries attached to a log line. We discussed adding other limits like `max_structured_metadata_label_name_length` and `max_structured_metadata_label_value_length` but discarded this idea for now.	2 years ago
Christian Haudum	2b6bdc2a29	chore(storage): Simplify store interfaces and abstractions (pt 2) (#10454 ) What this PR does / why we need it: * Remove preemptive interface from indexReaderWriter and make it an exported struct * Compose IngesterStore and QuerierStore The ingester and querier do not require a store that implements the full interface, but can accept stores that are a subset of the interfaces. Therefore we can make specific interfaces that are accepted by the ingester and querier constructors. * Split up ChunkFetcher interface This commit splits the ChunkFetcher interface into a ChunkFetcher and a ChunkFetcherProvider interface. This has the advantage that methods can allow accepting smaller interfaces. This commit also renames the `GetChunkRefs()` method of the ChunkFetcher interface into `GetChunks()`, because it returns a `[][]chunk.Chunk`, not a `[]chunk.ChunkRef` type. Signed-off-by: Christian Haudum <christian.haudum@gmail.com>	2 years ago
Christian Haudum	ebef2fe1cd	Remove backup index writer from TSDB configuration (#10534 ) The setting `use_boltdb_shipper_as_backup` (`-tsdb.shipper.use-boltdb-shipper-as-backup`) was a remnant from the development of the TSDB storage. It was used to allow writing to both TSDB and BoltDB when TSDB was still highly experimental. Since TSDB is now stable and the recommended index type, the setting has become irrelevant and therefore was removed. The previous default value `false` is applied. Signed-off-by: Christian Haudum <christian.haudum@gmail.com>	2 years ago
Salva Corts	5ab9515020	Fix regression when parsing numbers in Push request (#10550 ) What this PR does / why we need it: Even though the[ Loki HTTP API docs for the push endpoint][1] state that the stream label values should be strings, we previously didn't enforce this requirement. With https://github.com/grafana/loki/pull/9694, we started enforcing this requirement, and that broke some users. In this PR we are reverting this type of assertion and adding a bunch of tests to avoid the regression in the future. [1]: https://grafana.com/docs/loki/latest/reference/api/#push-log-entries-to-loki	2 years ago
Periklis Tsirakidis	2750985cc7	operator: Update Loki operand to v2.9.0 (#10539 )	2 years ago
Sheikh-Abubaker	4456b7561b	Corrected typo in index.md file (#10546 ) What this PR does / why we need it: Hi there I was just going through some documentations to get some idea over an issue currently I am trying to resolve, it was then when I came across this typo and corrected it.	2 years ago
J Stickler	247c312dd8	[Docs] Separate command and result (#10463 ) Now that we've implemented a copy-paste feature for code blocks on the docs website, put commands and the results in separate code blocks in the docs for ease of use. --------- Co-authored-by: Christian Haudum <christian.haudum@gmail.com> Co-authored-by: Ashwanth <iamashwanth@gmail.com>	2 years ago
J Stickler	29c7ead5ed	[Docs] Revise Loki Overview topic (#10457 ) What this PR does / why we need it: Revising/Refreshing the Loki overview as first phase of addressing #9228.	2 years ago
Christian Haudum	4d96393935	Reflect 2.9 upgrade notes on main (#10536 ) Cleanup upgrade notes from 2.9 release on main branch. Signed-off-by: Christian Haudum <christian.haudum@gmail.com>	2 years ago
Pablo	5034b1b337	Add support for EventBridge s3 events in lambda-promtail (#10449 ) What this PR does / why we need it: Lambda promtail supports s3 events, which are used for scraping several log sources such as ALB access logs. This works by configuring at the S3 bucket level "s3 event notification", that are configured to target the lambda deployment of lambda-promtail. However, if one is configuring this through CloudFormation, there's a known issue with AWS that doesn't allow to configure both the lambda, the bucket, and the notifications in the same stack. See [this issue](https://github.com/aws-cloudformation/cloudformation-coverage-roadmap/issues/79) for details. For that, AWS introduced EventBridge notifications, which can be used to ship s3 events to a lambda deployment as well. This flow looks like: s3 -> eventbridge bus -> eventbridge rule -> lambda EventBridge has it's own message structure for s3 notifications. This PR adds a translation layer, just for `Object created` events (since they are the only ones we should take into account), so that EventBridge events can be received, and trigger the lambda as if they were from s3. Which issue(s) this PR fixes: Fixes https://github.com/grafana/loki/issues/10209 Special notes for your reviewer: - [x] Pending testing this with an actual deployment of the s3 -> event bridge -> lambda flow - [x] ~~Add CF template for the `s3 -> event bridge -> lambda` deployment~~ Follow up PR Checklist - [ ] Reviewed the [`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md) guide (required) - [ ] Documentation added - [x] Tests updated - [ ] `CHANGELOG.md` updated - [ ] If the change is worth mentioning in the release notes, add `add-to-release-notes` label - [ ] Changes that require user attention or interaction to upgrade are documented in `docs/sources/setup/upgrade/_index.md` - [ ] For Helm chart changes bump the Helm chart version in `production/helm/loki/Chart.yaml` and update `production/helm/loki/CHANGELOG.md` and `production/helm/loki/README.md`. [Example PR](`d10549e3ec`)	2 years ago
Grot (@grafanabot)	bfe558c67c	[CI/CD] Update yaml file `./production/helm/loki/values.yaml` (+1 other) (#10517 ) Here is a summary of the updates contained in this PR: * Update attribute `$.enterprise.version` in yaml file `./production/helm/loki/values.yaml` to the following value: `v1.8.0` * Bump version of Helm Chart Add changelog entry to `./production/helm/loki/CHANGELOG.md` Re-generate docs Co-authored-by: Kaviraj Kanagaraj <kavirajkanagaraj@gmail.com>	2 years ago
J Stickler	dbfd81b868	Fix broken redirect (#10510 ) Fix a broken alias that is generating 404 errors	2 years ago
Poyzan	be2ecdc241	(Fix) cpu calculation queries in loki / operational (#10477 ) What this PR does / why we need it: irate is debated to show finer details, however it discards samples and only uses last two in range. This results in some smoothing out in our CPU utilisation graphs which mislead us during an investigation a while back. See the screenshot below from a random ingester. ![Screenshot 2023-09-06 at 15 58 45](https://github.com/grafana/loki/assets/31743851/d57483f2-e207-485b-b4b5-fd43c1d0ddca) Which issue(s) this PR fixes: Fixes #<issue number> Special notes for your reviewer: I choose to change the entire query to be in line with rest of the Loki Read/Write Resources dashboards. Checklist - [x] Reviewed the [`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md) guide (required) - [ ] Documentation added - [ ] Tests updated - [ ] `CHANGELOG.md` updated - [ ] If the change is worth mentioning in the release notes, add `add-to-release-notes` label - [ ] Changes that require user attention or interaction to upgrade are documented in `docs/sources/setup/upgrade/_index.md` - [ ] For Helm chart changes bump the Helm chart version in `production/helm/loki/Chart.yaml` and update `production/helm/loki/CHANGELOG.md` and `production/helm/loki/README.md`. [Example PR](`d10549e3ec`)	2 years ago
Jack Baldry	b5aae6985c	Fix alias to redirect `/docs/loki/<LOKI_VERSION>/best-practices/` to `/docs/loki/latest/get-started/labels/bp-labels/` (#10506 ) Tested locally with `make docs` Signed-off-by: Jack Baldry <jack.baldry@grafana.com> Signed-off-by: Jack Baldry <jack.baldry@grafana.com>	2 years ago
Antoine Colombier	c0f96a34eb	Loki Helm: add sidecar to find loki rules in configmaps and secrets (#9399 ) Disclaimer: this work has been done by @Jeinhaus in the [legacy repo](https://github.com/grafana/helm-charts/pull/1625), I have simply migrated here and tested it in our own infrastructure --- What this PR does / why we need it: This PR adds a sidecar to the loki chart that queries the Kubernetes API to find Configmaps and/or Secrets that contain loki rules, similar to the way the grafana chart does it for dashboards, plugins or sources. Special notes for your reviewer: Checklist - [X] Reviewed the [`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md) guide (required) - [X] Documentation added - [ ] ~Tests updated~ - [x] `CHANGELOG.md` updated - [ ] ~Changes that require user attention or interaction to upgrade are documented in `docs/sources/upgrading/_index.md`~ `CHANGELOG.md` updated: I have not update the CHANGELOG as I can see that it hasn't been updated since the start of version 5. As this been automated now? Or should I had the change logs somewhere else? Or perhaps simply format my commit in a certain way so it gets generated? Apologies if this was documented somewhere and I missed it. --------- Co-authored-by: Julian Einhaus <julianeinhaus@gmx.de> Co-authored-by: bwlkr <billynwalker@gmail.com> Co-authored-by: Michel Hollands <42814411+MichelHollands@users.noreply.github.com>	2 years ago
Travis Patterson	cc1b0dbc24	Don't use unsafe string for label names in the UnpackParser (#10487 ) We've had [some reports](https://github.com/grafana/loki/issues/8936) that `unpack` is mangling label names. We're using `unsafeGetString` to generate these so, when the underlying byte slice changes, the label name is also changed. This change incurs an alloc to make sure the generated string wont change when the underlying bytes do. fixes #8936	2 years ago
Jack Baldry	5408ec1c9c	Fix alias to redirect `/docs/loki/<LOKI_VERSION>/configuration/` to `/docs/loki/<LOKI_VERSION>/configure/` (#10503 ) Tested locally with `make docs` Signed-off-by: Jack Baldry <jack.baldry@grafana.com> Signed-off-by: Jack Baldry <jack.baldry@grafana.com>	2 years ago
J Stickler	951dd66775	update README links (#10490 ) What this PR does / why we need it: Some of the docs files have moved. Updating links in the README now that 2.9 has been released.	2 years ago
Bayan Taani	53c2b2c22a	operator: Use a condition to warn when labels for zone-awareness are empty (#10418 )	2 years ago
Kaviraj Kanagaraj	0d542af45f	changelog: move promtail specific entries to it's own section (#10498 ) What this PR does / why we need it: Follow up to #10497 Move promtail changelog to it's own section. Which issue(s) this PR fixes: Fixes #NA Special notes for your reviewer: Checklist - [x] Reviewed the [`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md) guide (required) - [x] `CHANGELOG.md` updated Signed-off-by: Kaviraj <kavirajkanagaraj@gmail.com>	2 years ago
Ruben Van Wanzeele	f0762b32dd	Helm: Add service labels and annotations (#10214 ) What this PR does / why we need it: Support setting labels and annotations for all services in the Helm chart. Move label configuration for read, write and gateway services to #component.service.labels to improve consistency between services. Which issue(s) this PR fixes: Fixes #8627 Special notes for your reviewer: Checklist - [x] Reviewed the [`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md) guide (required) - [x] `CHANGELOG.md` updated - [x] For Helm chart changes bump the Helm chart version in `production/helm/loki/Chart.yaml` and update `production/helm/loki/CHANGELOG.md` and `production/helm/loki/README.md`. [Example PR](`d10549e3ec`)	2 years ago
Kaviraj Kanagaraj	7f6c9104e2	post-release: Changelog adjustments on `main` branch (#10497 ) What this PR does / why we need it: Adjust changelog on `main` branch post release of v2.9.0 Cleanup unreleased changes w.r.t changes went into v2.9.0 Which issue(s) this PR fixes: Related: #10468 Special notes for your reviewer: Checklist - [x] Reviewed the [`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md) guide (required) - [x] `CHANGELOG.md` updated --------- Signed-off-by: Kaviraj <kavirajkanagaraj@gmail.com> Co-authored-by: Christian Haudum <christian.haudum@gmail.com>	2 years ago
Ashwanth	d7493ad044	chore: add 2.9.x entry to release-notes index page (#10491 ) What this PR does / why we need it: Adds 2.9.x release to release notes page Which issue(s) this PR fixes: Fixes #<issue number> Special notes for your reviewer: Checklist - [ ] Reviewed the [`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md) guide (required) - [ ] Documentation added - [ ] Tests updated - [ ] `CHANGELOG.md` updated - [ ] If the change is worth mentioning in the release notes, add `add-to-release-notes` label - [ ] Changes that require user attention or interaction to upgrade are documented in `docs/sources/setup/upgrade/_index.md` - [ ] For Helm chart changes bump the Helm chart version in `production/helm/loki/Chart.yaml` and update `production/helm/loki/CHANGELOG.md` and `production/helm/loki/README.md`. [Example PR](`d10549e3ec`)	2 years ago
Marcus Morris	e8f4b42eaa	Lambda Promtail: Add logs:PutSubscriptionFilter to Lambda Promtail role (#10426 ) What this PR does / why we need it: I ran into the below error during the CloudFormation stack creation when trying to use the [example CloudFormation config](https://github.com/grafana/loki/blob/main/tools/lambda-promtail/template.yaml): `"Could not execute the lambda function. Make sure you have given CloudWatch Logs permission to execute your function"` The issue is that the config does not give the `logs:PutSubscriptionFilter` permission to the Lambda Promtail Role. This adds that permission to fix the error Which issue(s) this PR fixes: Special notes for your reviewer: Checklist - [x] Reviewed the [`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md) guide (required) - [ ] Documentation added - [ ] Tests updated - [ ] `CHANGELOG.md` updated - [ ] If the change is worth mentioning in the release notes, add `add-to-release-notes` label - [ ] Changes that require user attention or interaction to upgrade are documented in `docs/sources/setup/upgrade/_index.md` - [ ] For Helm chart changes bump the Helm chart version in `production/helm/loki/Chart.yaml` and update `production/helm/loki/CHANGELOG.md` and `production/helm/loki/README.md`. [Example PR](`d10549e3ec`)	2 years ago
J Stickler	1f408305c4	[Docs] Create top level Manage section (#10461 ) What this PR does / why we need it: Part of the Epic to create a consistent documentation information architecture across the three DB projects. Which issue(s) this PR fixes: Fixes #8718 Special notes for your reviewer: The original Issue called for moving files under Manage, but rather than rename the folder and break links, I chose to make the "move" by updating the topic metadata. This PR also includes updates to topic metadata and removes or updates page weights.	2 years ago

... 2 3 4 5 6 ...

5769 Commits (86455195f31185d44b328b08a2e7b4e0d3c9ae9a) All Branches Search

5769 Commits (86455195f31185d44b328b08a2e7b4e0d3c9ae9a)

All Branches