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

Make sure new cluster's max compaction lag is 14 days #146

Merged
merged 1 commit into from
May 6, 2020

Conversation

rcillo
Copy link
Contributor

@rcillo rcillo commented May 6, 2020

As part of the global project for customer data deletion, log
compaction has a new maximum lag of 14 days, after which compaction
happens.

This change has already been applied dynamically to our production and
test cluster, but in the case a new cluster gets deployed in the
future, in order to avoid a "regression" from sneaking in, we better
build bubuku with this property by default, that way our current
solution becomes future proof.

As part of the global project for customer data deletion, log
compaction has a new maximum lag of 14 days, after which compaction
happens.

This change has already been applied dynamically to our production and
test cluster, but in the case a new cluster gets deployed in the
future, in order to avoid a "regression" from sneaking in, we better
build bubuku with this property by default, that way our current
solution becomes future proof.
@ferbncode
Copy link
Contributor

👍

1 similar comment
@rcillo
Copy link
Contributor Author

rcillo commented May 6, 2020

👍

@rcillo rcillo merged commit 822b3d7 into master May 6, 2020
@rcillo rcillo deleted the change-default-max-compaction-lag branch May 6, 2020 16:02
@rcillo
Copy link
Contributor Author

rcillo commented May 22, 2020

@ferbncode is it right to say that this PR was done to the wrong repo? right now this gets overriden by https://github.bus.zalan.do/aruha/bubuku-appliance/blob/master/server.properties if I'm not mistaken.

I'm checking what's needed in order to deliver https://jira.zalando.net/browse/ARUHA-2614 so I might have to resolve some conflicts on files that are present on both repos but have diverged for some reason. We would be deploying straight from the open source repo, if I got the idea right.

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

2 participants