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 configuration for noStrictOffsetReset #1620

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

pyama86
Copy link
Contributor

@pyama86 pyama86 commented Dec 4, 2024

Introduce a new configuration option noStrictOffsetReset in the Snuba replacer deployment template and values file. This allows users to optionally disable strict offset reset behavior by setting noStrictOffsetReset to true in the values.yaml file. This change provides more flexibility in handling offset resets, which can be useful in certain deployment scenarios where strict offset management is not required or desired.

@Mokto
Copy link
Contributor

Mokto commented Dec 4, 2024

Thanks! You still have to rebase

@Mokto
Copy link
Contributor

Mokto commented Dec 4, 2024

👋 Hi, @pyama86,
I detected conflicts against the base branch 🙊
You'll want to sync 🔄 your branch with upstream!

@Mokto Mokto added the conflicts label Dec 4, 2024
…r noStrictOffsetReset

Introduce a new configuration option `noStrictOffsetReset` in the Snuba
replacer deployment template and values file. This allows users to
optionally disable strict offset reset behavior by setting
`noStrictOffsetReset` to true in the `values.yaml` file. This change
provides more flexibility in handling offset resets, which can be
useful in certain deployment scenarios where strict offset management
is not required or desired.
@pyama86 pyama86 force-pushed the no-strict-replacer branch from 6632c96 to 3c212a1 Compare December 4, 2024 12:40
@pyama86
Copy link
Contributor Author

pyama86 commented Dec 4, 2024

@Mokto
Thank you for your reply. I have made the corrections.

@Mokto Mokto merged commit 234552e into sentry-kubernetes:develop Dec 4, 2024
1 of 3 checks passed
@Mokto Mokto mentioned this pull request Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants