Skip to content
This repository has been archived by the owner on Apr 18, 2018. It is now read-only.

SnapshotTime as '0000' #11

Open
roening opened this issue Jul 12, 2017 · 1 comment
Open

SnapshotTime as '0000' #11

roening opened this issue Jul 12, 2017 · 1 comment

Comments

@roening
Copy link

roening commented Jul 12, 2017

Hi,

I'm testing this script with a snapshot time configured as '0000', but it'snt working. The snapshot never are created.
After verify the script, I was left with a doubt with the below code:

if snapshot_time >= str(now_max) and snapshot_time <= str(now) and active_day is True:

Let's suppose that 'now = 0001' and 'snapshot_time=0000'. As 'time_delta = datetime.timedelta(minutes=4)', 'now_max' will be '2357'. In this time the variables are with this values:

  • snapshot_time = 0000
  • now = 0001
  • now_max = 2357

Based on this, the if statement result will be 'false', because the 'snapshot_time' is not greater or equal than 'now_max'.

Can you help me to clarify or to correct this?

Thanks.

@jweyrich
Copy link

jweyrich commented Jul 13, 2017

Good catch! 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants