[fuel-menu]Fuel-menu save-only uses eth0 as default interface regardless iface state

Bug #1534517 reported by Dmitry Kalashnik
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Fedor Zhadaev
8.0.x
Fix Released
High
Fedor Zhadaev
Mitaka
Fix Released
High
Fedor Zhadaev

Bug Description

Steps to reproduce:
1. Create VM with 1 network interface in down state
2. Attach ISO and deploy master

Effects:
1. In case of default installation deployment of master node will fail with because of fail of fuel-menu (see screenshot)
2. In case of adding "showmenu=yes" fuel-menu will configure repos improperly, with ${mos_version} instead of actual version.

Revision history for this message
Dmitry Kalashnik (dkalashnik) wrote :
description: updated
Ilya Kutukov (ikutukov)
Changed in fuel:
status: New → Confirmed
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Alexander Saprykin (cutwater)
Changed in fuel:
assignee: Alexander Saprykin (cutwater) → Fuel Python Team (fuel-python)
Fedor Zhadaev (fzhadaev)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Fedor Zhadaev (fzhadaev)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-menu (master)

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

Fedor Zhadaev (fzhadaev)
Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-main (master)

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

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

Reviewed: https://review.openstack.org/270135
Committed: https://git.openstack.org/cgit/openstack/fuel-main/commit/?id=47d85af8994ea27b9dfcde30c59bfdf540cce5c9
Submitter: Jenkins
Branch: master

commit 47d85af8994ea27b9dfcde30c59bfdf540cce5c9
Author: slava <email address hidden>
Date: Wed Jan 20 14:18:32 2016 +0300

    Check astute.yaml file was generated succesfully

    Fuelmenu may not generate astute.yaml for some reason.
    It may be crushing of fuelmenu or etc. To protect us of
    this problem additional check has been introduced.

    Change-Id: If661f27c5116e60fb7ded1fa5ed33bf396ef1a9f
    Related-Bug: #1534517

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-main (stable/8.0)

Related fix proposed to branch: stable/8.0
Review: https://review.openstack.org/271957

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

Reviewed: https://review.openstack.org/271957
Committed: https://git.openstack.org/cgit/openstack/fuel-main/commit/?id=dde4ad5471bc6bd8230613c2580424f81b070b53
Submitter: Jenkins
Branch: stable/8.0

commit dde4ad5471bc6bd8230613c2580424f81b070b53
Author: slava <email address hidden>
Date: Wed Jan 20 14:18:32 2016 +0300

    Check astute.yaml file was generated succesfully

    Fuelmenu may not generate astute.yaml for some reason.
    It may be crushing of fuelmenu or etc. To protect us of
    this problem additional check has been introduced.

    Change-Id: If661f27c5116e60fb7ded1fa5ed33bf396ef1a9f
    Related-Bug: #1534517
    (cherry picked from commit 47d85af8994ea27b9dfcde30c59bfdf540cce5c9)

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

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

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

Reviewed: https://review.openstack.org/272571
Committed: https://git.openstack.org/cgit/openstack/fuel-menu/commit/?id=2a72d9e3b0a7373f570ac706e96f40288d6196d0
Submitter: Jenkins
Branch: master

commit 2a72d9e3b0a7373f570ac706e96f40288d6196d0
Author: Fedor Zhadaev <email address hidden>
Date: Tue Jan 26 17:56:44 2016 +0300

    Remove direct copying default settings.yaml to astute.yaml

      The astute.yaml should be generated from settings.yaml
      using fuelmenu. Without this change astute.yaml contains
      incorrect values if fuelmenu fails.

    Change-Id: I73ed3c9b871e3dcc462bd968db7a234cab9e5458
    Related-Bug: #1534517

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-menu (stable/8.0)

Related fix proposed to branch: stable/8.0
Review: https://review.openstack.org/273651

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-menu (stable/8.0)

Change abandoned by Fedor Zhadaev (<email address hidden>) on branch: stable/8.0
Review: https://review.openstack.org/273651
Reason: commit for master was reverted

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

Reviewed: https://review.openstack.org/269043
Committed: https://git.openstack.org/cgit/openstack/fuel-menu/commit/?id=ab6855439376ae4ab2410d427e4d96b25b3d3e94
Submitter: Jenkins
Branch: master

