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

Adjust nssp max_age #1997

Merged
merged 1 commit into from
Jul 16, 2024
Merged

Adjust nssp max_age #1997

merged 1 commit into from
Jul 16, 2024

Conversation

minhkhul
Copy link
Contributor

Description

nssp source produces weekly data. The way the source records timestamp of data is with week_end column, while our db does the same thing using time_value column. Thing is, nssp source week_end column uses the last day of an epi week (example: 2024-07-06) to mark data from that week, while time_value uses the first day of the same epi week (example: 2024-06-30), hence this discrepancy in pre-release calculation of max_age in number of days for nssp in sircal. This PR to fix will add 6 days to the original max_age to account for this discrepancy, which will shut sircal up in system-monitoring.

Changelog

  • Change max age of nssp from 13 to 19 in params.

@minhkhul minhkhul requested a review from melange396 July 15, 2024 20:33
@melange396 melange396 merged commit b3fe2cb into main Jul 16, 2024
16 checks passed
@melange396 melange396 deleted the adjust-nssp-sir-complains branch July 16, 2024 21:29
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.

2 participants