)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"d80290e10f2e819eeb3d30b4fcd3fc038c837f04","unresolved":true,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Add LOCI patch to remove explicit virtualenv upgrade that"},{"line_number":15,"context_line":"breaks on Python 3.13."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I9483066c5d56e0d393f6bb7a63be111320cea9be"},{"line_number":18,"context_line":"Signed-off-by: Amit Mahajan \u003cAmit.Mahajan@windriver.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ea7c41b1_4819791c","line":16,"updated":"2026-04-28 18:47:33.000000000","message":"Missing TEST PLAN\n\nMissing  story/task - \nStory: 2010995\nTask: 54298","commit_id":"a4f3c7ff3c0e13f64bdf9ed3369bf92099dc48ea"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"01fd03816ddc22ce00a8523a4ae08eb970026d4c","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Add LOCI patch to remove explicit virtualenv upgrade that"},{"line_number":15,"context_line":"breaks on Python 3.13."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I9483066c5d56e0d393f6bb7a63be111320cea9be"},{"line_number":18,"context_line":"Signed-off-by: Amit Mahajan \u003cAmit.Mahajan@windriver.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ac6cda19_7b06f4e8","line":16,"in_reply_to":"8caf8570_56944f1d","updated":"2026-04-30 16:14:42.000000000","message":"Done","commit_id":"a4f3c7ff3c0e13f64bdf9ed3369bf92099dc48ea"},{"author":{"_account_id":38728,"name":"Amit Mahajan","display_name":"Amit Mahajan","email":"Amit.Mahajan@windriver.com","username":"amahajan"},"change_message_id":"cf4e3148f3906a754a45408b856cba1a8828da28","unresolved":true,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Add LOCI patch to remove explicit virtualenv upgrade that"},{"line_number":15,"context_line":"breaks on Python 3.13."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I9483066c5d56e0d393f6bb7a63be111320cea9be"},{"line_number":18,"context_line":"Signed-off-by: Amit Mahajan \u003cAmit.Mahajan@windriver.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"8caf8570_56944f1d","line":16,"in_reply_to":"ea7c41b1_4819791c","updated":"2026-04-29 14:23:12.000000000","message":"Added the required information.","commit_id":"a4f3c7ff3c0e13f64bdf9ed3369bf92099dc48ea"},{"author":{"_account_id":37173,"name":"Alex Fernandes Figueirêdo","display_name":"Alex Figueiredo","email":"alex.fernandesfigueiredo@windriver.com","username":"afernand"},"change_message_id":"49ead07dd0ec757d3b7e02602643c3001ad013b1","unresolved":true,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Add LOCI patch to remove explicit virtualenv upgrade that"},{"line_number":15,"context_line":"breaks on Python 3.13."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I9483066c5d56e0d393f6bb7a63be111320cea9be"},{"line_number":18,"context_line":"Signed-off-by: Amit Mahajan \u003cAmit.Mahajan@windriver.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"eb1ee70a_e2d6e9fe","line":16,"updated":"2026-04-30 21:29:15.000000000","message":"Please include:\n- Test Plan specifying the procedure used to validate this change\n- Starlingx Story/Task","commit_id":"88529e0e3ae12f1f0fdbfd3668097f99968a2f06"},{"author":{"_account_id":38728,"name":"Amit Mahajan","display_name":"Amit Mahajan","email":"Amit.Mahajan@windriver.com","username":"amahajan"},"change_message_id":"bc09cd0d448b96013e8dbf0a2e48250271b30d98","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Add LOCI patch to remove explicit virtualenv upgrade that"},{"line_number":15,"context_line":"breaks on Python 3.13."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I9483066c5d56e0d393f6bb7a63be111320cea9be"},{"line_number":18,"context_line":"Signed-off-by: Amit Mahajan \u003cAmit.Mahajan@windriver.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"a2802aa4_dc23092d","line":16,"in_reply_to":"eb1ee70a_e2d6e9fe","updated":"2026-04-30 21:38:34.000000000","message":"Added the required info.","commit_id":"88529e0e3ae12f1f0fdbfd3668097f99968a2f06"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":38728,"name":"Amit Mahajan","display_name":"Amit Mahajan","email":"Amit.Mahajan@windriver.com","username":"amahajan"},"change_message_id":"876870435e4fa7a64b09b5e5f6533162c848da38","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"01c25f8e_c3d46a87","updated":"2026-04-27 14:24:54.000000000","message":"Not to be merged on master, but  rather cherry-picked to f/gapzacho later.","commit_id":"a4f3c7ff3c0e13f64bdf9ed3369bf92099dc48ea"},{"author":{"_account_id":37173,"name":"Alex Fernandes Figueirêdo","display_name":"Alex Figueiredo","email":"alex.fernandesfigueiredo@windriver.com","username":"afernand"},"change_message_id":"fe981c6aec80b22fe6fec9229578d8840a813613","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"efe845b4_8b1e19f2","in_reply_to":"01c25f8e_c3d46a87","updated":"2026-04-30 22:15:16.000000000","message":"Please add a Workflow -1 to your change to make it clear that it should not be merged.","commit_id":"a4f3c7ff3c0e13f64bdf9ed3369bf92099dc48ea"},{"author":{"_account_id":38728,"name":"Amit Mahajan","display_name":"Amit Mahajan","email":"Amit.Mahajan@windriver.com","username":"amahajan"},"change_message_id":"0ac871e397b174ad13b18e0adccc065a5d8138ce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"744c22e3_d367d208","in_reply_to":"efe845b4_8b1e19f2","updated":"2026-05-01 11:22:00.000000000","message":"Done","commit_id":"a4f3c7ff3c0e13f64bdf9ed3369bf92099dc48ea"},{"author":{"_account_id":28410,"name":"Scott Little","email":"scott.little@windriver.com","username":"slittle1"},"change_message_id":"d62376df7c0386ea617e8fddcafdf4529ec2766f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7b2ee145_93a2b066","updated":"2026-04-29 20:37:56.000000000","message":"Not to be merged on master, but rather cherry-picked to f/gapzacho later.","commit_id":"fb9e7a5a1557926c06d79053be716b03e9c45a45"},{"author":{"_account_id":28410,"name":"Scott Little","email":"scott.little@windriver.com","username":"slittle1"},"change_message_id":"9837c1bbb7cf5a51142daf2af7f47e0930e390df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d52f5006_8c3b26d5","updated":"2026-04-30 13:59:12.000000000","message":"The Dockerfile update appear to be generic for trixie.\nIs the Loci path backward compatible for bullseye?   If not, perhaps split this review into two parts?","commit_id":"fb9e7a5a1557926c06d79053be716b03e9c45a45"},{"author":{"_account_id":28410,"name":"Scott Little","email":"scott.little@windriver.com","username":"slittle1"},"change_message_id":"8da45d942b9825179d2df70932860cf95921cbfb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9ba6293c_fec86389","in_reply_to":"2761f47b_8eac5c15","updated":"2026-05-01 13:47:07.000000000","message":"Done","commit_id":"fb9e7a5a1557926c06d79053be716b03e9c45a45"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"01fd03816ddc22ce00a8523a4ae08eb970026d4c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2761f47b_8eac5c15","in_reply_to":"7b2ee145_93a2b066","updated":"2026-04-30 16:14:42.000000000","message":"@scott.little@windriver.com that is not quite true.\nAlthough this is being done as part of the openstack epoxy-\u003egazpacho work (stx-openstack), this base image WILL DEF need to be ported to trixie, as part of the platform upgrade.\nAmit did it already, just because openstack-gazpacho images REQUIRE a new python version available on trixie.\n\nSo, for trixie this goes to master (and cherry-pick to f/gazpacho, if branch is created before  the merge).\n\nMakes sense?","commit_id":"fb9e7a5a1557926c06d79053be716b03e9c45a45"},{"author":{"_account_id":38728,"name":"Amit Mahajan","display_name":"Amit Mahajan","email":"Amit.Mahajan@windriver.com","username":"amahajan"},"change_message_id":"f03de6fd60e10d8989fb119143cbb60440b76a9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d76b43af_851484a0","in_reply_to":"bd937aee_75e31d35","updated":"2026-04-30 18:34:09.000000000","message":"Done, as discussed with Scott.","commit_id":"fb9e7a5a1557926c06d79053be716b03e9c45a45"},{"author":{"_account_id":38728,"name":"Amit Mahajan","display_name":"Amit Mahajan","email":"Amit.Mahajan@windriver.com","username":"amahajan"},"change_message_id":"d53f33996705f1b7ea645075c0cff89411dfbc18","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bd937aee_75e31d35","in_reply_to":"d52f5006_8c3b26d5","updated":"2026-04-30 15:14:32.000000000","message":"LOCI change done in 0004-*.patch is to remove pinning virtualenv to 20.7.2, not sure why it was done. How about if application of this patch or any trixie specific patch is made conditional (if OS_CODENAME is trixie) in cgcs-root/build-tools/build-docker-images/build-stx-images.sh - get_loci() function?","commit_id":"fb9e7a5a1557926c06d79053be716b03e9c45a45"},{"author":{"_account_id":38728,"name":"Amit Mahajan","display_name":"Amit Mahajan","email":"Amit.Mahajan@windriver.com","username":"amahajan"},"change_message_id":"d53f33996705f1b7ea645075c0cff89411dfbc18","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0f33d0e8_048b10d1","in_reply_to":"d52f5006_8c3b26d5","updated":"2026-04-30 15:14:32.000000000","message":"Regarding LOCI patch, not sure why the virtualenv version was pinned to a specific version. It might or might not break bullseye. What if we have all the patches 0001-0004*.patch and future trixie specific patches in a new trixie specific folder - cgcs-root/build-tools/build-docker-images/loci/patches/trixie and make change in cgcs-root/build-tools/build-docker-images/build-stx-images.sh - get_loci function to apply the patches conditionally, if OS_CODENAME is bullseye or trixie? For the bullseye patches 0001-0003*.patch can go in bullseye directory.","commit_id":"fb9e7a5a1557926c06d79053be716b03e9c45a45"},{"author":{"_account_id":37173,"name":"Alex Fernandes Figueirêdo","display_name":"Alex Figueiredo","email":"alex.fernandesfigueiredo@windriver.com","username":"afernand"},"change_message_id":"49ead07dd0ec757d3b7e02602643c3001ad013b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"971acead_20874f66","updated":"2026-04-30 21:29:15.000000000","message":"Missing \"Test Plan\" description and Starlingx \"Story/Task\" reference.","commit_id":"abac95d0e486fdb7fcda9e8b86d9f71a971153c1"}],"build-tools/build-docker-images/stx-debian-trixie/Dockerfile.stable":[{"author":{"_account_id":28410,"name":"Scott Little","email":"scott.little@windriver.com","username":"slittle1"},"change_message_id":"81e0579ff5fbb5ca73f895468c5c019830c976a3","unresolved":true,"context_lines":[{"line_number":110,"context_line":"RUN apt-get update -y \u0026\u0026 \\"},{"line_number":111,"context_line":"    apt-get upgrade -y \u0026\u0026 \\"},{"line_number":112,"context_line":"    apt-get install -y \\"},{"line_number":113,"context_line":"#        python3-thriftpy2 \\"},{"line_number":114,"context_line":"        python3-nss \\"},{"line_number":115,"context_line":"        \u0026\u0026 \\"},{"line_number":116,"context_line":"    apt-get clean \u0026\u0026 \\"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"5977735f_d69aa86c","line":113,"range":{"start_line":113,"start_character":0,"end_line":113,"end_character":28},"updated":"2026-04-29 20:35:44.000000000","message":"why not just remove ?","commit_id":"fb9e7a5a1557926c06d79053be716b03e9c45a45"},{"author":{"_account_id":38728,"name":"Amit Mahajan","display_name":"Amit Mahajan","email":"Amit.Mahajan@windriver.com","username":"amahajan"},"change_message_id":"f03de6fd60e10d8989fb119143cbb60440b76a9c","unresolved":false,"context_lines":[{"line_number":110,"context_line":"RUN apt-get update -y \u0026\u0026 \\"},{"line_number":111,"context_line":"    apt-get upgrade -y \u0026\u0026 \\"},{"line_number":112,"context_line":"    apt-get install -y \\"},{"line_number":113,"context_line":"#        python3-thriftpy2 \\"},{"line_number":114,"context_line":"        python3-nss \\"},{"line_number":115,"context_line":"        \u0026\u0026 \\"},{"line_number":116,"context_line":"    apt-get clean \u0026\u0026 \\"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"74141509_aa3cf76a","line":113,"range":{"start_line":113,"start_character":0,"end_line":113,"end_character":28},"in_reply_to":"2f6b469d_de37d6a1","updated":"2026-04-30 18:34:09.000000000","message":"Done.","commit_id":"fb9e7a5a1557926c06d79053be716b03e9c45a45"},{"author":{"_account_id":38728,"name":"Amit Mahajan","display_name":"Amit Mahajan","email":"Amit.Mahajan@windriver.com","username":"amahajan"},"change_message_id":"96bbb3172da1b09c5628da6a1f9b7e9f75c55415","unresolved":true,"context_lines":[{"line_number":110,"context_line":"RUN apt-get update -y \u0026\u0026 \\"},{"line_number":111,"context_line":"    apt-get upgrade -y \u0026\u0026 \\"},{"line_number":112,"context_line":"    apt-get install -y \\"},{"line_number":113,"context_line":"#        python3-thriftpy2 \\"},{"line_number":114,"context_line":"        python3-nss \\"},{"line_number":115,"context_line":"        \u0026\u0026 \\"},{"line_number":116,"context_line":"    apt-get clean \u0026\u0026 \\"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"2f6b469d_de37d6a1","line":113,"range":{"start_line":113,"start_character":0,"end_line":113,"end_character":28},"in_reply_to":"5977735f_d69aa86c","updated":"2026-04-30 13:25:18.000000000","message":"I\u0027ll remove it.","commit_id":"fb9e7a5a1557926c06d79053be716b03e9c45a45"}]}
