)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"eb3f8262e73dce8f90a7551899ab70f37c5295b5","unresolved":false,"context_lines":[{"line_number":35,"context_line":"- the defautl value is updated to 1G to maintain stable"},{"line_number":36,"context_line":"  branch behavior"},{"line_number":37,"context_line":"- the ci jobs are updated to use 2G to match master"},{"line_number":38,"context_line":"- the release note is updated to reflect this."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Closes-Bug: #2116852"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"752dad9b_1c6f928d","line":38,"updated":"2026-07-14 07:20:25.000000000","message":"thanks, yepp, this is the nicest solution.","commit_id":"8595b8eed4d0cabf6a7db04b0d946cf764187965"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"dd2f3510bda068168c3506556fe4445d7c57cc9d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c22a5fdf_d65d60a6","updated":"2026-07-13 12:42:11.000000000","message":"my only concern regarding this patch is that it changes default qemu config value, which we usually don\u0027t allow on stable branches. on the other hand, currently the gate is not stable without this fix (or without skipping `test_extend_attached_encrypted_volume_luksv1` test case with e.g. tempest_exclude_regex). but as Sean pointed out on IRC, this is needed for production systems as well, otherwise they might experience the same issue... so meh...\n\nmaybe one option is to set the default to 1G (which is the qemu default) and suggest changing it to 2G to operators if the error occurs (and setting it in nova-ceph-multistore job config to 2G to make the gate stable)\n\nopinions?","commit_id":"4f15360c6d466f64973a05f1b99afff561195c94"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"dfdd87f9423feb4b991a9134a9edbce099827170","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7caedc6b_378cbe89","in_reply_to":"66640533_fbf565fc","updated":"2026-07-14 10:18:12.000000000","message":"ack ill update the rest so by cherry picking this to the older branches","commit_id":"4f15360c6d466f64973a05f1b99afff561195c94"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"eb3f8262e73dce8f90a7551899ab70f37c5295b5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"66640533_fbf565fc","in_reply_to":"90dd8c02_8a404113","updated":"2026-07-14 07:20:25.000000000","message":"thanks, melwitt! This is the best way-forward I think. \u0026\u0026 thanks Sean for updating the patch accordingly! Backport looks perfect with the introduced changes \u0026 noted in commit message and release notes. LGTM. I think this is now good to go.","commit_id":"4f15360c6d466f64973a05f1b99afff561195c94"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f8ce158194180daad5bd39eb530614f6e051ad12","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f38b2f93_420e91b3","in_reply_to":"c22a5fdf_d65d60a6","updated":"2026-07-13 13:24:59.000000000","message":"\"(which is the qemu default)\" not quite\n\nthis is a new cofnig option for a value that was previously hardcoded\n\nqemu-img does not have a paramter for this we are basiclly usign the prlimit kernel api to encorce a memory limit on the coammnd we sapwn via oslo.\n\nso it woudl be more correct to say in the backport we could keep the default behvior constient and only allow 1G of adress space and then allow the ci job or opertors to overried that\n\nbut upstream qemu-img does not expect this type of sandboxign too be sued in genreal so it not qorrect to think fo this as a qmeu default avlue change\n\nit the introduction of a nova config option and update of our hardcoded valued via a different default then the internal costant for the config option.\n\nim pretty nutral on this either way i mainly wanted to get ci results hence the patch to confirm the bevhior","commit_id":"4f15360c6d466f64973a05f1b99afff561195c94"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"20b1f645010c82c564fecf64d4a501b2f5ef8970","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"90dd8c02_8a404113","in_reply_to":"f38b2f93_420e91b3","updated":"2026-07-13 16:19:52.000000000","message":"I think you are both on the same page and it\u0027s just a matter of terminology maybe. I think Elod meant that we (Nova) had been using 1G before this patch as a default.\n\nI think this is a fair point for stable and support setting 1G as the conf option default for stable backports so as not to change default behavior.\n\nFor the jobs, I suggest leaving them as the standard Ubuntu nodeset and then set 2G in the job definition to most closely model what a real life cloud operator would do in this situation (IMHO). Rather than change the OS to suit the job.","commit_id":"4f15360c6d466f64973a05f1b99afff561195c94"}]}