Skip to content
This repository was archived by the owner on Jul 28, 2026. It is now read-only.

update loki to include grafana/loki#3285 - #387

Merged
rfratto merged 1 commit into
grafana-cold-storage:masterfrom
rfratto:update-loki
Feb 5, 2021
Merged

rfratto merged 1 commit into
grafana-cold-storage:masterfrom
rfratto:update-loki

Conversation

@rfratto

@rfratto rfratto commented Feb 5, 2021

Copy link
Copy Markdown
Contributor

PR Description

Hopefully last update of Promtail to the gcplogger target before the 0.12.0 release.

Which issue(s) this PR fixes

Notes to the Reviewer

Loki updated its Cortex dependency again, which changed the name of IngesterDesc and moved the logging functionality to its own package.

PR Checklist

  • CHANGELOG updated
  • Documentation added
  • Tests updated

Comment thread CHANGELOG.md
- [ENHANCEMENT] The `/agent/api/v1/targets` API will now include discovered labels
on the target pre-relabeling in a `discovered_labels` field. (@rfratto)

- [ENHANCEMENT] Update to Cortex v1.7.0-rc.0 (@rfratto)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed the Cortex entry after thinking about it and realizing Cortex isn't really directly user-facing, but Loki is (via Promtail)

@rfratto
rfratto merged commit 2806c75 into grafana-cold-storage:master Feb 5, 2021
@rfratto
rfratto deleted the update-loki branch February 22, 2021 15:39
@mattdurham mattdurham mentioned this pull request Sep 7, 2021
3 tasks
mattdurham pushed a commit that referenced this pull request Nov 11, 2021
@github-actions github-actions Bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label Apr 17, 2024
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants