)]}'
{"specs/rocky/approved/granular-resource-requests.rst":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"f50660b20fac6c700abfd90cd14d942562dcb180","unresolved":false,"context_lines":[{"line_number":419,"context_line":"Developers of modules supplying Resource Provider representations (e.g. virt"},{"line_number":420,"context_line":"drivers) will need to be aware of this feature in order to model their RPs"},{"line_number":421,"context_line":"appropriately."},{"line_number":422,"context_line":""},{"line_number":423,"context_line":"Implementation"},{"line_number":424,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":425,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_5894d044","line":422,"updated":"2018-02-05 17:42:27.000000000","message":"pep8 error here...\n\n Section \u0027Proposed change\u0027 is missing subsections: [u\u0027Upgrade impact\u0027]","commit_id":"4d48872ebb18204d79b40e7a5a9ebe28ded4f5db"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"f50660b20fac6c700abfd90cd14d942562dcb180","unresolved":false,"context_lines":[{"line_number":514,"context_line":"     - Introduced, approved, implementation started"},{"line_number":515,"context_line":"   * - Rocky"},{"line_number":516,"context_line":"     - Reproposed"},{"line_number":517,"context_line":"   "}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_988258c8","line":517,"updated":"2018-02-05 17:42:27.000000000","message":"the horror. the horror.","commit_id":"4d48872ebb18204d79b40e7a5a9ebe28ded4f5db"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"7f6907aeab15e547dec8c9c05bbc7af4e7b4dfdb","unresolved":false,"context_lines":[{"line_number":69,"context_line":"            |   /           \\   |"},{"line_number":70,"context_line":"          +-+--+-+         +-+--+-+"},{"line_number":71,"context_line":"          | NET1 |         | NET2 |"},{"line_number":72,"context_line":"          +------+         +------+"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Assume this is modeled in Placement as:"},{"line_number":75,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_d8183c1f","line":72,"updated":"2018-02-12 12:00:17.000000000","message":"This would probably make a good t-shirt. But only if the ascii-ness was maintained.","commit_id":"0c4cc9f3d27366d83fcc66d06e6f68c5d8da59ae"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"7f6907aeab15e547dec8c9c05bbc7af4e7b4dfdb","unresolved":false,"context_lines":[{"line_number":347,"context_line":""},{"line_number":348,"context_line":"* We could allow the \"number\" suffixes to be any arbitrary string.  However,"},{"line_number":349,"context_line":"  using integers is easy to understand and validate, and obviates worries about"},{"line_number":350,"context_line":"  escaping/encoding special characters, etc."},{"line_number":351,"context_line":""},{"line_number":352,"context_line":"* There has been discussion over time about the need for a JSON payload-based"},{"line_number":353,"context_line":"  API to enable richer expression to request allocation candidates.  While this"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_f8717845","line":350,"updated":"2018-02-12 12:00:17.000000000","message":"our brains are also good at comparing numbers when looking at lots of them, so it is easier to grok an entry in a request log\n\n(not a suggested change, just confirming that I like the choice)","commit_id":"0c4cc9f3d27366d83fcc66d06e6f68c5d8da59ae"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"7f6907aeab15e547dec8c9c05bbc7af4e7b4dfdb","unresolved":false,"context_lines":[{"line_number":446,"context_line":"* Negotiate microversion capabilities with the `Placement API`_."},{"line_number":447,"context_line":"* Recognize and parse the new `Syntax In Flavors`_."},{"line_number":448,"context_line":"* If the new flavor extra_specs syntax is recognized and the `Placement API`_"},{"line_number":449,"context_line":"  is not capable of the appropriate microversion, error."},{"line_number":450,"context_line":"* Construct the ``GET /allocation_candidates`` querystring according to the"},{"line_number":451,"context_line":"  flavor extra_specs."},{"line_number":452,"context_line":"* Send the ``GET /allocation_candidates`` request to Placement, specifying the"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_98b954cd","line":449,"updated":"2018-02-12 12:00:17.000000000","message":"These first two entries seem to assume that we may have a situation where the the scheduler or flavors are newer than the placement service. If I remember right, this is not something we\u0027ve said is okay?\n\nBasically, I thought the rule was \"placement gets upgraded early\"?","commit_id":"0c4cc9f3d27366d83fcc66d06e6f68c5d8da59ae"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"8b95997cdb98123d59cfa865454dc45483164cc5","unresolved":false,"context_lines":[{"line_number":446,"context_line":"* Negotiate microversion capabilities with the `Placement API`_."},{"line_number":447,"context_line":"* Recognize and parse the new `Syntax In Flavors`_."},{"line_number":448,"context_line":"* If the new flavor extra_specs syntax is recognized and the `Placement API`_"},{"line_number":449,"context_line":"  is not capable of the appropriate microversion, error."},{"line_number":450,"context_line":"* Construct the ``GET /allocation_candidates`` querystring according to the"},{"line_number":451,"context_line":"  flavor extra_specs."},{"line_number":452,"context_line":"* Send the ``GET /allocation_candidates`` request to Placement, specifying the"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_99b4ef81","line":449,"in_reply_to":"3fa0c359_98b954cd","updated":"2018-02-12 18:02:56.000000000","message":"Cool.  The check wouldn\u0027t hurt, though.  And might save us if we ever change our mind about ^\n\nBut yeah, we can discuss in the code review if it comes to that.","commit_id":"0c4cc9f3d27366d83fcc66d06e6f68c5d8da59ae"}]}