)]}'
{".zuul.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5a68b7097d962f66a6dde98572e5b9b19a50dd24","unresolved":true,"context_lines":[{"line_number":603,"context_line":"        GLANCE_STANDALONE: True"},{"line_number":604,"context_line":"        GLANCE_USE_IMPORT_WORKFLOW: True"},{"line_number":605,"context_line":"        DEVSTACK_PARALLEL: True"},{"line_number":606,"context_line":"        IMG_IMAGES_MIN_SIZE: 1073741824"},{"line_number":607,"context_line":"        GLANCE_LIMIT_IMAGE_SIZE_TOTAL: 2048"},{"line_number":608,"context_line":"      # NOTE(danms): This job is pretty heavy as it is, so we disable some"},{"line_number":609,"context_line":"      # services that are not relevant to the nova-glance-ceph scenario"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"3e9f1059_46319b41","line":606,"range":{"start_line":606,"start_character":29,"end_line":606,"end_character":39},"updated":"2022-11-03 14:55:14.000000000","message":"ok this is 1G that is proably ok.\n\ni tought it was 10 for a sec which likely would break things but 1G should be ok.\n\nwe have 80GB total avaiable in the ci but we cant rely on all of it bing in the same partion.\n\nusing an addtional 1G image shoudl fit within the limits we have.","commit_id":"c1e1e5842bed5f44525a4e53f61e57109f3fdee9"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"4cf29e79a55e3f577233c796d3b77297684f3368","unresolved":true,"context_lines":[{"line_number":603,"context_line":"        GLANCE_STANDALONE: True"},{"line_number":604,"context_line":"        GLANCE_USE_IMPORT_WORKFLOW: True"},{"line_number":605,"context_line":"        DEVSTACK_PARALLEL: True"},{"line_number":606,"context_line":"        IMG_IMAGES_MIN_SIZE: 1073741824"},{"line_number":607,"context_line":"        GLANCE_LIMIT_IMAGE_SIZE_TOTAL: 2048"},{"line_number":608,"context_line":"      # NOTE(danms): This job is pretty heavy as it is, so we disable some"},{"line_number":609,"context_line":"      # services that are not relevant to the nova-glance-ceph scenario"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"8f6db870_87f273e8","line":606,"range":{"start_line":606,"start_character":29,"end_line":606,"end_character":39},"in_reply_to":"3e9f1059_46319b41","updated":"2022-11-03 17:25:09.000000000","message":"Yeah, I wanted 2G originally actually, but it ends up being too large on a single job with multiple images and volumes. I could have expanded other limits as well, but I figure 1G is much better than 0.016G in terms of reality :)","commit_id":"c1e1e5842bed5f44525a4e53f61e57109f3fdee9"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"f29eac76cf4674dee79b4aab9defe6f10823e87e","unresolved":false,"context_lines":[{"line_number":603,"context_line":"        GLANCE_STANDALONE: True"},{"line_number":604,"context_line":"        GLANCE_USE_IMPORT_WORKFLOW: True"},{"line_number":605,"context_line":"        DEVSTACK_PARALLEL: True"},{"line_number":606,"context_line":"        IMG_IMAGES_MIN_SIZE: 1073741824"},{"line_number":607,"context_line":"        GLANCE_LIMIT_IMAGE_SIZE_TOTAL: 2048"},{"line_number":608,"context_line":"      # NOTE(danms): This job is pretty heavy as it is, so we disable some"},{"line_number":609,"context_line":"      # services that are not relevant to the nova-glance-ceph scenario"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"b7a51b1e_e0583fee","line":606,"range":{"start_line":606,"start_character":29,"end_line":606,"end_character":39},"in_reply_to":"8f6db870_87f273e8","updated":"2022-11-11 16:06:58.000000000","message":"Done","commit_id":"c1e1e5842bed5f44525a4e53f61e57109f3fdee9"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f8916c26cd74f30fc503f2714acc94bd7c467a37","unresolved":true,"context_lines":[{"line_number":598,"context_line":"    vars:"},{"line_number":599,"context_line":"      # NOTE(danms): Increase our swap size since we\u0027re dealing with"},{"line_number":600,"context_line":"      # larger images and trigger OOMs."},{"line_number":601,"context_line":"      configure_swap_size: 4096"},{"line_number":602,"context_line":"      # NOTE(danms): These tests create an empty non-raw image, which nova"},{"line_number":603,"context_line":"      # will refuse because we set never_download_image_if_on_rbd in this job."},{"line_number":604,"context_line":"      # Just skip these tests for this case."}],"source_content_type":"text/x-yaml","patch_set":25,"id":"f9b4a917_5a87303e","line":601,"updated":"2022-11-11 16:18:08.000000000","message":"i think we set this to 8192 in some of the jobs already so ya im fine with rasing it\n\ni think the default is 1024?","commit_id":"010983e80374ef6538c965d3db3247cda1bc6edc"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"4f0166d545e418bad7108988b22c12af2e55f807","unresolved":true,"context_lines":[{"line_number":598,"context_line":"    vars:"},{"line_number":599,"context_line":"      # NOTE(danms): Increase our swap size since we\u0027re dealing with"},{"line_number":600,"context_line":"      # larger images and trigger OOMs."},{"line_number":601,"context_line":"      configure_swap_size: 4096"},{"line_number":602,"context_line":"      # NOTE(danms): These tests create an empty non-raw image, which nova"},{"line_number":603,"context_line":"      # will refuse because we set never_download_image_if_on_rbd in this job."},{"line_number":604,"context_line":"      # Just skip these tests for this case."}],"source_content_type":"text/x-yaml","patch_set":25,"id":"5aa7a729_2756a8d7","line":601,"in_reply_to":"f9b4a917_5a87303e","updated":"2022-11-11 16:27:01.000000000","message":"Yup.","commit_id":"010983e80374ef6538c965d3db3247cda1bc6edc"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"efe3c74ecf2b32247078a12ec12ebce5fc2c322f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f56af8b0_fd7446ad","updated":"2022-10-10 22:54:29.000000000","message":"recheck dep changed","commit_id":"cb9046dfb7faa950cdaff87276c0b74776b07c68"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"ba97ea8d79ae066a8b61a651b14fee9f920fe253","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1e7e7e80_b0ef97f7","updated":"2022-10-10 22:27:29.000000000","message":"recheck glance fixed","commit_id":"cb9046dfb7faa950cdaff87276c0b74776b07c68"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"644de713d908c4c5622c4fd229dc63340f789682","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2d9932d0_ffbbe7c2","updated":"2022-10-12 14:32:38.000000000","message":"recheck several unrelated POST_FAILUREs","commit_id":"a4a016d1edc49eacd074111e4784f7c7f65eddd7"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"432c44529d0729aacbf7d172c7c9adc1e62ca782","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7a9225a2_34e85f7c","updated":"2022-10-12 17:29:02.000000000","message":"recheck dep changed","commit_id":"3bc3aae84787f5a6fb5ae5c0361e0b4cda9122ce"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"2296e4d3616f2de524027959fa720c877fec80dd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a81b78f7_6ad8b903","updated":"2022-10-12 20:25:41.000000000","message":"recheck dep changed","commit_id":"057ffe53a9f1b1aec49d28fe0e290a1c55cd430b"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"4cf29e79a55e3f577233c796d3b77297684f3368","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"28c33553_6044a1bd","updated":"2022-11-03 17:25:09.000000000","message":"Thanks!","commit_id":"c1e1e5842bed5f44525a4e53f61e57109f3fdee9"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"a7f0fef7cf4c9b21a95d01d9344e85df5b8aafd5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"b0d74ab9_afd7ef88","updated":"2022-10-28 16:44:08.000000000","message":"recheck devstack changed","commit_id":"c1e1e5842bed5f44525a4e53f61e57109f3fdee9"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"b501267c621a7225857e74932ad78e0bf35d60b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"ae7637cc_3ea9169e","updated":"2022-11-07 15:30:28.000000000","message":"Taking a different approach with this","commit_id":"8b8a1398ec33c5c97c486837f81ff4cc063b453b"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"202936969287408b86ab01e5a55055809508fd9e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"23f6162f_e239f892","updated":"2022-11-03 20:10:41.000000000","message":"recheck nova-next unrelated test failure","commit_id":"8b8a1398ec33c5c97c486837f81ff4cc063b453b"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"52f60042fb12dfbb8953993b49a36b4c76d89cb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"f2d50928_e7ced62d","updated":"2022-11-09 18:14:48.000000000","message":"The ceph-multistore config is now doing the right thing. One test failed because it hung forever, for no reason I can determine. There was also another unrelated failure, so I\u0027m going to recheck for another data point.","commit_id":"6ddfe1bea462e2dbf380db398b4a1feab37d6486"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"7239897d539c574f0266d3c9bc37c0a6f32eb7c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"2c847f5d_7a69f27a","updated":"2022-11-09 18:15:00.000000000","message":"recheck another data point for debugging","commit_id":"6ddfe1bea462e2dbf380db398b4a1feab37d6486"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"4f0166d545e418bad7108988b22c12af2e55f807","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"36d4eaaa_e0bd56c3","updated":"2022-11-11 16:27:01.000000000","message":"Thanks Sean!","commit_id":"010983e80374ef6538c965d3db3247cda1bc6edc"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"49a183f4842e03461dff4487b517f3af9660fa68","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"6948e2f2_6fbcd9fb","updated":"2022-11-09 21:04:51.000000000","message":"The fix was to increase swap as it looks like something was using more memory with the larger image and we were tripping over an OOM. (thanks to rosmaita for that). I\u0027m going to recheck again to make sure it wasn\u0027t a fluke.","commit_id":"010983e80374ef6538c965d3db3247cda1bc6edc"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f8916c26cd74f30fc503f2714acc94bd7c467a37","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"67c98e5f_8dc3564f","updated":"2022-11-11 16:18:08.000000000","message":"im ok with this change.\n\ni  had considerd if this is something that should be done in devstack but i dont think we need to always expect the cirror image for other jobs so doing it in nova for the ceph-multistore job is preoably a better fit so +2","commit_id":"010983e80374ef6538c965d3db3247cda1bc6edc"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"b43842ee1239aee680b8c5e5d186da82a2c3c7ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"d943a337_a44f9d11","updated":"2022-11-11 17:30:02.000000000","message":"lgtm, agree to do it in nova side if that is the only job testing it. later we can move to devstack if needed for multiple job or extending testing with real image.","commit_id":"010983e80374ef6538c965d3db3247cda1bc6edc"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"3456ac2cf0843d9fa4ec8cbba8620886303d17cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"4c1dd5cf_76fdceff","updated":"2022-11-09 21:04:59.000000000","message":"recheck another data point","commit_id":"010983e80374ef6538c965d3db3247cda1bc6edc"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"4f0166d545e418bad7108988b22c12af2e55f807","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"cf2f3409_65adad5a","in_reply_to":"67c98e5f_8dc3564f","updated":"2022-11-11 16:27:01.000000000","message":"Yeah, I had a devstack change before, but it was an extra step, a very weird thing to expose for just this one job, and was introducing extra discussion and work to get landed. So, easier to just do it ourselves here.","commit_id":"010983e80374ef6538c965d3db3247cda1bc6edc"}],"playbooks/ceph/glance-copy-policy.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5a68b7097d962f66a6dde98572e5b9b19a50dd24","unresolved":true,"context_lines":[{"line_number":15,"context_line":"              if openstack image list -f value -c Status | grep active; then"},{"line_number":16,"context_line":"                  break"},{"line_number":17,"context_line":"              else"},{"line_number":18,"context_line":"                  echo Waiting for all images to become active:"},{"line_number":19,"context_line":"                  openstack image list"},{"line_number":20,"context_line":"                  sleep 1"},{"line_number":21,"context_line":"              fi"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"91377040_6eb886b8","line":18,"range":{"start_line":18,"start_character":35,"end_line":18,"end_character":38},"updated":"2022-11-03 14:55:14.000000000","message":"nit: any","commit_id":"c1e1e5842bed5f44525a4e53f61e57109f3fdee9"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"4cf29e79a55e3f577233c796d3b77297684f3368","unresolved":true,"context_lines":[{"line_number":15,"context_line":"              if openstack image list -f value -c Status | grep active; then"},{"line_number":16,"context_line":"                  break"},{"line_number":17,"context_line":"              else"},{"line_number":18,"context_line":"                  echo Waiting for all images to become active:"},{"line_number":19,"context_line":"                  openstack image list"},{"line_number":20,"context_line":"                  sleep 1"},{"line_number":21,"context_line":"              fi"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"a58d152d_3b26b8e6","line":18,"range":{"start_line":18,"start_character":35,"end_line":18,"end_character":38},"in_reply_to":"91377040_6eb886b8","updated":"2022-11-03 17:25:09.000000000","message":"It\u0027s kinda both because we really only have one image. But yeah, I say any in the comment and in the logic, so I\u0027ll fix it :)","commit_id":"c1e1e5842bed5f44525a4e53f61e57109f3fdee9"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"f8fb7fe56035f2f446641333a2e288c5adfd900d","unresolved":false,"context_lines":[{"line_number":15,"context_line":"              if openstack image list -f value -c Status | grep active; then"},{"line_number":16,"context_line":"                  break"},{"line_number":17,"context_line":"              else"},{"line_number":18,"context_line":"                  echo Waiting for all images to become active:"},{"line_number":19,"context_line":"                  openstack image list"},{"line_number":20,"context_line":"                  sleep 1"},{"line_number":21,"context_line":"              fi"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"ca420ef0_799fa446","line":18,"range":{"start_line":18,"start_character":35,"end_line":18,"end_character":38},"in_reply_to":"a58d152d_3b26b8e6","updated":"2022-11-07 15:30:14.000000000","message":"Done","commit_id":"c1e1e5842bed5f44525a4e53f61e57109f3fdee9"}],"playbooks/ceph/glance-setup.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f8916c26cd74f30fc503f2714acc94bd7c467a37","unresolved":true,"context_lines":[{"line_number":7,"context_line":"        create: True"},{"line_number":8,"context_line":"        mode: 0777"},{"line_number":9,"context_line":"        block: |"},{"line_number":10,"context_line":"          # Delete all existing images"},{"line_number":11,"context_line":"          source /opt/stack/devstack/openrc admin"},{"line_number":12,"context_line":"          for img in $(openstack image list -f value -c ID); do"},{"line_number":13,"context_line":"              openstack image show $img"}],"source_content_type":"text/x-yaml","patch_set":25,"id":"2c340237_40477070","line":10,"updated":"2022-11-11 16:18:08.000000000","message":"so when i first saw this i was wondering if we coudl update the images in place\nwithout deleateing them but then i reaslied we might also be changing the format \nand im unsure if we can chagne the iamge size when we relace the data so for those reasons i agree deleteing them and recreating them makes sine in the context of this job.","commit_id":"010983e80374ef6538c965d3db3247cda1bc6edc"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"4f0166d545e418bad7108988b22c12af2e55f807","unresolved":true,"context_lines":[{"line_number":7,"context_line":"        create: True"},{"line_number":8,"context_line":"        mode: 0777"},{"line_number":9,"context_line":"        block: |"},{"line_number":10,"context_line":"          # Delete all existing images"},{"line_number":11,"context_line":"          source /opt/stack/devstack/openrc admin"},{"line_number":12,"context_line":"          for img in $(openstack image list -f value -c ID); do"},{"line_number":13,"context_line":"              openstack image show $img"}],"source_content_type":"text/x-yaml","patch_set":25,"id":"bac3f56a_94474f78","line":10,"in_reply_to":"2c340237_40477070","updated":"2022-11-11 16:27:01.000000000","message":"Yeah, we can\u0027t convert them in place. We have to delete and re-upload. It\u0027s much easier to do this than to have multiple images and configure tempest to use one. Otherwise it just picks the first.","commit_id":"010983e80374ef6538c965d3db3247cda1bc6edc"}]}