-
-
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
15 min filecheck added #783
Conversation
@peterdudfield is this what you were looking for? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for taking this, on i left a comment which should help
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great, just one small change
"but the satellite consumer should also do this. " | ||
"No out of office hours support is required." | ||
satellite_both_files_missing_error_message = ( | ||
"⚠️ Tried to update the latest Satellite data, but could not find " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You ok to update this, then I think its ready to go
Tried to update the database to show when the latest satellite data was collected, but could not find the 5-min or the 15-min satellite files.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You ok to update this, then I think its ready to go
Tried to update the database to show when the latest satellite data was collected, but could not find the 5-min or the 15-min satellite files.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry i cant see this change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tried to update the database to show when the latest satellite data was collected, but could not find the 5-min or the 15-min satellite files.
my bad updated it with the wrong thing, you should see it now
Happy for me to merge? |
Yes I'm happy with this |
Pull Request
Description
pipeline will check both the 5-minute and 15-minute satellite files each run, and will fail early and alert Slack if one is missing
Fixes #764
Checklist: