Brett - While I agree we could simplify the proposal to only do repeatable playbooks I disagree with including the scheduling information inside the playbook that would run on the schedule. This makes playbooks less reusable and is much less flexible if you want to run the same playbook (e.g. checking status on various infrastructure systems) at different intervals or even on-demand (during an incident vs slower more regular checks once a day during non-incident times).
I think its important the definition of the schedule is kept separate from the playbook for this reason.
So this leaves us with "where" the definition of the schedule occurs.
Currently there are 2 proposals and I think your proposal is really just a variant of those choices.
1) Define a new object/step that defines the schedule of an action step (which could include launching a playbook or any other step if you wanted that flexibility), where this new schedule object introduces all the various parameters needed to define what schedule the contained action is executed on.
- This option is effectively what both you and Lucas proposals are.
2) Define new properties as part of the while loop step instead of a completely separate object. Still executes the contained action within the while but avoids a new object for the schedule itself.
- This option is the one that I had proposed.
At this point (cause we've been talking about this for weeks) I'm neutral on either option if we can get consensus on one of the options. However, I'm not for the option you proposed where the schedule is defined within the playbook itself.