diff options
author | Bryn M. Reeves <bmr@redhat.com> | 2019-03-25 11:59:16 +0000 |
---|---|---|
committer | Bryn M. Reeves <bmr@redhat.com> | 2019-03-25 11:59:16 +0000 |
commit | 41d48770c0c34b58862be57b1aa14618b7422ae9 (patch) | |
tree | 17b150b84d9c084462f51c4133f9b807affd5620 /docs | |
parent | cb9b166e08be51bff9b0a98ce266507c702740eb (diff) | |
download | sos-41d48770c0c34b58862be57b1aa14618b7422ae9.tar.gz |
[Plugin] handle predicates correctly in get_cmd_output_now()
The get_command_output_now() method is used for two distinct
purposes:
* Called directly from user plugin code, to immediately collect
the output of a command during the setup() phase (e.g. to then
process the output to inform further collection, while also
storing the result in the report).
* Internally from the _collect_cmd_output() callback, in order
to collect the commands added at setup() time by user plugins
during the collect() phase.
In the former case, predicates should be applied as normal and
evaluated prior to command execution. In the latter, predicates
have already been applied at the time of the add_cmd_output()
call; any predicate that was not set at the time of that call
must not apply to the collection of the command.
Split the method in two: _g_c_o_n() now collects command output
unconditionally, and is used directly by internal callers, and
g_c_o_n() becomes a wrapper to perform predicate evaluation.
Resolves: #1616
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
Diffstat (limited to 'docs')
0 files changed, 0 insertions, 0 deletions