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

Add curator config for CLO #230

Merged
merged 1 commit into from
Feb 10, 2021
Merged

Conversation

4n4nd
Copy link
Contributor

@4n4nd 4n4nd commented Feb 9, 2021

Set default log retention to 7 days.
Docs: https://docs.openshift.com/container-platform/4.5/logging/config/cluster-logging-curator.html#cluster-logging-curator-yaml_cluster-logging-curator

Currently the 200Gig PVC for cluster logging is at >90% usage. This is resulting in Elasticsearch not pushing in any new logs.

Related #operate-first/support#83

@sesheta sesheta requested review from ipolonsk and naved001 February 9, 2021 21:52
@4n4nd 4n4nd requested a review from tumido February 9, 2021 21:52
@sesheta sesheta added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Feb 9, 2021
@anishasthana
Copy link
Member

/lgtm

@sesheta sesheta added the lgtm Indicates that a PR is ready to be merged. label Feb 9, 2021
@4n4nd 4n4nd force-pushed the clo-retention-config branch from 1fa2d30 to a3bd987 Compare February 10, 2021 02:40
@sesheta sesheta removed the lgtm Indicates that a PR is ready to be merged. label Feb 10, 2021
@4n4nd 4n4nd force-pushed the clo-retention-config branch from a3bd987 to 127e4ea Compare February 10, 2021 17:14
@sesheta sesheta added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Feb 10, 2021
@4n4nd 4n4nd requested a review from tumido February 10, 2021 17:14
@4n4nd 4n4nd force-pushed the clo-retention-config branch from 127e4ea to 6298ed6 Compare February 10, 2021 17:16
Copy link
Member

@tumido tumido left a comment

Choose a reason for hiding this comment

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

/lgtm

@sesheta sesheta added the lgtm Indicates that a PR is ready to be merged. label Feb 10, 2021
@HumairAK
Copy link
Member

/lgtm

@HumairAK
Copy link
Member

/approve

@sesheta
Copy link
Member

sesheta commented Feb 10, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: HumairAK

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sesheta sesheta added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 10, 2021
@sesheta sesheta merged commit 5bff9a0 into operate-first:master Feb 10, 2021
@HumairAK
Copy link
Member

Configmap updated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants