Skip to content

Commit 028de8e

Browse files
committed
Fix broken metadata function calls
The patch [1] cleanup so that the database connection was re-used but forgot to update all function calls causing functionality to be broken. [1] https://review.opendev.org/c/openstack/glance/+/889380 Change-Id: Ic56d897c441004d710ec89a536ad57b2fe480a41 Closes-Bug: #2094816 (cherry picked from commit e791ed5) (cherry picked from commit e74b91d)
1 parent 6925177 commit 028de8e

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

glance/db/sqlalchemy/metadata.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -280,7 +280,7 @@ def _populate_metadata(meta, conn, metadata_path=None, merge=False,
280280

281281
if db_namespace and overwrite:
282282
LOG.info(_LI("Overwriting namespace %s"), values['namespace'])
283-
_clear_namespace_metadata(meta, db_namespace[0])
283+
_clear_namespace_metadata(meta, conn, db_namespace[0])
284284
db_namespace = None
285285

286286
if not db_namespace:
@@ -303,7 +303,7 @@ def _populate_metadata(meta, conn, metadata_path=None, merge=False,
303303
continue
304304
elif prefer_new:
305305
values.update({'updated_at': timeutils.utcnow()})
306-
_update_data_in_db(namespaces_table, values,
306+
_update_data_in_db(conn, namespaces_table, values,
307307
namespaces_table.c.id, db_namespace[0])
308308

309309
namespace_id = db_namespace[0]
@@ -321,7 +321,7 @@ def _populate_metadata(meta, conn, metadata_path=None, merge=False,
321321
meta, conn, resource_type['name'])
322322
elif prefer_new:
323323
val = {'updated_at': timeutils.utcnow()}
324-
_update_data_in_db(resource_types_table, val,
324+
_update_data_in_db(conn, resource_types_table, val,
325325
resource_types_table.c.id, rt_id)
326326

327327
values = {
@@ -338,7 +338,7 @@ def _populate_metadata(meta, conn, metadata_path=None, merge=False,
338338
_insert_data_to_db(conn, namespace_rt_table, values)
339339
elif prefer_new:
340340
values.update({'updated_at': timeutils.utcnow()})
341-
_update_rt_association(namespace_rt_table, values,
341+
_update_rt_association(conn, namespace_rt_table, values,
342342
rt_id, namespace_id)
343343

344344
for name, schema in metadata.get('properties', {}).items():
@@ -355,7 +355,7 @@ def _populate_metadata(meta, conn, metadata_path=None, merge=False,
355355
_insert_data_to_db(conn, properties_table, values)
356356
elif prefer_new:
357357
values.update({'updated_at': timeutils.utcnow()})
358-
_update_data_in_db(properties_table, values,
358+
_update_data_in_db(conn, properties_table, values,
359359
properties_table.c.id, property_id)
360360

361361
for object in metadata.get('objects', []):
@@ -373,7 +373,7 @@ def _populate_metadata(meta, conn, metadata_path=None, merge=False,
373373
_insert_data_to_db(conn, objects_table, values)
374374
elif prefer_new:
375375
values.update({'updated_at': timeutils.utcnow()})
376-
_update_data_in_db(objects_table, values,
376+
_update_data_in_db(conn, objects_table, values,
377377
objects_table.c.id, object_id)
378378

379379
for tag in metadata.get('tags', []):
@@ -388,7 +388,7 @@ def _populate_metadata(meta, conn, metadata_path=None, merge=False,
388388
_insert_data_to_db(conn, tags_table, values)
389389
elif prefer_new:
390390
values.update({'updated_at': timeutils.utcnow()})
391-
_update_data_in_db(tags_table, values,
391+
_update_data_in_db(conn, tags_table, values,
392392
tags_table.c.id, tag_id)
393393

394394
LOG.info(_LI("File %s loaded to database."), file)

0 commit comments

Comments
 (0)