)]}'
{"README.rst":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"2c1ff5fa517fd7988aa264e2dba5ef52ce6c2ec5","unresolved":false,"context_lines":[{"line_number":91,"context_line":"There are certain API operations which will use the `elastic-recheck`_ project"},{"line_number":92,"context_line":"to pull in additional information about failures that occur during a run."},{"line_number":93,"context_line":"However, since elastic-recheck is not widely deployed this is an optional"},{"line_number":94,"context_line":"feature and is only enabled if elastic-recheck is installed. (and importable"},{"line_number":95,"context_line":"by the API server) Also note that elastic-recheck is not published on pypi and"},{"line_number":96,"context_line":"must be manually installed via git. Additionally, after you install"},{"line_number":97,"context_line":"elastic-recheck you also need to configure the location of the queries by"}],"source_content_type":"text/x-rst","patch_set":14,"id":"bab6814e_6e5645cf","line":94,"range":{"start_line":94,"start_character":59,"end_line":94,"end_character":60},"updated":"2016-05-20 18:34:21.000000000","message":"NIT: \".\" should be after the \")\"","commit_id":"1b1b3af0549890f8cfa71817ed51d00f1cfb7fe8"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"2c1ff5fa517fd7988aa264e2dba5ef52ce6c2ec5","unresolved":false,"context_lines":[{"line_number":109,"context_line":"^^^^^^^^^^^^^^^^^^^^^"},{"line_number":110,"context_line":"Since the introduction of elastic recheck querying dogpile.cache has been"},{"line_number":111,"context_line":"used to cache any request that hits elasticsearch. This is because the"},{"line_number":112,"context_line":"query times for using elastic-recheck are quite variable and often very slow."},{"line_number":113,"context_line":"(at least for talking to openstack-infra\u0027s elasticsearch) To enable reasonable"},{"line_number":114,"context_line":"interactive response times we cache the api response from requests using"},{"line_number":115,"context_line":"elasticsearch data. Note, that this caching is enabled regardless of whether"}],"source_content_type":"text/x-rst","patch_set":14,"id":"bab6814e_2ed55d1b","line":112,"range":{"start_line":112,"start_character":76,"end_line":112,"end_character":77},"updated":"2016-05-20 18:34:21.000000000","message":"ditto","commit_id":"1b1b3af0549890f8cfa71817ed51d00f1cfb7fe8"}],"app/js/controllers/home.js":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"126ce6797784b84f1b54aa69d219b914219eb493","unresolved":false,"context_lines":[{"line_number":139,"context_line":"          vm.recentRuns[test.link].bugs \u003d [];"},{"line_number":140,"context_line":"          vm.recentRuns[test.link].fails \u003d [];"},{"line_number":141,"context_line":"        }"},{"line_number":142,"context_line":"        vm.recentRuns[test.link].bugs.push(vm.recentTests.bugs[test.uuid]);"},{"line_number":143,"context_line":"        vm.recentRuns[test.link].bugs \u003d unique(vm.recentRuns[test.link].bugs);"},{"line_number":144,"context_line":"        vm.recentRuns[test.link].fails.push(test);"},{"line_number":145,"context_line":"      });"}],"source_content_type":"text/javascript","patch_set":11,"id":"bab6814e_bf176353","line":142,"updated":"2016-05-18 05:59:42.000000000","message":"This push is making length\u003e0 arrays. So, \u0027Likely bugs: \u0027 without bug numbers shows up at the home page.","commit_id":"2ca0426ab5a39ff7ec3e19201eb1c7816e1ba8ce"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"ab919441db3a23f67b6cc17114444d925cb465fa","unresolved":false,"context_lines":[{"line_number":139,"context_line":"          vm.recentRuns[test.link].bugs \u003d [];"},{"line_number":140,"context_line":"          vm.recentRuns[test.link].fails \u003d [];"},{"line_number":141,"context_line":"        }"},{"line_number":142,"context_line":"        vm.recentRuns[test.link].bugs.push(vm.recentTests.bugs[test.uuid]);"},{"line_number":143,"context_line":"        vm.recentRuns[test.link].bugs \u003d unique(vm.recentRuns[test.link].bugs);"},{"line_number":144,"context_line":"        vm.recentRuns[test.link].fails.push(test);"},{"line_number":145,"context_line":"      });"}],"source_content_type":"text/javascript","patch_set":11,"id":"bab6814e_4f2b0dec","line":142,"in_reply_to":"bab6814e_bf176353","updated":"2016-05-18 13:42:48.000000000","message":"Done","commit_id":"2ca0426ab5a39ff7ec3e19201eb1c7816e1ba8ce"},{"author":{"_account_id":5623,"name":"David Lyle","email":"dklyle0@gmail.com","username":"david-lyle"},"change_message_id":"4c492f57d8bbb84506bd0a59d9740f65b649eb64","unresolved":false,"context_lines":[{"line_number":143,"context_line":"          vm.recentRuns[test.link].bugs.push(vm.recentTests.bugs[test.uuid]);"},{"line_number":144,"context_line":"          vm.recentRuns[test.link].bugs \u003d unique(vm.recentRuns[test.link].bugs);"},{"line_number":145,"context_line":"          vm.recentRuns[test.link].fails.push(test);"},{"line_number":146,"context_line":"        }"},{"line_number":147,"context_line":"      });"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"      for (var link in vm.recentRuns) {"}],"source_content_type":"text/javascript","patch_set":12,"id":"bab6814e_877e6ae2","line":146,"updated":"2016-05-18 18:25:15.000000000","message":"this should be up a line, you still want to push fails.","commit_id":"041937797c20a46ef49a1735ad624c5c810e5535"},{"author":{"_account_id":5623,"name":"David Lyle","email":"dklyle0@gmail.com","username":"david-lyle"},"change_message_id":"a78c27763df6e985e323ab3f129382f636c925d3","unresolved":false,"context_lines":[{"line_number":143,"context_line":"          vm.recentRuns[test.link].bugs.push(vm.recentTests.bugs[test.uuid]);"},{"line_number":144,"context_line":"          vm.recentRuns[test.link].bugs \u003d unique(vm.recentRuns[test.link].bugs);"},{"line_number":145,"context_line":"          vm.recentRuns[test.link].fails.push(test);"},{"line_number":146,"context_line":"        }"},{"line_number":147,"context_line":"      });"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"      for (var link in vm.recentRuns) {"}],"source_content_type":"text/javascript","patch_set":12,"id":"bab6814e_12cbd654","line":146,"in_reply_to":"bab6814e_877e6ae2","updated":"2016-05-18 18:25:52.000000000","message":"Done","commit_id":"041937797c20a46ef49a1735ad624c5c810e5535"}],"app/views/home.html":[{"author":{"_account_id":17001,"name":"Tim Buckley","email":"timothy.jas.buckley@hpe.com","username":"timothy.jas.buckley"},"change_message_id":"c1229b5452bb60de71757b09adb2ba93d0a13890","unresolved":false,"context_lines":[{"line_number":68,"context_line":"                        \u003cdiv ng-repeat\u003d\"(key, value) in home.recentRuns\"\u003e"},{"line_number":69,"context_line":"                            \u003cuib-accordion-group"},{"line_number":70,"context_line":"                             template-url\u003d\"templates/accordion-group-run.html\""},{"line_number":71,"context_line":"                             heading\u003d\"{{ key }} {{ value.bugs }}\""},{"line_number":72,"context_line":"                             is-open\u003d\"false\"\u003e"},{"line_number":73,"context_line":"                                \u003ctable table-sort data\u003d\"value.fails\""},{"line_number":74,"context_line":"                                       class\u003d\"table table-hover default-cols\"\u003e"}],"source_content_type":"text/html","patch_set":11,"id":"dab17558_ccea900e","line":71,"range":{"start_line":71,"start_character":48,"end_line":71,"end_character":64},"updated":"2016-05-17 21:00:14.000000000","message":"So, the reason this is including the \"likely bugs\" text in the URL is that the template assumes whatever is passed as heading\u003d\"...\" is the log URL. If we\u0027re going to use this template, we\u0027ll need to display the bugs text elsewhere.\n\nThat said, IMO the accordion group / nested panel used here is fairly ugly anyway, I\u0027d be in favor of a redesign that could also accommodate E-R info better.","commit_id":"2ca0426ab5a39ff7ec3e19201eb1c7816e1ba8ce"}],"etc/openstack-health-api.conf":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"2194d780c6ee94671bab980e54b85dddc5b16e5f","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  build_zuul_url"},{"line_number":12,"context_line":"host \u003d 127.0.0.1"},{"line_number":13,"context_line":"port \u003d 5000"},{"line_number":14,"context_line":"query_dir \u003d /home/computertreker/git/elastic-recheck/queries"}],"source_content_type":"text/plain","patch_set":13,"id":"bab6814e_5df07c67","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":32},"updated":"2016-05-20 05:53:26.000000000","message":"heh, this is your home directory :)","commit_id":"d7c3f31670d5a845de03e1a4b3143e2c3582f661"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"b839f15baf2c8fcd669639076ca715088a88031f","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  build_zuul_url"},{"line_number":12,"context_line":"host \u003d 127.0.0.1"},{"line_number":13,"context_line":"port \u003d 5000"},{"line_number":14,"context_line":"query_dir \u003d /home/computertreker/git/elastic-recheck/queries"}],"source_content_type":"text/plain","patch_set":13,"id":"bab6814e_de58d08c","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":32},"in_reply_to":"bab6814e_5df07c67","updated":"2016-05-20 13:21:28.000000000","message":"You\u0027re right, this was a mistake we really don\u0027t need a default in the sample. I was just sloppy with git commit -a --amend at some point","commit_id":"d7c3f31670d5a845de03e1a4b3143e2c3582f661"}],"test/unit/controllers/home_spec.js":[{"author":{"_account_id":17001,"name":"Tim Buckley","email":"timothy.jas.buckley@hpe.com","username":"timothy.jas.buckley"},"change_message_id":"52294bdba9c5f200a7a57f138ee0436e7097957a","unresolved":false,"context_lines":[{"line_number":14,"context_line":"      }"},{"line_number":15,"context_line":"    }"},{"line_number":16,"context_line":"  };"},{"line_number":17,"context_line":"  var mockRecentFailed \u003d {"},{"line_number":18,"context_line":"    \u0027bugs\u0027: {},"},{"line_number":19,"context_line":"    \u0027test_runs\u0027: ["},{"line_number":20,"context_line":"      {"}],"source_content_type":"text/javascript","patch_set":11,"id":"dab17558_cf756298","line":17,"range":{"start_line":17,"start_character":2,"end_line":17,"end_character":22},"updated":"2016-05-17 21:25:02.000000000","message":"Looks like this is failing unit tests because this mocked callback should be wrapped inside a `data` field. That is, it should be:\n\n    var mockRecentFailed \u003d {\n        data: { ... }\n    }\n\nThis probably should be part of the mocked service implementation (e.g. `callback({ data: mockRecentFailed })` or so) since the `data` field is part of angular\u0027s $http response and not the actual server response, but the other two mocks do it like this already.\n\n(It\u0027s worth noting, this has technically been broken for a while, but until now no code ever actually touched the response in the home controller)","commit_id":"2ca0426ab5a39ff7ec3e19201eb1c7816e1ba8ce"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"ab919441db3a23f67b6cc17114444d925cb465fa","unresolved":false,"context_lines":[{"line_number":14,"context_line":"      }"},{"line_number":15,"context_line":"    }"},{"line_number":16,"context_line":"  };"},{"line_number":17,"context_line":"  var mockRecentFailed \u003d {"},{"line_number":18,"context_line":"    \u0027bugs\u0027: {},"},{"line_number":19,"context_line":"    \u0027test_runs\u0027: ["},{"line_number":20,"context_line":"      {"}],"source_content_type":"text/javascript","patch_set":11,"id":"bab6814e_0fbaa523","line":17,"range":{"start_line":17,"start_character":2,"end_line":17,"end_character":22},"in_reply_to":"dab17558_cf756298","updated":"2016-05-18 13:42:48.000000000","message":"Done","commit_id":"2ca0426ab5a39ff7ec3e19201eb1c7816e1ba8ce"}]}