)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"c872651fcf6771b1fc55f6bf315dde589024720b","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Do we really need to test the other six modes I\u0027m removing"},{"line_number":14,"context_line":"here?  Dual-NIC/dual-stack tests everything, and if it\u0027s"},{"line_number":15,"context_line":"not working then the single-NIC case doesn\u0027t matter."},{"line_number":16,"context_line":"Is not testing that base case actually going to miss something?"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Closes-bug: #1489962"},{"line_number":19,"context_line":"Change-Id: I378710382fb01c5afaf10c5e892600244be02449"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7a740942_7f41b70a","line":16,"updated":"2015-12-10 22:20:23.000000000","message":"This is a classic explosion of test combinations. It does sound expensive to validate data path sanity this way involving actual vm\u0027s etc. For this depth of coverage we should move these tests into the fullstack framework, which is a lot cheaper to run.\n\nThat said, I would be surprised to see that a regression would sneak in due to the reduced coverage, so I am not sure the tests being removed are worth running periodically.","commit_id":"5c8b8b2f01b4b39a0ebad96ec3e5fe1fd11526b0"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"472f12f7a20a54c5443ccd9d2485fb4330651622","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Do we really need to test the other six modes I\u0027m removing"},{"line_number":14,"context_line":"here?  Dual-NIC/dual-stack tests everything, and if it\u0027s"},{"line_number":15,"context_line":"not working then the single-NIC case doesn\u0027t matter."},{"line_number":16,"context_line":"Is not testing that base case actually going to miss something?"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Closes-bug: #1489962"},{"line_number":19,"context_line":"Change-Id: I378710382fb01c5afaf10c5e892600244be02449"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5a710552_4779dffb","line":16,"in_reply_to":"7a740942_7f41b70a","updated":"2015-12-15 03:29:21.000000000","message":"I\u0027m fine either removing these tests (and moving to fullstack) or marking them \"slow\", but I tend to lean towards the latter, maybe until the end of Mitaka, since thinking they\u0027re not finding bugs and knowing they\u0027re not are different animals.\n\nMaybe someone can vote other than a \"0\" on that :)","commit_id":"5c8b8b2f01b4b39a0ebad96ec3e5fe1fd11526b0"}]}