Fuel for OpenStack

deplopyment fails on murano upload_package

Bug #1622627 reported by Fedor Tarasenko
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Maksim Malchuk
Fuel for OpenStack 10.0
Mitaka
Fix Released
High
Maksim Malchuk
Fuel for OpenStack 9.1

Bug Description

Fuel 9.0 with detached keystone plugin fails while running openstack_tasks::murano::upload_murano_package

File deployment/puppet/openstack_tasks/manifests/murano/murano.pp has following line:
  $management_ip = hiera('management_vip')

Variable $management_ip is not used further in code.
Instead, on lines 37 & 40 we see $management_vip which is not defined.

  $internal_auth_address = get_ssl_property($ssl_hash, {}, 'keystone', 'internal', 'hostname', [hiera('keystone_endpoint', ''), $service_endpoint, $management_vip])
  $admin_auth_address = get_ssl_property($ssl_hash, {}, 'keystone', 'admin', 'hostname', [hiera('keystone_endpoint', ''), $service_endpoint, $management_vip])

We need proper fix for this bug.

Revision history for this message
Dmitry Klenov (dklenov) wrote :

@Fedor Tarasenko, please provide steps to reproduce and diagnostic snapshot.

tags: added: area-library
Changed in fuel:
milestone: none → 9.1
assignee: nobody → Fuel Sustaining (fuel-sustaining-team)
Maksim Malchuk (mmalchuk)
no longer affects: fuel/newton
Changed in fuel:
status: Incomplete → Confirmed
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/369427

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

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/369429

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

Reviewed: https://review.openstack.org/369429
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=2debf786e9d5dca2b0d70c9bae64ebe945cc022a
Submitter: Jenkins
Branch: stable/mitaka

commit 2debf786e9d5dca2b0d70c9bae64ebe945cc022a
Author: Maksim Malchuk <email address hidden>
Date: Tue Sep 13 15:43:49 2016 +0300

    Fix typo for management_vip variable

    Change-Id: I53ee4abb49f3cc1cd6fcb14c3a913aebd7257a93
    Closes-Bug: #1622627
    Signed-off-by: Maksim Malchuk <email address hidden>

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

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

commit edac9c60f8dc688608e1f2ca9091474a95c16822
Author: Maksim Malchuk <email address hidden>
Date: Tue Sep 13 15:43:49 2016 +0300

    Fix typo for management_vip variable

    Change-Id: I53ee4abb49f3cc1cd6fcb14c3a913aebd7257a93
    Closes-Bug: #1622627
    Signed-off-by: Maksim Malchuk <email address hidden>

Changed in fuel:
status: In Progress → Fix Committed
Andrey Lavrentyev (alavrentyev)
tags: added: on-verification
Revision history for this message
Andrey Lavrentyev (alavrentyev) wrote :

Verified on 9.1 snapshot #268

[root@nailgun ~]# shotgun2 short-report
cat /etc/fuel_build_id:
 495
cat /etc/fuel_build_number:
 495
cat /etc/fuel_release:
 9.0
cat /etc/fuel_openstack_version:
 mitaka-9.0

UBUNTU_MIRROR_ID=ubuntu-2016-09-14-213640
CENTOS_MIRROR_ID=centos-7.2.1511-2016-05-31-083834
MOS_UBUNTU_MIRROR_ID=9.0-2016-09-15-164325
MOS_CENTOS_OS_MIRROR_ID=os-2016-06-23-135731
MOS_CENTOS_PROPOSED_MIRROR_ID=proposed-2016-09-15-174322
MOS_CENTOS_UPDATES_MIRROR_ID=updates-2016-06-23-135916
MOS_CENTOS_HOLDBACK_MIRROR_ID=holdback-2016-06-23-140047
MOS_CENTOS_HOTFIX_MIRROR_ID=hotfix-2016-09-15-144323
MOS_CENTOS_SECURITY_MIRROR_ID=security-2016-06-23-140002

tags: removed: on-verification
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library 10.0.0rc1

This issue was fixed in the openstack/fuel-library 10.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library 10.0.0

This issue was fixed in the openstack/fuel-library 10.0.0 release.

TatyanaGladysheva (tgladysheva)
tags: added: on-verification
Revision history for this message
TatyanaGladysheva (tgladysheva) wrote :

Verified on 10.0 build #1569.

$internal_auth_address, $admin_auth_address have $management_ip variable in file murano.pp. Task 'murano' is passed during deployment.
Deployment is failed, but it is covered by separate bug https://bugs.launchpad.net/fuel/+bug/1653073.

tags: removed: on-verification
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.
Subscribing...

Other bug subscribers

Remote bug watches

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