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

Update refresh interval #5241

Merged
merged 1 commit into from
Oct 4, 2022
Merged

Conversation

cnlucas
Copy link
Member

@cnlucas cnlucas commented Sep 21, 2022

Summary (required)

Updates the refresh interval for AO's and cases to 10 hours, to ensure we are looking at recently modified during the full time the script isn't running (10-23 UTC note: crontax syntax is through the hour, so it's 10-23:59 UTC)

Also updates the slack message bot to include the AO's and MUR's published in the off hours and adds a keyword to search in the kibana logs for the next time we are having celery issues.

Required reviewers

1-2 devs

Impacted areas of the application

General components of the application that this PR will affect:

  • Refresh statements for publishing AO's and Cases

How to test

-Pushed to dev in a test branch to deploy overnight

@cnlucas cnlucas requested review from hcaofec and removed request for pkfec September 30, 2022 14:38
@cnlucas cnlucas force-pushed the feature/5238-update-refresh-time-interval branch from e9ffb2f to 7d6f1e2 Compare September 30, 2022 16:00
@codecov-commenter
Copy link

Codecov Report

Merging #5241 (7d6f1e2) into develop (a3e2e09) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #5241   +/-   ##
========================================
  Coverage    85.82%   85.82%           
========================================
  Files           81       81           
  Lines         8089     8089           
========================================
  Hits          6942     6942           
  Misses        1147     1147           
Impacted Files Coverage Δ
webservices/tasks/legal_docs.py 0.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Improve docs,

Edit slack message bot,

Add celery keyword
@cnlucas cnlucas force-pushed the feature/5238-update-refresh-time-interval branch from 13f164d to d404676 Compare October 3, 2022 22:35
Copy link
Contributor

@fec-jli fec-jli left a comment

Choose a reason for hiding this comment

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

good job. Thanks working on this.

Copy link
Contributor

@hcaofec hcaofec left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for working on this.

@fec-jli fec-jli merged commit 0e8cead into develop Oct 4, 2022
@cnlucas cnlucas deleted the feature/5238-update-refresh-time-interval branch December 21, 2022 17:15
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.

Update ADR/AF/MUR refresh lookup time interval
4 participants