From 305fa9f64081b73c51718935fde5d4b052ae4d01 Mon Sep 17 00:00:00 2001 From: "James E. Blair" Date: Mon, 3 Dec 2012 08:51:25 -0800 Subject: [PATCH] Flush gerrit caches after the db commit. Change-Id: I5eeffdafdcb0b7fc1d09885fa14088a58b9f3cac Reviewed-on: https://review.openstack.org/17383 Reviewed-by: Jeremy Stanley Approved: Clark Boylan Reviewed-by: Clark Boylan Tested-by: Jenkins --- modules/launchpad_sync/files/update_gerrit_users.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/modules/launchpad_sync/files/update_gerrit_users.py b/modules/launchpad_sync/files/update_gerrit_users.py index 76636ae410..6e9e60b5d8 100755 --- a/modules/launchpad_sync/files/update_gerrit_users.py +++ b/modules/launchpad_sync/files/update_gerrit_users.py @@ -724,10 +724,10 @@ if not options.prep_only: log.info('Syncing') sync.sync() +conn.commit() + if not options.skip_dump: os.system("ssh -i %s -p29418 %s@localhost gerrit flush-caches" % (GERRIT_SSH_KEY, GERRIT_USER)) -conn.commit() - log.info('Gerrit user sync stop ' + str(datetime.now()))