]> review.fuel-infra Code Review - openstack-build/heat-build.git/commitdiff
Delete user creds from DB when no longer required
authorZane Bitter <zbitter@redhat.com>
Tue, 3 Jul 2012 10:32:02 +0000 (12:32 +0200)
committerZane Bitter <zbitter@redhat.com>
Wed, 4 Jul 2012 15:20:39 +0000 (17:20 +0200)
The user credentials are only for a particular stack and should be deleted
when that stack is deleted.

Fixes #162.

Change-Id: Id5d1f101df17ede1843e4f2fca51903c956667ca
Signed-off-by: Zane Bitter <zbitter@redhat.com>
heat/db/sqlalchemy/api.py
heat/db/sqlalchemy/models.py

index f68560ccc4bc48b9f75e21b682752a671c94db2a..dd800fe5be5831f51de167812ffb4d76b17b6c17 100644 (file)
@@ -166,9 +166,11 @@ def stack_delete(context, stack_id):
         session.delete(r)
 
     rt = s.raw_template
+    uc = s.user_creds
 
     session.delete(s)
     session.delete(rt)
+    session.delete(uc)
 
     session.flush()
 
index 337ca728b55ae4f7bc5116962cb70fb4ea4068e0..25f63b0891b733845a0483642ad9baf8859c32b2 100644 (file)
@@ -172,6 +172,8 @@ class UserCreds(BASE, HeatBase):
     aws_auth_url = Column(String)
     tenant_id = Column(String)
     aws_creds = Column(String)
+    stack = relationship(Stack,
+        backref=backref('user_creds'))
 
 
 class Event(BASE, HeatBase):