)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"5fcb3e5c6d508967e6970299779c0b71340ef602","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"aec4d724_23ab441a","updated":"2022-10-31 08:59:34.000000000","message":"FWIW, this looks like a very good idea.\n\nI feel like my spec around ironic partitions needs similar logic to detect \"strange\" changes, like the ones described here.","commit_id":"82affbfe5973236f8c1de417259a838a1f9bea56"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"a4fad41c72f33a3bd77f59f782806dc88cf57501","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"2cf38efb_6ee1d989","updated":"2022-11-15 10:49:48.000000000","message":"This is a backlog spec, not a real spec, so we don\u0027t need to be nitpicky here, this is rather just a documented direction.\n\n","commit_id":"3256eda85f6de908702482f2ef65ad15c58ea867"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"faa81539ef2654c3ff0b3eea542fc1c6aa571ebb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"dfa70075_c1600594","updated":"2022-11-15 13:14:04.000000000","message":"i agree with sylvain\n\nlet merge this as a direction to take and  disucss each part in detail in the follow up specs.\n\nwe shoudl feel free to reviese this ether in teh follow up specs or over time based on changes in direction and future ptg/mailing list conversations but in terms of captureing the problem statement and posible direction to imporve thing si htink this is fine as is.","commit_id":"3256eda85f6de908702482f2ef65ad15c58ea867"}],"specs/antelope/approved/robustify-compute-hostnames.rst":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"5fcb3e5c6d508967e6970299779c0b71340ef602","unresolved":true,"context_lines":[{"line_number":16,"context_line":"compute nodes. This spec aims to address this limitation, at least"},{"line_number":17,"context_line":"from the perspective of being able to detect an accidental change and"},{"line_number":18,"context_line":"avoiding catastrophe in the database that can currently result from a"},{"line_number":19,"context_line":"hostname change, whether intentional or not."},{"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":"b7a2f138_0fcad6ed","line":19,"updated":"2022-10-31 08:59:34.000000000","message":"Ah, nice.","commit_id":"82affbfe5973236f8c1de417259a838a1f9bea56"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"5fcb3e5c6d508967e6970299779c0b71340ef602","unresolved":true,"context_lines":[{"line_number":27,"context_line":"#. As the routing key for communicating with a compute node over RPC"},{"line_number":28,"context_line":"#. As the link between the instance, service and compute node objects"},{"line_number":29,"context_line":"   in the database"},{"line_number":30,"context_line":"#. For neutron to bind ports to the proper hostname"},{"line_number":31,"context_line":"#. For cinder to export a volume to the proper host"},{"line_number":32,"context_line":"#. As the resource provider name in placement (this actually comes"},{"line_number":33,"context_line":"   from libvirt\u0027s notion of the hostname, not ``CONF.host``)"}],"source_content_type":"text/x-rst","patch_set":2,"id":"e95a8002_ba9f8b02","line":30,"updated":"2022-10-31 08:59:34.000000000","message":"Maybe worth nothing that has to match neutron agent config, in some cases?","commit_id":"82affbfe5973236f8c1de417259a838a1f9bea56"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"21f015f4bd5ea79c3916d3ea20f8bd8f0fb8f74e","unresolved":false,"context_lines":[{"line_number":27,"context_line":"#. As the routing key for communicating with a compute node over RPC"},{"line_number":28,"context_line":"#. As the link between the instance, service and compute node objects"},{"line_number":29,"context_line":"   in the database"},{"line_number":30,"context_line":"#. For neutron to bind ports to the proper hostname"},{"line_number":31,"context_line":"#. For cinder to export a volume to the proper host"},{"line_number":32,"context_line":"#. As the resource provider name in placement (this actually comes"},{"line_number":33,"context_line":"   from libvirt\u0027s notion of the hostname, not ``CONF.host``)"}],"source_content_type":"text/x-rst","patch_set":2,"id":"c12eff49_f6d928f9","line":30,"in_reply_to":"e95a8002_ba9f8b02","updated":"2022-10-31 17:33:31.000000000","message":"Ack","commit_id":"82affbfe5973236f8c1de417259a838a1f9bea56"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"5fcb3e5c6d508967e6970299779c0b71340ef602","unresolved":true,"context_lines":[{"line_number":131,"context_line":""},{"line_number":132,"context_line":"We already link PCI allocations for instances to the compute node by"},{"line_number":133,"context_line":"id, even though the instance itself is linked via hostname. This"},{"line_number":134,"context_line":"discrepancy makes it easy to get one out of sync with the other."},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"Potential Changes in the future"},{"line_number":137,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":2,"id":"925d2560_af231e58","line":134,"updated":"2022-10-31 08:59:34.000000000","message":"Hmm, good point!","commit_id":"82affbfe5973236f8c1de417259a838a1f9bea56"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"21f015f4bd5ea79c3916d3ea20f8bd8f0fb8f74e","unresolved":true,"context_lines":[{"line_number":131,"context_line":""},{"line_number":132,"context_line":"We already link PCI allocations for instances to the compute node by"},{"line_number":133,"context_line":"id, even though the instance itself is linked via hostname. This"},{"line_number":134,"context_line":"discrepancy makes it easy to get one out of sync with the other."},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"Potential Changes in the future"},{"line_number":137,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":2,"id":"62c3f091_76d5ca05","line":134,"in_reply_to":"925d2560_af231e58","updated":"2022-10-31 17:33:31.000000000","message":"Ask me how I know :)","commit_id":"82affbfe5973236f8c1de417259a838a1f9bea56"}]}
