openstack
/
trove
Code Issues Proposed changes

Logging audit for trove/db module 

Adjust log messages to conform to logging standards.
Cleanup some messages that were unclear.

Change-Id: I0a67acd6aa7bef8cb9adb32cf0f821ba24a4cb92
Partial-Bug: #1324206
This commit is contained in:
Amrith Kumar 2014-07-01 13:22:37 -04:00
parent 76c2e6a887
commit 1fc852b08c
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
trove/db/models.py
View File

@ -95,15 +95,19 @@ class DatabaseModelBase(models.ModelBase):
model = cls.get_by(**conditions)
if model is None:
raise exception.ModelNotFoundError(_("%s Not Found") %
cls.__name__)
raise exception.ModelNotFoundError(_("%(s_name)s Not Found") %
{"s_name": cls.__name__})
if ((context and not context.is_admin and hasattr(model, 'tenant_id')
and model.tenant_id != context.tenant)):
LOG.error("Tenant %s tried to access %s, owned by %s."
% (context.tenant, cls.__name__, model.tenant_id))
raise exception.ModelNotFoundError(_("%s Not Found") %
cls.__name__)
msg = _("Tenant %(s_tenant)s tried to access "
"%(s_name)s, owned by %(s_owner)s.")
LOG.error(msg % (
{"s_tenant": context.tenant, "s_name": cls.__name__,
"s_owner": model.tenant_id}))
raise exception.ModelNotFoundError(
_("Tenant %(s_tenant)s cannot access %(s_name)s") % (
{"s_tenant": context.tenant, "s_name": cls.__name__}))
return model