On 12/3/18 12:06 PM, Daniel P. Berrangé wrote:
On Mon, Dec 03, 2018 at 01:38:45PM +0300, Vasiliy Tolstov wrote:
> пн, 3 дек. 2018 г. в 12:27, Daniel P. Berrangé <berrange(a)redhat.com>:
>>
>> We don't have any plans to add it, but if someone's interested we'd
>> accept patches. It would need to integrate with our virStream APIs
>> to allow download, in similar way to how virStorageVolDownload() works
>
> What args this function need to receive? VolDownload this args:
> virStorageVolDownload(virStorageVolPtr vol,
> virStreamPtr stream,
> unsigned long long offset,
> unsigned long long length,
> unsigned int flags)
>
> I think that we can use all of this, but instead of vol, what we need to recv?
We need the domain object, and the device alias name, eg
virDomainOpenDeviceLogFile(virDomainPtr dom,
virStreamPtr stream,
const char *name,
unsigned long long offset,
unsigned long long length,
unsigned int flags);
We can have it used the same way as virStorageVolDownload:
stream = virStreamNew();
virDomainOpenDeviceLogFile(dom, stream, alias, 0, 0, 0);
virStreamRecvAll(stream, ...);
Michal