)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"17b3854136580a8f94952a73214069e13063b0c0","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Fiorella Yanac \u003cfyanac@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-05-18 13:32:04 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Tune QoS GW/FIP bandwidth limit ratios in scenario tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Use round(max_kbps * 0.75) for gateway limits and round(max_kbps * 0.5)"},{"line_number":10,"context_line":"for floating IP limits instead of integer halves and thirds, so the"},{"line_number":11,"context_line":"test traffic stays below the port cap."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Assisted-By: Cursor-composer-2-fast"},{"line_number":14,"context_line":"Change-Id: Icefd31266d0cf261ed645990e6fb8ead30799c60"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1c885d4f_b0053978","line":12,"range":{"start_line":7,"start_character":0,"end_line":12,"end_character":1},"updated":"2026-05-19 08:30:42.000000000","message":"We should make a reference to the LP bug: https://bugs.launchpad.net/neutron/+bug/2153122\n\nThis commit message should also describe the rationale behind this BW increase. Something like this:\n```\nIncrease QoS GW/FIP bandwidth limits to avoid TBF packet drops\n\nThe OVS Token Bucket Filter (TBF) algorithm drops all packets when\nthe bucket size (max-burst) is smaller than the MTU size. In the\nsouth-north BW limit test, the GW and FIP limits were derived from\nthe port limit (250 kbps) using integer division:\n  - GW:  250 // 2 \u003d 125 kbps, burst \u003d 100 kbits (12500 bytes)\n  - FIP: 250 // 3 \u003d  83 kbps, burst \u003d  66 kbits (8250 bytes)\n\nWith jumbo frames (MTU 9000), a single packet (72000 bits) exceeds\nthe FIP burst bucket (66000 bits), causing all traffic to be\ndropped instead of shaped.\n\nIncrease the ratios to 0.75 (GW) and 0.5 (FIP) so that the resulting\nburst values always exceed the MTU:\n  - GW: round(250 * 0.75) \u003d 188 kbps, burst \u003d 150 kbits (18750 bytes)\n  - FIP: round(250 * 0.5)  \u003d 125 kbps, burst \u003d 100 kbits (12500 bytes)\n\nCloses-Bug: #2153122\n```","commit_id":"edd3e8af60b8a52de7f2af3fa9d6cafe70e8faa6"},{"author":{"_account_id":34777,"name":"Fiorella Yanac","email":"fyanac@redhat.com","username":"fyanac"},"change_message_id":"870ed66e95ff2dc4e9003ad26ad8ec6e09c127c4","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Fiorella Yanac \u003cfyanac@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-05-18 13:32:04 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Tune QoS GW/FIP bandwidth limit ratios in scenario tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Use round(max_kbps * 0.75) for gateway limits and round(max_kbps * 0.5)"},{"line_number":10,"context_line":"for floating IP limits instead of integer halves and thirds, so the"},{"line_number":11,"context_line":"test traffic stays below the port cap."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Assisted-By: Cursor-composer-2-fast"},{"line_number":14,"context_line":"Change-Id: Icefd31266d0cf261ed645990e6fb8ead30799c60"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5fbcfc2a_782d06ab","line":12,"range":{"start_line":7,"start_character":0,"end_line":12,"end_character":1},"in_reply_to":"1c885d4f_b0053978","updated":"2026-05-19 08:47:39.000000000","message":"Done, Thank you","commit_id":"edd3e8af60b8a52de7f2af3fa9d6cafe70e8faa6"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"eb870001cb2e2b6b41601b3722cde32143f74fa3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"526adbda_98643005","updated":"2026-05-18 15:15:32.000000000","message":"As we traced neutron errors to bug [1], and that lead to this test change too, lets wait with any change to tests.\nAnyways later on as was chatted, a generic increase in tested limits would be great and is a nice change idea instead.\n[1] https://redhat.atlassian.net/browse/OSPRH-29440","commit_id":"edd3e8af60b8a52de7f2af3fa9d6cafe70e8faa6"},{"author":{"_account_id":34777,"name":"Fiorella Yanac","email":"fyanac@redhat.com","username":"fyanac"},"change_message_id":"1ce4436dcd64b984996985bd486b3935ab2bb4c8","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"97c22bb9_c758ace3","in_reply_to":"526adbda_98643005","updated":"2026-05-18 15:46:57.000000000","message":"Hi,\nI think the bug [1] is not related to the issue, it is for maintenance task.\nAnd yes,I\u0027ll check it by increasing the values of:\n    bwlimit_kbps_net \u003d 1000\n    bwlimit_kbps_net_new \u003d 500\n    bwlimit_kbps_port \u003d 250\nThank you","commit_id":"edd3e8af60b8a52de7f2af3fa9d6cafe70e8faa6"},{"author":{"_account_id":34777,"name":"Fiorella Yanac","email":"fyanac@redhat.com","username":"fyanac"},"change_message_id":"870ed66e95ff2dc4e9003ad26ad8ec6e09c127c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d42da66e_9866f7e5","in_reply_to":"97c22bb9_c758ace3","updated":"2026-05-19 08:47:39.000000000","message":"Done","commit_id":"edd3e8af60b8a52de7f2af3fa9d6cafe70e8faa6"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"4e04f9fae12f7965d83058c6ddb3197130814cf7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d97cfbb1_42f4cac0","updated":"2026-05-19 09:04:46.000000000","message":"Was the code tested on reproducer OSP setup?","commit_id":"ac79225910a76cb9f2e8f05bee61d44019ec33d0"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"3352310e6e56d4e970fa18c5d3269eed1c6d856b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ff42fa46_9998359c","updated":"2026-05-19 10:06:04.000000000","message":"Thanks.\nWas test class verified with latest patchset?","commit_id":"3db3e8be70647b305bff27997aa076b068c09361"},{"author":{"_account_id":34777,"name":"Fiorella Yanac","email":"fyanac@redhat.com","username":"fyanac"},"change_message_id":"48145d16e892cfa0050af8695bbfc649ce0ad8b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"f35e2b47_bd90e3d7","in_reply_to":"ff42fa46_9998359c","updated":"2026-05-19 10:23:04.000000000","message":"yes","commit_id":"3db3e8be70647b305bff27997aa076b068c09361"}],"whitebox_neutron_tempest_plugin/tests/scenario/test_qos.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"17b3854136580a8f94952a73214069e13063b0c0","unresolved":true,"context_lines":[{"line_number":61,"context_line":"    IPERF_PORT \u003d 4321"},{"line_number":62,"context_line":"    bwlimit_kbps_net \u003d 1000"},{"line_number":63,"context_line":"    bwlimit_kbps_net_new \u003d 500"},{"line_number":64,"context_line":"    bwlimit_kbps_port \u003d 250"},{"line_number":65,"context_line":"    sriov_test \u003d False"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":1,"id":"2a62b806_4f87d13d","line":64,"range":{"start_line":64,"start_character":24,"end_line":64,"end_character":27},"updated":"2026-05-19 08:30:42.000000000","message":"We should also rise this value to 350, for example.","commit_id":"edd3e8af60b8a52de7f2af3fa9d6cafe70e8faa6"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"51dcab02ca0602d7f02ba3b7dc0e882bea22aa49","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    IPERF_PORT \u003d 4321"},{"line_number":62,"context_line":"    bwlimit_kbps_net \u003d 1000"},{"line_number":63,"context_line":"    bwlimit_kbps_net_new \u003d 500"},{"line_number":64,"context_line":"    bwlimit_kbps_port \u003d 250"},{"line_number":65,"context_line":"    sriov_test \u003d False"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":1,"id":"f7492aaf_b2b3552e","line":64,"range":{"start_line":64,"start_character":24,"end_line":64,"end_character":27},"in_reply_to":"2a62b806_4f87d13d","updated":"2026-05-19 08:39:22.000000000","message":"Disregard this comment.","commit_id":"edd3e8af60b8a52de7f2af3fa9d6cafe70e8faa6"},{"author":{"_account_id":34777,"name":"Fiorella Yanac","email":"fyanac@redhat.com","username":"fyanac"},"change_message_id":"870ed66e95ff2dc4e9003ad26ad8ec6e09c127c4","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    IPERF_PORT \u003d 4321"},{"line_number":62,"context_line":"    bwlimit_kbps_net \u003d 1000"},{"line_number":63,"context_line":"    bwlimit_kbps_net_new \u003d 500"},{"line_number":64,"context_line":"    bwlimit_kbps_port \u003d 250"},{"line_number":65,"context_line":"    sriov_test \u003d False"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":1,"id":"5229dc33_7a3407d3","line":64,"range":{"start_line":64,"start_character":24,"end_line":64,"end_character":27},"in_reply_to":"f7492aaf_b2b3552e","updated":"2026-05-19 08:47:39.000000000","message":"Done","commit_id":"edd3e8af60b8a52de7f2af3fa9d6cafe70e8faa6"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c8ed7c299b9f0fcec0f889498389502fa4a06f87","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    IPERF_PORT \u003d 4321"},{"line_number":62,"context_line":"    bwlimit_kbps_net \u003d 1000"},{"line_number":63,"context_line":"    bwlimit_kbps_net_new \u003d 500"},{"line_number":64,"context_line":"    bwlimit_kbps_port \u003d 350"},{"line_number":65,"context_line":"    sriov_test \u003d False"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":4,"id":"d639cb28_a81e2685","line":64,"range":{"start_line":64,"start_character":24,"end_line":64,"end_character":27},"updated":"2026-05-19 09:07:38.000000000","message":"Let\u0027s keep the original value to match the commit message.","commit_id":"1d40e15e551c5884c34333734b66baee1e72932a"},{"author":{"_account_id":34777,"name":"Fiorella Yanac","email":"fyanac@redhat.com","username":"fyanac"},"change_message_id":"bd74d365803e697041d36164b16d2038776e5471","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    IPERF_PORT \u003d 4321"},{"line_number":62,"context_line":"    bwlimit_kbps_net \u003d 1000"},{"line_number":63,"context_line":"    bwlimit_kbps_net_new \u003d 500"},{"line_number":64,"context_line":"    bwlimit_kbps_port \u003d 350"},{"line_number":65,"context_line":"    sriov_test \u003d False"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":4,"id":"7ac31476_0dad6971","line":64,"range":{"start_line":64,"start_character":24,"end_line":64,"end_character":27},"in_reply_to":"189fc6c0_b210b377","updated":"2026-05-19 09:40:12.000000000","message":"I\u0027ve updated the commit message as well.","commit_id":"1d40e15e551c5884c34333734b66baee1e72932a"},{"author":{"_account_id":34777,"name":"Fiorella Yanac","email":"fyanac@redhat.com","username":"fyanac"},"change_message_id":"8428ba23234141e3842ee3777caffd2586106eb7","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    IPERF_PORT \u003d 4321"},{"line_number":62,"context_line":"    bwlimit_kbps_net \u003d 1000"},{"line_number":63,"context_line":"    bwlimit_kbps_net_new \u003d 500"},{"line_number":64,"context_line":"    bwlimit_kbps_port \u003d 350"},{"line_number":65,"context_line":"    sriov_test \u003d False"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":4,"id":"189fc6c0_b210b377","line":64,"range":{"start_line":64,"start_character":24,"end_line":64,"end_character":27},"in_reply_to":"d639cb28_a81e2685","updated":"2026-05-19 09:28:56.000000000","message":"I\u0027ve added this change in the commit message, so it has to be modified too then \u003d)","commit_id":"1d40e15e551c5884c34333734b66baee1e72932a"}]}