)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"ff39031c79e87845116eb3a43f23f6c66cc232d7","unresolved":false,"context_lines":[{"line_number":10,"context_line":"use the Bifrost dynamic inventory. This runs bifrost_inventory.py which"},{"line_number":11,"context_line":"gathers node and port information for all bare metal hosts. Because this"},{"line_number":12,"context_line":"gets executed for each node in our inventory, it produces a large number"},{"line_number":13,"context_line":"of API requests to Ironic, which grows exponentially as the number of"},{"line_number":14,"context_line":"hosts increases."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Halve the number of calls using the BIFROST_NODE_NAMES environment"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"58445bc6_499dbd38","line":13,"range":{"start_line":13,"start_character":39,"end_line":13,"end_character":52},"updated":"2023-05-15 12:20:21.000000000","message":"with the square of the number of nodes? 😋 (don\u0027t need to change it)","commit_id":"ea9b7050b0e3e71f2b49d89d7d9e5f0d7bbe6774"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"869696f6ae9084562aeafd864d6cf5b2d21c5044","unresolved":false,"context_lines":[{"line_number":10,"context_line":"use the Bifrost dynamic inventory. This runs bifrost_inventory.py which"},{"line_number":11,"context_line":"gathers node and port information for all bare metal hosts. Because this"},{"line_number":12,"context_line":"gets executed for each node in our inventory, it produces a large number"},{"line_number":13,"context_line":"of API requests to Ironic, which grows exponentially as the number of"},{"line_number":14,"context_line":"hosts increases."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Halve the number of calls using the BIFROST_NODE_NAMES environment"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"e5566c97_1d37c59b","line":13,"range":{"start_line":13,"start_character":39,"end_line":13,"end_character":52},"in_reply_to":"58445bc6_499dbd38","updated":"2023-05-15 19:28:26.000000000","message":"Good point 😄","commit_id":"ea9b7050b0e3e71f2b49d89d7d9e5f0d7bbe6774"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"8c0402be0cf1bb2e5a69001cdb6f92e000ae8c52","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b5caa569_51322de2","updated":"2023-05-11 13:58:12.000000000","message":"Is this also useful in ``openstack-inventory-discover.yml``, as it is calling ``bifrost_inventory.py`` here too? \n\nhttps://opendev.org/openstack/kayobe/src/commit/805414ef9d29deddd432fd3f591734a5f05da174/ansible/overcloud-inventory-discover.yml#L16","commit_id":"63b1aba5f2e825ef7c5182ab4f6d36bc5d0e7fa3"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"499381d37001b4a5c3eaf02ed29d87eb7776390a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0365dd98_c9087f84","updated":"2023-05-11 19:38:20.000000000","message":"Needs more testing.","commit_id":"63b1aba5f2e825ef7c5182ab4f6d36bc5d0e7fa3"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"f71b0014860215c034916ced3b2c912b3509c81f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0a2d0bf0_1ba5c865","in_reply_to":"0365dd98_c9087f84","updated":"2023-05-11 19:46:15.000000000","message":"One task in overcloud-provision.yml shouldn\u0027t have been using BIFROST_NODE_NAMES. This was breaking provisioning of multiple nodes at once. Fixed in the updated patch.","commit_id":"63b1aba5f2e825ef7c5182ab4f6d36bc5d0e7fa3"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"0d630c8631e1d84aa7852a69b413ad40c9301774","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ca8ad7ce_e54baba4","in_reply_to":"b5caa569_51322de2","updated":"2023-05-11 14:35:26.000000000","message":"No, because this is called just once (hosts: seed) and we want the full inventory to be returned to us.","commit_id":"63b1aba5f2e825ef7c5182ab4f6d36bc5d0e7fa3"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"f1d17f8be2f4f40cce29a9126e60ab9fa86a2077","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b75dfa8d_0afd3f78","updated":"2023-08-29 15:08:59.000000000","message":"Accounting for previous +2","commit_id":"3e03884c476604a717c1fa5fb35771f60e39dd8f"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"8cbae14fc969136616df6564e8bfa63d952c42d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"cdc560b1_7a5914e3","updated":"2023-08-30 10:11:30.000000000","message":"recheck\n\nCI error is fixed now","commit_id":"3e03884c476604a717c1fa5fb35771f60e39dd8f"}]}