-
Notifications
You must be signed in to change notification settings - Fork 1
Description
What happened:
Here there
is a field Actions []string on the Executor struct.
And here there is an assertion that that the user specified action is in the toiExecutor's list of actions.
The documentation here doesn't cover the above Actions field.
Perhaps this is by design, since not specifying any Actions on the toiExecutor seems to make it applicable for any user specified action. So maybe the field is actually deprecated or not ready for user adoption yet. Or maybe the doc above is just failing to describe the field.
In any case, the doc is unclear. I am looking at the code because the doc doesn't make it clear how the various config, template, scheme, credential request and parameters of toiPackage and toiExecutor are used to make the input for the the install program that will ultimately be run.
For sure part of the problem is the reliance on spiff++. Not many people are familiar with this and the docs for spiff++ aren't great.
I guess what I am saying is that there are possibly two doc defects here :
- For sure the doc should be clear enough so that I don't feel the need to resort to reading the code, and
- Perhaps the doc needs to describe the Actions field
What you expected to happen:
Head shouldn't hurt after trying to figure out how this stuff all works :)
How to reproduce it (as minimally and precisely as possible):
See above
Anything else we need to know:
Nope
Environment:
OCM and doc site as of 2024-12-16.
Metadata
Metadata
Assignees
Labels
Type
Projects
Status