)]}'
{".zuul.yaml":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"59df0106256448e703757aa99576852aeda38fff","unresolved":true,"context_lines":[{"line_number":691,"context_line":"    vars:"},{"line_number":692,"context_line":"      # revert this when bug #1940425 is fixed in neutron"},{"line_number":693,"context_line":"      # FIXME(sean-k-mooney) skip the test_image_formats for now"},{"line_number":694,"context_line":"      # as we have not configured this job correctly to run them"},{"line_number":695,"context_line":"      tempest_exclude_regex: \u003e"},{"line_number":696,"context_line":"        test_live_migration_with_trunk |"},{"line_number":697,"context_line":"        tempest.api.image.v2.test_images_formats"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"de2b61a8_1d9244ed","line":694,"updated":"2024-08-13 14:26:29.000000000","message":"It\u0027s not so much that it\u0027s not configured correctly as the basic job layout won\u0027t pass these tests since nova never gets to inspect the image. But, I think we probably need a tempest conf knob to gate these, so if I add that, this comment will become correct...right before we remove it :)","commit_id":"bb67d3a0a46614dbf3d93750079994e8d20687e5"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"76fc6781f7b957be15ea255237d78cdae4297829","unresolved":true,"context_lines":[{"line_number":691,"context_line":"    vars:"},{"line_number":692,"context_line":"      # revert this when bug #1940425 is fixed in neutron"},{"line_number":693,"context_line":"      # FIXME(sean-k-mooney) skip the test_image_formats for now"},{"line_number":694,"context_line":"      # as we have not configured this job correctly to run them"},{"line_number":695,"context_line":"      tempest_exclude_regex: \u003e"},{"line_number":696,"context_line":"        test_live_migration_with_trunk |"},{"line_number":697,"context_line":"        tempest.api.image.v2.test_images_formats"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ddb85c68_fcae79c9","line":694,"in_reply_to":"de2b61a8_1d9244ed","updated":"2024-08-13 14:30:01.000000000","message":"ack so if i undersand correctly the current test could work in the multi store job but only if we stored the image in the file store correct rather then the ceph one.\n\nright now they are being stroed in whatever the default store is which is why we get the shallow copy behavior and not what we expect.\n\nbut ya for now i want ot skip these to unblock the gate until we have time to decied the best path forward. be that a tempest config option or some other approch.","commit_id":"bb67d3a0a46614dbf3d93750079994e8d20687e5"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c8b406829934001ed2090b4b3c5c57af59d4b541","unresolved":true,"context_lines":[{"line_number":694,"context_line":"      # as we have not configured this job correctly to run them"},{"line_number":695,"context_line":"      tempest_exclude_regex: \u003e"},{"line_number":696,"context_line":"        test_live_migration_with_trunk |"},{"line_number":697,"context_line":"        tempest.api.image.v2.test_images_formats"},{"line_number":698,"context_line":"      # NOTE(danms): Increase our swap size since we\u0027re dealing with"},{"line_number":699,"context_line":"      # larger images and trigger OOMs."},{"line_number":700,"context_line":"      configure_swap_size: 8192"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f93b7f72_d0e98da8","line":697,"updated":"2024-08-13 18:30:49.000000000","message":"this rendered effectivly as i expected but i wonder if we need to wrap this in ()\n\nhttps://zuul.opendev.org/t/openstack/build/caa401e0a9a24b20a97767e4559d5de6/log/job-output.txt#31293\n\n```\n2024-08-13 16:11:13.167034 | controller | \u0027 --exclude-list\u003d/home/zuul/src/opendev.org/openstack/devstack-plugin-ceph/tempest_skiplist.txt --concurrency\u003d3 \u0027--exclude-regex\u003dtest_live_migration_with_trunk | tempest.api.image.v2.test_images_formats\n```","commit_id":"bb67d3a0a46614dbf3d93750079994e8d20687e5"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"fd40132dd15b188989eb1e6b74de16226fe521ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"70ffa884_dac812f5","updated":"2024-08-13 18:32:39.000000000","message":"If there\u0027s a question, you can also unset the `[images]/images_manifest_file` config in tempest.conf and effectively stop these tests as well.","commit_id":"bb67d3a0a46614dbf3d93750079994e8d20687e5"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"f1cb41da64de77f1644b9b9e07339714d536760d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d1755e29_041ff6db","updated":"2024-08-13 18:35:27.000000000","message":"Re-approving since it\u0027s just a format change","commit_id":"c69a9eb87f1823b2a06d5bdd48e97f4a73c9adf7"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"7de491be1ea734c30be7a6fe01da2b661a71e9d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2d1b58da_25a854ec","updated":"2024-08-14 13:29:52.000000000","message":"recheck failed an unrelated volume test","commit_id":"c69a9eb87f1823b2a06d5bdd48e97f4a73c9adf7"}]}