)]}'
{"doc/source/devref/index.rst":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"c6204e6610dca719262fd858006c731dfac972c4","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    effective_neutron"},{"line_number":43,"context_line":"    development.environment"},{"line_number":44,"context_line":"    contribute"},{"line_number":45,"context_line":"    infra"},{"line_number":46,"context_line":"    neutron_api"},{"line_number":47,"context_line":"    client_command_extensions"},{"line_number":48,"context_line":"    alembic_migrations"}],"source_content_type":"text/x-rst","patch_set":9,"id":"7a740942_cea12dab","line":45,"updated":"2015-12-05 00:57:43.000000000","message":"Doesn\u0027t seem like this is the place. Isn\u0027t this better located under policies?","commit_id":"2fd169c6b0cb27e7b58315afae58c196d0741364"},{"author":{"_account_id":11682,"name":"Ryan Moats","email":"rmoats@us.ibm.com","username":"regXboi"},"change_message_id":"183ce316010b693e754aadcbbdb77cbdb3585078","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    effective_neutron"},{"line_number":43,"context_line":"    development.environment"},{"line_number":44,"context_line":"    contribute"},{"line_number":45,"context_line":"    infra"},{"line_number":46,"context_line":"    neutron_api"},{"line_number":47,"context_line":"    client_command_extensions"},{"line_number":48,"context_line":"    alembic_migrations"}],"source_content_type":"text/x-rst","patch_set":9,"id":"7a740942_7085ef57","line":45,"in_reply_to":"7a740942_0e8d72d5","updated":"2015-12-08 20:19:08.000000000","message":"so moved","commit_id":"2fd169c6b0cb27e7b58315afae58c196d0741364"},{"author":{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},"change_message_id":"f80b3a100bf52d6f2db27f92525a69f5adb5960d","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    effective_neutron"},{"line_number":43,"context_line":"    development.environment"},{"line_number":44,"context_line":"    contribute"},{"line_number":45,"context_line":"    infra"},{"line_number":46,"context_line":"    neutron_api"},{"line_number":47,"context_line":"    client_command_extensions"},{"line_number":48,"context_line":"    alembic_migrations"}],"source_content_type":"text/x-rst","patch_set":9,"id":"7a740942_0e8d72d5","line":45,"in_reply_to":"7a740942_cea12dab","updated":"2015-12-08 18:40:01.000000000","message":"indeed.","commit_id":"2fd169c6b0cb27e7b58315afae58c196d0741364"}],"doc/source/devref/infra.rst":[{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"change_message_id":"8cbb4d9f7274b1795d426e22f6b051355d5502d4","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"* (may be out of scope of this devref, but don\u0027t want to forget): Today jobs"},{"line_number":41,"context_line":"  run current common test sets.  This may need to be revisited to"},{"line_number":42,"context_line":"  ensure that (for example) new tests for multinode don\u0027t break singlenode "},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"\"I\u0027d like to see us use this to add tips and tricks, and also to document the workflow of escalating things to infra, which should start with a sub-team of focused people from Neutron.\" (agreed, plan to add this, noting here as todo.)"},{"line_number":45,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"1a4dcd0f_f0e181fd","line":42,"updated":"2015-08-15 01:07:12.000000000","message":"I know is a WIP but just in case you have missed this one","commit_id":"2d2571dee52b7d6e19913834710032a702e52dd0"},{"author":{"_account_id":11682,"name":"Ryan Moats","email":"rmoats@us.ibm.com","username":"regXboi"},"change_message_id":"ab7b421c366e37c24657cd4339af79eb966cccdf","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"* (may be out of scope of this devref, but don\u0027t want to forget): Today jobs"},{"line_number":41,"context_line":"  run current common test sets.  This may need to be revisited to"},{"line_number":42,"context_line":"  ensure that (for example) new tests for multinode don\u0027t break singlenode "},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"\"I\u0027d like to see us use this to add tips and tricks, and also to document the workflow of escalating things to infra, which should start with a sub-team of focused people from Neutron.\" (agreed, plan to add this, noting here as todo.)"},{"line_number":45,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"1a4dcd0f_b1d531b8","line":42,"in_reply_to":"1a4dcd0f_f0e181fd","updated":"2015-08-16 03:40:56.000000000","message":"Done","commit_id":"2d2571dee52b7d6e19913834710032a702e52dd0"},{"author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"change_message_id":"541f17dab4e1b243a245e575153da0b814ea2dd3","unresolved":false,"context_lines":[{"line_number":12,"context_line":"      under the License."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"      Convention for heading levels in Neutron devref:"},{"line_number":16,"context_line":"      \u003d\u003d\u003d\u003d\u003d\u003d\u003d  Heading 0 (reserved for the title in a document)"},{"line_number":17,"context_line":"      -------  Heading 1"},{"line_number":18,"context_line":"      ~~~~~~~  Heading 2"}],"source_content_type":"text/x-rst","patch_set":8,"id":"fa1b9901_bc81cd91","line":15,"updated":"2015-08-17 23:33:37.000000000","message":"This all needs to be changed before this merges. Specifically, lines 15-22.","commit_id":"feab502f8585a09e389a85bb4ef857bdf6a1664e"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"d8276cadc3181ff02261e9c9590196617dfae333","unresolved":false,"context_lines":[{"line_number":12,"context_line":"      under the License."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"      Convention for heading levels in Neutron devref:"},{"line_number":16,"context_line":"      \u003d\u003d\u003d\u003d\u003d\u003d\u003d  Heading 0 (reserved for the title in a document)"},{"line_number":17,"context_line":"      -------  Heading 1"},{"line_number":18,"context_line":"      ~~~~~~~  Heading 2"}],"source_content_type":"text/x-rst","patch_set":8,"id":"fa1b9901_1683edcd","line":15,"in_reply_to":"fa1b9901_bc81cd91","updated":"2015-08-18 05:03:52.000000000","message":"I instigated this revolt via [1]. In RST markup, everything between \u0027..\u0027 (line 1) and the first line with zero indent (line 24) is a comment.\n\n[1] https://review.openstack.org/210769","commit_id":"feab502f8585a09e389a85bb4ef857bdf6a1664e"},{"author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"change_message_id":"541f17dab4e1b243a245e575153da0b814ea2dd3","unresolved":false,"context_lines":[{"line_number":24,"context_line":"Neutron Infra Processes"},{"line_number":25,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"With the proliferation of repositories and subcores under the neutron stadium,"},{"line_number":28,"context_line":"the load that neutron places on infra has increased by a large margin. In"},{"line_number":29,"context_line":"response to feedback, we are putting some policies in place to make sure"},{"line_number":30,"context_line":"that folks are first leveraging infra help from inside neutron, and second,"}],"source_content_type":"text/x-rst","patch_set":8,"id":"fa1b9901_7c4a5585","line":27,"updated":"2015-08-17 23:33:37.000000000","message":"\"subcores\"? Can we instead use \"sub-team core reviewers\"?","commit_id":"feab502f8585a09e389a85bb4ef857bdf6a1664e"},{"author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"change_message_id":"541f17dab4e1b243a245e575153da0b814ea2dd3","unresolved":false,"context_lines":[{"line_number":31,"context_line":"to coordinate larger efforts among a few new infra lieutenants, in order to"},{"line_number":32,"context_line":"minimize the cross-talk that can occur otherwise."},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"The lieutenants can be found here \u003cinsert link\u003e."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"While reading through this doc, please be aware that from now on, any requests"},{"line_number":37,"context_line":"for emergency intervention or ninja merges must go through the PTL or an infra"}],"source_content_type":"text/x-rst","patch_set":8,"id":"fa1b9901_9c364902","line":34,"updated":"2015-08-17 23:33:37.000000000","message":"The link is:\n\nhttp://docs.openstack.org/developer/neutron/policies/core-reviewers.html#core-review-hierarchy","commit_id":"feab502f8585a09e389a85bb4ef857bdf6a1664e"},{"author":{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},"change_message_id":"8d6f9a0f46f1d20d042bba0d5b45d44aafe7c06c","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Interactions with infra"},{"line_number":41,"context_line":"-----------------------"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"This policy is in no way intended to stop the free flow of commmunication,"},{"line_number":44,"context_line":"but rather to document best practices that will save everyone time. In that"},{"line_number":45,"context_line":"vein, normally all that\u0027s needed is submitting a well-documented and tested"}],"source_content_type":"text/x-rst","patch_set":8,"id":"fa1b9901_7de24cd7","line":42,"updated":"2015-08-18 02:50:34.000000000","message":"Need a section for common interactions that are not patches, like:\n\n- bootstrapping a gerrit group\n- requesting a CI node not be destroyed\n- another really common one that i\u0027ve forgotten, but i\u0027m leaving this line to drive myself mad.","commit_id":"feab502f8585a09e389a85bb4ef857bdf6a1664e"},{"author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"change_message_id":"541f17dab4e1b243a245e575153da0b814ea2dd3","unresolved":false,"context_lines":[{"line_number":52,"context_line":"* two"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"* three"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Escalation workflow"},{"line_number":57,"context_line":"-------------------"},{"line_number":58,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"fa1b9901_dc03610e","line":55,"updated":"2015-08-17 23:33:37.000000000","message":"Add a section like this:\n\n\"Please note that changes to Neutron\u0027s job setup or configuration will require a Neutron Infra Lieutenant or the PTL to +1 that change before infra will merge the change. In an extreme case, infra may go around this policy and merge such a change. But this is a rare event and the normal workflow is to require a +1 from a Neutron Infra Lieutenant or the PTL.\"","commit_id":"feab502f8585a09e389a85bb4ef857bdf6a1664e"},{"author":{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},"change_message_id":"8d6f9a0f46f1d20d042bba0d5b45d44aafe7c06c","unresolved":false,"context_lines":[{"line_number":61,"context_line":"\"I\u0027d like to see us use this to add tips and tricks, and also to document the workflow of escalating things to infra, which should start with a sub-team of focused people from Neutron.\" (agreed, plan to add this, noting here as todo.)"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"* todo, infra to not merge patch related to neutron without a +1 from "},{"line_number":64,"context_line":"  one of the infra lieutenants"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Repo creation"},{"line_number":67,"context_line":"-------------"}],"source_content_type":"text/x-rst","patch_set":8,"id":"fa1b9901_bdef84af","line":64,"updated":"2015-08-18 02:50:34.000000000","message":"nit: space","commit_id":"feab502f8585a09e389a85bb4ef857bdf6a1664e"},{"author":{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},"change_message_id":"b376ac5b5e63b05244d2e858dd670ce4f4befc8d","unresolved":false,"context_lines":[{"line_number":61,"context_line":"\"I\u0027d like to see us use this to add tips and tricks, and also to document the workflow of escalating things to infra, which should start with a sub-team of focused people from Neutron.\" (agreed, plan to add this, noting here as todo.)"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"* todo, infra to not merge patch related to neutron without a +1 from "},{"line_number":64,"context_line":"  one of the infra lieutenants"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Repo creation"},{"line_number":67,"context_line":"-------------"}],"source_content_type":"text/x-rst","patch_set":8,"id":"fa1b9901_609b7fca","line":64,"updated":"2015-08-18 03:09:52.000000000","message":"this needs some wording love (I think I did this text initially). The intent is not that infra is required to get approval from us, but rather we will volunteer to serve as the first line of screening neutron related reviews.","commit_id":"feab502f8585a09e389a85bb4ef857bdf6a1664e"},{"author":{"_account_id":8873,"name":"Assaf Muller","email":"amuller@redhat.com","username":"amuller"},"change_message_id":"d1d395138ef02f39703d61384b1d1e19fc4f8359","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":".. [#]_ http://docs.openstack.org/developer/neutron/policies/core-reviewers.html#core-review-hierarchy"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"While reading through this doc, please be aware that from now on, any requests"},{"line_number":40,"context_line":"for emergency intervention or ninja merges must go through the PTL or an infra"},{"line_number":41,"context_line":"lieutenant, and they will be rare."},{"line_number":42,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"7a740942_c5b343e8","line":39,"updated":"2015-12-05 16:05:11.000000000","message":"I\u0027m not following. I\u0027ve ninja-merged gate-failure fixes in the past. You\u0027re saying I was wrong in doing that?","commit_id":"2fd169c6b0cb27e7b58315afae58c196d0741364"},{"author":{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},"change_message_id":"8ce6d9bb49c53adf824429dd35c45efe60744afd","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":".. [#]_ http://docs.openstack.org/developer/neutron/policies/core-reviewers.html#core-review-hierarchy"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"While reading through this doc, please be aware that from now on, any requests"},{"line_number":40,"context_line":"for emergency intervention or ninja merges must go through the PTL or an infra"},{"line_number":41,"context_line":"lieutenant, and they will be rare."},{"line_number":42,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"7a740942_aed9be58","line":39,"in_reply_to":"7a740942_c5b343e8","updated":"2015-12-08 18:31:53.000000000","message":"A ninja merge is an infra merge without CI. What you\u0027re referring to is a rage merge (single approver), which is still a viable tool.","commit_id":"2fd169c6b0cb27e7b58315afae58c196d0741364"},{"author":{"_account_id":7776,"name":"Paul Carver","email":"pcarver@att.com","username":"pcarver"},"change_message_id":"0ee88117e733dc6285e0ae285abadadc5232c222","unresolved":false,"context_lines":[{"line_number":37,"context_line":".. [#]_ http://docs.openstack.org/developer/neutron/policies/core-reviewers.html#core-review-hierarchy"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"While reading through this doc, please be aware that from now on, any requests"},{"line_number":40,"context_line":"for emergency intervention or ninja merges must go through the PTL or an infra"},{"line_number":41,"context_line":"lieutenant, and they will be rare."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"Interactions with infra"}],"source_content_type":"text/x-rst","patch_set":9,"id":"7a740942_c32b46ea","line":40,"updated":"2015-12-05 00:09:29.000000000","message":"Not really even a nit, but should I know what a \"ninja merge\" is? A Google search didn\u0027t really turn up anything relevant.","commit_id":"2fd169c6b0cb27e7b58315afae58c196d0741364"},{"author":{"_account_id":11682,"name":"Ryan Moats","email":"rmoats@us.ibm.com","username":"regXboi"},"change_message_id":"183ce316010b693e754aadcbbdb77cbdb3585078","unresolved":false,"context_lines":[{"line_number":37,"context_line":".. [#]_ http://docs.openstack.org/developer/neutron/policies/core-reviewers.html#core-review-hierarchy"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"While reading through this doc, please be aware that from now on, any requests"},{"line_number":40,"context_line":"for emergency intervention or ninja merges must go through the PTL or an infra"},{"line_number":41,"context_line":"lieutenant, and they will be rare."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"Interactions with infra"}],"source_content_type":"text/x-rst","patch_set":9,"id":"7a740942_f0e0ff2a","line":40,"in_reply_to":"7a740942_217b9f29","updated":"2015-12-08 20:19:08.000000000","message":"the paragraph now has the difference between the two terms","commit_id":"2fd169c6b0cb27e7b58315afae58c196d0741364"},{"author":{"_account_id":8873,"name":"Assaf Muller","email":"amuller@redhat.com","username":"amuller"},"change_message_id":"e19bfb5b0c86ba3dc16c7c34e9aebb8656215e8c","unresolved":false,"context_lines":[{"line_number":37,"context_line":".. [#]_ http://docs.openstack.org/developer/neutron/policies/core-reviewers.html#core-review-hierarchy"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"While reading through this doc, please be aware that from now on, any requests"},{"line_number":40,"context_line":"for emergency intervention or ninja merges must go through the PTL or an infra"},{"line_number":41,"context_line":"lieutenant, and they will be rare."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"Interactions with infra"}],"source_content_type":"text/x-rst","patch_set":9,"id":"7a740942_217b9f29","line":40,"in_reply_to":"7a740942_6edf563a","updated":"2015-12-08 18:49:20.000000000","message":"Please do, I was not aware that there was a difference.","commit_id":"2fd169c6b0cb27e7b58315afae58c196d0741364"},{"author":{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},"change_message_id":"8ce6d9bb49c53adf824429dd35c45efe60744afd","unresolved":false,"context_lines":[{"line_number":37,"context_line":".. [#]_ http://docs.openstack.org/developer/neutron/policies/core-reviewers.html#core-review-hierarchy"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"While reading through this doc, please be aware that from now on, any requests"},{"line_number":40,"context_line":"for emergency intervention or ninja merges must go through the PTL or an infra"},{"line_number":41,"context_line":"lieutenant, and they will be rare."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"Interactions with infra"}],"source_content_type":"text/x-rst","patch_set":9,"id":"7a740942_6edf563a","line":40,"in_reply_to":"7a740942_c32b46ea","updated":"2015-12-08 18:31:53.000000000","message":"Likely we should define ninja vs rage merges.  :-)","commit_id":"2fd169c6b0cb27e7b58315afae58c196d0741364"}],"doc/source/policies/infra.rst":[{"author":{"_account_id":17711,"name":"Margaret Frances","email":"margaret_frances@cable.comcast.com","username":"margaret-frances"},"change_message_id":"7a989b4c392108f52ca9728f05521088a39141fc","unresolved":false,"context_lines":[{"line_number":25,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"With the proliferation of repositories and sub-team core reviewers under"},{"line_number":28,"context_line":"the Neutron stadium, the load that Neutron places on infra has increased"},{"line_number":29,"context_line":"by a large margin. In response to feedback, we are putting some policies"},{"line_number":30,"context_line":"in place to make sure that folks are first leveraging infra help from"},{"line_number":31,"context_line":"inside Neutron, and second, to coordinate larger efforts among a few new"}],"source_content_type":"text/x-rst","patch_set":10,"id":"9a68dd71_f7e74c10","line":28,"range":{"start_line":28,"start_character":53,"end_line":28,"end_character":58},"updated":"2016-01-20 21:54:56.000000000","message":"Could/should you define this term?","commit_id":"11b83eb4e7c1d4439ce0ce67001044b4a8743ecb"}]}