commit ab6855439376ae4ab2410d427e4d96b25b3d3e94
Author: Fedor Zhadaev <email address hidden>
Date: Mon Jan 18 16:32:01 2016 +0300

    Change behavior in case when management interface is down

     * Add check for management interface has IP

    Change-Id: Ic64c9dc1d931a131152a84070900b747a5311a3c
    Related-Bug: #1534517

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-menu (stable/8.0)

Related fix proposed to branch: stable/8.0
Review: https://review.openstack.org/274579

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

Reviewed: https://review.openstack.org/274033
Committed: https://git.openstack.org/cgit/openstack/fuel-menu/commit/?id=8fe1220566487079869c4815a59dd4804a8af2c9
Submitter: Jenkins
Branch: master

commit 8fe1220566487079869c4815a59dd4804a8af2c9
Author: Fedor Zhadaev <email address hidden>
Date: Tue Jan 26 17:56:44 2016 +0300

    Remove direct copying default settings.yaml to astute.yaml

      The astute.yaml should be generated from settings.yaml
      using fuelmenu. Without this change astute.yaml contains
      incorrect values if fuelmenu fails.

    Change-Id: I5c0885c8991bc41cce5396e3c587f466ef0d3f92
    Related-Bug: #1534517

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-menu (stable/8.0)

Related fix proposed to branch: stable/8.0
Review: https://review.openstack.org/274650

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

Reviewed: https://review.openstack.org/274579
Committed: https://git.openstack.org/cgit/openstack/fuel-menu/commit/?id=4a2b817eb6f67c6399e8f748375b560f9fd47f75
Submitter: Jenkins
Branch: stable/8.0

commit 4a2b817eb6f67c6399e8f748375b560f9fd47f75
Author: Fedor Zhadaev <email address hidden>
Date: Mon Jan 18 16:32:01 2016 +0300

    Change behavior in case when management interface is down

     * Add check for management interface has IP

    Change-Id: Ic64c9dc1d931a131152a84070900b747a5311a3c
    Related-Bug: #1534517
    (cherry picked from commit ab6855439376ae4ab2410d427e4d96b25b3d3e94)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/274650
Committed: https://git.openstack.org/cgit/openstack/fuel-menu/commit/?id=15bdbceb4485e058737e56b4e30966791ad55976
Submitter: Jenkins
Branch: stable/8.0

commit 15bdbceb4485e058737e56b4e30966791ad55976
Author: Fedor Zhadaev <email address hidden>
Date: Tue Jan 26 17:56:44 2016 +0300

    Remove direct copying default settings.yaml to astute.yaml

      The astute.yaml should be generated from settings.yaml
      using fuelmenu. Without this change astute.yaml contains
      incorrect values if fuelmenu fails.

    Change-Id: I5c0885c8991bc41cce5396e3c587f466ef0d3f92
    Related-Bug: #1534517
    (cherry picked from commit 8fe1220566487079869c4815a59dd4804a8af2c9)

tags: added: on-verification
Revision history for this message
Artem Panchenko (apanchenko-8) wrote :

verified

VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "8.0"
  api: "1.0"
  build_number: "525"
  build_id: "525"
  fuel-nailgun_sha: "baec8643ca624e52b37873f2dbd511c135d236d9"
  python-fuelclient_sha: "4f234669cfe88a9406f4e438b1e1f74f1ef484a5"
  fuel-agent_sha: "658be72c4b42d3e1436b86ac4567ab914bfb451b"
  fuel-nailgun-agent_sha: "b2bb466fd5bd92da614cdbd819d6999c510ebfb1"
  astute_sha: "b81577a5b7857c4be8748492bae1dec2fa89b446"
  fuel-library_sha: "f2ed8e4503dd9641b70b94381d30e21689b8e5c9"
  fuel-ostf_sha: "3bc76a63a9e7d195ff34eadc29552f4235fa6c52"
  fuel-mirror_sha: "fb45b80d7bee5899d931f926e5c9512e2b442749"
  fuelmenu_sha: "e071216cb214e34b4d861478033425ee6a54a3be"
  shotgun_sha: "63645dea384a37dde5c01d4f8905566978e5d906"
  network-checker_sha: "a43cf96cd9532f10794dce736350bf5bed350e9d"
  fuel-upgrade_sha: "616a7490ec7199f69759e97e42f9b97dfc87e85b"
  fuelmain_sha: "a365f05b903368225da3fea9aa42afc1d50dc9b4"

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.