)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8b1a464907a696aaa43c1a7aa9962f8de41e99bc","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"We propose to add ``host`` field to"},{"line_number":18,"context_line":"``InstanceActionEvent`` object and related API responses."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I8534bd637e19eda66a901444f36394794078926b"},{"line_number":21,"context_line":"Blueprint: add-host-to-instance-action-events"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa0c359_3abb46c5","line":19,"updated":"2018-02-12 22:37:18.000000000","message":"Add the APIImpact tag here, per the template.","commit_id":"cb7ff3fbedfe08927eceaa4ba77c2d5dca1e89d5"},{"author":{"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},"change_message_id":"a7328f57505c1007ba539dc8132018917e3aa45a","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"We propose to add ``host`` field to"},{"line_number":18,"context_line":"``InstanceActionEvent`` object and related API responses."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I8534bd637e19eda66a901444f36394794078926b"},{"line_number":21,"context_line":"Blueprint: add-host-to-instance-action-events"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df7087c5_3e8dcdd2","line":19,"in_reply_to":"3fa0c359_3abb46c5","updated":"2018-03-13 06:48:54.000000000","message":"Done","commit_id":"cb7ff3fbedfe08927eceaa4ba77c2d5dca1e89d5"}],"specs/rocky/approved/add-host-to-instance-action-events.rst":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8b1a464907a696aaa43c1a7aa9962f8de41e99bc","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"* Add ``host`` field to ``InstanceActionEvent`` object."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Record the host info when the event is recorded."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* Add an API microversion to the"},{"line_number":46,"context_line":"  ``GET /servers/{UUID}/os-instance-actions/{REQ_ID}`` to include ``host`` in"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3fa0c359_ba58b687","line":43,"range":{"start_line":43,"start_character":2,"end_line":43,"end_character":50},"updated":"2018-02-12 22:37:18.000000000","message":"There are some events created in the nova-api and nova-conductor services, so what would the host be in those cases? Just CONF.host? Maybe that\u0027s what we\u0027ll use everywhere, I just thought about this being mostly useful for compute hosts, but we create events from other services too.","commit_id":"cb7ff3fbedfe08927eceaa4ba77c2d5dca1e89d5"},{"author":{"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},"change_message_id":"a7328f57505c1007ba539dc8132018917e3aa45a","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"* Add ``host`` field to ``InstanceActionEvent`` object."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Record the host info when the event is recorded."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* Add an API microversion to the"},{"line_number":46,"context_line":"  ``GET /servers/{UUID}/os-instance-actions/{REQ_ID}`` to include ``host`` in"}],"source_content_type":"text/x-rst","patch_set":2,"id":"df7087c5_72f85e7a","line":43,"range":{"start_line":43,"start_character":2,"end_line":43,"end_character":50},"in_reply_to":"3fa0c359_ba58b687","updated":"2018-03-13 06:48:54.000000000","message":"Yes, I think just CONF.host is OK, and all service should record the host info, because the actual api/conductor host will also make it easier for admins to locate where the api/conductor error happens, especially in the cluster deployment.","commit_id":"cb7ff3fbedfe08927eceaa4ba77c2d5dca1e89d5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8b1a464907a696aaa43c1a7aa9962f8de41e99bc","unresolved":false,"context_lines":[{"line_number":68,"context_line":"Security impact"},{"line_number":69,"context_line":"---------------"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"None"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"Notifications impact"},{"line_number":74,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3fa0c359_ba067651","line":71,"updated":"2018-02-12 22:37:18.000000000","message":"You should mention something about policy rules here. There is some history here too.\n\nInstance action events could only be shown, by default policy, for admins until microversion 2.51:\n\nhttps://developer.openstack.org/api-ref/compute/#show-server-action-details\n\nThen the admin or owner of the instance could see the events, but not the traceback (that still defaults to admin-only policy).\n\nSo I think we want the same logic for exposing the \u0027host\u0027 field in the action as we have for exposing the \u0027traceback\u0027 which is admin-only by default.\n\nhttps://github.com/openstack/nova/blob/a5a569d6670c29f995b1e8a2a2013471d57469d7/nova/api/openstack/compute/instance_actions.py#L140","commit_id":"cb7ff3fbedfe08927eceaa4ba77c2d5dca1e89d5"},{"author":{"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},"change_message_id":"a7328f57505c1007ba539dc8132018917e3aa45a","unresolved":false,"context_lines":[{"line_number":68,"context_line":"Security impact"},{"line_number":69,"context_line":"---------------"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"None"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"Notifications impact"},{"line_number":74,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"df7087c5_72ca9e4e","line":71,"in_reply_to":"3fa0c359_ba067651","updated":"2018-03-13 06:48:54.000000000","message":"Done","commit_id":"cb7ff3fbedfe08927eceaa4ba77c2d5dca1e89d5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8b1a464907a696aaa43c1a7aa9962f8de41e99bc","unresolved":false,"context_lines":[{"line_number":78,"context_line":"Other end user impact"},{"line_number":79,"context_line":"---------------------"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Python-novaclient will be modified to handle the new microversion to"},{"line_number":82,"context_line":"include ``host`` field in the response."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"Performance Impact"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3fa0c359_9af7fa59","line":81,"updated":"2018-02-12 22:37:18.000000000","message":"We likely don\u0027t need to even change anything in python-novaclient except say that the new microversion is supported, because the \u0027nova instance-action\u0027 CLI just dumps whatever is in the events response.","commit_id":"cb7ff3fbedfe08927eceaa4ba77c2d5dca1e89d5"},{"author":{"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},"change_message_id":"a7328f57505c1007ba539dc8132018917e3aa45a","unresolved":false,"context_lines":[{"line_number":78,"context_line":"Other end user impact"},{"line_number":79,"context_line":"---------------------"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Python-novaclient will be modified to handle the new microversion to"},{"line_number":82,"context_line":"include ``host`` field in the response."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"Performance Impact"}],"source_content_type":"text/x-rst","patch_set":2,"id":"df7087c5_d2de0a10","line":81,"in_reply_to":"3fa0c359_9af7fa59","updated":"2018-03-13 06:48:54.000000000","message":"OK, I will remove this.","commit_id":"cb7ff3fbedfe08927eceaa4ba77c2d5dca1e89d5"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"6878d0add43637f1658b3cff360c448bcd2a6fa2","unresolved":false,"context_lines":[{"line_number":56,"context_line":"------------"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Leave it as is."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Data model impact"},{"line_number":61,"context_line":"-----------------"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"df7087c5_cb1f2359","line":59,"updated":"2018-03-13 12:52:02.000000000","message":"The upper-layer management software (or the admins) can listen to the versioned instance notifications [1] . The notification contains the node and the host of the instance [2]\n\n[1] https://docs.openstack.org/nova/latest/reference/notifications.html#existing-versioned-notifications\n[2] https://github.com/openstack/nova/blob/master/doc/notification_samples/common_payloads/InstanceActionPayload.json#L13","commit_id":"d14d9eba22c5c47ae9c1a4c0d532d245995a7d4b"},{"author":{"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},"change_message_id":"943409cb09682656e5e606674a53d7d47d6b9ef9","unresolved":false,"context_lines":[{"line_number":56,"context_line":"------------"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Leave it as is."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Data model impact"},{"line_number":61,"context_line":"-----------------"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"df7087c5_ab6cfdcd","line":59,"in_reply_to":"df7087c5_917f1e14","updated":"2018-03-15 02:31:08.000000000","message":"@gibi Thanks, :), Done.","commit_id":"d14d9eba22c5c47ae9c1a4c0d532d245995a7d4b"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"b613a8feefbff549e67b4672453b55518f9a76cc","unresolved":false,"context_lines":[{"line_number":56,"context_line":"------------"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Leave it as is."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Data model impact"},{"line_number":61,"context_line":"-----------------"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"df7087c5_917f1e14","line":59,"in_reply_to":"df7087c5_cb1f2359","updated":"2018-03-13 14:00:55.000000000","message":"+1","commit_id":"d14d9eba22c5c47ae9c1a4c0d532d245995a7d4b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"14590b3236e64a7c0cc08b261a2c31d831629865","unresolved":false,"context_lines":[{"line_number":84,"context_line":"            \"start_time\": \"2017-12-07T11:07:06.251280\","},{"line_number":85,"context_line":"            \"traceback\": null,"},{"line_number":86,"context_line":"            \"host\": \"host1\","},{"line_number":87,"context_line":"            \"hostId\": \"8a8d66db9eed58f2b1283d23acc9a32691290b603a716d81d8ed8c4e\""},{"line_number":88,"context_line":"            }"},{"line_number":89,"context_line":"        ],"},{"line_number":90,"context_line":"        \"instance_uuid\": \"b48316c5-71e8-45e4-9884-6c78055b9b13\","}],"source_content_type":"text/x-rst","patch_set":4,"id":"df7087c5_4b755331","line":87,"updated":"2018-03-13 13:08:45.000000000","message":"OK so this would be generated the same way that the hostId response parameter is generated for showing server details:\n\nhttps://github.com/openstack/nova/blob/9a497671342f0f15538b68bb7ee8f0c7ba3e66fe/nova/api/openstack/compute/views/servers.py#L236\n\nWe have the host and project_id information in the event (and parent action) so we should be able to do this.\n\nI don\u0027t really understand *why* we need to add a hashed hostId to the response, however. I know it came up during the PTG but I didn\u0027t really understand the reason for it, or if there was a misunderstanding.","commit_id":"d14d9eba22c5c47ae9c1a4c0d532d245995a7d4b"},{"author":{"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},"change_message_id":"943409cb09682656e5e606674a53d7d47d6b9ef9","unresolved":false,"context_lines":[{"line_number":84,"context_line":"            \"start_time\": \"2017-12-07T11:07:06.251280\","},{"line_number":85,"context_line":"            \"traceback\": null,"},{"line_number":86,"context_line":"            \"host\": \"host1\","},{"line_number":87,"context_line":"            \"hostId\": \"8a8d66db9eed58f2b1283d23acc9a32691290b603a716d81d8ed8c4e\""},{"line_number":88,"context_line":"            }"},{"line_number":89,"context_line":"        ],"},{"line_number":90,"context_line":"        \"instance_uuid\": \"b48316c5-71e8-45e4-9884-6c78055b9b13\","}],"source_content_type":"text/x-rst","patch_set":4,"id":"df7087c5_4b8a016a","line":87,"in_reply_to":"df7087c5_4b755331","updated":"2018-03-15 02:31:08.000000000","message":"Agree with @matt, I prefer to keep consistent with GET /servers/{id}, and the format like [1] using project_id and host name.\n\nhttps://github.com/openstack/nova/blob/master/nova/api/openstack/compute/views/servers.py#L236-L242","commit_id":"d14d9eba22c5c47ae9c1a4c0d532d245995a7d4b"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"b613a8feefbff549e67b4672453b55518f9a76cc","unresolved":false,"context_lines":[{"line_number":84,"context_line":"            \"start_time\": \"2017-12-07T11:07:06.251280\","},{"line_number":85,"context_line":"            \"traceback\": null,"},{"line_number":86,"context_line":"            \"host\": \"host1\","},{"line_number":87,"context_line":"            \"hostId\": \"8a8d66db9eed58f2b1283d23acc9a32691290b603a716d81d8ed8c4e\""},{"line_number":88,"context_line":"            }"},{"line_number":89,"context_line":"        ],"},{"line_number":90,"context_line":"        \"instance_uuid\": \"b48316c5-71e8-45e4-9884-6c78055b9b13\","}],"source_content_type":"text/x-rst","patch_set":4,"id":"df7087c5_519c2669","line":87,"in_reply_to":"df7087c5_4b755331","updated":"2018-03-13 14:00:55.000000000","message":"If I remembered correctly, this is because that \"host\" info is available for normal users in \"HostID\" format, so it is better to do the same here. One thing I want to ask here is that do we have to do exactly the same thing here? I mean in the instance response, we have both \"host\" and \"hostID\" in the response for admin and only \"hostID\" for non-admin. Could we just have one \"host\" field here, but the value is \"host\" for admin and \"hostID\" for non-admin?","commit_id":"d14d9eba22c5c47ae9c1a4c0d532d245995a7d4b"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"cecb2749efaa6066c4404370e1cff94fa86c851d","unresolved":false,"context_lines":[{"line_number":84,"context_line":"            \"start_time\": \"2017-12-07T11:07:06.251280\","},{"line_number":85,"context_line":"            \"traceback\": null,"},{"line_number":86,"context_line":"            \"host\": \"host1\","},{"line_number":87,"context_line":"            \"hostId\": \"8a8d66db9eed58f2b1283d23acc9a32691290b603a716d81d8ed8c4e\""},{"line_number":88,"context_line":"            }"},{"line_number":89,"context_line":"        ],"},{"line_number":90,"context_line":"        \"instance_uuid\": \"b48316c5-71e8-45e4-9884-6c78055b9b13\","}],"source_content_type":"text/x-rst","patch_set":4,"id":"df7087c5_b5440476","line":87,"in_reply_to":"df7087c5_4b8a016a","updated":"2018-03-19 14:14:21.000000000","message":"What I was describing is how I thought it worked in GET on /servers/id, fwiw, but looking I see now that the real hostname is an extended attribute instead. So, makes sense now.","commit_id":"d14d9eba22c5c47ae9c1a4c0d532d245995a7d4b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ce7ceaebfdcb245d3ed6d04609a4d1035e7872e1","unresolved":false,"context_lines":[{"line_number":84,"context_line":"            \"start_time\": \"2017-12-07T11:07:06.251280\","},{"line_number":85,"context_line":"            \"traceback\": null,"},{"line_number":86,"context_line":"            \"host\": \"host1\","},{"line_number":87,"context_line":"            \"hostId\": \"8a8d66db9eed58f2b1283d23acc9a32691290b603a716d81d8ed8c4e\""},{"line_number":88,"context_line":"            }"},{"line_number":89,"context_line":"        ],"},{"line_number":90,"context_line":"        \"instance_uuid\": \"b48316c5-71e8-45e4-9884-6c78055b9b13\","}],"source_content_type":"text/x-rst","patch_set":4,"id":"df7087c5_806b4583","line":87,"in_reply_to":"df7087c5_519c2669","updated":"2018-03-13 15:01:18.000000000","message":"\u003e Could we just have one \"host\" field here, but the value is \"host\" for admin and \"hostID\" for non-admin?\n\nI think that\u0027s confusing and we\u0027re better off just doing the same thing as we have in the server GET response with distinct response parameters.","commit_id":"d14d9eba22c5c47ae9c1a4c0d532d245995a7d4b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"eb96469644a2157141861c4113445bcce0a4c739","unresolved":false,"context_lines":[{"line_number":84,"context_line":"            \"start_time\": \"2017-12-07T11:07:06.251280\","},{"line_number":85,"context_line":"            \"traceback\": null,"},{"line_number":86,"context_line":"            \"host\": \"host1\","},{"line_number":87,"context_line":"            \"hostId\": \"8a8d66db9eed58f2b1283d23acc9a32691290b603a716d81d8ed8c4e\""},{"line_number":88,"context_line":"            }"},{"line_number":89,"context_line":"        ],"},{"line_number":90,"context_line":"        \"instance_uuid\": \"b48316c5-71e8-45e4-9884-6c78055b9b13\","}],"source_content_type":"text/x-rst","patch_set":4,"id":"df7087c5_8be35dd9","line":87,"in_reply_to":"df7087c5_542f2f8d","updated":"2018-03-13 21:47:50.000000000","message":"\u003e Are you saying only have one host field and hash it or not based on is-admin policy?\n\nYes I think that\u0027s what Kevin is suggesting, and I don\u0027t like that idea. If we\u0027re going to have two fields, I\u0027d rather just be consistent with how these fields work in the GET /servers/{id} response.","commit_id":"d14d9eba22c5c47ae9c1a4c0d532d245995a7d4b"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"21b3528cac0197d386fdeb4c65e76489b4a00334","unresolved":false,"context_lines":[{"line_number":84,"context_line":"            \"start_time\": \"2017-12-07T11:07:06.251280\","},{"line_number":85,"context_line":"            \"traceback\": null,"},{"line_number":86,"context_line":"            \"host\": \"host1\","},{"line_number":87,"context_line":"            \"hostId\": \"8a8d66db9eed58f2b1283d23acc9a32691290b603a716d81d8ed8c4e\""},{"line_number":88,"context_line":"            }"},{"line_number":89,"context_line":"        ],"},{"line_number":90,"context_line":"        \"instance_uuid\": \"b48316c5-71e8-45e4-9884-6c78055b9b13\","}],"source_content_type":"text/x-rst","patch_set":4,"id":"df7087c5_542f2f8d","line":87,"in_reply_to":"df7087c5_806b4583","updated":"2018-03-13 17:15:16.000000000","message":"I\u0027m not sure I understand this. Are you saying only have one host field and hash it or not based on is-admin policy?","commit_id":"d14d9eba22c5c47ae9c1a4c0d532d245995a7d4b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e519c4af6fc1dcc38dd0b9e1c099e8ea98aff386","unresolved":false,"context_lines":[{"line_number":52,"context_line":"  ``GET /servers/{UUID}/os-instance-actions/{REQ_ID}`` to include ``host`` in"},{"line_number":53,"context_line":"  the response."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"Alternatives"},{"line_number":56,"context_line":"------------"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"As alternative the upper-layer management software (or the admins) can listen"}],"source_content_type":"text/x-rst","patch_set":5,"id":"df7087c5_7512ed64","line":55,"updated":"2018-03-19 13:36:45.000000000","message":"Or if you\u0027re indexing logs in logstash by the hostname, and you have the request ID (from the action record) then you could query logstash for the logs for the failed action, but I won\u0027t presume to know how people are doing everything or using the action event records.","commit_id":"afdc828db3c9d0205b6ded268db24f5cdf857fa6"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"cecb2749efaa6066c4404370e1cff94fa86c851d","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"Add a new microversion to"},{"line_number":72,"context_line":"``GET /servers/{server_id}/os\u003dinstance-actions/{req_id}`` API to include"},{"line_number":73,"context_line":"the ``host`` field for admin and an obfuscated hashed host id \"hostId\" for"},{"line_number":74,"context_line":"admin and non-admin users."},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"* For admin users::"}],"source_content_type":"text/x-rst","patch_set":5,"id":"df7087c5_d53e58d9","line":73,"range":{"start_line":73,"start_character":62,"end_line":73,"end_character":70},"updated":"2018-03-19 14:14:21.000000000","message":"Shouldn\u0027t this be ``hostId`` for consistency with ``host``?","commit_id":"afdc828db3c9d0205b6ded268db24f5cdf857fa6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8155a531d606f82e55e266285c9ddbf5374e09b9","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"Add a new microversion to"},{"line_number":72,"context_line":"``GET /servers/{server_id}/os\u003dinstance-actions/{req_id}`` API to include"},{"line_number":73,"context_line":"the ``host`` field for admin and an obfuscated hashed host id \"hostId\" for"},{"line_number":74,"context_line":"admin and non-admin users."},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"* For admin users::"}],"source_content_type":"text/x-rst","patch_set":5,"id":"df7087c5_d811adae","line":73,"range":{"start_line":73,"start_character":62,"end_line":73,"end_character":70},"in_reply_to":"df7087c5_d53e58d9","updated":"2018-03-19 14:37:56.000000000","message":"I don\u0027t understand the question. This is just using the same parameter names (and meanings) as in GET /servers/detail, well except the former in the servers API is actually \"OS-EXT-SRV-ATTR:host\". The description for hostId in the servers API is:\n\n\"An ID string representing the host. This is a hashed value so will not actually look like a hostname, and is hashed with data from the project_id, so the same physical host as seen by two different project_ids, will be different. It is useful when within the same project you need to determine if two instances are on the same or different physical hosts for the purposes of availability or performance.\"\n\nWhich, again, I\u0027m not really sure why people at the PTG wanted this on the action events API since it\u0027s usage in the servers API really sounds like it\u0027s just for the user to use with the same_host/different_host scheduler hints.","commit_id":"afdc828db3c9d0205b6ded268db24f5cdf857fa6"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"cecb2749efaa6066c4404370e1cff94fa86c851d","unresolved":false,"context_lines":[{"line_number":181,"context_line":"  host that the event occurs on."},{"line_number":182,"context_line":"* Add a new microversion to"},{"line_number":183,"context_line":"  ``GET /servers/{server_id}/os\u003dinstance-actions/{req_id}`` API to include"},{"line_number":184,"context_line":"  the ``host`` field for admin and an ``obfuscated-hashed-host-id`` for"},{"line_number":185,"context_line":"  non-admin users."},{"line_number":186,"context_line":"* Adopt the new microversion in python-novaclient."},{"line_number":187,"context_line":"* Add related tests."}],"source_content_type":"text/x-rst","patch_set":5,"id":"df7087c5_d5173851","line":184,"range":{"start_line":184,"start_character":40,"end_line":184,"end_character":65},"updated":"2018-03-19 14:14:21.000000000","message":"Shouldn\u0027t this be hostId (i.e. the name of the actual key you\u0027re going to add)?","commit_id":"afdc828db3c9d0205b6ded268db24f5cdf857fa6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8155a531d606f82e55e266285c9ddbf5374e09b9","unresolved":false,"context_lines":[{"line_number":181,"context_line":"  host that the event occurs on."},{"line_number":182,"context_line":"* Add a new microversion to"},{"line_number":183,"context_line":"  ``GET /servers/{server_id}/os\u003dinstance-actions/{req_id}`` API to include"},{"line_number":184,"context_line":"  the ``host`` field for admin and an ``obfuscated-hashed-host-id`` for"},{"line_number":185,"context_line":"  non-admin users."},{"line_number":186,"context_line":"* Adopt the new microversion in python-novaclient."},{"line_number":187,"context_line":"* Add related tests."}],"source_content_type":"text/x-rst","patch_set":5,"id":"df7087c5_38e309a7","line":184,"range":{"start_line":184,"start_character":40,"end_line":184,"end_character":65},"in_reply_to":"df7087c5_d5173851","updated":"2018-03-19 14:37:56.000000000","message":"Yeah this should be hostId like above.","commit_id":"afdc828db3c9d0205b6ded268db24f5cdf857fa6"}]}
