)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"23ddd11adece38df2a349a458014676397310c78","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3f834481_197b1f39","updated":"2025-05-23 16:19:54.000000000","message":"looks good! thanks!","commit_id":"9057be2b912c73bad34ca0914d88ee71f267fb75"}],"roles/copy-release-tools-scripts/files/release-tools/update_constraints.sh":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"dc7f185b8a1866eb7b3e60a7933d4443e37ac8f1","unresolved":true,"context_lines":[{"line_number":125,"context_line":"    if git commit -a -m \"update constraint for $dist_name to new release $VERSION"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"$TAG_META"},{"line_number":128,"context_line":"\" -s --trailer\u003d\"Generated-By:openstack/project-config:roles/copy-release-tools-scripts/files/release-tools/update_constraints.sh\""},{"line_number":129,"context_line":"    then"},{"line_number":130,"context_line":"        echo \"Sleeping 10 minutes to avoid issues with the pypi cache\""},{"line_number":131,"context_line":"        sleep 600"}],"source_content_type":"text/x-sh","patch_set":1,"id":"4f450e6b_19aea76e","line":128,"updated":"2025-05-23 16:13:21.000000000","message":"I guess you don\u0027t need the ; anymore because we\u0027re using the line break now.","commit_id":"9057be2b912c73bad34ca0914d88ee71f267fb75"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"7f526fca9ca536fa845431f932f5dad728aecefa","unresolved":true,"context_lines":[{"line_number":125,"context_line":"    if git commit -a -m \"update constraint for $dist_name to new release $VERSION"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"$TAG_META"},{"line_number":128,"context_line":"\" -s --trailer\u003d\"Generated-By:openstack/project-config:roles/copy-release-tools-scripts/files/release-tools/update_constraints.sh\""},{"line_number":129,"context_line":"    then"},{"line_number":130,"context_line":"        echo \"Sleeping 10 minutes to avoid issues with the pypi cache\""},{"line_number":131,"context_line":"        sleep 600"}],"source_content_type":"text/x-sh","patch_set":1,"id":"aefeae9f_3c9180ab","line":128,"updated":"2025-05-24 08:30:53.000000000","message":"are multiple generated-by headers allowed? IMO it would be very helpful to also identify the job that ran to generate this change. ideally that would be a link to the zuul build, but a reference to the job definition might be fine, too, if that\u0027s wasier","commit_id":"9057be2b912c73bad34ca0914d88ee71f267fb75"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"d4b156d0a630392747e2710fdcd8656e2e6b116a","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    if git commit -a -m \"update constraint for $dist_name to new release $VERSION"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"$TAG_META"},{"line_number":128,"context_line":"\" -s --trailer\u003d\"Generated-By:openstack/project-config:roles/copy-release-tools-scripts/files/release-tools/update_constraints.sh\""},{"line_number":129,"context_line":"    then"},{"line_number":130,"context_line":"        echo \"Sleeping 10 minutes to avoid issues with the pypi cache\""},{"line_number":131,"context_line":"        sleep 600"}],"source_content_type":"text/x-sh","patch_set":1,"id":"9f18db1c_f13f5f18","line":128,"in_reply_to":"4f450e6b_19aea76e","updated":"2025-05-24 20:27:41.000000000","message":"Correct, there was no longer any reason like conciseness/brevity to smash the \"then\" keyword into the previous lone.","commit_id":"9057be2b912c73bad34ca0914d88ee71f267fb75"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"d5284f352a5f5cdd256f54955c49def1db477201","unresolved":true,"context_lines":[{"line_number":125,"context_line":"    if git commit -a -m \"update constraint for $dist_name to new release $VERSION"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"$TAG_META"},{"line_number":128,"context_line":"\" -s --trailer\u003d\"Generated-By:openstack/project-config:roles/copy-release-tools-scripts/files/release-tools/update_constraints.sh\""},{"line_number":129,"context_line":"    then"},{"line_number":130,"context_line":"        echo \"Sleeping 10 minutes to avoid issues with the pypi cache\""},{"line_number":131,"context_line":"        sleep 600"}],"source_content_type":"text/x-sh","patch_set":1,"id":"c7fec8e1_cc8c7eed","line":128,"in_reply_to":"aefeae9f_3c9180ab","updated":"2025-05-24 20:26:44.000000000","message":"Sure, you can add as many as you like by just tacking on more --trailer options, however I went with the script location because it\u0027s hard to know what job(s) may call any particular script and remember to update the script any time the job name changes or a new job starts using it.","commit_id":"9057be2b912c73bad34ca0914d88ee71f267fb75"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"3de0b287ae0f70cab51376f198cc0d64f03f68f5","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    if git commit -a -m \"update constraint for $dist_name to new release $VERSION"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"$TAG_META"},{"line_number":128,"context_line":"\" -s --trailer\u003d\"Generated-By:openstack/project-config:roles/copy-release-tools-scripts/files/release-tools/update_constraints.sh\""},{"line_number":129,"context_line":"    then"},{"line_number":130,"context_line":"        echo \"Sleeping 10 minutes to avoid issues with the pypi cache\""},{"line_number":131,"context_line":"        sleep 600"}],"source_content_type":"text/x-sh","patch_set":1,"id":"d8817067_285c2a3e","line":128,"in_reply_to":"c7fec8e1_cc8c7eed","updated":"2025-05-25 12:19:57.000000000","message":"right, it would just be very helpful for someone looking at the generated patch to be able to answer the question \"where does this come from?\"\n\nbut not a blocker for sure, we can add more info in a follow-up","commit_id":"9057be2b912c73bad34ca0914d88ee71f267fb75"}]}