)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5bbae9cfeb76429001d1867a27123a0bba83caa7","unresolved":false,"context_lines":[{"line_number":14,"context_line":"-----------------------------------------------"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Related-Bug: 1387543"},{"line_number":17,"context_line":"Change-Id: Ica74dd6c35e6bd17eac285e2dc2900c1ff23073f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa1b9901_c88962cc","line":17,"updated":"2015-08-17 20:24:38.000000000","message":"I\u0027d prefer that you use the same change ID that the other changes to oslo-incubator and oslo.concurrency had for this:\n\nI22b2d7bde8797276f7670bc289d915dab5122481\n\nThat way we can make sure all of the related projects and branches are updated as necessary via the gerrit dashboard:\n\nhttps://review.openstack.org/#/q/I22b2d7bde8797276f7670bc289d915dab5122481,n,z","commit_id":"219ba3434e15ce49bde73f4c3057810f90706b59"}],"nova/openstack/common/__init__.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"562149a229b6f649c0150e8054004694684b6998","unresolved":false,"context_lines":[{"line_number":14,"context_line":"import six"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"six.add_move(six.MovedModule(\u0027mox\u0027, \u0027mox\u0027, \u0027mox3.mox\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa1b9901_03cadba1","side":"PARENT","line":17,"updated":"2015-08-17 20:34:02.000000000","message":"This is change I59d5799283233f8411044ddb15c8abfc8850014c which has nothing to do with what you\u0027re syncing from what I can tell, so why is it part of this?","commit_id":"cc6c50ee23f336d08e4fe833070943c0025a205d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6fd2e8dff0cc7e2493cd320289f7466af596518b","unresolved":false,"context_lines":[{"line_number":14,"context_line":"import six"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"six.add_move(six.MovedModule(\u0027mox\u0027, \u0027mox\u0027, \u0027mox3.mox\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa1b9901_e3cb2771","side":"PARENT","line":17,"in_reply_to":"fa1b9901_03cadba1","updated":"2015-08-17 20:38:04.000000000","message":"I guess because the oslo-incubator update script pulls this in as a dependency, I get the same when doing this:\n\nmriedem@ubuntu:~/git/oslo-incubator$ python update.py --dest-dir /home/mriedem/git/nova --base nova --module processutils\nCopying processutils under the nova module in /home/mriedem/git/nova\n\nCopying gettextutils under the nova module in /home/mriedem/git/nova\n\nCopying strutils under the nova module in /home/mriedem/git/nova\n\n\nmriedem@ubuntu:~/git/nova$ git status\nOn branch stable/juno\nYour branch is up-to-date with \u0027origin/stable/juno\u0027.\n\nChanges not staged for commit:\n  (use \"git add \u003cfile\u003e...\" to update what will be committed)\n  (use \"git checkout -- \u003cfile\u003e...\" to discard changes in working directory)\n\n\tmodified:   nova/openstack/common/__init__.py\n\tmodified:   nova/openstack/common/processutils.py\n\nno changes added to commit (use \"git add\" and/or \"git commit -a\")\n\n\nSo that\u0027s fine, but we have to list all of the changes in the commit message, not just the two that we are targeting.","commit_id":"cc6c50ee23f336d08e4fe833070943c0025a205d"}],"nova/openstack/common/processutils.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"562149a229b6f649c0150e8054004694684b6998","unresolved":false,"context_lines":[{"line_number":18,"context_line":"\"\"\""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import errno"},{"line_number":21,"context_line":"import logging"},{"line_number":22,"context_line":"import multiprocessing"},{"line_number":23,"context_line":"import os"},{"line_number":24,"context_line":"import random"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa1b9901_23811f4b","line":21,"updated":"2015-08-17 20:34:02.000000000","message":"This is change I245750f9f397a231891514f6ed1ea9626513cbbb which isn\u0027t in your list of changes in the commit message.  It also looks like that was already half synced into nova stable/juno already via change Ie0caf32469126dd9feb44867adf27acb6e383958.\n\nWe should have full details about the changes here in the commit message for clarity to know that we\u0027re not missing something in the sync.","commit_id":"219ba3434e15ce49bde73f4c3057810f90706b59"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"562149a229b6f649c0150e8054004694684b6998","unresolved":false,"context_lines":[{"line_number":153,"context_line":"        check_exit_code \u003d [check_exit_code]"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"    if kwargs:"},{"line_number":156,"context_line":"        raise UnknownArgumentError(_(\u0027Got unknown keyword args: %r\u0027) % kwargs)"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"    if run_as_root and hasattr(os, \u0027geteuid\u0027) and os.geteuid() !\u003d 0:"},{"line_number":159,"context_line":"        if not root_helper:"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa1b9901_03573b03","line":156,"updated":"2015-08-17 20:34:02.000000000","message":"This is change Ia5bb4180dbf0df92bfb986c2ddf8327d5bc08e7c which isn\u0027t in your list of changes in the commit message.  All changes in the sync should be called out in the commit message.","commit_id":"219ba3434e15ce49bde73f4c3057810f90706b59"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"dfa8056d36548adb260a71388f25b4bd2703df0b","unresolved":false,"context_lines":[{"line_number":30,"context_line":"import six"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"from nova.openstack.common.gettextutils import _"},{"line_number":33,"context_line":"from nova.openstack.common import strutils"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"LOG \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa1b9901_9ffff4a9","line":33,"updated":"2015-08-18 16:15:56.000000000","message":"As previously noted, this was sort of a half sync from commit 853d8f9897f8563851441108a9be26b10908c076 previously, so that\u0027s why it\u0027s not changed as part of the sync of change I245750f .","commit_id":"bf23643e36c8764b4bd532546a2cc04385fe0cff"}]}