)]}'
{"nova/virt/ironic/driver.py":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"b08471116fa1c0a59b93001da6fe4cc15f57c80e","unresolved":true,"context_lines":[{"line_number":862,"context_line":"        # _refresh_cache() when the cache is recent enough.  The TTL is"},{"line_number":863,"context_line":"        # configurable via [ironic] node_cache_min_refresh_interval; the"},{"line_number":864,"context_line":"        # periodic update_available_resource task always falls outside that"},{"line_number":865,"context_line":"        # window so it still gets a fresh view on every invocation."},{"line_number":866,"context_line":"        min_interval \u003d CONF.ironic.node_cache_min_refresh_interval"},{"line_number":867,"context_line":"        if (min_interval \u003e 0 and"},{"line_number":868,"context_line":"                self.node_cache and"}],"source_content_type":"text/x-python","patch_set":1,"id":"759acae4_971b4cd4","line":865,"updated":"2026-03-16 16:26:55.000000000","message":"This is going to have really nasty side effects around when Ironic nodes return to being available after cleaning. I think this would only really be safe if paired with an Ironic callback of sorts when cleaning was completed.","commit_id":"9fed2d15eb6ce09a9508d43e61b89534d7d4228d"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"fcf97d823b50fb20f7d3cfacb9e60d3094dafd04","unresolved":true,"context_lines":[{"line_number":862,"context_line":"        # _refresh_cache() when the cache is recent enough.  The TTL is"},{"line_number":863,"context_line":"        # configurable via [ironic] node_cache_min_refresh_interval; the"},{"line_number":864,"context_line":"        # periodic update_available_resource task always falls outside that"},{"line_number":865,"context_line":"        # window so it still gets a fresh view on every invocation."},{"line_number":866,"context_line":"        min_interval \u003d CONF.ironic.node_cache_min_refresh_interval"},{"line_number":867,"context_line":"        if (min_interval \u003e 0 and"},{"line_number":868,"context_line":"                self.node_cache and"}],"source_content_type":"text/x-python","patch_set":1,"id":"44353aae_7124c6d4","line":865,"in_reply_to":"759acae4_971b4cd4","updated":"2026-03-16 16:41:55.000000000","message":"ya im not sure about this my inclination is to fix the note above or remove the cache entirly since im not sure we need this if we finally remove the peer_list which is overdue.\n\n\nanyway https://review.opendev.org/c/openstack/nova/+/980676/1 and \nhttps://review.opendev.org/c/openstack/nova/+/980679/1 are the real performance wins this was just somethjing that was obseved as beign called multiple time durign agent startup.","commit_id":"9fed2d15eb6ce09a9508d43e61b89534d7d4228d"}]}