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

Feature/set crontab #87

Merged
merged 2 commits into from
Jul 19, 2022

Conversation

MateoLostanlen
Copy link
Member

@MateoLostanlen MateoLostanlen commented Jul 19, 2022

In pyro-engine we have two crontab runnig the two scripts involved in this PR:

on crontab run check_internet_connection.sh to ping google every 10mn to check if the pi is still have internet access, if it fail it reboot the network and then the pi if it's still failling

the second one run update_script.sh to check pyro-engine master branch for update every day at 3am

In this PR we move these two script out of the python package and specify how to setup the crontabs

@MateoLostanlen MateoLostanlen requested a review from frgfm July 19, 2022 22:10
@MateoLostanlen MateoLostanlen self-assigned this Jul 19, 2022
@MateoLostanlen MateoLostanlen added the invalid This doesn't seem right label Jul 19, 2022
@codecov
Copy link

codecov bot commented Jul 19, 2022

Codecov Report

Merging #87 (d8cb3f5) into develop (bed9bfb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #87   +/-   ##
========================================
  Coverage    63.68%   63.68%           
========================================
  Files            5        5           
  Lines          179      179           
========================================
  Hits           114      114           
  Misses          65       65           
Flag Coverage Δ
unittests 63.68% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@MateoLostanlen MateoLostanlen merged commit 6c657cc into pyronear:develop Jul 19, 2022
@MateoLostanlen MateoLostanlen deleted the feature/set_crontab branch July 24, 2022 17:21
@frgfm frgfm added this to the 0.2.0 milestone Aug 7, 2022
@frgfm frgfm added type: enhancement New feature or request ext: scripts and removed invalid This doesn't seem right labels Aug 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ext: scripts type: enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants