)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"fbd521472bec4951559163ad1ddb68a777fdd27c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f4f2529a_d2c13320","updated":"2023-03-13 22:00:53.000000000","message":"Looks obvious enough to me.","commit_id":"4b7da146c201d02e89634cacc29f17bf4528368b"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"3393cb77e892cd244ce516c64f57c40d638f1194","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"71e97515_90b2e944","updated":"2023-02-09 19:41:55.000000000","message":"Setting +RP since this could have significant stability/security impact.","commit_id":"4b7da146c201d02e89634cacc29f17bf4528368b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"eaf294fa69919a996f705d97454edf536eab28ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4731536e_0c64a115","updated":"2023-03-02 08:17:17.000000000","message":"changes look good, one comment inline.","commit_id":"4b7da146c201d02e89634cacc29f17bf4528368b"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"028a6f16440cb0bf3bac45fd37cbfd9d5b2952c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"05ea77bb_f947d175","updated":"2025-04-13 16:16:09.000000000","message":"LGTM except that this missed the epoxy release, so the \u0027deprecated_since\u0027 strings will need to be updated.  Raising the priority because this patch was discussed at the Flamingo PTG.","commit_id":"9c7542d578d450500f329cfdb80f9b4fa8582a46"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"60867069037b5c174c89688596b0cbc6cfa35fe0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"78c05809_314b1b9e","updated":"2025-04-14 10:11:49.000000000","message":"@rosmaita.fossdev@gmail.com I updated deprecation time frame related strings.","commit_id":"63c2d1e14f66f6cb6e769527532476f99bb7a045"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"9493ddcd5142d6519603344c867757e626074369","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a4b7c474_5ea94852","updated":"2026-07-14 10:24:32.000000000","message":"Note, this might become a bit urgent. In our recent testing with latest Ceph (tentacle), we\u0027ve found that now 1 GB of address space is reliably not enough, and one has to edit the default value in the code as setting the config option does not work w/o this patch.","commit_id":"ee523236f01e82db4917e4cf27d2501376b1af6b"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"e5ccb301e191d20df3c8dc79cf1192d5c71dfb36","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"436586f0_965fc80b","updated":"2026-09-04 14:54:32.000000000","message":"Raising the priority ... we need this in Hibiscus RC-1, the issue is happening more frequently in the gate with the ceph encrypted vol tests, and we need the configuration changes to actually work.","commit_id":"ee523236f01e82db4917e4cf27d2501376b1af6b"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"9dc1c649bb8fa302e51ad5e71d63e49e8f0723a0","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"c5add67e_7c9f7110","updated":"2026-09-04 22:35:03.000000000","message":"See inline for a comment about the option rename.  I have a different question, namely at some point you made the prlimits a singleton that got created on the first use of qemu_img_info().  I notice that in PS 7 it\u0027s no longer a singleton; instead we create a new ProcessLimits object each time.  Is this because the config is reloadable without restarting the service, or did you have another reason?","commit_id":"ee523236f01e82db4917e4cf27d2501376b1af6b"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"90562e34e4ed7fce4e6cd31d24a36725966d8b0d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"4c804350_3a0e4cc2","updated":"2026-09-11 14:24:56.000000000","message":"This could possibly be improved to use a singleton ProcessLimits object with some kind of check for a change in the config values, but let\u0027s save that for a followup.","commit_id":"ee523236f01e82db4917e4cf27d2501376b1af6b"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"b872b7b6468719a85fa5f7c6279e42ab57c475c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"cbc0378d_e4848fc9","updated":"2026-07-15 18:41:05.000000000","message":"any upgrade impact?if yes, please update release notes.","commit_id":"ee523236f01e82db4917e4cf27d2501376b1af6b"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"063e2b3130f7cb65f43b89b5eeb276938481b185","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e5ec6d12_29da2b10","updated":"2026-09-11 14:12:21.000000000","message":"recheck","commit_id":"ee523236f01e82db4917e4cf27d2501376b1af6b"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"79308480409f5fa638fd75a5f1baeaec68def9c7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"108bdd37_a37ea151","in_reply_to":"cbc0378d_e4848fc9","updated":"2026-07-22 11:56:45.000000000","message":"I don\u0027t think there\u0027s any real impact. Saying \"now this option starts to actually work\" does not have much sense.\nIf you as an operator have set that option to a non-default value - that\u0027s most likely because something did not work. But then, it still wouldn\u0027t work if you changed that value because it had no effect 🤷","commit_id":"ee523236f01e82db4917e4cf27d2501376b1af6b"}],"cinder/image/image_utils.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"eaf294fa69919a996f705d97454edf536eab28ca","unresolved":true,"context_lines":[{"line_number":67,"context_line":"                default\u003dTrue,"},{"line_number":68,"context_line":"                help\u003d\u0027When possible, compress images uploaded \u0027"},{"line_number":69,"context_line":"                \u0027to the image service\u0027),"},{"line_number":70,"context_line":"    cfg.IntOpt(\u0027image_conversion_cpu_limit\u0027,"},{"line_number":71,"context_line":"               default\u003d60,"},{"line_number":72,"context_line":"               help\u003d\u0027CPU time limit in seconds to introspect the image\u0027),"},{"line_number":73,"context_line":"    cfg.IntOpt(\u0027image_conversion_address_space_limit\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3ac9c009_4642180f","line":70,"range":{"start_line":70,"start_character":16,"end_line":70,"end_character":42},"updated":"2023-03-02 08:17:17.000000000","message":"would be good to update these config options and deprecate the old ones as they\u0027re misleading here, the new options could be,\n\nimage_introspect_cpu_limit\nimage_introspect_address_space_limit","commit_id":"4b7da146c201d02e89634cacc29f17bf4528368b"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"db4dd9286e11e958c1520d3503737ba13c3d1a42","unresolved":false,"context_lines":[{"line_number":67,"context_line":"                default\u003dTrue,"},{"line_number":68,"context_line":"                help\u003d\u0027When possible, compress images uploaded \u0027"},{"line_number":69,"context_line":"                \u0027to the image service\u0027),"},{"line_number":70,"context_line":"    cfg.IntOpt(\u0027image_conversion_cpu_limit\u0027,"},{"line_number":71,"context_line":"               default\u003d60,"},{"line_number":72,"context_line":"               help\u003d\u0027CPU time limit in seconds to introspect the image\u0027),"},{"line_number":73,"context_line":"    cfg.IntOpt(\u0027image_conversion_address_space_limit\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"f4465d9e_41c46a12","line":70,"range":{"start_line":70,"start_character":16,"end_line":70,"end_character":42},"in_reply_to":"3ac9c009_4642180f","updated":"2024-12-19 14:42:16.000000000","message":"Done","commit_id":"4b7da146c201d02e89634cacc29f17bf4528368b"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"bae57718c63349d88eb81ae3c83e0236bbccd35d","unresolved":true,"context_lines":[{"line_number":154,"context_line":"    return QEMU_IMG_FORMAT_MAP_INV.get(disk_format, disk_format)"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"def qemu_limits():"},{"line_number":158,"context_line":"    return processutils.ProcessLimits("},{"line_number":159,"context_line":"        cpu_time\u003dCONF.image_conversion_cpu_limit,"},{"line_number":160,"context_line":"        address_space\u003dCONF.image_conversion_address_space_limit * units.Gi)"}],"source_content_type":"text/x-python","patch_set":3,"id":"a4eb71c7_c15da0be","line":157,"updated":"2024-11-04 21:04:52.000000000","message":"Probably better to make this a singleton rather than creating a new object each time.","commit_id":"4b7da146c201d02e89634cacc29f17bf4528368b"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"db4dd9286e11e958c1520d3503737ba13c3d1a42","unresolved":false,"context_lines":[{"line_number":154,"context_line":"    return QEMU_IMG_FORMAT_MAP_INV.get(disk_format, disk_format)"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"def qemu_limits():"},{"line_number":158,"context_line":"    return processutils.ProcessLimits("},{"line_number":159,"context_line":"        cpu_time\u003dCONF.image_conversion_cpu_limit,"},{"line_number":160,"context_line":"        address_space\u003dCONF.image_conversion_address_space_limit * units.Gi)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9e072f62_f29f015d","line":157,"in_reply_to":"a4eb71c7_c15da0be","updated":"2024-12-19 14:42:16.000000000","message":"Done","commit_id":"4b7da146c201d02e89634cacc29f17bf4528368b"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"028a6f16440cb0bf3bac45fd37cbfd9d5b2952c6","unresolved":true,"context_lines":[{"line_number":68,"context_line":"                \u0027to the image service\u0027),"},{"line_number":69,"context_line":"    cfg.IntOpt(\u0027image_conversion_cpu_limit\u0027,"},{"line_number":70,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":71,"context_line":"               deprecated_since\u003d\"2025.1/Epoxy\","},{"line_number":72,"context_line":"               deprecated_reason\u003d\"Renamed to \u0027image_introspection_cpu_limit\u0027\","},{"line_number":73,"context_line":"               default\u003d60,"},{"line_number":74,"context_line":"               help\u003d\u0027CPU time limit in seconds to convert the image\u0027),"}],"source_content_type":"text/x-python","patch_set":5,"id":"0b9b9c74_a6d8b734","line":71,"range":{"start_line":71,"start_character":33,"end_line":71,"end_character":45},"updated":"2025-04-13 16:16:09.000000000","message":"This will have to change to 2025.2/Flamingo.  We won\u0027t be able to remove the old opt names until 2026.2/H (so that a deprecation notice will appear in the sample config for the next SLURP release, which is 2026.1).  Because that is a long time off, when you update the \u0027deprecated since\u0027, please add a TODO comment after line 68 reminding us to remove the opts in 2026.2.","commit_id":"9c7542d578d450500f329cfdb80f9b4fa8582a46"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"60867069037b5c174c89688596b0cbc6cfa35fe0","unresolved":false,"context_lines":[{"line_number":68,"context_line":"                \u0027to the image service\u0027),"},{"line_number":69,"context_line":"    cfg.IntOpt(\u0027image_conversion_cpu_limit\u0027,"},{"line_number":70,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":71,"context_line":"               deprecated_since\u003d\"2025.1/Epoxy\","},{"line_number":72,"context_line":"               deprecated_reason\u003d\"Renamed to \u0027image_introspection_cpu_limit\u0027\","},{"line_number":73,"context_line":"               default\u003d60,"},{"line_number":74,"context_line":"               help\u003d\u0027CPU time limit in seconds to convert the image\u0027),"}],"source_content_type":"text/x-python","patch_set":5,"id":"4b963934_c871a9ee","line":71,"range":{"start_line":71,"start_character":33,"end_line":71,"end_character":45},"in_reply_to":"0b9b9c74_a6d8b734","updated":"2025-04-14 10:11:49.000000000","message":"Done","commit_id":"9c7542d578d450500f329cfdb80f9b4fa8582a46"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"98420204ce9145343e520a670ea7e2ebd07d3610","unresolved":true,"context_lines":[{"line_number":68,"context_line":"                default\u003dTrue,"},{"line_number":69,"context_line":"                help\u003d\u0027When possible, compress images uploaded \u0027"},{"line_number":70,"context_line":"                \u0027to the image service\u0027),"},{"line_number":71,"context_line":"    # TODO(pas-ha): remove in 2027.2/J release"},{"line_number":72,"context_line":"    cfg.IntOpt(\u0027image_conversion_cpu_limit\u0027,"},{"line_number":73,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":74,"context_line":"               deprecated_since\u003d\"2026.2/Hibiscus\","}],"source_content_type":"text/x-python","patch_set":7,"id":"fac9c68f_c33fc4bb","line":71,"range":{"start_line":71,"start_character":0,"end_line":71,"end_character":46},"updated":"2026-09-04 15:30:37.000000000","message":"I don\u0027t think we should rename those variables. It\u0027s preferable that we keep the same naming use it in Nova. If I understood correctly, your rationale is that \u0027introspection\u0027 reflects better other operations that are performed by the qemu_img function, even if that\u0027s the case, I still believe that keeping parity with no is better than having a different name variable that does the same thing.","commit_id":"ee523236f01e82db4917e4cf27d2501376b1af6b"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"05bf259d563f66f4246bc4409dd917d7cd8be33d","unresolved":true,"context_lines":[{"line_number":68,"context_line":"                default\u003dTrue,"},{"line_number":69,"context_line":"                help\u003d\u0027When possible, compress images uploaded \u0027"},{"line_number":70,"context_line":"                \u0027to the image service\u0027),"},{"line_number":71,"context_line":"    # TODO(pas-ha): remove in 2027.2/J release"},{"line_number":72,"context_line":"    cfg.IntOpt(\u0027image_conversion_cpu_limit\u0027,"},{"line_number":73,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":74,"context_line":"               deprecated_since\u003d\"2026.2/Hibiscus\","}],"source_content_type":"text/x-python","patch_set":7,"id":"ea3a8b79_acbd3486","line":71,"range":{"start_line":71,"start_character":0,"end_line":71,"end_character":46},"in_reply_to":"21c3396d_8f1e6526","updated":"2026-09-04 18:05:04.000000000","message":"hey @rajatdhasmana@gmail.com, can you reconsider your statement on on this option naming? \u0027Introspection\u0027 doesn\u0027t lead me to a direct correlation to what this parameter does, furthermore I think we should keep parity with the Nova naming since they means the same.","commit_id":"ee523236f01e82db4917e4cf27d2501376b1af6b"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"5f28715266e8c5d3c8a240c86e140c0b83084458","unresolved":true,"context_lines":[{"line_number":68,"context_line":"                default\u003dTrue,"},{"line_number":69,"context_line":"                help\u003d\u0027When possible, compress images uploaded \u0027"},{"line_number":70,"context_line":"                \u0027to the image service\u0027),"},{"line_number":71,"context_line":"    # TODO(pas-ha): remove in 2027.2/J release"},{"line_number":72,"context_line":"    cfg.IntOpt(\u0027image_conversion_cpu_limit\u0027,"},{"line_number":73,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":74,"context_line":"               deprecated_since\u003d\"2026.2/Hibiscus\","}],"source_content_type":"text/x-python","patch_set":7,"id":"ea091bb1_75a047bc","line":71,"range":{"start_line":71,"start_character":0,"end_line":71,"end_character":46},"in_reply_to":"4629d82f_b8982cab","updated":"2026-09-09 18:55:26.000000000","message":"Considering that you also leans towards renaming it, I\u0027m fine with the rename as well.","commit_id":"ee523236f01e82db4917e4cf27d2501376b1af6b"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"9dc1c649bb8fa302e51ad5e71d63e49e8f0723a0","unresolved":true,"context_lines":[{"line_number":68,"context_line":"                default\u003dTrue,"},{"line_number":69,"context_line":"                help\u003d\u0027When possible, compress images uploaded \u0027"},{"line_number":70,"context_line":"                \u0027to the image service\u0027),"},{"line_number":71,"context_line":"    # TODO(pas-ha): remove in 2027.2/J release"},{"line_number":72,"context_line":"    cfg.IntOpt(\u0027image_conversion_cpu_limit\u0027,"},{"line_number":73,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":74,"context_line":"               deprecated_since\u003d\"2026.2/Hibiscus\","}],"source_content_type":"text/x-python","patch_set":7,"id":"4629d82f_b8982cab","line":71,"range":{"start_line":71,"start_character":0,"end_line":71,"end_character":46},"in_reply_to":"ea3a8b79_acbd3486","updated":"2026-09-04 22:35:03.000000000","message":"We can see what Rajat thinks, but these options don\u0027t really have anything to do with image conversion, which happens only after we have checked the image to see what it is.  We need the prlimits for the \u0027qemu-img info\u0027 call, because it\u0027s unsafe to give it unrestricted time/space to examine the image (there are some CVEs around this).  Plus, IIRC, we had some complaints from operators who were trying to use the options to control image conversion, which you can\u0027t do, because they aren\u0027t used in the image conversion call.  So even though they will differ from nova, I think the rename is a good idea.","commit_id":"ee523236f01e82db4917e4cf27d2501376b1af6b"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"d7cee075c4463545ed5752d036b37b25035252c2","unresolved":true,"context_lines":[{"line_number":68,"context_line":"                default\u003dTrue,"},{"line_number":69,"context_line":"                help\u003d\u0027When possible, compress images uploaded \u0027"},{"line_number":70,"context_line":"                \u0027to the image service\u0027),"},{"line_number":71,"context_line":"    # TODO(pas-ha): remove in 2027.2/J release"},{"line_number":72,"context_line":"    cfg.IntOpt(\u0027image_conversion_cpu_limit\u0027,"},{"line_number":73,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":74,"context_line":"               deprecated_since\u003d\"2026.2/Hibiscus\","}],"source_content_type":"text/x-python","patch_set":7,"id":"21c3396d_8f1e6526","line":71,"range":{"start_line":71,"start_character":0,"end_line":71,"end_character":46},"in_reply_to":"fac9c68f_c33fc4bb","updated":"2026-09-04 16:25:57.000000000","message":"this was changed in response to one of previous comments\nhttps://review.opendev.org/c/openstack/cinder/+/873249/7#message-eaf294fa69919a996f705d97454edf536eab28ca\n\nI personally don\u0027t have any preferrence, keeping the old name is easier for sure (even though the name might be a bit misleading).","commit_id":"ee523236f01e82db4917e4cf27d2501376b1af6b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"aadeec048efd7a9d531f0d7c5c797cb432c99f48","unresolved":true,"context_lines":[{"line_number":69,"context_line":"                help\u003d\u0027When possible, compress images uploaded \u0027"},{"line_number":70,"context_line":"                \u0027to the image service\u0027),"},{"line_number":71,"context_line":"    # TODO(pas-ha): remove in 2027.2/J release"},{"line_number":72,"context_line":"    cfg.IntOpt(\u0027image_conversion_cpu_limit\u0027,"},{"line_number":73,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":74,"context_line":"               deprecated_since\u003d\"2026.2/Hibiscus\","},{"line_number":75,"context_line":"               deprecated_reason\u003d\"Renamed to \u0027image_introspection_cpu_limit\u0027\","}],"source_content_type":"text/x-python","patch_set":7,"id":"8d03d63a_a6e2e914","line":72,"range":{"start_line":72,"start_character":16,"end_line":72,"end_character":42},"updated":"2026-09-12 17:46:54.000000000","message":"You don\u0027t really have to keep these deprecated names. oslo.config should take care of relations defined by the deprecated_name arg, and list these as deprecated names...","commit_id":"ee523236f01e82db4917e4cf27d2501376b1af6b"}],"cinder/tests/unit/test_image_utils.py":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"fbd521472bec4951559163ad1ddb68a777fdd27c","unresolved":true,"context_lines":[{"line_number":37,"context_line":"        limits \u003d image_utils.qemu_limits()"},{"line_number":38,"context_line":"        self.assertIsInstance(limits, processutils.ProcessLimits)"},{"line_number":39,"context_line":"        self.assertEqual(0, limits.cpu_time)"},{"line_number":40,"context_line":"        self.assertEqual(0, limits.address_space)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @mock.patch(\u0027os.name\u0027, new\u003d\u0027posix\u0027)"},{"line_number":43,"context_line":"    @mock.patch.object(image_utils, \"qemu_limits\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"556bd321_6de59ff4","line":40,"updated":"2023-03-13 22:00:53.000000000","message":"You know, it you used some other value, the multiplier would\u0027ve been evident too.","commit_id":"4b7da146c201d02e89634cacc29f17bf4528368b"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"db4dd9286e11e958c1520d3503737ba13c3d1a42","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        limits \u003d image_utils.qemu_limits()"},{"line_number":38,"context_line":"        self.assertIsInstance(limits, processutils.ProcessLimits)"},{"line_number":39,"context_line":"        self.assertEqual(0, limits.cpu_time)"},{"line_number":40,"context_line":"        self.assertEqual(0, limits.address_space)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @mock.patch(\u0027os.name\u0027, new\u003d\u0027posix\u0027)"},{"line_number":43,"context_line":"    @mock.patch.object(image_utils, \"qemu_limits\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"362e3319_9de177ac","line":40,"in_reply_to":"556bd321_6de59ff4","updated":"2024-12-19 14:42:16.000000000","message":"Done","commit_id":"4b7da146c201d02e89634cacc29f17bf4528368b"}]}