theforeman / foreman_expire_hosts

Foreman plugin for limiting host lifetime
GNU General Public License v3.0
5 stars 12 forks source link

do not notify about already powered down hosts #37

Closed timogoebel closed 5 years ago

laugmanuel commented 5 years ago

I've tested the patch and it works as expected. Notifications are no longer sent if the host was shutdown already.

gqueri commented 5 years ago

Yes I confirm also it works as expected. The only remaining issue is that the actual deletion date is one day before the date mentioned in the mail.

timogoebel commented 5 years ago

The only remaining issue is that the actual deletion date is one day before the date mentioned in the mail.

That's interesting to know. Do you mind opening a separate issue for this here? Preferably with some mir information attached?