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@xxxxxxxxxx>: >>> >>> 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 _______________________________________________ libvirt-users mailing list libvirt-users@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvirt-users
