-
Notifications
You must be signed in to change notification settings - Fork 21
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
chore: add watch config to exemption watch #682
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mjnagel
commented
Aug 20, 2024
rjferguson21
previously approved these changes
Aug 21, 2024
mjnagel
changed the title
chore: update operator watch config to react to failures faster
chore: add env for Aug 21, 2024
PEPR_LAST_SEEN_LIMIT_SECONDS
in operator config chart
mjnagel
changed the title
chore: add env for
chore: add watch config to exemption watch
Aug 21, 2024
PEPR_LAST_SEEN_LIMIT_SECONDS
in operator config chart
mjnagel
commented
Aug 21, 2024
rjferguson21
approved these changes
Aug 21, 2024
UnicornChance
approved these changes
Aug 21, 2024
mjnagel
pushed a commit
that referenced
this pull request
Aug 21, 2024
🤖 I have created a release *beep* *boop* --- ## [0.26.0](v0.25.2...v0.26.0) (2024-08-21) ### ⚠ BREAKING CHANGES * client attribute allow list ([#676](#676)) ### Features * **azure:** azure blob storage support for velero ([#644](#644)) ([eff9a82](eff9a82)) * support authservice with redis, switch to pepr helm chart ([#658](#658)) ([e2fe58a](e2fe58a)) ### Bug Fixes * client attribute allow list ([#676](#676)) ([100321e](100321e)) * handle client id names with special characters ([#659](#659)) ([a84769e](a84769e)) * pull lula main for threshold update ([#638](#638)) ([5a34ce8](5a34ce8)) * release-please config bump minor pre-major ([#680](#680)) ([3f824c1](3f824c1)) ### Miscellaneous * add watch config to exemption watch ([#682](#682)) ([7714ff8](7714ff8)) * **deps:** update grafana helm chart to v8.4.4 ([#664](#664)) ([77ea6f5](77ea6f5)) * **deps:** update pepr to 0.34.1 ([#654](#654)) ([6d4655d](6d4655d)) * **deps:** update promtail to v3.1.1 ([#657](#657)) ([c009e5f](c009e5f)) * **deps:** update test-infra ([#412](#412)) ([a4c8fe9](a4c8fe9)) * **deps:** update test-infra (kms) to v0.0.5 ([#667](#667)) ([bd68637](bd68637)) * **deps:** update test-infra KMS to v0.0.4 ([#663](#663)) ([3c30b9f](3c30b9f)) * **deps:** update uds to v0.14.1 ([#677](#677)) ([12ec8a1](12ec8a1)) * **deps:** update velero kubectl image to v1.31.0 ([#669](#669)) ([d6b2f12](d6b2f12)) * **deps:** update velero to v7.1.5 ([#671](#671)) ([10ab714](10ab714)) * **deps:** update zarf to v0.38.1 ([#616](#616)) ([e0cb85d](e0cb85d)) * **deps:** update zarf to v0.38.2 ([#668](#668)) ([3328925](3328925)) * generate a schema for keycloak helm chart ([#627](#627)) ([cf3a9e7](cf3a9e7)) * mute pepr on deploy action for migrating to helm chart ([#683](#683)) ([9d05ddd](9d05ddd)) * **neuvector:** update source for unicorn images ([#675](#675)) ([568efa2](568efa2)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Adds the watch config to exemption watch. Minimal change but ensures that the watch is configurable for all places we are using watches.
PEPR_LAST_SEEN_LIMIT_SECONDS
is also added to the default envs for the config chart, intending to set this to 60s to shrink the window/tolerance for failure in staging environment.Related Issue
N/A
Type of change
Checklist before merging