sot / jobwatch

Watch files, database tables, and log files to ensure valid cron processing
3 stars 0 forks source link

Updates for Python 3 compatibility #35

Closed taldcroft closed 4 years ago

taldcroft commented 4 years ago

This moves Ska watch processing into a new skawatch3 namespace in data and share to make this a softer transition.

The actual Py3 changes were expected and trivial.

I ended up making more changes than expected to get tests to pass. I don't think they were still passing on the Py2 side at this point, but I didn't actually check.

Testing

In task schedule

Ran as per cron job schedule:

/proj/sot/ska3/flight/bin/skare task_schedule3.pl -config /proj/sot/ska3/flight/data/skawatch3/task_schedule_hourly.cfg
/proj/sot/ska3/flight/bin/skare task_schedule3.pl -config /proj/sot/ska3/flight/data/skawatch3/task_schedule_daily.cfg

Outputs:

To do:

jeanconn commented 4 years ago

Thanks for the work on this @taldcroft ! Looking good. Are you running it manually now as you are testing? I was wondering about the cadence of the warnings showing up in my inbox.

taldcroft commented 4 years ago

I have been running manually and via cron jobs. I'll update the aspect cron page, but this is the current setup as me on kady:

## Skawatch
40 10 * * * /proj/sot/ska3/flight/bin/skare task_schedule3.pl -config /proj/sot/ska3/flight/data/skawatch3/task_schedule_daily.cfg

# Hourly skawatch/jobwatch
5 */2 * * * /proj/sot/ska3/flight/bin/skare task_schedule3.pl -config /proj/sot/ska3/flight/data/skawatch3/task_schedule_hourly.cfg

I disabled the Ska2 hourly watch to stop the hourly spamming. The Ska3 hourly seems to be giving good results now.

After we have let this soak for a few days my plan would be to: