Unable to download file > 100kb using murano action

Bug #1563318 reported by Artem Akulshin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Murano
Won't Fix
High
Nikolay Starodubtsev

Bug Description

I have used the Murano Action like mentioned here
http://docs.openstack.org/developer/murano/draft/appdev-guide/murano_pl/actions.html

But replaced the script for the file by my own. My script archives content of /var/log, everything else is just like by link above.

For the reference I have used the code from stable/kilo fork of Kubernetes cluster and action ExportConfig. Everything the same except the file not config, but logs.
https://github.com/openstack/murano-apps/tree/stable/kilo/Docker/Kubernetes/KubernetesCluster

The problem is that my archive of 6+ megabytes and it fail to start downloading. When i try to do the same with file 10kb, it downloads just fine.

I have Mirantis OpenStack 7 and Kilo OpenStack release.

tags: added: download
removed: dow
affects: openstack-ansible → murano
tags: added: engine muranopl
removed: action download file murano
Changed in murano:
milestone: none → mitaka-rc2
importance: Undecided → High
Changed in murano:
assignee: nobody → Nikolay Starodubtsev (starodubcevna)
status: New → Won't Fix
Revision history for this message
Nikolay Starodubtsev (starodubcevna) wrote :

Hi, Artem. After investigation I see that it's not the problem with murano client or dashboard, the error occurs because task result is too long for database field. I'll create a bp related this problem.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to murano (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/299461

Revision history for this message
Kirill Zaitsev (kzaitsev) wrote :

Closing this as won't fix, because fixing the problem requires careful design decisions about how and where to store the files. It's not a good idea to store the files on controller nodes as this would pose a security hole (I would be able to drain all the space by pushing multiple large files). Probably storing them in swift would be a good idea.

Revision history for this message
Artem Akulshin (aakulshin) wrote :

Thank you, guys! I have successfully make it work by changing type of result column from task table at murano database from text to mediumtext which allows up to 16 mb content size.

I agree that this is not production-ready solution, there should be some other storage for these files, the temporary one. Normally you would not need to download that file one more time, you would start action again to have an update.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to murano (master)

Reviewed: https://review.openstack.org/299461
Committed: https://git.openstack.org/cgit/openstack/murano/commit/?id=8e29b3474878a2a1dd609f8010f01d0293018764
Submitter: Jenkins
Branch: master

commit 8e29b3474878a2a1dd609f8010f01d0293018764
Author: Nikolay Starodubtsev <email address hidden>
Date: Wed Mar 30 18:04:52 2016 +0300

    Add note about big files download

    Murano doesn't support big files download during actions execution. This
    problem wasn't covered in documentation previously.

    Change-Id: I1575bcead668eb415f0fdbd308e4c237b9b024cc
    Related-Bug: #1563318

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers