)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"5e4cc55abbf5dab27df74995d3ee819fbca24a55","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8b9ffd9d_792cb8a7","updated":"2022-06-22 21:42:45.000000000","message":"Bumping priority as we are seeing job failures from it.\nhttps://zuul.opendev.org/t/openstack/build/2ffd8bf119b5490585d1af046989f975/log/job-output.txt#21816","commit_id":"600191275b8a28b499e0353d69a5d1f28ee2f69a"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"c537b3645c99e5158836cbaee8ad5f90ddabe3a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"80954723_7db3765d","updated":"2022-06-24 15:33:39.000000000","message":"LGTM","commit_id":"600191275b8a28b499e0353d69a5d1f28ee2f69a"}],"designate_tempest_plugin/data_utils.py":[{"author":{"_account_id":19298,"name":"Nicolas Bock","email":"nicolas.bock@canonical.com","username":"nicolasbock"},"change_message_id":"a17429f588881b3aaddd690bd26d6930d2953dda","unresolved":true,"context_lines":[{"line_number":213,"context_line":"    # when doing sorted comparisons"},{"line_number":214,"context_line":"    for i in range(0, 2):"},{"line_number":215,"context_line":"        ns_records.append({\"hostname\": \"ns%s.%s\" % (i, ns_zone),"},{"line_number":216,"context_line":"                           \"priority\": (random.randint(1, 999) + i)})"},{"line_number":217,"context_line":"    return ns_records"},{"line_number":218,"context_line":""},{"line_number":219,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"6d8bbda8_b0acba13","line":216,"updated":"2022-09-15 18:40:58.000000000","message":"I don\u0027t understand how that guarantees that we will never have two identical priorities. Why not simply:\n\n\"priority\": i","commit_id":"600191275b8a28b499e0353d69a5d1f28ee2f69a"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"0fc36e4f181dbbb58dd72697a3e6c2fb97d4d671","unresolved":true,"context_lines":[{"line_number":213,"context_line":"    # when doing sorted comparisons"},{"line_number":214,"context_line":"    for i in range(0, 2):"},{"line_number":215,"context_line":"        ns_records.append({\"hostname\": \"ns%s.%s\" % (i, ns_zone),"},{"line_number":216,"context_line":"                           \"priority\": (random.randint(1, 999) + i)})"},{"line_number":217,"context_line":"    return ns_records"},{"line_number":218,"context_line":""},{"line_number":219,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5469d604_60ade398","line":216,"in_reply_to":"6d8bbda8_b0acba13","updated":"2023-01-09 10:01:21.000000000","message":"1) I\u0027m agree with Nicolas, yep something seems to be buggy here, for example it could be:\n Random value + index \u003d priority\n      10         0        10\n      9          1        10\n\n\n2) Nit, but if we are already modifying this function, maybe we can add \"number_of_recordsets\" parameter that will set to 2 by default.","commit_id":"600191275b8a28b499e0353d69a5d1f28ee2f69a"}]}