)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"aa00e7717e8cc66a4d029b941953683859934e6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f29994dd_0330a545","updated":"2025-10-08 10:43:39.000000000","message":"/me wonders if this might be one of those settings that’s better left to operators to tune, as a power sync config would unlikely go unnoticed. Perhaps a docs update... I imagine you likely considered these points before settling on this change, and I’d love to understand the thinking behind it.","commit_id":"aa6059eb5a0f2e2acd24bf5f3085dd284e76236d"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"df3916c92a4feb6dfdfb1d9022be8a52a37d7649","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d2769688_0a2c4a5f","updated":"2025-09-25 17:18:10.000000000","message":"Consider this a +2, but want to ensure there is discussion.","commit_id":"aa6059eb5a0f2e2acd24bf5f3085dd284e76236d"},{"author":{"_account_id":14228,"name":"Clif Houck","email":"me@clifhouck.com","username":"clif_h"},"change_message_id":"c2664bb9b79428539f35192c189e8c2263a615d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"86fec559_eb43a084","updated":"2025-10-02 17:01:49.000000000","message":"Seems fine on the surface. \n\nIs it possible to re-use BMC connections if connection slots are limited?","commit_id":"aa6059eb5a0f2e2acd24bf5f3085dd284e76236d"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"766382ec79c272b3ea29994745506ac25bfcbe3c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"04765daa_d24f2914","in_reply_to":"86fec559_eb43a084","updated":"2025-10-03 16:22:23.000000000","message":"The code in the sushy driver does this when in auto or session based auth mode.","commit_id":"aa6059eb5a0f2e2acd24bf5f3085dd284e76236d"},{"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":"35b8878727b34f74e5611b832374946210ba2293","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"39c37ba1_343c2525","in_reply_to":"f29994dd_0330a545","updated":"2025-10-08 16:03:24.000000000","message":"So this is basically a \"do a thing for each node\" action. This means at scale, a conductor has to loop through each node it manages -- ideally before $power_sync_interval has passed.\n\nGiven how well the rest of the stack scales, this setting should help us ensure that we scale reasonable well -- the worse case tradeoff of not noticing a non-ironic power state change for an extra minute is not a major issue IMO.","commit_id":"aa6059eb5a0f2e2acd24bf5f3085dd284e76236d"}],"ironic/conf/conductor.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":"bc389efbbd4df382aef3f4a4518ed6efb118e00e","unresolved":true,"context_lines":[{"line_number":52,"context_line":"                      \u0027of a conductor. A conductor is considered inactive \u0027"},{"line_number":53,"context_line":"                      \u0027when this time has been exceeded.\u0027)),"},{"line_number":54,"context_line":"    cfg.IntOpt(\u0027sync_power_state_interval\u0027,"},{"line_number":55,"context_line":"               default\u003d300,"},{"line_number":56,"context_line":"               help\u003d_(\u0027Interval between syncing the node power state to the \u0027"},{"line_number":57,"context_line":"                      \u0027database, in seconds. Set to 0 to disable syncing.\u0027)),"},{"line_number":58,"context_line":"    cfg.IntOpt(\u0027check_provision_state_interval\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"226f2aa5_0f118268","line":55,"updated":"2025-09-17 21:27:00.000000000","message":"obviously may be controversial, but wanted to have the discussion.\n\nCost of power_state being wrong for a short period of time: small\nCost of querying a BMC every minute when some BMCs have limited connection counts: huge","commit_id":"3ec6c907343dc4d692653ca67d52b8963f12c9ef"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"8c078d43741d998f282bef6584f9fa49e3410536","unresolved":true,"context_lines":[{"line_number":54,"context_line":"    cfg.IntOpt(\u0027sync_power_state_interval\u0027,"},{"line_number":55,"context_line":"               default\u003d300,"},{"line_number":56,"context_line":"               help\u003d_(\u0027Interval between syncing the node power state to the \u0027"},{"line_number":57,"context_line":"                      \u0027database, in seconds. Set to 0 to disable syncing.\u0027)),"},{"line_number":58,"context_line":"    cfg.IntOpt(\u0027check_provision_state_interval\u0027,"},{"line_number":59,"context_line":"               default\u003d60,"},{"line_number":60,"context_line":"               min\u003d0,"}],"source_content_type":"text/x-python","patch_set":1,"id":"2c79a311_03a5085d","line":57,"updated":"2025-09-17 21:32:02.000000000","message":"I feel like this is a bit of a jump, maybe we should provide basic guidance on the interval in an expanded help? I honestly feel like the interval might be better at 90-120 seconds.","commit_id":"3ec6c907343dc4d692653ca67d52b8963f12c9ef"}]}