Skip to content
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

squid: cephadm: change loki/promtail default image tags #57473

Open
wants to merge 1 commit into
base: squid
Choose a base branch
from

Conversation

guits
Copy link
Contributor

@guits guits commented May 15, 2024

backport tracker: https://tracker.ceph.com/issues/66023


backport of #57164
parent tracker: https://tracker.ceph.com/issues/65784

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

This commit changes the default images for both loki and promtail
containers.
Also, to allow this update we need to update the configuration of loki
in order to add a new storage schema configuration:

```
    - from: 2024-05-03
      store: tsdb
      object_store: filesystem
      schema: v13
      index:
        prefix: index_
        period: 24h
```

This is because 3.0.0 uses new structured metadata.

See [1][2].

[1] https://grafana.com/docs/loki/latest/operations/storage/schema/
[2] https://grafana.com/docs/loki/latest/get-started/labels/structured-metadata/#when-to-use-structured-metadata

Fixes: https://tracker.ceph.com/issues/65784

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit 00f5257)
@guits guits requested a review from a team as a code owner May 15, 2024 07:26
@guits guits added this to the squid milestone May 15, 2024
@guits guits added the cephadm label May 15, 2024
@guits
Copy link
Contributor Author

guits commented May 15, 2024

jenkins test api

Copy link
Contributor

@phlogistonjohn phlogistonjohn left a comment

Choose a reason for hiding this comment

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

looks ok for backport

@guits
Copy link
Contributor Author

guits commented May 21, 2024

jenkins test api

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants