)]}'
{"template.rst":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"6fb3e40ee7fac2f05abe3aaed756a906c52381fa","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Include the URL of your launchpad blueprint:"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"https://blueprints.launchpad.net/tempest/+spec/example"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Introduction paragraph -- why are we doing anything?"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F%2B5Vk%3D","line":30,"updated":"2014-03-22 00:59:46.000000000","message":"Do we really need this? If we link it in the commit message, the bp itself has a link to the spec, and the names match I\u0027m not sure this is actually needed. That being said I don\u0027t think it really matters.","commit_id":"345ed6f552096ed539548e0754749ec07e436034"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"b0551a5a2ed8da90d02f8a5955ff4eba6a299a96","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Include the URL of your launchpad blueprint:"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"https://blueprints.launchpad.net/tempest/+spec/example"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Introduction paragraph -- why are we doing anything?"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F%2B0KQ%3D","line":30,"in_reply_to":"AAAAWH%2F%2B5Vk%3D","updated":"2014-03-24 04:42:01.000000000","message":"I think its handy for those who find the specification once its been merged. With the names matching its no biggie, though I think convenient. Am ok with removing it if others think its redundant.","commit_id":"345ed6f552096ed539548e0754749ec07e436034"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"6fb3e40ee7fac2f05abe3aaed756a906c52381fa","unresolved":false,"context_lines":[{"line_number":50,"context_line":"If the change is designed to test other OpenStack projects then list which ones"},{"line_number":51,"context_line":"it is targeted at."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"Alternatives"},{"line_number":54,"context_line":"------------"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"What other ways could we do this thing? Why aren\u0027t we using those? This doesn\u0027t"}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F%2B4e8%3D","line":53,"updated":"2014-03-22 00:59:46.000000000","message":"I\u0027m not sure this always applies should we put it in the template because of that, or somehow mark it as optional?","commit_id":"345ed6f552096ed539548e0754749ec07e436034"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"b0551a5a2ed8da90d02f8a5955ff4eba6a299a96","unresolved":false,"context_lines":[{"line_number":50,"context_line":"If the change is designed to test other OpenStack projects then list which ones"},{"line_number":51,"context_line":"it is targeted at."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"Alternatives"},{"line_number":54,"context_line":"------------"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"What other ways could we do this thing? Why aren\u0027t we using those? This doesn\u0027t"}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F%2B0KI%3D","line":53,"in_reply_to":"AAAAWH%2F%2B4e8%3D","updated":"2014-03-24 04:42:01.000000000","message":"Yea I agree it should be optional. If someone is just adding tests for say Neutron, we don\u0027t need it. But if its something which requires or is infrastructure improvements then I think its relevant. I\u0027ll try some wording around it being optional","commit_id":"345ed6f552096ed539548e0754749ec07e436034"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"6fb3e40ee7fac2f05abe3aaed756a906c52381fa","unresolved":false,"context_lines":[{"line_number":60,"context_line":"Implementation"},{"line_number":61,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Assignee(s)"},{"line_number":64,"context_line":"-----------"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Who is leading the writing of the code? Or is this a blueprint where you\u0027re"}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F%2B4es%3D","line":63,"updated":"2014-03-22 00:59:46.000000000","message":"Do we need this section? I think whoever is submitting the BP proposal is normally the primary assignee and the point of contact. This will also be tracked in launchpad. It also really shouldn\u0027t affect the proposal by itself. I think the point of the repo was to have real review of bp proposals before we accepting a bp. Knowing the assignees shouldn\u0027t be part of that.","commit_id":"345ed6f552096ed539548e0754749ec07e436034"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"6fb3e40ee7fac2f05abe3aaed756a906c52381fa","unresolved":false,"context_lines":[{"line_number":72,"context_line":"Primary assignee:"},{"line_number":73,"context_line":"  \u003claunchpad-id or None\u003e"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"Other contributors:"},{"line_number":76,"context_line":"  \u003claunchpad-id or None\u003e"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"Milestones"}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F%2B4ec%3D","line":75,"updated":"2014-03-22 00:59:46.000000000","message":"For the other contributors I don\u0027t think we need to list it here because that\u0027s done through code submissions. I think the list is often too dynamic, depending on the bp, to be in the proposal. For example, look at the unit-tests bp I\u0027ve seen plenty of contributions on that bp from people who I had no expectation of seeing commits.","commit_id":"345ed6f552096ed539548e0754749ec07e436034"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"b0551a5a2ed8da90d02f8a5955ff4eba6a299a96","unresolved":false,"context_lines":[{"line_number":72,"context_line":"Primary assignee:"},{"line_number":73,"context_line":"  \u003claunchpad-id or None\u003e"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"Other contributors:"},{"line_number":76,"context_line":"  \u003claunchpad-id or None\u003e"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"Milestones"}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F%2B0J8%3D","line":75,"in_reply_to":"AAAAWH%2F%2B4ec%3D","updated":"2014-03-24 04:42:01.000000000","message":"ok I think it was just designed to get a feel for how much support there is upfront, but I\u0027m ok with removing it.","commit_id":"345ed6f552096ed539548e0754749ec07e436034"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"6fb3e40ee7fac2f05abe3aaed756a906c52381fa","unresolved":false,"context_lines":[{"line_number":76,"context_line":"  \u003claunchpad-id or None\u003e"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"Milestones"},{"line_number":79,"context_line":"----------"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Originally approved for:"},{"line_number":82,"context_line":"  Juno-1"}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F%2B5Wo%3D","line":79,"updated":"2014-03-22 00:59:46.000000000","message":"So I think leaving this part in the bp is fine. I don\u0027t expect us to come in and update the templates upon approval and on completion we just to move them over to the implemented dir instead of updating them.\n\nWhat I\u0027d rather see here is just what the intended target for the work is. Because that\u0027s something that can be discussed in a review.","commit_id":"345ed6f552096ed539548e0754749ec07e436034"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"b0551a5a2ed8da90d02f8a5955ff4eba6a299a96","unresolved":false,"context_lines":[{"line_number":76,"context_line":"  \u003claunchpad-id or None\u003e"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"Milestones"},{"line_number":79,"context_line":"----------"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Originally approved for:"},{"line_number":82,"context_line":"  Juno-1"}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F%2B0JM%3D","line":79,"in_reply_to":"AAAAWH%2F%2B5Wo%3D","updated":"2014-03-24 04:42:01.000000000","message":"Done","commit_id":"345ed6f552096ed539548e0754749ec07e436034"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"0016f2f0e62710db27d6fcacc4874b5987f6116c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"::"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"#  Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#  not use this file except in compliance with the License. You may obtain"},{"line_number":5,"context_line":"#  a copy of the License at"},{"line_number":6,"context_line":"#"}],"source_content_type":"text/x-rst","patch_set":2,"id":"AAAAWH%2F%2BjtA%3D","line":3,"updated":"2014-03-25 22:32:58.000000000","message":"Also this should be CC see:\n\nhttps://review.openstack.org/#/c/82934/","commit_id":"2187995e07285c8e47ac1ad4be8153f61a06e124"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"unresolved":false,"context_lines":[{"line_number":1,"context_line":"::"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"#  Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#  not use this file except in compliance with the License. You may obtain"},{"line_number":5,"context_line":"#  a copy of the License at"},{"line_number":6,"context_line":"#"}],"source_content_type":"text/x-rst","patch_set":2,"id":"AAAAWH%2F%2BjO0%3D","line":3,"in_reply_to":"AAAAWH%2F%2BjtA%3D","updated":"2014-03-26 00:23:45.000000000","message":"Done","commit_id":"2187995e07285c8e47ac1ad4be8153f61a06e124"}]}
