)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"4c33f8074bcc29d929c1fd12e4935f3c9705dfe0","unresolved":false,"context_lines":[{"line_number":14,"context_line":"The second is a yet unsolved bug around the use of autospec"},{"line_number":15,"context_line":"https://github.com/testing-cabal/mock/issues/264"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"The skip is because something has changed with mock.open that is"},{"line_number":18,"context_line":"causing the vhduils test to fail. We don\u0027t know why, but it\u0027s one test"},{"line_number":19,"context_line":"to skip."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"ba3cc151_5ca8b9db","line":17,"updated":"2015-07-10 14:35:10.000000000","message":"The only change I see in the mock NEWS file for mock_open since 1.1.0 is this:\n\nhttps://github.com/testing-cabal/mock/commit/a98a5badfca50222a48e82eb8e2df68f0656d701","commit_id":"4442979e71bb09ff9b71604c0dddd37c2bede04d"}],"nova/tests/unit/compute/test_compute_mgr.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"625e9cd897e3780e46fe46a21c7d0af2940462f8","unresolved":false,"context_lines":[{"line_number":1453,"context_line":"            self.assertEqual(2, updates[\u0027volume_size\u0027])"},{"line_number":1454,"context_line":"            return fake_bdm"},{"line_number":1455,"context_line":""},{"line_number":1456,"context_line":"        self.stubs.Set(self.compute.volume_api, \u0027roll_detaching\u0027,"},{"line_number":1457,"context_line":"                       fake_vol_api_roll_detaching)"},{"line_number":1458,"context_line":"        self.stubs.Set(self.compute.volume_api, \u0027get\u0027, fake_vol_get)"},{"line_number":1459,"context_line":"        self.stubs.Set(self.compute.volume_api, \u0027initialize_connection\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_8f012cb2","side":"PARENT","line":1456,"updated":"2015-07-10 12:52:39.000000000","message":"oops, some unrelated changes.","commit_id":"22c3e7437a2757c9b55a0245c63a1f349209c99a"}],"nova/tests/unit/virt/hyperv/test_vmops.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"85f42f51945ca0b9d2d01b704f2ffc237b79049d","unresolved":false,"context_lines":[{"line_number":974,"context_line":"            mock.sentinel.FAKE_PATH, mock.sentinel.FAKE_PATH_ARCHIVED)"},{"line_number":975,"context_line":""},{"line_number":976,"context_line":"        with mock.patch(\u0027nova.virt.hyperv.vmops.open\u0027,"},{"line_number":977,"context_line":"                        mock.mock_open(read_data\u003dself.FAKE_LOG * 2),"},{"line_number":978,"context_line":"                        create\u003dTrue):"},{"line_number":979,"context_line":"            instance_log \u003d self._vmops.get_console_output(mock_instance)"},{"line_number":980,"context_line":"            # get_vm_console_log_paths returns 2 paths."}],"source_content_type":"text/x-python","patch_set":5,"id":"ba3cc151_7ea81421","line":977,"updated":"2015-07-10 14:27:26.000000000","message":"I assume this is changed b/c of the assertion on line 981?","commit_id":"4442979e71bb09ff9b71604c0dddd37c2bede04d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"4c33f8074bcc29d929c1fd12e4935f3c9705dfe0","unresolved":false,"context_lines":[{"line_number":974,"context_line":"            mock.sentinel.FAKE_PATH, mock.sentinel.FAKE_PATH_ARCHIVED)"},{"line_number":975,"context_line":""},{"line_number":976,"context_line":"        with mock.patch(\u0027nova.virt.hyperv.vmops.open\u0027,"},{"line_number":977,"context_line":"                        mock.mock_open(read_data\u003dself.FAKE_LOG * 2),"},{"line_number":978,"context_line":"                        create\u003dTrue):"},{"line_number":979,"context_line":"            instance_log \u003d self._vmops.get_console_output(mock_instance)"},{"line_number":980,"context_line":"            # get_vm_console_log_paths returns 2 paths."}],"source_content_type":"text/x-python","patch_set":5,"id":"ba3cc151_7c357da6","line":977,"in_reply_to":"ba3cc151_7ea81421","updated":"2015-07-10 14:35:10.000000000","message":"https://docs.python.org/3/library/unittest.mock.html#mock-open","commit_id":"4442979e71bb09ff9b71604c0dddd37c2bede04d"}],"tox.ini":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"263149a7944f8561689f76fc6d9ceec79ed5d62d","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"[testenv:docs]"},{"line_number":110,"context_line":"commands \u003d"},{"line_number":111,"context_line":"  pip install \"sphinx!\u003d1.2.0,!\u003d1.3b1,\u003c1.3,\u003e\u003d1.1.2\""},{"line_number":112,"context_line":"  python setup.py build_sphinx"},{"line_number":113,"context_line":"  bash -c \u0027! find doc/ -type f -name *.json | xargs -t -n1 python -m json.tool 2\u003e\u00261 \u003e /dev/null | grep -B1 -v ^python\u0027"},{"line_number":114,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"ba3cc151_edf3209e","line":111,"updated":"2015-07-10 13:31:46.000000000","message":"Let\u0027s put a TODO comment in here to remove this once we figure out what\u0027s busted with pip/pbr/whatever so we\u0027re not stuck with this pin.\n\nAlso, not sure if the order matters here or not, but in g-r this is listed as:\n\nsphinx\u003e\u003d1.1.2,!\u003d1.2.0,!\u003d1.3b1,\u003c1.3","commit_id":"d3e866e7145756e09cccd36734ea33d56d92184c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5c4dbd528ce923d00f3eac684fdd3a097fc7d8c4","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"[testenv:docs]"},{"line_number":110,"context_line":"commands \u003d"},{"line_number":111,"context_line":"  # TODO(sdague) this is a brute force work around for pip not respecting test-requirements.txt"},{"line_number":112,"context_line":"  pip install \"sphinx!\u003d1.2.0,!\u003d1.3b1,\u003c1.3,\u003e\u003d1.1.2\""},{"line_number":113,"context_line":"  python setup.py build_sphinx"},{"line_number":114,"context_line":"  bash -c \u0027! find doc/ -type f -name *.json | xargs -t -n1 python -m json.tool 2\u003e\u00261 \u003e /dev/null | grep -B1 -v ^python\u0027"}],"source_content_type":"text/x-properties","patch_set":4,"id":"ba3cc151_08ae7281","line":111,"updated":"2015-07-10 13:45:43.000000000","message":"mock 1.1.0 is pulling in sphinx:\n\nhttps://github.com/testing-cabal/mock/blob/1.1.0/setup.cfg#L34\n\nAnd in nova\u0027s test-requirements.txt, mock comes before sphinx so we\u0027re not processing the pin in our test-requirements.txt, so we should re-order those in nova\u0027s test-requirements.txt.  I\u0027m trying that locally to confirm and if that works I\u0027ll update this change.","commit_id":"cf6cbd073a03e495081bbd56e10857dd2184a551"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ea5658237873eca8c08e0b93634faf44895b83f5","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"[testenv:docs]"},{"line_number":110,"context_line":"commands \u003d"},{"line_number":111,"context_line":"  # TODO(sdague) this is a brute force work around for pip not respecting test-requirements.txt"},{"line_number":112,"context_line":"  pip install \"sphinx!\u003d1.2.0,!\u003d1.3b1,\u003c1.3,\u003e\u003d1.1.2\""},{"line_number":113,"context_line":"  python setup.py build_sphinx"},{"line_number":114,"context_line":"  bash -c \u0027! find doc/ -type f -name *.json | xargs -t -n1 python -m json.tool 2\u003e\u00261 \u003e /dev/null | grep -B1 -v ^python\u0027"}],"source_content_type":"text/x-properties","patch_set":4,"id":"ba3cc151_63771fa0","line":111,"in_reply_to":"ba3cc151_08ae7281","updated":"2015-07-10 13:55:33.000000000","message":"Re-ordering didn\u0027t help:\n\nhttp://paste.openstack.org/show/362003/\n\nSomething else must be pulling in uncapped sphinx.","commit_id":"cf6cbd073a03e495081bbd56e10857dd2184a551"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"3b2cf1ab974638a5fd2822dd1ee2ad1d8197e5f4","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"[testenv:docs]"},{"line_number":110,"context_line":"commands \u003d"},{"line_number":111,"context_line":"  # TODO(sdague) this is a brute force work around for pip not respecting test-requirements.txt"},{"line_number":112,"context_line":"  pip install \"sphinx!\u003d1.2.0,!\u003d1.3b1,\u003c1.3,\u003e\u003d1.1.2\""},{"line_number":113,"context_line":"  python setup.py build_sphinx"},{"line_number":114,"context_line":"  bash -c \u0027! find doc/ -type f -name *.json | xargs -t -n1 python -m json.tool 2\u003e\u00261 \u003e /dev/null | grep -B1 -v ^python\u0027"}],"source_content_type":"text/x-properties","patch_set":4,"id":"ba3cc151_83c26333","line":111,"in_reply_to":"ba3cc151_63771fa0","updated":"2015-07-10 14:02:10.000000000","message":"mordred pointed out that it\u0027s oslo.versionedobjects that\u0027s pull in mock via it\u0027s requirements.txt which is then pulling in uncapped sphinx, see:\n\nhttps://review.openstack.org/#/c/200544/","commit_id":"cf6cbd073a03e495081bbd56e10857dd2184a551"}]}