Master node migration failed with on 'lvcreate os -n root -L 9,77g'

Bug #1538226 reported by Egor Kotko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Peter Zhurba
8.0.x
Fix Released
High
Peter Zhurba

Bug Description

Baremetal. Master node migration failed with on 'lvcreate os -n root -L 9,77g'

Steps to reproduce:
1. Deploy environment 3Controllers, 2Computes
2. Migrate master node on one of the Compute nodes:
#fuel-migrate <ip_of_compute_node>

Actual result:
failed with error message:

Warning: Permanently added '172.16.39.131' (ECDSA) to the list of known hosts.
Create VM fuel_master on 172.16.39.131
Create partition table
Create lvm volumes and file-systems
ERROR: look at /var/log/fuel-migrate.log
+ ssh 172.16.39.137 vgcreate os /dev/vda5
Warning: Permanently added '172.16.39.137' (ECDSA) to the list of known hosts.
  No physical volume label read from /dev/vda5
  Physical volume "/dev/vda5" successfully created
  Volume group "os" successfully created
+ ssh 172.16.39.137 lvcreate os -n root -L 9,77g
Warning: Permanently added '172.16.39.137' (ECDSA) to the list of known hosts.
  Invalid argument for --size: 9,77g
  Error during parsing of command line.
+ set +x

But if go on created VM and create lv volume with different size it will pass:

root@bootstrap:~# lvcreate os -n root -L 8g
Logical volume "root" created

Snapshot:
https://drive.google.com/file/d/0BzWDM1PONYEuMXN3OGxtdjFPZTA/view?usp=sharing

Dmitry Klenov (dklenov)
tags: added: area-library
no longer affects: fuel/mitaka
tags: added: feature-reduced-footprint
tags: added: team-telco
Revision history for this message
Dmitry Bilunov (dbilunov) wrote :

Try adding export LC_NUMERIC=C at the top of fuel-library/files/fuel-migrate/fuel-migrate

Revision history for this message
Peter Zhurba (pzhurba) wrote :

Sure it should help
Thank you

Dmitry Pyzhov (dpyzhov)
no longer affects: fuel/mitaka
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/273164
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=77be30697000d9b50851b9ce7b4e2248ff4fb47b
Submitter: Jenkins
Branch: master

commit 77be30697000d9b50851b9ce7b4e2248ff4fb47b
Author: Peter Zhurba <email address hidden>
Date: Wed Jan 27 19:27:53 2016 +0200

    Set LC_ALL=C at the beginning performing fuel_migrate

    Change-Id: I7405564f3c33471c6d57983157abf42c37fe72d7
    Closes-Bug: #1538226

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/8.0)

Fix proposed to branch: stable/8.0
Review: https://review.openstack.org/274008

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/8.0)

Reviewed: https://review.openstack.org/274008
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=2c9890d3b4150010259a911cd2174886096c92c9
Submitter: Jenkins
Branch: stable/8.0

commit 2c9890d3b4150010259a911cd2174886096c92c9
Author: Peter Zhurba <email address hidden>
Date: Wed Jan 27 19:27:53 2016 +0200

    Set LC_ALL=C at the beginning performing fuel_migrate

    Change-Id: I7405564f3c33471c6d57983157abf42c37fe72d7
    Closes-Bug: #1538226
    (cherry picked from commit 77be30697000d9b50851b9ce7b4e2248ff4fb47b)

tags: added: on-verification
Revision history for this message
Dmitriy Kruglov (dkruglov) wrote :

Verified on MOS 8.0, build 539. The issue is fixed.

ISO details:
VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "8.0"
  api: "1.0"
  build_number: "539"
  build_id: "539"
  fuel-nailgun_sha: "baec8643ca624e52b37873f2dbd511c135d236d9"
  python-fuelclient_sha: "4f234669cfe88a9406f4e438b1e1f74f1ef484a5"
  fuel-agent_sha: "658be72c4b42d3e1436b86ac4567ab914bfb451b"
  fuel-nailgun-agent_sha: "b2bb466fd5bd92da614cdbd819d6999c510ebfb1"
  astute_sha: "b81577a5b7857c4be8748492bae1dec2fa89b446"
  fuel-library_sha: "e2d79330d5d708796330fac67722c21f85569b87"
  fuel-ostf_sha: "3bc76a63a9e7d195ff34eadc29552f4235fa6c52"
  fuel-mirror_sha: "fb45b80d7bee5899d931f926e5c9512e2b442749"
  fuelmenu_sha: "78ffc73065a9674b707c081d128cb7eea611474f"
  shotgun_sha: "63645dea384a37dde5c01d4f8905566978e5d906"
  network-checker_sha: "a43cf96cd9532f10794dce736350bf5bed350e9d"
  fuel-upgrade_sha: "616a7490ec7199f69759e97e42f9b97dfc87e85b"
  fuelmain_sha: "87dfb6bc25d4650264f09c338ed77c21a3d6fe87"

tags: removed: on-verification
Revision history for this message
Ksenia Svechnikova (kdemina) wrote :

Verified on fuel-9.0-mos-392-2016-05-23, fuel migration pass successfully

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.