Add systemd timer for periodic inspector #352
Conversation
pdmars
requested review from
msherman64,
Mark-Powers and
codyhammock
as code owners
roles/chameleon_periodic_inspector/templates/periodic-inspector.service.j2
Outdated
Show resolved
Hide resolved
| Description=Run Chameleon Periodic Inspector daily | ||
|
|
||
| [Timer] | ||
| OnCalendar=daily |
There was a problem hiding this comment.
we'll probably want to play with this timing in practice
running more frequently allows the script to attempt to "fix" nodes that failed to inspect a previous time.
|
|
||
| # Periodic Inspector | ||
| [chameleon-periodic-inspector:children] | ||
| control |
There was a problem hiding this comment.
all this script needs is access to the ironic API.
in practice "control" is fine, longer term we may want explicitly target the "deployment" group, and/or something like "control[0]" to avoid running it multiple times in the multinode case.
tldr leaving it as "control" is ok because we'll have to fix all sorts of other things before that assumption becomes invalid.
There was a problem hiding this comment.
Looks great to me, thanks!
FYI, this can be tested without the timer by executing systemctl start <unitname> instead of having the timer do it.
Once deployed, we'll also want to remove my scripts from the "rally-executor" vm so that we're not doubling up on it
No description provided.