Complete change 218359

Code Issues Proposed changes

Completes/fixes change 218359.
A portion of code was not tested by the unit test which
does not test whether the context is admin or not, which
is the intent since policy ought to be controlled through
the policy.json framework only.
Change-Id: I3b9a5b60ac47dbb0454c435112fdcb7312a36bdf

This commit is contained in:

Thomas Morin

2015年08月31日 11:18:23 +02:00

parent a841001c79

commit 03264af2bd

1 changed files with 0 additions and 5 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download Patch File Download Diff File Expand all files Collapse all files

5
networking_bgpvpn/neutron/db/bgpvpn_db.py

View File

@@ -148,11 +148,6 @@ class BGPVPNPluginDb(common_db_mixin.CommonDbMixin):

bgpvpn_connection_db = self._get_bgpvpn_connection(context, id)

LOG.debug("get_bgpvpn_connection called with fields = %s" % fields)

if not context.is_admin:

fields = self._get_user_readable_fields(fields)

LOG.debug("get_bgpvpn_connection called for user,"

"readable fields = %s" % fields)

return self._make_bgpvpn_connection_dict(bgpvpn_connection_db, fields)

def update_bgpvpn_connection(self, context, id, bgpvpn_connection):

Complete change 218359

5 networking_bgpvpn/neutron/db/bgpvpn_db.py Unescape Escape View File

5
networking_bgpvpn/neutron/db/bgpvpn_db.py

View File