)]}'
{"doc/source/dev/vendor-passthru.rst":[{"author":{"_account_id":5805,"name":"Chris Krelle","email":"nobodycam@gmail.com","username":"nobodycam"},"change_message_id":"8f3e91581380bad5ae5b42dc693bc0e5fa844e9f","unresolved":false,"context_lines":[{"line_number":140,"context_line":"If you are changing the exceptions being raised, you might want to ensure"},{"line_number":141,"context_line":"that the same HTTP code is being returned to the user."},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"For non-backwards compatibility, please make sure you add a release"},{"line_number":144,"context_line":"note that indicates this."}],"source_content_type":"text/x-rst","patch_set":1,"id":"da6ed579_b9b96f17","line":143,"range":{"start_line":143,"start_character":18,"end_line":143,"end_character":31},"updated":"2016-01-13 21:06:35.000000000","message":"suggestion: maybe change compatibility to \"compatible changes\" here?","commit_id":"c12cc036ebb693083392ceb64895233e5045a05e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"c23075a3296072229bd0b09b6f5abbc04b318b72","unresolved":false,"context_lines":[{"line_number":140,"context_line":"If you are changing the exceptions being raised, you might want to ensure"},{"line_number":141,"context_line":"that the same HTTP code is being returned to the user."},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"For non-backwards compatibility, please make sure you add a release"},{"line_number":144,"context_line":"note that indicates this."}],"source_content_type":"text/x-rst","patch_set":1,"id":"da6ed579_19fc3b71","line":143,"range":{"start_line":143,"start_character":18,"end_line":143,"end_character":31},"in_reply_to":"da6ed579_b9b96f17","updated":"2016-01-13 21:11:10.000000000","message":"Or \u0027For changes that are not backwards compatible\u0027...  ?\n\nI\u0027ll change it if I do another revision.","commit_id":"c12cc036ebb693083392ceb64895233e5045a05e"},{"author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"change_message_id":"b57c11e844caf74f7b8e33693e6011757111f2a6","unresolved":false,"context_lines":[{"line_number":141,"context_line":"that the same HTTP code is being returned to the user."},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"For non-backwards compatibility, please make sure you add a release"},{"line_number":144,"context_line":"note that indicates this."}],"source_content_type":"text/x-rst","patch_set":1,"id":"da6ed579_d1c2ef33","line":144,"updated":"2016-01-14 03:09:26.000000000","message":"I prefer to write a new vendor passthru method and deprecate the previous one if there will be an incompatibility change. The vendor passthru method is still Ironic\u0027s API to some extends, Ironic should have a commitment to a backward compatible API, do we?","commit_id":"c12cc036ebb693083392ceb64895233e5045a05e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"bd14f307a1aa3aa2ea5e93b50ca2a79e10d8f3c3","unresolved":false,"context_lines":[{"line_number":141,"context_line":"that the same HTTP code is being returned to the user."},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"For non-backwards compatibility, please make sure you add a release"},{"line_number":144,"context_line":"note that indicates this."}],"source_content_type":"text/x-rst","patch_set":1,"id":"da6ed579_b2239d57","line":144,"in_reply_to":"da6ed579_d1c2ef33","updated":"2016-01-14 14:13:33.000000000","message":"This is documenting ironic\u0027s commitment -- which is that vendor passthru methods do not have to be backwards compatible. (They are exempt from the normal constraints we have on our API).\n\nI agree, it would be nice to deprecate etc. but we don\u0027t impose that now. I suppose one could suggest that they might consider writing new methods and deprecating previous ones, but I don\u0027t want to think about how that might work and how to document it here. I just wanted to get down what the commitment was so that I\u0027d remember when reviewing patches.","commit_id":"c12cc036ebb693083392ceb64895233e5045a05e"}]}