)]}'
{"specs/rocky/approved/add-req-id-for-migrations-obj.rst":[{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"f420e9e1006ea74663e7980398164fbcbf8808d6","unresolved":false,"context_lines":[{"line_number":23,"context_line":"the details about why it failed."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"We may use instance action API to check for failure details, showing"},{"line_number":26,"context_line":"instance-action details will need req-id as an input parameter, But"},{"line_number":27,"context_line":"migrations obj does not include req-id field, it will be hard to connect"},{"line_number":28,"context_line":"to instance-actions."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Use Cases"},{"line_number":31,"context_line":"---------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_e72acde5","line":28,"range":{"start_line":26,"start_character":64,"end_line":28,"end_character":20},"updated":"2018-02-06 04:15:34.000000000","message":"Users and administrators can receive the request ID in the HTTP header of the response.\nSo is it really hard?","commit_id":"573b12005f952fea91f447cab9a808515ff4d3f3"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7e32ed5e24cd904a36994712538c036016f1909a","unresolved":false,"context_lines":[{"line_number":23,"context_line":"the details about why it failed."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"We may use instance action API to check for failure details, showing"},{"line_number":26,"context_line":"instance-action details will need req-id as an input parameter, But"},{"line_number":27,"context_line":"migrations obj does not include req-id field, it will be hard to connect"},{"line_number":28,"context_line":"to instance-actions."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Use Cases"},{"line_number":31,"context_line":"---------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_fff694f9","line":28,"range":{"start_line":26,"start_character":64,"end_line":28,"end_character":20},"in_reply_to":"3fa0c359_e72acde5","updated":"2018-02-08 03:15:11.000000000","message":"Maybe the problem description isn\u0027t clear here. I think what this is saying is that at time t1, user A performs a migration of server X with request id foo.\n\nLater, admin B wants to get instance action details about that migration, but to get the instance action details, they need a request id for the action, which isn\u0027t stored on the migration record.\n\nSo this proposes to store the original request id with the migration record so it can be used later to lookup the action details.","commit_id":"573b12005f952fea91f447cab9a808515ff4d3f3"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"f420e9e1006ea74663e7980398164fbcbf8808d6","unresolved":false,"context_lines":[{"line_number":61,"context_line":"REST API impact"},{"line_number":62,"context_line":"---------------"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"Add ``request_id`` field to ``migrations`` object and its DB schema."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Add an API microversion to the following APIs to include ``request_id`` in"},{"line_number":67,"context_line":"their response:"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_c7af9192","line":64,"range":{"start_line":64,"start_character":0,"end_line":64,"end_character":68},"updated":"2018-02-06 04:15:34.000000000","message":"It should be in the \"Data model impact\" section.","commit_id":"573b12005f952fea91f447cab9a808515ff4d3f3"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"f420e9e1006ea74663e7980398164fbcbf8808d6","unresolved":false,"context_lines":[{"line_number":133,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"Would need new in-tree functional and unit tests."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"Documentation Impact"},{"line_number":138,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":139,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_07a6d9c2","line":136,"updated":"2018-02-06 04:15:34.000000000","message":"Tempest tests are required because the microversion changes the response schema.\n\nhttps://docs.openstack.org/nova/latest/contributor/code-review.html#microversion-api","commit_id":"573b12005f952fea91f447cab9a808515ff4d3f3"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"f420e9e1006ea74663e7980398164fbcbf8808d6","unresolved":false,"context_lines":[{"line_number":141,"context_line":""},{"line_number":142,"context_line":"References"},{"line_number":143,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":""},{"line_number":146,"context_line":"History"},{"line_number":147,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_877ae921","line":144,"updated":"2018-02-06 04:15:34.000000000","message":"None? IMO, it is better to add the reference about request IDs.\n\ne.g.\nhttps://developer.openstack.org/api-guide/compute/faults.html","commit_id":"573b12005f952fea91f447cab9a808515ff4d3f3"}]}