Make the output of `bios get` valid YAML

Cray-HPE / gru

A utility for reading and modifying BMCs (e.g. iLO, RMMC) using RedFish (gofish).

MIT License

6 stars 2 forks source link

Make the output of `bios get` valid YAML #46

Closed dev-zero closed 8 months ago

dev-zero commented 8 months ago

SUMMARY

To use the information in a Unix-type pipe-chain it would be useful if bios get would output valid YAML. At the moment it does not due to log messages interlaced with the output (fixed by #44), when using IP addresses for hosts (since those are not valid YAML keys) and some keys are not quoted.

ISSUE TYPE

ADDITIONAL INFORMATION

rustydb commented 8 months ago

Would it be sufficient if there was a global YAML output option (similar to the JSON option)?

rustydb commented 8 months ago

Would it be sufficient if there was a global YAML output option (similar to the JSON option)?

I just saw your PR, I think I can rescind my previous question given your PR satisfies the RFE you reported. A global YAML toggle would still be nice.