ocp: fix telemetry string log output file handling

[Karthikmct]

ocp: fix telemetry string log output file handling

Fixed the code to avoid binary file creation by tool for ocp

Signed-off-by: Karthik [email protected]

[Karthikmct]

@igaw , This fix is addressing your comment on get_c9_log_page_data in [https://github.com//pull/2586]

[igaw]

This drops the support for the binary output into a file completely. I'd say it would be nice to get all the use cases properly working instead just dropping this feature. Someone might want to us it.

In nvme.c we have couple example how it could be done. get_log for example has the --raw-output argument. But that was introduces before we had the global --output-format=binary. Hmm, so I think the most canonical way would be to get --output-format=binary properly working for all commands in ocp and use the shell redirect to write into a file. For those commands which want to write the output into a file an additional command line argument could be added. I thinke we have this for the submit_io based command.

Anyway, my point is that I'd like to see the ocp plugin to adapt the same type of input/output handling pattern we have in the main code base. Having two different philosophy how to things is just confusing.