)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"b437aaae56d0b036fc17413890ad8129cc48a812","unresolved":true,"context_lines":[{"line_number":3,"context_line":"AuthorDate: 2023-10-03 14:05:25 +0000"},{"line_number":4,"context_line":"Commit:     Luigi Dino Tamagnone \u003cltamagno@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-11-07 14:27:59 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adding a new config option in tempest.conf as the prefix for the random name resources"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"With this new option all the resources created by tempest will have the prefix defined by resource_name_prefix option in tempest.conf."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"22122a60_790f23b8","line":6,"updated":"2023-11-10 15:15:45.000000000","message":"shorten the lines please:\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure\n\n^^\n\"he first line should be limited to 50 characters and should not end with a period.\nSubsequent lines should be wrapped at 72 characters\"","commit_id":"48ccdb3a15975a195cece351920d52f77ce72778"},{"author":{"_account_id":33983,"name":"Luigi Dino Tamagnone","display_name":"luigi","email":"ltamagno@redhat.com","username":"ltamagno"},"change_message_id":"28038262739c3333afc53b034e0708e89d8df6f1","unresolved":false,"context_lines":[{"line_number":3,"context_line":"AuthorDate: 2023-10-03 14:05:25 +0000"},{"line_number":4,"context_line":"Commit:     Luigi Dino Tamagnone \u003cltamagno@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-11-07 14:27:59 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adding a new config option in tempest.conf as the prefix for the random name resources"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"With this new option all the resources created by tempest will have the prefix defined by resource_name_prefix option in tempest.conf."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"4504d382_5f4b20ff","line":6,"in_reply_to":"22122a60_790f23b8","updated":"2023-11-14 13:36:10.000000000","message":"Done","commit_id":"48ccdb3a15975a195cece351920d52f77ce72778"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"70afaacc1bf813eda61acdac78f1c6c2e236e811","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"694ecc49_9dfbfacf","updated":"2023-10-16 09:02:02.000000000","message":"Thanks for working on this! There seems to be pep8 issues (try running tox -epep8 before pushing next change).","commit_id":"77942c5405a82f0d0418f1252f56163221101fae"},{"author":{"_account_id":33983,"name":"Luigi Dino Tamagnone","display_name":"luigi","email":"ltamagno@redhat.com","username":"ltamagno"},"change_message_id":"2a0a995450cefb8adb0db12b268abaca8f414314","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5f81fdd4_46d3b05c","updated":"2023-10-30 14:05:29.000000000","message":"yes. I already modified it locally but didn\u0027t update it. I\u0027ll look to the other errors in these days.","commit_id":"77942c5405a82f0d0418f1252f56163221101fae"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"7e032707f527e753236cb1e885d3cdd05de88102","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"0a791e85_83a5e08c","updated":"2023-12-01 10:25:38.000000000","message":"recheck unrelated failure","commit_id":"46c302739efb5bcca33f10936eafa2c09b452500"},{"author":{"_account_id":33983,"name":"Luigi Dino Tamagnone","display_name":"luigi","email":"ltamagno@redhat.com","username":"ltamagno"},"change_message_id":"caefc6556af1e02b49fd21e0704ad0d658b13653","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"743ce9be_86e1c721","updated":"2023-12-01 15:35:02.000000000","message":"recheck was working in previous try","commit_id":"46c302739efb5bcca33f10936eafa2c09b452500"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"201939ced296890e79752d5d06573b76407a114c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"4efa4408_8b64765c","updated":"2023-12-05 20:24:42.000000000","message":"thanks, almost looks good. a few comments inline and please add the release note for the new config option.  example - https://github.com/openstack/tempest/blob/8def25cbb5885e91899793f8575d3919816a0d12/releasenotes/notes/add-floating-ip-config-option-e5774bf77702ce9f.yaml","commit_id":"46c302739efb5bcca33f10936eafa2c09b452500"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"45f58babff1dbecdd45210a66e097bbab6ddfe38","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"2624b4c2_ff41275d","updated":"2023-12-07 10:47:53.000000000","message":"recheck post failure","commit_id":"17053d69e57d0131e795ba6f35c6ed7d77ba8bcf"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"cd369c6f28b4efdbc55e008eb8af15bd3901e3a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"c6098b39_0c9478c8","updated":"2023-12-07 00:17:06.000000000","message":"thanks. lgtm","commit_id":"17053d69e57d0131e795ba6f35c6ed7d77ba8bcf"}],"tempest/config.py":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"41885660fba3e04cc57ca318ded5279c84db0dfa","unresolved":true,"context_lines":[{"line_number":981,"context_line":"                      \"migration, in seconds. \""},{"line_number":982,"context_line":"                      \"When the measured downtime exceeds this value, an \""},{"line_number":983,"context_line":"                      \"exception is raised.\"),"},{"line_number":984,"context_line":"    cfg.StrOpt(\u0027resource_name_prefix\u0027,"},{"line_number":985,"context_line":"               default\u003d\u0027tempest-test\u0027,"},{"line_number":986,"context_line":"               help\u003d\u0027Define the prefix name for the resources created \u0027"},{"line_number":987,"context_line":"                    \u0027by tempest.\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"0bb36a19_41be2b3f","line":984,"range":{"start_line":984,"start_character":16,"end_line":984,"end_character":36},"updated":"2023-10-31 13:27:29.000000000","message":"considering that we can\u0027t use CONF in tempest/lib and there isn\u0027t any other place where we wanna read this, we don\u0027t need to add a new option","commit_id":"541ae8780cabc0ba9b46233d4028f964c40f0abc"},{"author":{"_account_id":33983,"name":"Luigi Dino Tamagnone","display_name":"luigi","email":"ltamagno@redhat.com","username":"ltamagno"},"change_message_id":"3557a32f43ac01da1634c81018441e2a7faff4cb","unresolved":false,"context_lines":[{"line_number":981,"context_line":"                      \"migration, in seconds. \""},{"line_number":982,"context_line":"                      \"When the measured downtime exceeds this value, an \""},{"line_number":983,"context_line":"                      \"exception is raised.\"),"},{"line_number":984,"context_line":"    cfg.StrOpt(\u0027resource_name_prefix\u0027,"},{"line_number":985,"context_line":"               default\u003d\u0027tempest-test\u0027,"},{"line_number":986,"context_line":"               help\u003d\u0027Define the prefix name for the resources created \u0027"},{"line_number":987,"context_line":"                    \u0027by tempest.\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"49eba395_c7eddc60","line":984,"range":{"start_line":984,"start_character":16,"end_line":984,"end_character":36},"in_reply_to":"0bb36a19_41be2b3f","updated":"2023-11-07 15:59:47.000000000","message":"Done","commit_id":"541ae8780cabc0ba9b46233d4028f964c40f0abc"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"235bfe71dbec3fe9203fb4fdab5302e81f82895d","unresolved":true,"context_lines":[{"line_number":979,"context_line":"                      \"When the measured downtime exceeds this value, an \""},{"line_number":980,"context_line":"                      \"exception is raised.\"),"},{"line_number":981,"context_line":"    cfg.StrOpt(\u0027resource_name_prefix\u0027,"},{"line_number":982,"context_line":"               default\u003d\u0027tempest-test\u0027,"},{"line_number":983,"context_line":"               help\u003d\u0027Define the prefix name for the resources created \u0027"},{"line_number":984,"context_line":"                    \u0027by tempest.\u0027)"},{"line_number":985,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":9,"id":"31ecfd9d_15cefb59","line":982,"range":{"start_line":982,"start_character":23,"end_line":982,"end_character":37},"updated":"2023-12-05 20:38:01.000000000","message":"the below comment in other change using this config option makes me think that we should default it to only \u0027tempest\u0027\n\nhttps://review.opendev.org/c/openstack/tempest/+/902252/3/tempest/api/compute/admin/test_aggregates_negative.py#48","commit_id":"46c302739efb5bcca33f10936eafa2c09b452500"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"00e8f7ce341c62d9a9d4953d3f5c601287acc3f3","unresolved":false,"context_lines":[{"line_number":979,"context_line":"                      \"When the measured downtime exceeds this value, an \""},{"line_number":980,"context_line":"                      \"exception is raised.\"),"},{"line_number":981,"context_line":"    cfg.StrOpt(\u0027resource_name_prefix\u0027,"},{"line_number":982,"context_line":"               default\u003d\u0027tempest-test\u0027,"},{"line_number":983,"context_line":"               help\u003d\u0027Define the prefix name for the resources created \u0027"},{"line_number":984,"context_line":"                    \u0027by tempest.\u0027)"},{"line_number":985,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":9,"id":"f1ce5ad8_da169b36","line":982,"range":{"start_line":982,"start_character":23,"end_line":982,"end_character":37},"in_reply_to":"31ecfd9d_15cefb59","updated":"2023-12-06 20:25:57.000000000","message":"Done","commit_id":"46c302739efb5bcca33f10936eafa2c09b452500"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"201939ced296890e79752d5d06573b76407a114c","unresolved":true,"context_lines":[{"line_number":981,"context_line":"    cfg.StrOpt(\u0027resource_name_prefix\u0027,"},{"line_number":982,"context_line":"               default\u003d\u0027tempest-test\u0027,"},{"line_number":983,"context_line":"               help\u003d\u0027Define the prefix name for the resources created \u0027"},{"line_number":984,"context_line":"                    \u0027by tempest.\u0027)"},{"line_number":985,"context_line":"]"},{"line_number":986,"context_line":""},{"line_number":987,"context_line":"volume_group \u003d cfg.OptGroup(name\u003d\u0027volume\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"5b82f761_570de01d","line":984,"range":{"start_line":984,"start_character":31,"end_line":984,"end_character":32},"updated":"2023-12-05 20:24:42.000000000","message":"lets add more detail here.\n\n....Tempest cleanup CLI will use this config option to cleanup the resources created by Tempest. Make sure this prefix does not match with the resource name you do not want Tempest cleanup CLI to delete.","commit_id":"46c302739efb5bcca33f10936eafa2c09b452500"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"00e8f7ce341c62d9a9d4953d3f5c601287acc3f3","unresolved":false,"context_lines":[{"line_number":981,"context_line":"    cfg.StrOpt(\u0027resource_name_prefix\u0027,"},{"line_number":982,"context_line":"               default\u003d\u0027tempest-test\u0027,"},{"line_number":983,"context_line":"               help\u003d\u0027Define the prefix name for the resources created \u0027"},{"line_number":984,"context_line":"                    \u0027by tempest.\u0027)"},{"line_number":985,"context_line":"]"},{"line_number":986,"context_line":""},{"line_number":987,"context_line":"volume_group \u003d cfg.OptGroup(name\u003d\u0027volume\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"851df504_797be188","line":984,"range":{"start_line":984,"start_character":31,"end_line":984,"end_character":32},"in_reply_to":"5b82f761_570de01d","updated":"2023-12-06 20:25:57.000000000","message":"Done","commit_id":"46c302739efb5bcca33f10936eafa2c09b452500"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"201939ced296890e79752d5d06573b76407a114c","unresolved":true,"context_lines":[{"line_number":978,"context_line":"                      \"migration, in seconds. \""},{"line_number":979,"context_line":"                      \"When the measured downtime exceeds this value, an \""},{"line_number":980,"context_line":"                      \"exception is raised.\"),"},{"line_number":981,"context_line":"    cfg.StrOpt(\u0027resource_name_prefix\u0027,"},{"line_number":982,"context_line":"               default\u003d\u0027tempest-test\u0027,"},{"line_number":983,"context_line":"               help\u003d\u0027Define the prefix name for the resources created \u0027"},{"line_number":984,"context_line":"                    \u0027by tempest.\u0027)"},{"line_number":985,"context_line":"]"},{"line_number":986,"context_line":""},{"line_number":987,"context_line":"volume_group \u003d cfg.OptGroup(name\u003d\u0027volume\u0027,"},{"line_number":988,"context_line":"                            title\u003d\u0027Block Storage Options\u0027)"}],"source_content_type":"text/x-python","patch_set":9,"id":"c4216c02_4693e827","line":985,"range":{"start_line":981,"start_character":3,"end_line":985,"end_character":0},"updated":"2023-12-05 20:24:42.000000000","message":"I do not think this new config belong to validation group. we should add it in DefaultGroup","commit_id":"46c302739efb5bcca33f10936eafa2c09b452500"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"00e8f7ce341c62d9a9d4953d3f5c601287acc3f3","unresolved":false,"context_lines":[{"line_number":978,"context_line":"                      \"migration, in seconds. \""},{"line_number":979,"context_line":"                      \"When the measured downtime exceeds this value, an \""},{"line_number":980,"context_line":"                      \"exception is raised.\"),"},{"line_number":981,"context_line":"    cfg.StrOpt(\u0027resource_name_prefix\u0027,"},{"line_number":982,"context_line":"               default\u003d\u0027tempest-test\u0027,"},{"line_number":983,"context_line":"               help\u003d\u0027Define the prefix name for the resources created \u0027"},{"line_number":984,"context_line":"                    \u0027by tempest.\u0027)"},{"line_number":985,"context_line":"]"},{"line_number":986,"context_line":""},{"line_number":987,"context_line":"volume_group \u003d cfg.OptGroup(name\u003d\u0027volume\u0027,"},{"line_number":988,"context_line":"                            title\u003d\u0027Block Storage Options\u0027)"}],"source_content_type":"text/x-python","patch_set":9,"id":"2adbbb97_467a595c","line":985,"range":{"start_line":981,"start_character":3,"end_line":985,"end_character":0},"in_reply_to":"c4216c02_4693e827","updated":"2023-12-06 20:25:57.000000000","message":"Done","commit_id":"46c302739efb5bcca33f10936eafa2c09b452500"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"201939ced296890e79752d5d06573b76407a114c","unresolved":true,"context_lines":[{"line_number":1344,"context_line":"                    \"value keeps profiling disabled\"),"},{"line_number":1345,"context_line":"]"},{"line_number":1346,"context_line":""},{"line_number":1347,"context_line":"DefaultGroup \u003d ["},{"line_number":1348,"context_line":"    cfg.BoolOpt(\u0027pause_teardown\u0027,"},{"line_number":1349,"context_line":"                default\u003dFalse,"},{"line_number":1350,"context_line":"                help\u003d\"\"\"Whether to pause a test in global teardown."}],"source_content_type":"text/x-python","patch_set":9,"id":"15c2f2d9_8e614374","line":1347,"range":{"start_line":1347,"start_character":0,"end_line":1347,"end_character":13},"updated":"2023-12-05 20:24:42.000000000","message":"here","commit_id":"46c302739efb5bcca33f10936eafa2c09b452500"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"00e8f7ce341c62d9a9d4953d3f5c601287acc3f3","unresolved":false,"context_lines":[{"line_number":1344,"context_line":"                    \"value keeps profiling disabled\"),"},{"line_number":1345,"context_line":"]"},{"line_number":1346,"context_line":""},{"line_number":1347,"context_line":"DefaultGroup \u003d ["},{"line_number":1348,"context_line":"    cfg.BoolOpt(\u0027pause_teardown\u0027,"},{"line_number":1349,"context_line":"                default\u003dFalse,"},{"line_number":1350,"context_line":"                help\u003d\"\"\"Whether to pause a test in global teardown."}],"source_content_type":"text/x-python","patch_set":9,"id":"51cb215c_6deee1be","line":1347,"range":{"start_line":1347,"start_character":0,"end_line":1347,"end_character":13},"in_reply_to":"15c2f2d9_8e614374","updated":"2023-12-06 20:25:57.000000000","message":"Done","commit_id":"46c302739efb5bcca33f10936eafa2c09b452500"}],"tempest/lib/common/utils/data_utils.py":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"41885660fba3e04cc57ca318ded5279c84db0dfa","unresolved":true,"context_lines":[{"line_number":19,"context_line":"import uuid"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"from oslo_utils import uuidutils"},{"line_number":22,"context_line":"from tempest import config"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"CONF \u003d config.CONF"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"def rand_uuid():"},{"line_number":27,"context_line":"    \"\"\"Generate a random UUID string"}],"source_content_type":"text/x-python","patch_set":2,"id":"916bc983_580f9c9f","line":24,"range":{"start_line":22,"start_character":0,"end_line":24,"end_character":18},"updated":"2023-10-31 13:27:29.000000000","message":"unfortunately we can\u0027t import config in tempest/lib:\n\npep8 complains:\n./tempest/lib/common/utils/data_utils.py:22:1: T112: tempest.lib should not import local tempest code to avoid circular dependency\nfrom tempest import config\n^\n./tempest/lib/common/utils/data_utils.py:22:1: T114: tempest.lib can not have any dependency on tempest config.\nfrom tempest import config\n^\n\ndue to custom rules we set for tempest - https://opendev.org/openstack/tempest/src/branch/master/tempest/hacking/checks.py","commit_id":"541ae8780cabc0ba9b46233d4028f964c40f0abc"},{"author":{"_account_id":33983,"name":"Luigi Dino Tamagnone","display_name":"luigi","email":"ltamagno@redhat.com","username":"ltamagno"},"change_message_id":"3557a32f43ac01da1634c81018441e2a7faff4cb","unresolved":false,"context_lines":[{"line_number":19,"context_line":"import uuid"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"from oslo_utils import uuidutils"},{"line_number":22,"context_line":"from tempest import config"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"CONF \u003d config.CONF"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"def rand_uuid():"},{"line_number":27,"context_line":"    \"\"\"Generate a random UUID string"}],"source_content_type":"text/x-python","patch_set":2,"id":"a70d1bc5_c03f87fe","line":24,"range":{"start_line":22,"start_character":0,"end_line":24,"end_character":18},"in_reply_to":"916bc983_580f9c9f","updated":"2023-11-07 15:59:47.000000000","message":"Done","commit_id":"541ae8780cabc0ba9b46233d4028f964c40f0abc"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"41885660fba3e04cc57ca318ded5279c84db0dfa","unresolved":true,"context_lines":[{"line_number":41,"context_line":"    return uuid.uuid4().hex"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"def rand_name(name\u003d\u0027\u0027, prefix\u003dCONF.validation.resource_name_prefix):"},{"line_number":45,"context_line":"    \"\"\"Generate a random name that includes a random number"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    :param str name: The name that you want to include"}],"source_content_type":"text/x-python","patch_set":2,"id":"a8635ef3_14308e20","line":44,"range":{"start_line":44,"start_character":31,"end_line":44,"end_character":66},"updated":"2023-10-31 13:27:29.000000000","message":"the only way is have it set to not using resource_name_prefix conf variable, just set it to \u0027tempest-test\u0027","commit_id":"541ae8780cabc0ba9b46233d4028f964c40f0abc"},{"author":{"_account_id":33983,"name":"Luigi Dino Tamagnone","display_name":"luigi","email":"ltamagno@redhat.com","username":"ltamagno"},"change_message_id":"3557a32f43ac01da1634c81018441e2a7faff4cb","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    return uuid.uuid4().hex"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"def rand_name(name\u003d\u0027\u0027, prefix\u003dCONF.validation.resource_name_prefix):"},{"line_number":45,"context_line":"    \"\"\"Generate a random name that includes a random number"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    :param str name: The name that you want to include"}],"source_content_type":"text/x-python","patch_set":2,"id":"b18109a7_68163cf5","line":44,"range":{"start_line":44,"start_character":31,"end_line":44,"end_character":66},"in_reply_to":"a8635ef3_14308e20","updated":"2023-11-07 15:59:47.000000000","message":"Done","commit_id":"541ae8780cabc0ba9b46233d4028f964c40f0abc"}],"tempest/tests/lib/common/utils/test_data_utils.py":[{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"70afaacc1bf813eda61acdac78f1c6c2e236e811","unresolved":true,"context_lines":[{"line_number":39,"context_line":"    def test_rand_name_with_default_prefix(self):"},{"line_number":40,"context_line":"        actual \u003d data_utils.rand_name(\u0027foo\u0027)"},{"line_number":41,"context_line":"        self.assertIsInstance(actual, str)"},{"line_number":42,"context_line":"        self.assertTrue(actual.startswith(CONF.validation.resource_name_prefix+\u0027foo\u0027))"},{"line_number":43,"context_line":"        actual2 \u003d data_utils.rand_name(\u0027foo\u0027)"},{"line_number":44,"context_line":"        self.assertTrue(actual2.startswith(CONF.validation.resource_name_prefix+\u0027foo\u0027))"},{"line_number":45,"context_line":"        self.assertNotEqual(actual, actual2)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bd9bcc4e_4d99dd92","line":42,"range":{"start_line":42,"start_character":80,"end_line":42,"end_character":83},"updated":"2023-10-16 09:02:02.000000000","message":"Shouldn\u0027t it be \u0027-foo\u0027?","commit_id":"77942c5405a82f0d0418f1252f56163221101fae"},{"author":{"_account_id":33983,"name":"Luigi Dino Tamagnone","display_name":"luigi","email":"ltamagno@redhat.com","username":"ltamagno"},"change_message_id":"2a0a995450cefb8adb0db12b268abaca8f414314","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    def test_rand_name_with_default_prefix(self):"},{"line_number":40,"context_line":"        actual \u003d data_utils.rand_name(\u0027foo\u0027)"},{"line_number":41,"context_line":"        self.assertIsInstance(actual, str)"},{"line_number":42,"context_line":"        self.assertTrue(actual.startswith(CONF.validation.resource_name_prefix+\u0027foo\u0027))"},{"line_number":43,"context_line":"        actual2 \u003d data_utils.rand_name(\u0027foo\u0027)"},{"line_number":44,"context_line":"        self.assertTrue(actual2.startswith(CONF.validation.resource_name_prefix+\u0027foo\u0027))"},{"line_number":45,"context_line":"        self.assertNotEqual(actual, actual2)"}],"source_content_type":"text/x-python","patch_set":1,"id":"4ff145c9_cd37c7a1","line":42,"range":{"start_line":42,"start_character":80,"end_line":42,"end_character":83},"in_reply_to":"bd9bcc4e_4d99dd92","updated":"2023-10-30 14:05:29.000000000","message":"yes. I already modified it locally but didn\u0027t update it. I\u0027ll look to the other error today.","commit_id":"77942c5405a82f0d0418f1252f56163221101fae"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"70afaacc1bf813eda61acdac78f1c6c2e236e811","unresolved":true,"context_lines":[{"line_number":41,"context_line":"        self.assertIsInstance(actual, str)"},{"line_number":42,"context_line":"        self.assertTrue(actual.startswith(CONF.validation.resource_name_prefix+\u0027foo\u0027))"},{"line_number":43,"context_line":"        actual2 \u003d data_utils.rand_name(\u0027foo\u0027)"},{"line_number":44,"context_line":"        self.assertTrue(actual2.startswith(CONF.validation.resource_name_prefix+\u0027foo\u0027))"},{"line_number":45,"context_line":"        self.assertNotEqual(actual, actual2)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def test_rand_name_with_none_prefix(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"a0f4e9f8_43c63759","line":44,"range":{"start_line":44,"start_character":81,"end_line":44,"end_character":85},"updated":"2023-10-16 09:02:02.000000000","message":"ditto","commit_id":"77942c5405a82f0d0418f1252f56163221101fae"},{"author":{"_account_id":33983,"name":"Luigi Dino Tamagnone","display_name":"luigi","email":"ltamagno@redhat.com","username":"ltamagno"},"change_message_id":"2a0a995450cefb8adb0db12b268abaca8f414314","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        self.assertIsInstance(actual, str)"},{"line_number":42,"context_line":"        self.assertTrue(actual.startswith(CONF.validation.resource_name_prefix+\u0027foo\u0027))"},{"line_number":43,"context_line":"        actual2 \u003d data_utils.rand_name(\u0027foo\u0027)"},{"line_number":44,"context_line":"        self.assertTrue(actual2.startswith(CONF.validation.resource_name_prefix+\u0027foo\u0027))"},{"line_number":45,"context_line":"        self.assertNotEqual(actual, actual2)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def test_rand_name_with_none_prefix(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7c40ab0e_50d4b050","line":44,"range":{"start_line":44,"start_character":81,"end_line":44,"end_character":85},"in_reply_to":"a0f4e9f8_43c63759","updated":"2023-10-30 14:05:29.000000000","message":"yes. I already modified it locally but didn\u0027t update it. I\u0027ll look to the other error today.","commit_id":"77942c5405a82f0d0418f1252f56163221101fae"}]}