)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"410da104e22d34c23233cd55292422b504b866a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"470dea26_3eed3c59","updated":"2025-05-13 12:43:59.000000000","message":"can you please update the commit message explaining why that\u0027s not needed?","commit_id":"a0e4f06010c1931b194fb3131dc0207b6564c72f"},{"author":{"_account_id":37842,"name":"Habeeb Babasulaiman","display_name":"Freeman","email":"bossfreeman9@gmail.com","username":"FreemanBoss"},"change_message_id":"6661b61dc9c70da1884b4e4ecd07287993247256","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"af1ce6d2_994cbd09","in_reply_to":"470dea26_3eed3c59","updated":"2025-05-13 13:25:31.000000000","message":"Done now, also in the ironic IRC channel I was once told that it is irrelevant to restart Ironic cause it automatically restarts by the fault and if it doesn`t then it is a bug. Also, in one of by patch I got a review that I should remove this particular reference from the doc.","commit_id":"a0e4f06010c1931b194fb3131dc0207b6564c72f"},{"author":{"_account_id":37842,"name":"Habeeb Babasulaiman","display_name":"Freeman","email":"bossfreeman9@gmail.com","username":"FreemanBoss"},"change_message_id":"6e2d7dea702df7bce13c9467723cafa9fe8ea59b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"428a9db1_d36080e7","in_reply_to":"af1ce6d2_994cbd09","updated":"2025-05-13 13:42:28.000000000","message":"https://review.opendev.org/c/openstack/bifrost/+/946724/comment/1cd3672e_8e0f5ba0/\n\nhere is the link to the review you made where you said I should work on this","commit_id":"a0e4f06010c1931b194fb3131dc0207b6564c72f"},{"author":{"_account_id":37842,"name":"Habeeb Babasulaiman","display_name":"Freeman","email":"bossfreeman9@gmail.com","username":"FreemanBoss"},"change_message_id":"462207553b371b010b75dab90a7dace5fb057154","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"348b7cdd_3acf3187","updated":"2025-05-22 09:58:09.000000000","message":"#ironic-week-prio","commit_id":"f2fd9e1cd35353daab4d9841a711de007c218f4e"},{"author":{"_account_id":37842,"name":"Habeeb Babasulaiman","display_name":"Freeman","email":"bossfreeman9@gmail.com","username":"FreemanBoss"},"change_message_id":"590aca24bf666d1da01d2107eebdc2649e31c496","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"b1ceeab6_9a2904a7","updated":"2025-05-22 09:55:33.000000000","message":"#ironic-week-prio","commit_id":"f2fd9e1cd35353daab4d9841a711de007c218f4e"}],"doc/source/contributor/testenv.rst":[{"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":"de45968ebd556d10709ce93699110c53b5164000","unresolved":true,"context_lines":[{"line_number":19,"context_line":"       You still need to restart services to apply any changes, e.g.::"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"        sudo systemctl restart ironic"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"``--driver\u003d[ipmi|redfish]``"},{"line_number":24,"context_line":"    Choose the default driver for the generated nodes inventory."},{"line_number":25,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"b6fc0e93_ad03d467","side":"PARENT","line":22,"updated":"2025-05-16 18:09:50.000000000","message":"So, we can choose per config option if a restart is needed -\u003e https://opendev.org/openstack/ironic/src/branch/master/ironic/conf/conductor.py#L49 (this one with mutable\u003dTrue doesn\u0027t need a restart).\n\nI do think this advice to restart unconditionally is probably overkill in some cases; but never restarting will actually break folks. I\u0027d suggest this patch not move forward.","commit_id":"0052e2527956cd526836a747365db8314cf16205"}]}