Should all: be in playbook example in README? #51
Replies: 4 comments 5 replies
-
And in the other part of REAMD.md there is
so, maybe use |
Beta Was this translation helpful? Give feedback.
-
I think the example we currently have is too rich. It will be nice to have much simpler example and a hidden (expandable) text section for full example with all the options possible |
Beta Was this translation helpful? Give feedback.
-
@umputun I think that spot should not allow to use Here is an example:
I think that this is a bad behaviour that I have specified |
Beta Was this translation helpful? Give feedback.
-
Yes, thank you. I've re-run my previous example with the latest release of spot that has commit eb724ec and it is working as I expect:
I've also checked that in README.md there is no such invalid config, and everything is ok there:
I'm closing this discussion. |
Beta Was this translation helpful? Give feedback.
-
Now in README.md there is an example of playbook:
I'm not sure if the
all:
is a good example to be placed in playbook. Why anybody needs to defineall
in playbook, when cli--target=all
will always run on everything from inventory?Beta Was this translation helpful? Give feedback.
All reactions