)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2b7eb96ac5831b70343bf3a7c8355437ad9ea948","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Stephen Finucane \u003csfinucan@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-10-18 14:34:08 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"api-ref: \u0027vcpus_total\u0027 doesn\u0027t reflect overcommit ratio"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We could introduce an API microversion to correct this but,"},{"line_number":10,"context_line":"realistically, placement should be the go-to reference for this stuff"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_5253383b","line":7,"range":{"start_line":7,"start_character":31,"end_line":7,"end_character":55},"updated":"2018-10-18 13:58:55.000000000","message":"Is this also true for ram and disk?","commit_id":"08b1c71c90711bea10c6d50d13e27df5310b6c43"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"7d815abdfcef9e0112d085d4e97a53e51317453c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Stephen Finucane \u003csfinucan@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-10-18 14:34:08 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"api-ref: \u0027vcpus_total\u0027 doesn\u0027t reflect overcommit ratio"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We could introduce an API microversion to correct this but,"},{"line_number":10,"context_line":"realistically, placement should be the go-to reference for this stuff"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_f39a8373","line":7,"range":{"start_line":7,"start_character":31,"end_line":7,"end_character":55},"in_reply_to":"3f79a3b5_5253383b","updated":"2018-10-19 15:53:32.000000000","message":"Yup. Fixed.","commit_id":"08b1c71c90711bea10c6d50d13e27df5310b6c43"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"cc4b90bde5e66efb52359d14175dcf2a00018dc5","unresolved":false,"context_lines":[{"line_number":11,"context_line":"going forward."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Id8532d5a9480e4f71a9390e3b98886c85e27d003"},{"line_number":14,"context_line":"TrivialFix"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_4d163d4d","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":10},"updated":"2018-10-18 13:59:12.000000000","message":"I think you can remove this. I wouldn\u0027t consider this so trivial.","commit_id":"08b1c71c90711bea10c6d50d13e27df5310b6c43"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"7d815abdfcef9e0112d085d4e97a53e51317453c","unresolved":false,"context_lines":[{"line_number":11,"context_line":"going forward."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Id8532d5a9480e4f71a9390e3b98886c85e27d003"},{"line_number":14,"context_line":"TrivialFix"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_13a05fa4","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":10},"in_reply_to":"3f79a3b5_4d163d4d","updated":"2018-10-19 15:53:32.000000000","message":"Done","commit_id":"08b1c71c90711bea10c6d50d13e27df5310b6c43"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"1170eb6be2cb102887a3d4d5f50ed5e6d38053ee","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We could/should introduce an API microversion to correct this or remove"},{"line_number":10,"context_line":"it, given that placement should be the go-to reference for this stuff"},{"line_number":11,"context_line":"going forward. However, for now we take the path of least resistance and"},{"line_number":12,"context_line":"document the broken behavior."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Id8532d5a9480e4f71a9390e3b98886c85e27d003"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9fdfeff1_0b894c6d","line":12,"range":{"start_line":12,"start_character":13,"end_line":12,"end_character":29},"updated":"2019-03-01 02:57:39.000000000","message":"Didn\u0027t you have a related bug for this at one point? I remember working with someone from Red Hat about this, so maybe it was a downstream BZ.","commit_id":"f30aa064f9366a3164f44c0665d8d9ffb21b1276"}],"api-ref/source/os-hypervisors.inc":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"230bd63933990b002be0e92c1685217c16d8237b","unresolved":false,"context_lines":[{"line_number":147,"context_line":"this operation. Cloud providers can change these permissions through"},{"line_number":148,"context_line":"the ``policy.json`` file."},{"line_number":149,"context_line":""},{"line_number":150,"context_line":".. note::"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"   As noted, some of the parameters in the response representing totals do not"},{"line_number":153,"context_line":"   take allocation ratios into account. This can result in a disparity between"}],"source_content_type":"text/x-c++src","patch_set":2,"id":"3f79a3b5_6277bc8e","line":150,"range":{"start_line":150,"start_character":3,"end_line":150,"end_character":7},"updated":"2018-11-28 10:50:57.000000000","message":"or warning instead of just note ?","commit_id":"f30aa064f9366a3164f44c0665d8d9ffb21b1276"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0714c5941ed9081822c495951b44694418e6caa6","unresolved":false,"context_lines":[{"line_number":147,"context_line":"this operation. Cloud providers can change these permissions through"},{"line_number":148,"context_line":"the ``policy.json`` file."},{"line_number":149,"context_line":""},{"line_number":150,"context_line":".. note::"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"   As noted, some of the parameters in the response representing totals do not"},{"line_number":153,"context_line":"   take allocation ratios into account. This can result in a disparity between"}],"source_content_type":"text/x-c++src","patch_set":2,"id":"9fdfeff1_689343a4","line":150,"range":{"start_line":150,"start_character":3,"end_line":150,"end_character":7},"in_reply_to":"3f79a3b5_6277bc8e","updated":"2019-02-28 20:43:42.000000000","message":"humm, this note explains just normal behavior and I feel warning is a little bit strong for readers.\nI am OK to put this as a note.","commit_id":"f30aa064f9366a3164f44c0665d8d9ffb21b1276"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"50776c727bf6a2cb4cd12254a31500e5a99e8e21","unresolved":false,"context_lines":[{"line_number":152,"context_line":"   As noted, some of the parameters in the response representing totals do not"},{"line_number":153,"context_line":"   take allocation ratios into account. This can result in a disparity between"},{"line_number":154,"context_line":"   the totals and the usages. A more accurate representation of state can be"},{"line_number":155,"context_line":"   obtained using `placement`__."},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"   __ https://developer.openstack.org/api-ref/placement/#list-resource-provider-usages"},{"line_number":158,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":2,"id":"3f79a3b5_5c67d8ba","line":155,"range":{"start_line":155,"start_character":19,"end_line":155,"end_character":28},"updated":"2018-11-30 06:47:41.000000000","message":"nit: \u0027Placement API\u0027 would be better.","commit_id":"f30aa064f9366a3164f44c0665d8d9ffb21b1276"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"50776c727bf6a2cb4cd12254a31500e5a99e8e21","unresolved":false,"context_lines":[{"line_number":200,"context_line":"   As noted, some of the parameters in the response representing totals do not"},{"line_number":201,"context_line":"   take allocation ratios into account. This can result in a disparity between"},{"line_number":202,"context_line":"   the totals and the usages. A more accurate representation of state can be"},{"line_number":203,"context_line":"   obtained using `placement`__."},{"line_number":204,"context_line":""},{"line_number":205,"context_line":"   __ https://developer.openstack.org/api-ref/placement/#show-resource-provider-usages"},{"line_number":206,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":2,"id":"3f79a3b5_1c6160a3","line":203,"range":{"start_line":203,"start_character":19,"end_line":203,"end_character":28},"updated":"2018-11-30 06:47:41.000000000","message":"ditto","commit_id":"f30aa064f9366a3164f44c0665d8d9ffb21b1276"}],"api-ref/source/parameters.yaml":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2b7eb96ac5831b70343bf3a7c8355437ad9ea948","unresolved":false,"context_lines":[{"line_number":3595,"context_line":"  type: string"},{"line_number":3596,"context_line":"hypervisor_vcpus:"},{"line_number":3597,"context_line":"  description: |"},{"line_number":3598,"context_line":"    The number of vcpu in this hypervisor. This does not take overcommit"},{"line_number":3599,"context_line":"    ratio into account so there may be disparity between this and the used"},{"line_number":3600,"context_line":"    count."},{"line_number":3601,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_3219dc86","line":3598,"updated":"2018-10-18 13:58:55.000000000","message":"Would we want to reference https://developer.openstack.org/api-ref/placement/#show-resource-provider-inventory and mention the resource class is VCPU if you want to get the more detailed inventory information including allocation ratio?","commit_id":"08b1c71c90711bea10c6d50d13e27df5310b6c43"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"7d815abdfcef9e0112d085d4e97a53e51317453c","unresolved":false,"context_lines":[{"line_number":3595,"context_line":"  type: string"},{"line_number":3596,"context_line":"hypervisor_vcpus:"},{"line_number":3597,"context_line":"  description: |"},{"line_number":3598,"context_line":"    The number of vcpu in this hypervisor. This does not take overcommit"},{"line_number":3599,"context_line":"    ratio into account so there may be disparity between this and the used"},{"line_number":3600,"context_line":"    count."},{"line_number":3601,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_93c6af69","line":3598,"in_reply_to":"3f79a3b5_3219dc86","updated":"2018-10-19 15:53:32.000000000","message":"I stuck this in the API ref document instead","commit_id":"08b1c71c90711bea10c6d50d13e27df5310b6c43"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2b7eb96ac5831b70343bf3a7c8355437ad9ea948","unresolved":false,"context_lines":[{"line_number":3609,"context_line":"  in: body"},{"line_number":3610,"context_line":"  required: true"},{"line_number":3611,"context_line":"  type: integer"},{"line_number":3612,"context_line":"hypervisor_vcpus_used:"},{"line_number":3613,"context_line":"  description: |"},{"line_number":3614,"context_line":"    The number of vcpu used in this hypervisor."},{"line_number":3615,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_72fe7407","line":3612,"updated":"2018-10-18 13:58:55.000000000","message":"Similarly we have https://developer.openstack.org/api-ref/placement/#list-resource-provider-usages which would show the total VCPU allocations on this hypervisor, which would take into account allocation ratio.","commit_id":"08b1c71c90711bea10c6d50d13e27df5310b6c43"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"7d815abdfcef9e0112d085d4e97a53e51317453c","unresolved":false,"context_lines":[{"line_number":3609,"context_line":"  in: body"},{"line_number":3610,"context_line":"  required: true"},{"line_number":3611,"context_line":"  type: integer"},{"line_number":3612,"context_line":"hypervisor_vcpus_used:"},{"line_number":3613,"context_line":"  description: |"},{"line_number":3614,"context_line":"    The number of vcpu used in this hypervisor."},{"line_number":3615,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_33c31b5b","line":3612,"in_reply_to":"3f79a3b5_72fe7407","updated":"2018-10-19 15:53:32.000000000","message":"Ditto","commit_id":"08b1c71c90711bea10c6d50d13e27df5310b6c43"}]}