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

logs: moves statefulsets to loki mixins #525

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JoaoBraveCoding
Copy link
Contributor

@JoaoBraveCoding JoaoBraveCoding commented Apr 19, 2023

Changes

For each statefulset component we can now:

  • use the components struct to specify the pvcSize
  • use the components struct to specify overwrites to liveness and readiness Probes

Common to all components (compactor, index gateway, ingester and ruler)

  • updateStrategy now set to RollingUpdate, shouldn't be a problem since this is the default.
  • securityContext now set to 10001, since we were using loki images I'm suspecting this will also not be a problem

Compactor & Ingex Gateway

  • has gossip label at the spec.template.metadata.labels level, before it didn't

Ingester

Ruler

@JoaoBraveCoding JoaoBraveCoding force-pushed the move-ssets branch 7 times, most recently from 66b2ec9 to feccb01 Compare April 21, 2023 18:33
Signed-off-by: Joao Marcal <jmarcal@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant