)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"1c7eb630cc704be65745c938f47a725c04466537","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fixes indentation and ordering of interfaces, the same as in [1]."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"1. https://github.com/openstack/kayobe/commit/c8229e4a0b9a015e690e0021710132c518721a40"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Related-Bug: #2168529"},{"line_number":14,"context_line":"Change-Id: I3944e1f56dfc5d6db878f57e89a5db02553a2e9b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bc166491_35d671bd","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":86},"updated":"2026-10-27 18:52:19.000000000","message":"Can you link to Gerrit instead?","commit_id":"8d0b54a74b3ab5195868f4bba7fb463890b59e6a"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"50b7e5e44ac6e96b4c5e13162d115478fc54d66d","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fixes indentation and ordering of interfaces, the same as in [1]."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"1. https://github.com/openstack/kayobe/commit/c8229e4a0b9a015e690e0021710132c518721a40"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Related-Bug: #2168529"},{"line_number":14,"context_line":"Change-Id: I3944e1f56dfc5d6db878f57e89a5db02553a2e9b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1dc4aa21_b8e47464","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":86},"in_reply_to":"bc166491_35d671bd","updated":"2026-10-28 08:30:44.000000000","message":"Done","commit_id":"8d0b54a74b3ab5195868f4bba7fb463890b59e6a"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"91a1cc7b7c8e69b8f26fd46ec27b324c2226885c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ae8d851d_ba7ad400","updated":"2026-10-29 10:59:11.000000000","message":"Seems like sonic uses capitalized vlan interface names i.e Vlan vs vlan:\n\nhttps://www.dell.com/support/kbdoc/en-us/000217901/dell-networking-sonic-how-to-create-vlan-and-assign-it-to-trunk-access-switchports\n\nWorth making the match case insensitve?","commit_id":"788d8afa1098f956f45bd6fe0568592b62c99f1b"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"67b79767ffc9db1d3ff12e1bc1d3d52cea136eb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4cf27559_3ff8fc67","updated":"2026-10-28 13:46:03.000000000","message":"check experimental","commit_id":"788d8afa1098f956f45bd6fe0568592b62c99f1b"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"b06c2987527c5f8dca6804d9c405246521875594","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"67b5b04b_a69be039","in_reply_to":"ae8d851d_ba7ad400","updated":"2026-10-29 13:30:38.000000000","message":"I think sonic will automatically convert to upper case when the command is received, but yet this would be sensible anyway.","commit_id":"788d8afa1098f956f45bd6fe0568592b62c99f1b"}],"ansible/roles/dell-switch-sonic/templates/dell-sonic-config.j2":[{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"91a1cc7b7c8e69b8f26fd46ec27b324c2226885c","unresolved":true,"context_lines":[{"line_number":5,"context_line":"{% endfor %}"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"{% for interface, config in dell_switch_sonic_interface_config.items() %}"},{"line_number":8,"context_line":"{% if \u0027vlan\u0027 in interface %}"},{"line_number":9,"context_line":"interface {{ interface }}"},{"line_number":10,"context_line":"{% if config.description is defined %}"},{"line_number":11,"context_line":" description \"{{ config.description }}\""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"e0410372_ee5c2b43","line":8,"updated":"2026-10-29 10:59:11.000000000","message":"```suggestion\n{% if \u0027vlan\u0027 in interface | lower %}\n```","commit_id":"788d8afa1098f956f45bd6fe0568592b62c99f1b"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"b06c2987527c5f8dca6804d9c405246521875594","unresolved":false,"context_lines":[{"line_number":5,"context_line":"{% endfor %}"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"{% for interface, config in dell_switch_sonic_interface_config.items() %}"},{"line_number":8,"context_line":"{% if \u0027vlan\u0027 in interface %}"},{"line_number":9,"context_line":"interface {{ interface }}"},{"line_number":10,"context_line":"{% if config.description is defined %}"},{"line_number":11,"context_line":" description \"{{ config.description }}\""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"b683973d_67b81cbe","line":8,"in_reply_to":"e0410372_ee5c2b43","updated":"2026-10-29 13:30:38.000000000","message":"Fix applied.","commit_id":"788d8afa1098f956f45bd6fe0568592b62c99f1b"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"b06c2987527c5f8dca6804d9c405246521875594","unresolved":false,"context_lines":[{"line_number":5,"context_line":"{% endfor %}"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"{% for interface, config in dell_switch_sonic_interface_config.items() %}"},{"line_number":8,"context_line":"{% if \u0027vlan\u0027 in interface %}"},{"line_number":9,"context_line":"interface {{ interface }}"},{"line_number":10,"context_line":"{% if config.description is defined %}"},{"line_number":11,"context_line":" description \"{{ config.description }}\""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"fa1e8060_c16dd442","line":8,"in_reply_to":"e0410372_ee5c2b43","updated":"2026-10-29 13:30:38.000000000","message":"Fix applied.","commit_id":"788d8afa1098f956f45bd6fe0568592b62c99f1b"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"91a1cc7b7c8e69b8f26fd46ec27b324c2226885c","unresolved":true,"context_lines":[{"line_number":17,"context_line":"{% endfor %}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"{% for interface, config in dell_switch_sonic_interface_config.items() %}"},{"line_number":20,"context_line":"{% if \u0027vlan\u0027 not in interface %}"},{"line_number":21,"context_line":"interface {{ interface }}"},{"line_number":22,"context_line":"{% if config.description is defined %}"},{"line_number":23,"context_line":" description \"{{ config.description }}\""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"6ed311e8_2d4e71fc","line":20,"updated":"2026-10-29 10:59:11.000000000","message":"```suggestion\n{% if \u0027vlan\u0027 not in interface | lower %}\n```","commit_id":"788d8afa1098f956f45bd6fe0568592b62c99f1b"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"b06c2987527c5f8dca6804d9c405246521875594","unresolved":false,"context_lines":[{"line_number":17,"context_line":"{% endfor %}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"{% for interface, config in dell_switch_sonic_interface_config.items() %}"},{"line_number":20,"context_line":"{% if \u0027vlan\u0027 not in interface %}"},{"line_number":21,"context_line":"interface {{ interface }}"},{"line_number":22,"context_line":"{% if config.description is defined %}"},{"line_number":23,"context_line":" description \"{{ config.description }}\""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"8ec38a66_8c3d9383","line":20,"in_reply_to":"6ed311e8_2d4e71fc","updated":"2026-10-29 13:30:38.000000000","message":"Fix applied.","commit_id":"788d8afa1098f956f45bd6fe0568592b62c99f1b"}]}
