[Kimchi-devel] [PATCH 0/4 V5] Storage volume upload

[Crístian Viana]

That assumption didn't seem to work well with the current implementation of
the upload feature, where it works in the tests and it doesn't in "real
life" ☺

On Tue, May 12, 2015, 13:46 Aline Manera <alinefm at linux.vnet.ibm.com> wrote:

>
>
> On 12/05/2015 13:44, Crístian Deives wrote:
> > On 12-05-2015 13:11, Aline Manera wrote:
> >> On 12/05/2015 12:46, Crístian Deives wrote:
> >>> Shouldn't the UI be updated to re-enable file uploads now?
> >>
> >> I am working on UI right now.
> >> As upload is current disabled on UI I don't see problems on merge the
> >> backend first. Are you agree?
> >
> > Well, there's no problem in merging this code (as in "it won't break
> > anything"), but there's no way for anyone to actually test a file
> > upload and make sure the code works "in real life" (other than writing
> > a test function...). I can't use the REST API via command line because
> > this command requires passing a real object instead of JSON data in
> > the variable "chunk", and there's no UI to use it as well. So only
> > with the UI I'll be able to actually test if the implementation works.
>
> Well, we have a specific test case to use file upload. So if the test
> passes...
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/kimchi-devel/attachments/20150512/c53632c6/attachment.html>