Fuel for OpenStack

Wrong SQLAlchemy backend settings for Openstack

  1. Series 5.0.x
  2. Bug #1415811
Bug #1415811 reported by Aleksandr Didenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
In Progress
Medium
Aleksandr Didenko
Fuel for OpenStack 6.1
5.0.x
In Progress
Medium
Aleksandr Didenko
Fuel for OpenStack 5.0-updates
5.1.x
In Progress
Medium
Aleksandr Didenko
Fuel for OpenStack 5.1.1-updates
6.0.x
In Progress
Medium
Aleksandr Didenko
Fuel for OpenStack 6.0-updates
6.1.x
In Progress
Medium
Aleksandr Didenko
Fuel for OpenStack 6.1

Bug Description

Due to wrong variables scoping we use defaul SQLAlchemy backend values defined in openstack classes.

https://github.com/stackforge/fuel-library/blob/master/deployment/puppet/osnailyfacter/manifests/cluster_ha.pp#L478-L481

We should use proper scope for those variables there because there's double class encapsulation in cluster_ha.

See original description
Tags: library
Aleksandr Didenko (adidenko)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/6.0)

Fix proposed to branch: stable/6.0
Review: https://review.openstack.org/151161

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

Fix proposed to branch: stable/5.1
Review: https://review.openstack.org/151162

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

Fix proposed to branch: stable/5.0
Review: https://review.openstack.org/151165

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

Reviewed: https://review.openstack.org/148522
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=943ab6e4712d5edc5c69020116f85039e5c78e14
Submitter: Jenkins
Branch: master

commit 943ab6e4712d5edc5c69020116f85039e5c78e14
Author: Aleksandr Didenko <email address hidden>
Date: Tue Jan 20 14:05:46 2015 +0200

    Separate controller top-scope role

    * Move controller role out of legacy/cluster_ha manifests.
    * Update tasks.yaml accordingly.
    * Update existing classes to work properly with new task.
    * Removed 'role' from globals to avoid hiera loop.

    Related-bug: #1415811

    Partial-blueprint: fuel-library-modularization
    Change-Id: I88250421f7b607be4f839d21e5b540f1f2cd2748

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

Reviewed: https://review.openstack.org/151165
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=1c2ce9387d6ff37ec5ba1bdfe9b5ad7e58a561b5
Submitter: Jenkins
Branch: stable/5.0

commit 1c2ce9387d6ff37ec5ba1bdfe9b5ad7e58a561b5
Author: Aleksandr Didenko <email address hidden>
Date: Thu Jan 29 12:17:41 2015 +0200

    Fix scope for sqlalchemy parameters

    We should use proper scope for those variables there because there's
    double class encapsulation in cluster_ha.

    Change-Id: I914fc5783b0b980a4843ea0593b17edf7b1dacb8
    Partial-bug: #1415811

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

Reviewed: https://review.openstack.org/151162
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=b0b14c8610524a8d9ba5c70b51c8638de0e506da
Submitter: Jenkins
Branch: stable/5.1

commit b0b14c8610524a8d9ba5c70b51c8638de0e506da
Author: Aleksandr Didenko <email address hidden>
Date: Thu Jan 29 12:07:27 2015 +0200

    Fix scope for sqlalchemy parameters

    We should use proper scope for those variables there because there's
    double class encapsulation in cluster_ha.

    Change-Id: I130a66964f6fc4425d9f3b112802b358bcb61921
    Closes-bug: #1415811

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

Reviewed: https://review.openstack.org/151161
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=e2b985c351f1cc810a654d085745d7b298d796d0
Submitter: Jenkins
Branch: stable/6.0

commit e2b985c351f1cc810a654d085745d7b298d796d0
Author: Aleksandr Didenko <email address hidden>
Date: Thu Jan 29 12:07:27 2015 +0200

    Fix scope for sqlalchemy parameters

    We should use proper scope for those variables there because there's
    double class encapsulation in cluster_ha.

    Change-Id: I130a66964f6fc4425d9f3b112802b358bcb61921
    Closes-bug: #1415811

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.