)]}'
{"specs/backlog/api-v2-experimental-impl.rst":[{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"35ad936c303f69c7ec5781454bdb615a8d0749bd","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"There and back again, a roadmap to API v2"},{"line_number":9,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/sahara/+spec/v2-api-experimental-impl"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_44d9a982","line":9,"updated":"2015-08-14 17:22:07.000000000","message":"General note, plolicy suggestion\n\nShould we resolve that once we start the v2 API work, additional changes to the v1 API should be severely restricted?  That is, we don\u0027t make any changes to the v1 API except for critical, show-stopping defects in Sahara?\n\nOtherwise we have a moving target, and changes that have to be made in both places. However, this does require that as a team we make a conscious decision to move this off the backlog at a given time and then puruse it.","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"b5caee1a9a4c2d9b083555535d93e9a0b6438358","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"There and back again, a roadmap to API v2"},{"line_number":9,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/sahara/+spec/v2-api-experimental-impl"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_940063ac","line":9,"in_reply_to":"1a4dcd0f_3ca34db3","updated":"2015-08-17 22:03:50.000000000","message":"i dunno, i think it may take 6months to get all the v2 stuff done. it might be difficult to limit v1 stuff, i\u0027m hopeful that if we are diligent about keeping the new v1 stuff in shape it won\u0027t need too much modification to be included in v2. so, i guess i\u0027m against freezing v1 until v2 is done.","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":8090,"name":"Chad Roberts","email":"croberts@redhat.com","username":"croberts"},"change_message_id":"39240113bddcb21b5ce7bb5644fbb7c34090b015","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"There and back again, a roadmap to API v2"},{"line_number":9,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/sahara/+spec/v2-api-experimental-impl"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_3ca34db3","line":9,"in_reply_to":"1a4dcd0f_44d9a982","updated":"2015-08-14 20:10:06.000000000","message":"Not sure how much we\u0027ll be able to totally limit v1 changes.  I suppose it depends somewhat on how long we anticipate the \"lockdown\" period for v2 development to be.","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"35ad936c303f69c7ec5781454bdb615a8d0749bd","unresolved":false,"context_lines":[{"line_number":11,"context_line":"https://blueprints.launchpad.net/sahara/+spec/v2-api-experimental-impl"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"As sahara\u0027s API has evolved there have been several features introduced"},{"line_number":14,"context_line":"in the form of routes and methods and that could be crafted in a more"},{"line_number":15,"context_line":"consistent and predictable manner. Additionally, there are several new"},{"line_number":16,"context_line":"considerations and methodologies that can only be addressed by updating the"},{"line_number":17,"context_line":"major version of the API. This document serves as a roadmap to implement an"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_89234aa7","line":14,"updated":"2015-08-14 17:22:07.000000000","message":"total nit, remove \"and\" before \"that\"","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"b5caee1a9a4c2d9b083555535d93e9a0b6438358","unresolved":false,"context_lines":[{"line_number":11,"context_line":"https://blueprints.launchpad.net/sahara/+spec/v2-api-experimental-impl"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"As sahara\u0027s API has evolved there have been several features introduced"},{"line_number":14,"context_line":"in the form of routes and methods and that could be crafted in a more"},{"line_number":15,"context_line":"consistent and predictable manner. Additionally, there are several new"},{"line_number":16,"context_line":"considerations and methodologies that can only be addressed by updating the"},{"line_number":17,"context_line":"major version of the API. This document serves as a roadmap to implement an"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_d41d7b10","line":14,"in_reply_to":"1a4dcd0f_89234aa7","updated":"2015-08-17 22:03:50.000000000","message":"Done","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"35ad936c303f69c7ec5781454bdb615a8d0749bd","unresolved":false,"context_lines":[{"line_number":16,"context_line":"considerations and methodologies that can only be addressed by updating the"},{"line_number":17,"context_line":"major version of the API. This document serves as a roadmap to implement an"},{"line_number":18,"context_line":"experimental v2 API which will form the basis of the eventual stable version."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Problem description"},{"line_number":22,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_899caa5e","line":19,"updated":"2015-08-14 17:22:07.000000000","message":"I might say at the outset here that the vision is to have this be an overall spec, with suboordinate specs as neeed, just so that readers are clear on the way in.","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"b5caee1a9a4c2d9b083555535d93e9a0b6438358","unresolved":false,"context_lines":[{"line_number":16,"context_line":"considerations and methodologies that can only be addressed by updating the"},{"line_number":17,"context_line":"major version of the API. This document serves as a roadmap to implement an"},{"line_number":18,"context_line":"experimental v2 API which will form the basis of the eventual stable version."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Problem description"},{"line_number":22,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_941703ed","line":19,"in_reply_to":"1a4dcd0f_899caa5e","updated":"2015-08-17 22:03:50.000000000","message":"agree, i\u0027ll add something here","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"641d17ff83550e5bc420aacdee807d139fee6e36","unresolved":false,"context_lines":[{"line_number":52,"context_line":"* HREF embedding in responses to improve resource location discovery."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"These are just a few examples of issues which can be addressed in a new"},{"line_number":55,"context_line":"major version API implemenation."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da20952f_083833ee","line":55,"updated":"2015-08-30 02:42:50.000000000","message":"s/implemenation/implementation/","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"5758b571c5ae8dad61f31fe7f32eb4e4a8b02186","unresolved":false,"context_lines":[{"line_number":52,"context_line":"* HREF embedding in responses to improve resource location discovery."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"These are just a few examples of issues which can be addressed in a new"},{"line_number":55,"context_line":"major version API implemenation."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a8ffd7b_e77961fd","line":55,"in_reply_to":"da20952f_083833ee","updated":"2015-12-03 01:24:24.000000000","message":"Done","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"35ad936c303f69c7ec5781454bdb615a8d0749bd","unresolved":false,"context_lines":[{"line_number":94,"context_line":"The following list is an overview of all the changes that should be"},{"line_number":95,"context_line":"incorporated into the experimental API before it can be considered for"},{"line_number":96,"context_line":"migration to stable. These changes are not in order of precedence, and can"},{"line_number":97,"context_line":"be carried out in parallel. Some of these changes can be address with simple"},{"line_number":98,"context_line":"bugs, which should be marked with ``[APIv2]`` in their names. The more"},{"line_number":99,"context_line":"complex changes should be preceeded by specifications marked with the same"},{"line_number":100,"context_line":"``[APIv2]`` moniker in their names. For both types of changes, the"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_29dc7673","line":97,"updated":"2015-08-14 17:22:07.000000000","message":"typo, address -\u003e addressed","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"b5caee1a9a4c2d9b083555535d93e9a0b6438358","unresolved":false,"context_lines":[{"line_number":94,"context_line":"The following list is an overview of all the changes that should be"},{"line_number":95,"context_line":"incorporated into the experimental API before it can be considered for"},{"line_number":96,"context_line":"migration to stable. These changes are not in order of precedence, and can"},{"line_number":97,"context_line":"be carried out in parallel. Some of these changes can be address with simple"},{"line_number":98,"context_line":"bugs, which should be marked with ``[APIv2]`` in their names. The more"},{"line_number":99,"context_line":"complex changes should be preceeded by specifications marked with the same"},{"line_number":100,"context_line":"``[APIv2]`` moniker in their names. For both types of changes, the"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_74f10fcd","line":97,"in_reply_to":"1a4dcd0f_29dc7673","updated":"2015-08-17 22:03:50.000000000","message":"Done","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"35ad936c303f69c7ec5781454bdb615a8d0749bd","unresolved":false,"context_lines":[{"line_number":99,"context_line":"complex changes should be preceeded by specifications marked with the same"},{"line_number":100,"context_line":"``[APIv2]`` moniker in their names. For both types of changes, the"},{"line_number":101,"context_line":"commits should contain ``Partial-Implements: bp v2-api-experimental-impl``"},{"line_number":102,"context_line":"to aid in tracking the API conversion process."},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Overview of changes:"},{"line_number":105,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_49e30206","line":102,"updated":"2015-08-14 17:22:07.000000000","message":"Should we have sub-blueprints to track sub-specs, too, or is that nuts?\n\nSo, link the overall spec to the bp, and any subspecs to the same bp, or make bps for anything with its own spec and make the overall blueprint dependent on the subbps in launchpad?\n\nIn that case, the commits should list both bps -- the overall bp and the sub-bp.\nThis would make it easy to jump from a CR to both the specific spec for that change and the overall spec.","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"b5caee1a9a4c2d9b083555535d93e9a0b6438358","unresolved":false,"context_lines":[{"line_number":99,"context_line":"complex changes should be preceeded by specifications marked with the same"},{"line_number":100,"context_line":"``[APIv2]`` moniker in their names. For both types of changes, the"},{"line_number":101,"context_line":"commits should contain ``Partial-Implements: bp v2-api-experimental-impl``"},{"line_number":102,"context_line":"to aid in tracking the API conversion process."},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Overview of changes:"},{"line_number":105,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_7476ef3f","line":102,"in_reply_to":"1a4dcd0f_49e30206","updated":"2015-08-17 22:03:50.000000000","message":"i kinda thought we would just have the master bp and link all specs to that. but i\u0027m open to thoughts on this. i\u0027m not sure it would make things easier to have 2 sets of bps running, but i don\u0027t know.","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"35ad936c303f69c7ec5781454bdb615a8d0749bd","unresolved":false,"context_lines":[{"line_number":120,"context_line":"    favor of requesting a cancelled state on a PATCH to the new"},{"line_number":121,"context_line":"    /jobs/{job_id} endpoint."},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"  * /job-binary-internals should be removed in favor of /job-binaries as"},{"line_number":124,"context_line":"    the latter accepts interal database referenced items."},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"  * /job-executions/{job_execution_id}/refresh-status should be removed in"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_09e17a9a","line":123,"updated":"2015-08-14 17:22:07.000000000","message":"how do we disambiguate the creation of a job binary which simply contains a url, and a two-step operation where we upload a file, generate an internal url, and create the job binary with that url?\n\nCurrently we are using @rest.put_file for job binary internal creation, followed by a normal @rest.post method to create the job binary.\n\nWould we try to use POST for both?  Can POST optionally take a file, in which case the url should not be specified (would be an error)?","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"b5caee1a9a4c2d9b083555535d93e9a0b6438358","unresolved":false,"context_lines":[{"line_number":120,"context_line":"    favor of requesting a cancelled state on a PATCH to the new"},{"line_number":121,"context_line":"    /jobs/{job_id} endpoint."},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"  * /job-binary-internals should be removed in favor of /job-binaries as"},{"line_number":124,"context_line":"    the latter accepts interal database referenced items."},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"  * /job-executions/{job_execution_id}/refresh-status should be removed in"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_94d8631f","line":123,"in_reply_to":"1a4dcd0f_075cfe7b","updated":"2015-08-17 22:03:50.000000000","message":"Chad\u0027s point not withstanding, the way the put reads to me for the job binary internals now is that the body contains the file. i think there is a way we could allow a post of json data with the file body as part of that data. i will look into it a little more.\n\nultimately, i\u0027m thinking we would have a one POST operation that would create the internal job binary and upload the file.","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":8090,"name":"Chad Roberts","email":"croberts@redhat.com","username":"croberts"},"change_message_id":"39240113bddcb21b5ce7bb5644fbb7c34090b015","unresolved":false,"context_lines":[{"line_number":120,"context_line":"    favor of requesting a cancelled state on a PATCH to the new"},{"line_number":121,"context_line":"    /jobs/{job_id} endpoint."},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"  * /job-binary-internals should be removed in favor of /job-binaries as"},{"line_number":124,"context_line":"    the latter accepts interal database referenced items."},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"  * /job-executions/{job_execution_id}/refresh-status should be removed in"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_075cfe7b","line":123,"in_reply_to":"1a4dcd0f_09e17a9a","updated":"2015-08-14 20:10:06.000000000","message":"Perhaps it\u0027s time to get rid of the internal db binary store entirely.","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"641d17ff83550e5bc420aacdee807d139fee6e36","unresolved":false,"context_lines":[{"line_number":121,"context_line":"    /jobs/{job_id} endpoint."},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"  * /job-binary-internals should be removed in favor of /job-binaries as"},{"line_number":124,"context_line":"    the latter accepts interal database referenced items."},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"  * /job-executions/{job_execution_id}/refresh-status should be removed in"},{"line_number":127,"context_line":"    favor of using a GET on the new /jobs/{job_id} endpoint for running"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da20952f_482aab91","line":124,"updated":"2015-08-30 02:42:50.000000000","message":"s/interal/internal/","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"5758b571c5ae8dad61f31fe7f32eb4e4a8b02186","unresolved":false,"context_lines":[{"line_number":121,"context_line":"    /jobs/{job_id} endpoint."},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"  * /job-binary-internals should be removed in favor of /job-binaries as"},{"line_number":124,"context_line":"    the latter accepts interal database referenced items."},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"  * /job-executions/{job_execution_id}/refresh-status should be removed in"},{"line_number":127,"context_line":"    favor of using a GET on the new /jobs/{job_id} endpoint for running"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a8ffd7b_b33717fd","line":124,"in_reply_to":"da20952f_482aab91","updated":"2015-12-03 01:24:24.000000000","message":"Done","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"641d17ff83550e5bc420aacdee807d139fee6e36","unresolved":false,"context_lines":[{"line_number":136,"context_line":""},{"line_number":137,"context_line":"  * oozie_job_id should be changed to engine_job_id."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"  * all returned payalods should be wrapped in their type, this is currently"},{"line_number":140,"context_line":"    true for the API and should remain so for consistency."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"  * HREFs should be embedded in responses that contain references to other"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da20952f_081fd36c","line":139,"updated":"2015-08-30 02:42:50.000000000","message":"s/payalods/payloads/","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"5758b571c5ae8dad61f31fe7f32eb4e4a8b02186","unresolved":false,"context_lines":[{"line_number":136,"context_line":""},{"line_number":137,"context_line":"  * oozie_job_id should be changed to engine_job_id."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"  * all returned payalods should be wrapped in their type, this is currently"},{"line_number":140,"context_line":"    true for the API and should remain so for consistency."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"  * HREFs should be embedded in responses that contain references to other"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a8ffd7b_f3183f86","line":139,"in_reply_to":"da20952f_081fd36c","updated":"2015-12-03 01:24:24.000000000","message":"Done","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"742d5d855eac7f8bf3c1dc89ef55e61df95c1956","unresolved":false,"context_lines":[{"line_number":154,"context_line":"    compatibility features until the API has been declared stable. Once"},{"line_number":155,"context_line":"    the API has moved into the stable phase, the microversions will only"},{"line_number":156,"context_line":"    implement backward compatibility for version 2, and only for features"},{"line_number":157,"context_line":"    added after the stable release."},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"This list is not meant to contain all the possible future changes, but a"},{"line_number":160,"context_line":"window of the minimum necessary changes to be made before the new API can"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_dc17e1eb","line":157,"updated":"2015-08-17 23:17:07.000000000","message":"I have a bunch of suggestions for the new API:\n\n* version discovery (/ returns versions info, /2/ returns info about endpoints, micro versions supported and etc.)\n* adding hrefs as it done many other projects\n* what\u0027s about aggregating operations on clusters (and probably jobs) into the /clusters/\u003ccluster_id\u003e/action endpoint that will supported scaling, upgrading (someday) and etc. or some other way to be able to support new features that are operations on the cluster (backup? pause? heal?)","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"140ec5f8fcdf7a60bc63104d8d984ddcd153046a","unresolved":false,"context_lines":[{"line_number":154,"context_line":"    compatibility features until the API has been declared stable. Once"},{"line_number":155,"context_line":"    the API has moved into the stable phase, the microversions will only"},{"line_number":156,"context_line":"    implement backward compatibility for version 2, and only for features"},{"line_number":157,"context_line":"    added after the stable release."},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"This list is not meant to contain all the possible future changes, but a"},{"line_number":160,"context_line":"window of the minimum necessary changes to be made before the new API can"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_0872d3ad","line":157,"in_reply_to":"fa1b9901_dc17e1eb","updated":"2015-08-18 13:51:33.000000000","message":"in general i agree with your suggestions Sergey\n\n* for version discovery, i will add something about the root. for the v2 endpoint, were you thinking something like json-home[1]?\n\n* i mentioned hrefs on line 142, is that what you were thinking?\n\n* i want to add an endpoint for aggregating operations, and also helping with asynchronous operations, but i would like to create a generic /tasks, and /tasks/{task_id}, endpoints to handle this. how do you feel about having a more generic endpoint?\n\n[1]: https://tools.ietf.org/html/draft-nottingham-json-home-03","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"35ad936c303f69c7ec5781454bdb615a8d0749bd","unresolved":false,"context_lines":[{"line_number":182,"context_line":"``oozie_job_id``. As the data model will continue to be used for the v1.1"},{"line_number":183,"context_line":"API until it is deprecated, it is not advisable to rename these fields at"},{"line_number":184,"context_line":"this time. When the v2 API has been made stable, and the v1.1 API has been"},{"line_number":185,"context_line":"deprecated, these fields should be revisisted and changed in the data model."},{"line_number":186,"context_line":""},{"line_number":187,"context_line":"REST API impact"},{"line_number":188,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_69103e5a","line":185,"updated":"2015-08-14 17:22:07.000000000","message":"Maybe add a note that in the meantime, we can use whatever mapping magic we want to translate names in and out of the database without changing the actual field name stored in the db. This can let the code change without breaking the db for v1.\n\nWorth saying?","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":8090,"name":"Chad Roberts","email":"croberts@redhat.com","username":"croberts"},"change_message_id":"39240113bddcb21b5ce7bb5644fbb7c34090b015","unresolved":false,"context_lines":[{"line_number":182,"context_line":"``oozie_job_id``. As the data model will continue to be used for the v1.1"},{"line_number":183,"context_line":"API until it is deprecated, it is not advisable to rename these fields at"},{"line_number":184,"context_line":"this time. When the v2 API has been made stable, and the v1.1 API has been"},{"line_number":185,"context_line":"deprecated, these fields should be revisisted and changed in the data model."},{"line_number":186,"context_line":""},{"line_number":187,"context_line":"REST API impact"},{"line_number":188,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_87f6ce5b","line":185,"in_reply_to":"1a4dcd0f_69103e5a","updated":"2015-08-14 20:10:06.000000000","message":"Might be worth a mention of how that will be accomplished.  I think the above paragraph does sort of imply that there will be some magic going on.","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"b5caee1a9a4c2d9b083555535d93e9a0b6438358","unresolved":false,"context_lines":[{"line_number":182,"context_line":"``oozie_job_id``. As the data model will continue to be used for the v1.1"},{"line_number":183,"context_line":"API until it is deprecated, it is not advisable to rename these fields at"},{"line_number":184,"context_line":"this time. When the v2 API has been made stable, and the v1.1 API has been"},{"line_number":185,"context_line":"deprecated, these fields should be revisisted and changed in the data model."},{"line_number":186,"context_line":""},{"line_number":187,"context_line":"REST API impact"},{"line_number":188,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_d4f57b83","line":185,"in_reply_to":"1a4dcd0f_87f6ce5b","updated":"2015-08-17 22:03:50.000000000","message":"magic was definitely implied, i\u0027ll try to add a few more details about the magic.","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"35ad936c303f69c7ec5781454bdb615a8d0749bd","unresolved":false,"context_lines":[{"line_number":187,"context_line":"REST API impact"},{"line_number":188,"context_line":"---------------"},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"As this specification is talking to a high level change of the API, the"},{"line_number":191,"context_line":"following changes are enumerated in brief. Full details should be created"},{"line_number":192,"context_line":"for changes that will require more than just renaming an endpoint."},{"line_number":193,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_e9044e19","line":190,"updated":"2015-08-14 17:22:07.000000000","message":"nit, I would change \"talking to\" to either \"describing\", \"addressing\", or \"speaking to\".","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"b5caee1a9a4c2d9b083555535d93e9a0b6438358","unresolved":false,"context_lines":[{"line_number":187,"context_line":"REST API impact"},{"line_number":188,"context_line":"---------------"},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"As this specification is talking to a high level change of the API, the"},{"line_number":191,"context_line":"following changes are enumerated in brief. Full details should be created"},{"line_number":192,"context_line":"for changes that will require more than just renaming an endpoint."},{"line_number":193,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_b43fa724","line":190,"in_reply_to":"1a4dcd0f_479766eb","updated":"2015-08-17 22:03:50.000000000","message":"Done","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":8090,"name":"Chad Roberts","email":"croberts@redhat.com","username":"croberts"},"change_message_id":"39240113bddcb21b5ce7bb5644fbb7c34090b015","unresolved":false,"context_lines":[{"line_number":187,"context_line":"REST API impact"},{"line_number":188,"context_line":"---------------"},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"As this specification is talking to a high level change of the API, the"},{"line_number":191,"context_line":"following changes are enumerated in brief. Full details should be created"},{"line_number":192,"context_line":"for changes that will require more than just renaming an endpoint."},{"line_number":193,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_479766eb","line":190,"in_reply_to":"1a4dcd0f_e9044e19","updated":"2015-08-14 20:10:06.000000000","message":"+1","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":13953,"name":"Elise Gafford","email":"egafford@redhat.com","username":"egafford"},"change_message_id":"d13d330c400afcf6f6a99869723383395c5be57f","unresolved":false,"context_lines":[{"line_number":206,"context_line":"* creation of GET/PUT/DELETE to /images/{image_id}/tags/{tag_id}, this"},{"line_number":207,"context_line":"  should also be in the previously mentioned specification on tagging."},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"* move operations on /jobs to /job-templates"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"* move operations /job-executions to /jobs"},{"line_number":212,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"da20952f_1ee97280","line":209,"updated":"2015-08-28 16:57:50.000000000","message":"I want to add a full-fledged interface argument GET/PUT/DELETE API too, but that can rationally be a separate spec.","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"5758b571c5ae8dad61f31fe7f32eb4e4a8b02186","unresolved":false,"context_lines":[{"line_number":206,"context_line":"* creation of GET/PUT/DELETE to /images/{image_id}/tags/{tag_id}, this"},{"line_number":207,"context_line":"  should also be in the previously mentioned specification on tagging."},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"* move operations on /jobs to /job-templates"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"* move operations /job-executions to /jobs"},{"line_number":212,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a8ffd7b_290098aa","line":209,"in_reply_to":"da20952f_1ee97280","updated":"2015-12-03 01:24:24.000000000","message":"ack","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":13953,"name":"Elise Gafford","email":"egafford@redhat.com","username":"egafford"},"change_message_id":"d13d330c400afcf6f6a99869723383395c5be57f","unresolved":false,"context_lines":[{"line_number":213,"context_line":"* removal of POST to /jobs/{job_id}/execute"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"* creation of POST to /jobs, this should be defined in a specification"},{"line_number":216,"context_line":"  about restructuring the job execution endpoint."},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"* removal of GET to /job-executions/{job_execution_id}/cancel"},{"line_number":219,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"da20952f_9e1b421b","line":216,"updated":"2015-08-28 16:57:50.000000000","message":"Please note (in this bullet or a sub-note) that at minimum, this spec should include removal of the single-value INPUT and OUTPUT fields (which have never really been flexible enough to suit their use case, and should be supplanted by use of configs or interfaces.)","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"5758b571c5ae8dad61f31fe7f32eb4e4a8b02186","unresolved":false,"context_lines":[{"line_number":213,"context_line":"* removal of POST to /jobs/{job_id}/execute"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"* creation of POST to /jobs, this should be defined in a specification"},{"line_number":216,"context_line":"  about restructuring the job execution endpoint."},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"* removal of GET to /job-executions/{job_execution_id}/cancel"},{"line_number":219,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a8ffd7b_a9a7a8ba","line":216,"in_reply_to":"da20952f_9e1b421b","updated":"2015-12-03 01:24:24.000000000","message":"Done","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"641d17ff83550e5bc420aacdee807d139fee6e36","unresolved":false,"context_lines":[{"line_number":218,"context_line":"* removal of GET to /job-executions/{job_execution_id}/cancel"},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"* creation of PATCH to /jobs/{job_id}, this should be defined in the"},{"line_number":221,"context_line":"  specification about restructuring the job exection endpoint."},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"* removal of GET to /job-executions/{job_execution_id}/refresh-status"},{"line_number":224,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"da20952f_880be329","line":221,"updated":"2015-08-30 02:42:50.000000000","message":"s/excection/execution/","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"5758b571c5ae8dad61f31fe7f32eb4e4a8b02186","unresolved":false,"context_lines":[{"line_number":218,"context_line":"* removal of GET to /job-executions/{job_execution_id}/cancel"},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"* creation of PATCH to /jobs/{job_id}, this should be defined in the"},{"line_number":221,"context_line":"  specification about restructuring the job exection endpoint."},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"* removal of GET to /job-executions/{job_execution_id}/refresh-status"},{"line_number":224,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a8ffd7b_c9ac6ca1","line":221,"in_reply_to":"da20952f_880be329","updated":"2015-12-03 01:24:24.000000000","message":"Done","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"35ad936c303f69c7ec5781454bdb615a8d0749bd","unresolved":false,"context_lines":[{"line_number":222,"context_line":""},{"line_number":223,"context_line":"* removal of GET to /job-executions/{job_execution_id}/refresh-status"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"* removal of all /job-binary-internals endpoints."},{"line_number":226,"context_line":""},{"line_number":227,"context_line":"* refactor of PUT to /node-group-templates/{node_group_template_id} into"},{"line_number":228,"context_line":"  PATCH on same endpoint."}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_2439fdd8","line":225,"updated":"2015-08-14 17:22:07.000000000","message":"re earlier note on handling uploads, maybe we need mention here of the endpoint that will handle that","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"b5caee1a9a4c2d9b083555535d93e9a0b6438358","unresolved":false,"context_lines":[{"line_number":222,"context_line":""},{"line_number":223,"context_line":"* removal of GET to /job-executions/{job_execution_id}/refresh-status"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"* removal of all /job-binary-internals endpoints."},{"line_number":226,"context_line":""},{"line_number":227,"context_line":"* refactor of PUT to /node-group-templates/{node_group_template_id} into"},{"line_number":228,"context_line":"  PATCH on same endpoint."}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_f40bdf7b","line":225,"in_reply_to":"1a4dcd0f_2439fdd8","updated":"2015-08-17 22:03:50.000000000","message":"good point, i will add something once the research on that endpoint is complete.","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"641d17ff83550e5bc420aacdee807d139fee6e36","unresolved":false,"context_lines":[{"line_number":257,"context_line":"Developer impact"},{"line_number":258,"context_line":"----------------"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"As this change is targetted for experimental work, developers should know"},{"line_number":261,"context_line":"that the details of the v2 API will be constantly changing. There is no"},{"line_number":262,"context_line":"guarantee of stability."},{"line_number":263,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"da20952f_eb005903","line":260,"updated":"2015-08-30 02:42:50.000000000","message":"s/targetted/targeted/","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"5758b571c5ae8dad61f31fe7f32eb4e4a8b02186","unresolved":false,"context_lines":[{"line_number":257,"context_line":"Developer impact"},{"line_number":258,"context_line":"----------------"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"As this change is targetted for experimental work, developers should know"},{"line_number":261,"context_line":"that the details of the v2 API will be constantly changing. There is no"},{"line_number":262,"context_line":"guarantee of stability."},{"line_number":263,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a8ffd7b_69a460a5","line":260,"in_reply_to":"da20952f_eb005903","updated":"2015-12-03 01:24:24.000000000","message":"Done","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"35ad936c303f69c7ec5781454bdb615a8d0749bd","unresolved":false,"context_lines":[{"line_number":308,"context_line":""},{"line_number":309,"context_line":"Unit tests will be created to exercise the new endpoints. Additionally, the"},{"line_number":310,"context_line":"gabbi[5] testing framework should be investigated as a functional testing"},{"line_number":311,"context_line":"platform for the REST API. This investigation should be the result a"},{"line_number":312,"context_line":"further specification as it will deal with creating a platform for live"},{"line_number":313,"context_line":"testing a sahara api server."},{"line_number":314,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a4dcd0f_e45315e1","line":311,"updated":"2015-08-14 17:22:07.000000000","message":"looks like missing \"of\"","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"b5caee1a9a4c2d9b083555535d93e9a0b6438358","unresolved":false,"context_lines":[{"line_number":308,"context_line":""},{"line_number":309,"context_line":"Unit tests will be created to exercise the new endpoints. Additionally, the"},{"line_number":310,"context_line":"gabbi[5] testing framework should be investigated as a functional testing"},{"line_number":311,"context_line":"platform for the REST API. This investigation should be the result a"},{"line_number":312,"context_line":"further specification as it will deal with creating a platform for live"},{"line_number":313,"context_line":"testing a sahara api server."},{"line_number":314,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa1b9901_941dc3bf","line":311,"in_reply_to":"1a4dcd0f_e45315e1","updated":"2015-08-17 22:03:50.000000000","message":"Done","commit_id":"6c08046d6f87b535ce4bc14a9a6cff4ae9220def"},{"author":{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},"change_message_id":"9ca9bd46014d6e8d7897bba10bc0ec865ed861d8","unresolved":false,"context_lines":[{"line_number":117,"context_line":"  * /images/{image_id}/tag and /images/{image_id}/untag should be changed"},{"line_number":118,"context_line":"    to follow the guidelines on tags[3]."},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"  * /jobs should be renamed to /job-templates."},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"  * /job-executions should be renamed to /jobs."},{"line_number":123,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a740942_6aaaf786","line":120,"updated":"2015-12-08 12:48:34.000000000","message":"Actually I have thoughts that job-templates a little bit strange thing and mostly useless: in most cases it just collects main libs and additional libs in one place. Also having exact job execution (job) we can just re-run it (with probable config changes, but it looks like we cannot add some binary in here). BTW, it\u0027s probably harder to implement some logic to avoid job-templates.","commit_id":"c29a47ec3d0df130f33e093fb8c3d025bc6c8aeb"},{"author":{"_account_id":8090,"name":"Chad Roberts","email":"croberts@redhat.com","username":"croberts"},"change_message_id":"6294326996ce6b58e34a098755bd017714f6ac67","unresolved":false,"context_lines":[{"line_number":131,"context_line":""},{"line_number":132,"context_line":"  * /job-binary-internals should be removed in favor of /job-binaries as"},{"line_number":133,"context_line":"    the latter accepts internal database referenced items, an endpoint"},{"line_number":134,"context_line":"    under /job-binaries can be created for uploading files(if required)."},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"  * /job-executions/{job_execution_id}/refresh-status should be removed in"},{"line_number":137,"context_line":"    favor of using a GET on the new /jobs/{job_id} endpoint for running"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a740942_f0cd5f66","line":134,"updated":"2015-12-08 20:30:01.000000000","message":"It seems like the job-binaries/upload endpoint would just wind-up being a convenience wrapper of some sort.  It\u0027s probably not needed.","commit_id":"c29a47ec3d0df130f33e093fb8c3d025bc6c8aeb"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"11b7cad503c8e977fced640236c830f12dcdecf3","unresolved":false,"context_lines":[{"line_number":131,"context_line":""},{"line_number":132,"context_line":"  * /job-binary-internals should be removed in favor of /job-binaries as"},{"line_number":133,"context_line":"    the latter accepts internal database referenced items, an endpoint"},{"line_number":134,"context_line":"    under /job-binaries can be created for uploading files(if required)."},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"  * /job-executions/{job_execution_id}/refresh-status should be removed in"},{"line_number":137,"context_line":"    favor of using a GET on the new /jobs/{job_id} endpoint for running"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a68dd71_2545a916","line":134,"in_reply_to":"7a740942_f0cd5f66","updated":"2016-01-26 20:05:40.000000000","message":"How else would we initiate an upload, though?\n\nJob binaries are created with a reference to some other\nthing. We still need a way to get a blob in the internal\ndb.\n\nWe can also consider dropping internal-db. That would\nrequire swift or manila be up and running for all cases,\nincluding dev/test. (I find upload to internal convenient\nfor dev/test, but putting the stuff in swift wouldn\u0027t be\nbad)","commit_id":"c29a47ec3d0df130f33e093fb8c3d025bc6c8aeb"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"11b7cad503c8e977fced640236c830f12dcdecf3","unresolved":false,"context_lines":[{"line_number":140,"context_line":"  * all update operations should synchronize around using PATCH instead of"},{"line_number":141,"context_line":"    PUT for partial resource updates."},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"* JSON payload changes"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"  * hadoop_version should be changed to plugin_version."},{"line_number":146,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a68dd71_a5ff59a9","line":143,"updated":"2016-01-26 20:05:40.000000000","message":"I suggest an addition here, which is we should use\nOpenStack-style job type names like map-reduce instead\nMapReduce which is taken directly from Oozie.\n\nThis aligns with the mention of technology specifics\nabove. I wanted to do this anyway, but it should\nprobably be part of v2","commit_id":"c29a47ec3d0df130f33e093fb8c3d025bc6c8aeb"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"11b7cad503c8e977fced640236c830f12dcdecf3","unresolved":false,"context_lines":[{"line_number":177,"context_line":"    entrypoint for operations on those clusters. This endpoint should"},{"line_number":178,"context_line":"    initially allow operations such as scaling but will be used for"},{"line_number":179,"context_line":"    further improvements in the future. The actions endpoint will be"},{"line_number":180,"context_line":"    the subject a separate specification as it will describe the"},{"line_number":181,"context_line":"    removal of several verb-oriented endpoints that currently exists,"},{"line_number":182,"context_line":"    and the creation of a new mechanism for synchronous and asynchronous"},{"line_number":183,"context_line":"    operations."}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a68dd71_a5ed19f1","line":180,"updated":"2016-01-26 20:05:40.000000000","message":"typo, missing \"of\"","commit_id":"c29a47ec3d0df130f33e093fb8c3d025bc6c8aeb"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"11b7cad503c8e977fced640236c830f12dcdecf3","unresolved":false,"context_lines":[{"line_number":178,"context_line":"    initially allow operations such as scaling but will be used for"},{"line_number":179,"context_line":"    further improvements in the future. The actions endpoint will be"},{"line_number":180,"context_line":"    the subject a separate specification as it will describe the"},{"line_number":181,"context_line":"    removal of several verb-oriented endpoints that currently exists,"},{"line_number":182,"context_line":"    and the creation of a new mechanism for synchronous and asynchronous"},{"line_number":183,"context_line":"    operations."},{"line_number":184,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a68dd71_e5ca316e","line":181,"updated":"2016-01-26 20:05:40.000000000","message":"typo, should be exist","commit_id":"c29a47ec3d0df130f33e093fb8c3d025bc6c8aeb"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"11b7cad503c8e977fced640236c830f12dcdecf3","unresolved":false,"context_lines":[{"line_number":261,"context_line":"* removal of GET to /job-executions/{job_execution_id}/refresh-status"},{"line_number":262,"context_line":""},{"line_number":263,"context_line":"* removal of all /job-binary-internals endpoints with their functionality"},{"line_number":264,"context_line":"  being provides by /job-binaries, this may require creating a separate"},{"line_number":265,"context_line":"  sub-endpoint for uploading."},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"* refactor of PUT to /node-group-templates/{node_group_template_id} into"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a68dd71_e5c6b148","line":264,"updated":"2016-01-26 20:05:40.000000000","message":"typo, should be \"provided\"","commit_id":"c29a47ec3d0df130f33e093fb8c3d025bc6c8aeb"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"11b7cad503c8e977fced640236c830f12dcdecf3","unresolved":false,"context_lines":[{"line_number":280,"context_line":"---------------------"},{"line_number":281,"context_line":""},{"line_number":282,"context_line":"In the experimental phase, this change should have no noticeable affect on"},{"line_number":283,"context_line":"the end user. Once the API has been declared stabled, users will need to"},{"line_number":284,"context_line":"switch python-saharaclient versions as well as upgrade their horizon"},{"line_number":285,"context_line":"installations to make full use of renamed features."},{"line_number":286,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a68dd71_65e161d0","line":283,"updated":"2016-01-26 20:05:40.000000000","message":"typo, should be \"stable\"","commit_id":"c29a47ec3d0df130f33e093fb8c3d025bc6c8aeb"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"11b7cad503c8e977fced640236c830f12dcdecf3","unresolved":false,"context_lines":[{"line_number":281,"context_line":""},{"line_number":282,"context_line":"In the experimental phase, this change should have no noticeable affect on"},{"line_number":283,"context_line":"the end user. Once the API has been declared stabled, users will need to"},{"line_number":284,"context_line":"switch python-saharaclient versions as well as upgrade their horizon"},{"line_number":285,"context_line":"installations to make full use of renamed features."},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"Deployer impact"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a68dd71_c5f7ed8a","line":284,"updated":"2016-01-26 20:05:40.000000000","message":"upgrade their horizon installations, or maybe there is\na way to just upgrade the sahara-dashboard portion?","commit_id":"c29a47ec3d0df130f33e093fb8c3d025bc6c8aeb"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"11b7cad503c8e977fced640236c830f12dcdecf3","unresolved":false,"context_lines":[{"line_number":340,"context_line":"Dependencies"},{"line_number":341,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":342,"context_line":""},{"line_number":343,"context_line":"This change should not require new dependencies."},{"line_number":344,"context_line":""},{"line_number":345,"context_line":""},{"line_number":346,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a68dd71_0582e5c8","line":343,"updated":"2016-01-26 20:05:40.000000000","message":"I thought \"dependencies\" dealt with whether or not\nthis spec was dependent on another spec.\n\nMaybe it\u0027s both.","commit_id":"c29a47ec3d0df130f33e093fb8c3d025bc6c8aeb"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"16898234b723aec6547cb55ef5f1b52a783c7131","unresolved":false,"context_lines":[{"line_number":348,"context_line":""},{"line_number":349,"context_line":"Unit tests will be created to exercise the new endpoints. Additionally, the"},{"line_number":350,"context_line":"gabbi[8] testing framework should be investigated as a functional testing"},{"line_number":351,"context_line":"platform for the REST API."},{"line_number":352,"context_line":""},{"line_number":353,"context_line":"To improve security testing, tools such as Syntribos[9] and RestFuzz[10]"},{"line_number":354,"context_line":"should be investigated for use in directed testing efforts and as possible"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_d83b45ea","line":351,"updated":"2015-12-17 14:27:16.000000000","message":"And tempest? We need to add the tempest API tests in addition to the existing ones:\nhttp://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/data_processing\n\n(Please note that those tests could be moved to sahara repository at some point, but it does not change the previous question):","commit_id":"c29a47ec3d0df130f33e093fb8c3d025bc6c8aeb"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"11b7cad503c8e977fced640236c830f12dcdecf3","unresolved":false,"context_lines":[{"line_number":380,"context_line":"under question within the OpenStack documentation community[13]. The effort"},{"line_number":381,"context_line":"to make accurate documentation for sahara\u0027s API should also include the"},{"line_number":382,"context_line":"possibility of creating Swagger[14] output as the v2 API approaches stable"},{"line_number":383,"context_line":"status, this should be addressed in a more separate specification as that"},{"line_number":384,"context_line":"time approaches."},{"line_number":385,"context_line":""},{"line_number":386,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a68dd71_05d0a5c8","line":383,"updated":"2016-01-26 20:05:40.000000000","message":"nit, do we need \"more\" here?","commit_id":"c29a47ec3d0df130f33e093fb8c3d025bc6c8aeb"}]}