Support separate cache values for each tenant

Bug #1478525 reported by Kairat Kushaev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Undecided
Kairat Kushaev

Bug Description

Test case to reproduce:
0. create 2 tenants and users for them, configure memcache for heat.
1. create a flavor for tenant1
2. create a stack for tenant1 with server that refers to flavor from step 1
3. create a stack for tenant2 with server that refers to flavor from step 1 (but this flavor was not created for tenant2).
AR: validation passes
ER: validation should fail.
The reason is because current cache key doen't have tenant_id in it.

Changed in heat:
assignee: nobody → Kairat Kushaev (kkushaev)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

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

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

Reviewed: https://review.openstack.org/206006
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=386bf2a6bccb682eca23fe0801fef87102d5d58d
Submitter: Jenkins
Branch: master

commit 386bf2a6bccb682eca23fe0801fef87102d5d58d
Author: kairat_kushaev <email address hidden>
Date: Mon Jul 27 13:49:28 2015 +0300

    Add tenant_id to heat cache key

    Currently heat caches validation results with the following key:
    <service name> + <value>.
    The patch changes this to
    <service name> + <tenant_id> + <value>.
    It allows names to be safely used and not be confused with
    values received from other tenants.

    Change-Id: Idd3fcf6afb3990be7df25fa1c5ef0c58222a5967
    Closes-bug: #1478525

Changed in heat:
status: In Progress → Fix Committed
Changed in heat:
milestone: none → liberty-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in heat:
milestone: liberty-3 → 5.0.0
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.