)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"328b378746d86e6fbb7f47a3ebf72ee9a33c4464","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Matt Crees \u003cmattc@stackhpc.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-09-14 14:25:52 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support for configuring Dell SONiC switches"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds support for congiuring Dell Enterprise SONiC switches, using"},{"line_number":10,"context_line":"the ``dellemc.enterprise_sonic`` Ansible collection. Follows the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bb24ce7d_609f7a49","line":7,"range":{"start_line":7,"start_character":16,"end_line":7,"end_character":27},"updated":"2026-09-14 13:45:01.000000000","message":"nit:\n```suggestion\nAdd support for configuring Dell SONiC switches\n```","commit_id":"e438c00268e7903d5701bcae0ef757da6fb5f58c"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"d0370baa1dbf83c72fb28b77a5345a92c3fe15bd","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Matt Crees \u003cmattc@stackhpc.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-09-14 14:25:52 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support for configuring Dell SONiC switches"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds support for congiuring Dell Enterprise SONiC switches, using"},{"line_number":10,"context_line":"the ``dellemc.enterprise_sonic`` Ansible collection. Follows the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"16443a0e_24f5dd66","line":7,"range":{"start_line":7,"start_character":16,"end_line":7,"end_character":27},"in_reply_to":"bb24ce7d_609f7a49","updated":"2026-09-14 13:52:43.000000000","message":"Done","commit_id":"e438c00268e7903d5701bcae0ef757da6fb5f58c"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"dc11da55b391f32456b1da83291c2ad4b3fb4c09","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support for configuring Dell SONiC switches"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds support for congiuring Dell Enterprise SONiC switches, using"},{"line_number":10,"context_line":"the ``dellemc.enterprise_sonic`` Ansible collection. Follows the"},{"line_number":11,"context_line":"same conventions as the other Dell Switch roles."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"0f3ee63f_a4692a08","line":9,"range":{"start_line":9,"start_character":17,"end_line":9,"end_character":27},"updated":"2026-09-16 11:05:15.000000000","message":"configuring","commit_id":"c3ddcff1693a558c959c15255a1ec82605294790"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"cf8cdfe4dd82792983cf371f14d78eb943f2dfda","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support for configuring Dell SONiC switches"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds support for congiuring Dell Enterprise SONiC switches, using"},{"line_number":10,"context_line":"the ``dellemc.enterprise_sonic`` Ansible collection. Follows the"},{"line_number":11,"context_line":"same conventions as the other Dell Switch roles."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"17b78f9d_38b21e1a","line":9,"range":{"start_line":9,"start_character":17,"end_line":9,"end_character":27},"in_reply_to":"0f3ee63f_a4692a08","updated":"2026-09-16 12:05:37.000000000","message":"Done","commit_id":"c3ddcff1693a558c959c15255a1ec82605294790"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"678caa8ba980f4a81e051430109f5bb7380d87b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"04f86142_7ec5557a","updated":"2026-09-08 09:42:40.000000000","message":"I\u0027d like to make an exception for this patch and backport to stable releases.","commit_id":"ef74bc2275fa078409d019713413813bd034bd85"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"7a849c6a80ba0ba2edace3226ee9098d22fff397","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"52280ed2_5942ad42","updated":"2026-09-14 13:40:08.000000000","message":"Change looks good to me. I think we need to update supported switches in here: https://docs.openstack.org/kayobe/latest/configuration/reference/physical-network.html. Happy if that is in a follow up.","commit_id":"7c03f7d899a05ceef5d60e7eca8aaaecd55db64f"},{"author":{"_account_id":38288,"name":"Leonie Chamberlin-Medd","display_name":"Leonie Chamberlin-Medd","email":"leonie@stackhpc.com","username":"LChams"},"change_message_id":"291dedaf8d60a28d9f661ff74bb9d806e5489a91","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"35fc3241_41e0e0ac","updated":"2026-09-14 13:19:30.000000000","message":"Looking good mostly, just think we need something under ``doc/source/configuration/reference/physical-network.rst`` defining the required config.","commit_id":"7c03f7d899a05ceef5d60e7eca8aaaecd55db64f"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"4e129dbdbf4a1510d8bfa3944631806fa5c02fe8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"77b4c43e_ed3144f1","updated":"2026-09-14 12:51:01.000000000","message":"recheck","commit_id":"7c03f7d899a05ceef5d60e7eca8aaaecd55db64f"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"983e7bdaaa526c8ad558568dc45059d35dacdb2b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a4a55e98_30b7ba2c","updated":"2026-09-10 09:56:27.000000000","message":"recheck: ci should be fixed https://review.opendev.org/c/openstack/kayobe/+/1004785","commit_id":"7c03f7d899a05ceef5d60e7eca8aaaecd55db64f"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"3e244b2a9c594536783076cede885f08be9a06f8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f3ad2602_6666ed76","updated":"2026-09-08 15:38:11.000000000","message":"recheck: unrelated failure in infra vm job","commit_id":"7c03f7d899a05ceef5d60e7eca8aaaecd55db64f"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"5d50868bfc022273bf19ecc99f5d92f0cd30b7c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4ab6bfa2_5f7b0cd6","in_reply_to":"35fc3241_41e0e0ac","updated":"2026-09-14 13:26:22.000000000","message":"Thanks, I\u0027ve added these now.","commit_id":"7c03f7d899a05ceef5d60e7eca8aaaecd55db64f"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"a7a37ddfcae7ed6d361771197e54419abd53eb36","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"546faee7_78887c8c","in_reply_to":"52280ed2_5942ad42","updated":"2026-09-14 13:40:47.000000000","message":"Marking as unresolved","commit_id":"7c03f7d899a05ceef5d60e7eca8aaaecd55db64f"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"328b378746d86e6fbb7f47a3ebf72ee9a33c4464","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2192f433_c670d402","in_reply_to":"546faee7_78887c8c","updated":"2026-09-14 13:45:01.000000000","message":"Ignore me, I missed the docs...","commit_id":"7c03f7d899a05ceef5d60e7eca8aaaecd55db64f"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"2ad82ed7f72440f05b63a200f984499ae6dfb4a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"58b4c400_2e3b3b82","updated":"2026-09-14 13:50:19.000000000","message":"For NGS support, I think we need to map it to the netmiko device name:\n\nhttps://github.com/openstack/kayobe/blob/697b9920bf264523b85941f1edeb1b6239b77032/ansible/kolla-openstack.yml#L64-L73\n\nwhich should be netmiko_sonic if I\u0027m not mistaken, but that can be a follow up.","commit_id":"e438c00268e7903d5701bcae0ef757da6fb5f58c"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"7bdaa38a89412e67e8f489e259ae902696edd3f3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"933a0465_95d3da01","in_reply_to":"58b4c400_2e3b3b82","updated":"2026-09-14 13:58:35.000000000","message":"Best to be in a follow up I reckon, I haven\u0027t tested NGS support at all.","commit_id":"e438c00268e7903d5701bcae0ef757da6fb5f58c"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"3b2f381afa60db381a853e37c789177bd3f9666e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8f953830_b50b4931","updated":"2026-09-15 09:23:12.000000000","message":"recheck","commit_id":"c3ddcff1693a558c959c15255a1ec82605294790"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"c39d2b6e773d04ac1057c204b9bbb9f8b90efcf3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"bf95c24d_de35173c","updated":"2026-09-16 08:25:00.000000000","message":"recheck","commit_id":"c3ddcff1693a558c959c15255a1ec82605294790"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"5d7f0a7702c05c9c82acbd439244f6931c8ccea7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"acc19105_5b9f1e0a","updated":"2026-09-15 12:08:32.000000000","message":"recheck: tox ansible timed out","commit_id":"c3ddcff1693a558c959c15255a1ec82605294790"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"d07c1aabc726b415e68238d3f112afb924b20bc5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7e759d1c_9d014cf2","updated":"2026-09-16 23:40:41.000000000","message":"recheck timeouts","commit_id":"a16e973e64c3e9f214c4cb035d8a2ebb4debc5dc"}],"releasenotes/notes/add-dell-enterprise-sonic-switch-support-f7405df8695d9d23.yaml":[{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"328b378746d86e6fbb7f47a3ebf72ee9a33c4464","unresolved":true,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Adds support for configuring Dell Enterprise SONiC switches using the"},{"line_number":5,"context_line":"    ``sonic-cli``, via the ``dell_emc.enterprise_sonic`` Ansible collection."},{"line_number":6,"context_line":"    This is integrated into the ``kayobe physical network configure`` command."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"8314455a_4c0bab65","line":5,"range":{"start_line":5,"start_character":29,"end_line":5,"end_character":37},"updated":"2026-09-14 13:45:01.000000000","message":"nit:\ndellemc\n```suggestion\n    ``sonic-cli``, via the ``dellemc.enterprise_sonic`` Ansible collection.\n```","commit_id":"e438c00268e7903d5701bcae0ef757da6fb5f58c"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"569aabdacab79067c7c624bbc44efb20a6ab170c","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Adds support for configuring Dell Enterprise SONiC switches using the"},{"line_number":5,"context_line":"    ``sonic-cli``, via the ``dell_emc.enterprise_sonic`` Ansible collection."},{"line_number":6,"context_line":"    This is integrated into the ``kayobe physical network configure`` command."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"cf0ab60f_c0765b18","line":5,"range":{"start_line":5,"start_character":29,"end_line":5,"end_character":37},"in_reply_to":"8314455a_4c0bab65","updated":"2026-09-14 13:59:03.000000000","message":"Fix applied.","commit_id":"e438c00268e7903d5701bcae0ef757da6fb5f58c"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"c5541ae687cb713690242201be674c8766ea7123","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Adds support for configuring Dell Enterprise SONiC switches using the"},{"line_number":5,"context_line":"    ``sonic-cli``, via the ``dellemc.enterprise_sonic`` Ansible collection."},{"line_number":6,"context_line":"    This is integrated into the ``kayobe physical network configure`` command."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"09c324a0_30bb2d9a","line":4,"range":{"start_line":4,"start_character":70,"end_line":4,"end_character":73},"updated":"2026-09-16 11:07:37.000000000","message":"Remove \"the\"?","commit_id":"c3ddcff1693a558c959c15255a1ec82605294790"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"cf8cdfe4dd82792983cf371f14d78eb943f2dfda","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Adds support for configuring Dell Enterprise SONiC switches using the"},{"line_number":5,"context_line":"    ``sonic-cli``, via the ``dellemc.enterprise_sonic`` Ansible collection."},{"line_number":6,"context_line":"    This is integrated into the ``kayobe physical network configure`` command."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"aa5144e4_3036a7dd","line":4,"range":{"start_line":4,"start_character":70,"end_line":4,"end_character":73},"in_reply_to":"09c324a0_30bb2d9a","updated":"2026-09-16 12:05:37.000000000","message":"Done","commit_id":"c3ddcff1693a558c959c15255a1ec82605294790"}]}