virtualbox scripts fail memory check

Bug #1380687 reported by Vasiliy Pleshakov
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Stanislaw Bogatkin

Bug Description

script finds no 'top' or 'free' command and fails with awkward message.
OS: CYGWIN under Windows 7

Steps to reproduce:
1. get windows
2. install cygwin with default packages
3. download and unpack vbox scripts.
4. put MirantisOpenStack-5.1.iso in it's iso directory.
5. run launch.sh

Expected result: normal installing process.

Actual result: script fails with message:
"Your host has not enough memory.
You can allocate no more than -512MB, but trying to run VMs with 5632MB"

Workaround:
Install 'procps' package.

Impact: customer can be confused with negative amount of memory. Customer may not understand that scripts missing 'top' or 'free' command.

By default cygwin installs without 'top' or 'free' command. One should install 'procps' package manually in order to get those command.

Script should print warning message if it find no 'top' or 'free' command.
Script should print no negative amounts of memory in its message.

Tags: docs
Revision history for this message
Vasiliy Pleshakov (vpleshakov) wrote :
Changed in fuel:
milestone: none → 6.0
importance: Undecided → Medium
assignee: nobody → Fuel Library Team (fuel-library)
Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Stanislaw Bogatkin (sbogatkin)
Changed in fuel:
status: New → Triaged
Revision history for this message
Bartłomiej Piotrowski (bpiotrowski) wrote :

Fixed in 7c989124.

Changed in fuel:
status: Triaged → Fix Committed
tags: added: docs
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-docs (master)

Fix proposed to branch: master
Review: https://review.openstack.org/138238

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-docs (master)

Reviewed: https://review.openstack.org/138238
Committed: https://git.openstack.org/cgit/stackforge/fuel-docs/commit/?id=58e887e80c44eaa8d55be1ee9621da6edffceeab
Submitter: Jenkins
Branch: master

commit 58e887e80c44eaa8d55be1ee9621da6edffceeab
Author: Meg McRoberts <email address hidden>
Date: Mon Dec 1 19:30:21 2014 -0800

    VirtualBox: requires procps package on Windows/Cygwin

    Add procps to the list of Cygwin packages required
    to run the VirtualBox version of MOS on Windows systems.

    This fix is being applied to the 6.0 docs but it would
    be nice to cherrypick it to 5.1.1 as well.

    Change-Id: I1e484958f0f03ac3928dca095f16b4338b579bf7
    Closes-Bug:1380687

Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.