Change Vsock to VirFs to tranfer files between the VM and the node.

[musitdev]

Remove Vsock declarations in the Qemu VM start args.
Add VirtFs arguments.
Update file transfer logic.
Change the result management between VM and node to report more VM error in the submit result.

[tuommaki]

LGTM in general. Couple very minor nits / fix suggestions and one question about the tests prover's bigfile.

[tuommaki]

Should we put some intermediary directory here? Like data or txdata or something like that? Currently the transaction hashes are on the same level with images and log and it looks a bit weird. WDYT?

[musitdev]

Yes I agree with you. I add the same idea but I didn't change because I wanted to keep things like they were.
I can add a directory in this PR or open a new PR to do the change?
I prefer data or files.

[tuommaki]

open a new PR to do the change?

Open a new PR so it's clearer that way 👍

I prefer data or files.

Both work for me. Possibly prefix with "tx" so it's absolutely obvious what it's about. 👍

[musitdev]

I've created the #138 issue.

[tuommaki]

Should this file be generated during runtime?

[musitdev]

I wanted an easy way to transfer a big file from prover to verifier for test, so I was thinking of using the same file was easier.
It was only for test, and at some point perhaps we should revert to a more playful example.

[tuommaki]

Yeah, I get that. I was just thinking if this code works as-is on a new installation? I mean, would the end user need to put that "bigfile.bin" somewhere in order to make this work or does that somehow work already out-of-the-box?

[tuommaki]

If everything works, let's roll with this 🙂