openSeaChest icon indicating copy to clipboard operation
openSeaChest copied to clipboard

openSeaChest_Logs: FARM data in stdout

Open yaarith opened this issue 3 years ago • 8 comments

It would be nice if we can pipe the FARM data from openSeaChest_Logs to openSeaChest_LogParser and not go through a file on disk (i.e. openSeaChest_Logs --farm -o - | openSeaChest_LogParser ...). Currently, the openSeaChest_Logs saves the data to a file. That takes a little logic to find what the file name is (need the serial number and the time the log was generated), it's more automation friendly to allow piping.

yaarith avatar Mar 16 '22 14:03 yaarith

Hi @yaarith, The current method is due to how we've interacted with different customers in the past and had binaries shared directly back to Seagate. This has been changing over time and we've begun parsing more in the log parser. They are separate processes due to requirements we've had from data centers in the past. I think this is another good use-case to support though. I'll check our project schedule to see where we can fit this in.

vonericsen avatar Mar 16 '22 17:03 vonericsen

Thanks for clarifying, @vonericsen. When can we expect this feature?

yaarith avatar Mar 24 '22 20:03 yaarith

Hi @yaarith, It would take a few weeks.

Rungthida avatar Mar 30 '22 20:03 Rungthida

Any progress? Seems like there could be one tool that either only saves the binary format or does the extra step of parsing/output, but is anyone looking into this?

dmick avatar May 25 '22 19:05 dmick

Hello @dmick, We've started looking into it but it has been delayed due to other projects that we're working on in parallel. Hopefully, this should be ready for testing in a couple weeks.

Rungthida avatar May 26 '22 16:05 Rungthida

Hello @yaarith & @dmick, I'm so sorry it took so long. We have had a lot of internal customer request. However, the new code has been pushed to the develop branch for both openSeaChest_Logs and openSeaChest_LogsParser. We also added a bash script for Linux in"example" folder.

Example: ./openSeaChest_Logs -d /dev/sdb --farm --logMode pipe | ./openSeaChest_LogParser_x86_64 --inputLog fromPipe --logType farmLog --printType json --outputLog testLog.jsn

Rungthida avatar Aug 24 '22 21:08 Rungthida

Thanks for the update, @Rungthida. I do not see these commits in either repos under the develop branch, when were they merged?

yaarith avatar Aug 25 '22 15:08 yaarith

Hi @yaarith, I have merged @Rungthida's code into develop. We had a miscommunication, but it is now on develop in both openSeaChest and openSeaChest_LogParser. Let us know if you have any problem using this.

vonericsen avatar Aug 25 '22 17:08 vonericsen

Closing this as the code has been in develop and has now been released in https://github.com/Seagate/openSeaChest/releases/tag/v23.03 Please feel free to reopen this issue or create a new issue if you need any other help!

vonericsen avatar Mar 01 '23 21:03 vonericsen