Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Various improvements to the exported files controller #292

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Various improvements to the exported files controller #292

wants to merge 5 commits into from

Commits on Feb 9, 2016

  1. crowbar: Use temporary directory when exporting chef data 

    It's really horrible to put the files in the db directory, so don't do
that.

Also properly clean up files at the end.
    @vuntz
    vuntz committed Feb 9, 2016
    Configuration menu
    Copy the full SHA
    a706637 View commit details
    Browse the repository at this point in the history

  2. crowbar: Use temporary directory when exporting supportconfig 

    While not strictly needed, it's slightly better in order to avoid
non-crowbar owned files in the rails app tmp directory. We use a chown
at the end, of course, but if the rails app is stopped in the middle,
then the tmp directory contains files that can't be removed.

Using a proper temporary directory helps (since it may be cleaned up on
reboot, or by scripts).
    @vuntz
    vuntz committed Feb 9, 2016
    Configuration menu
    Copy the full SHA
    1eebb64 View commit details
    Browse the repository at this point in the history

  3. crowbar: Remove non-implemented /utils/{import,upload} routes 

    They were probably used a long time ago...
    @vuntz
    vuntz committed Feb 9, 2016
    Configuration menu
    Copy the full SHA
    38712d1 View commit details
    Browse the repository at this point in the history

  4. crowbar: Use proper delete methods for deleting exported files

    @vuntz
    vuntz committed Feb 9, 2016
    Configuration menu
    Copy the full SHA
    f4efe8f View commit details
    Browse the repository at this point in the history

  5. crowbar: Use Exported Files instead of Exported Items in navigation 

    This is the same as the title of the page.
    @vuntz
    vuntz committed Feb 9, 2016
    Configuration menu
    Copy the full SHA
    7b0cb79 View commit details
    Browse the repository at this point in the history