)]}'
{".zuul.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a3a5f0533a4d8ee99ca9d66695e11b0131d457e8","unresolved":true,"context_lines":[{"line_number":487,"context_line":"      devstack_localrc:"},{"line_number":488,"context_line":"        # For nested virt"},{"line_number":489,"context_line":"        LIBVIRT_TYPE: kvm"},{"line_number":490,"context_line":"        LIBVIRT_CPU_MODE: host-passthrough"},{"line_number":491,"context_line":"        DEFAULT_IMAGE_NAME: cirros-0.6.2-x86_64-disk"},{"line_number":492,"context_line":"        DEFAULT_IMAGE_FILE_NAME: cirros-0.6.2-x86_64-disk.img"},{"line_number":493,"context_line":"        Q_AGENT: openvswitch"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"cb19971c_2af55048","line":490,"updated":"2023-11-30 11:29:42.000000000","message":"we do not want to use host-passthough since we cant assume the host cpus will be the same so we should leave this at nehalem which is what devstack is setting.","commit_id":"ceb1874f6a3a8eef081d766fe7004a5b67d3722c"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"a7162e29acc5a602e2be84565e418820caa2c80f","unresolved":false,"context_lines":[{"line_number":487,"context_line":"      devstack_localrc:"},{"line_number":488,"context_line":"        # For nested virt"},{"line_number":489,"context_line":"        LIBVIRT_TYPE: kvm"},{"line_number":490,"context_line":"        LIBVIRT_CPU_MODE: host-passthrough"},{"line_number":491,"context_line":"        DEFAULT_IMAGE_NAME: cirros-0.6.2-x86_64-disk"},{"line_number":492,"context_line":"        DEFAULT_IMAGE_FILE_NAME: cirros-0.6.2-x86_64-disk.img"},{"line_number":493,"context_line":"        Q_AGENT: openvswitch"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"28a825d0_638ad13f","line":490,"in_reply_to":"cb19971c_2af55048","updated":"2023-11-30 18:36:34.000000000","message":"Done","commit_id":"ceb1874f6a3a8eef081d766fe7004a5b67d3722c"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a3a5f0533a4d8ee99ca9d66695e11b0131d457e8","unresolved":true,"context_lines":[{"line_number":532,"context_line":"    group-vars:"},{"line_number":533,"context_line":"      subnode:"},{"line_number":534,"context_line":"        devstack_localrc:"},{"line_number":535,"context_line":"          Q_AGENT: openvswitch"},{"line_number":536,"context_line":"          Q_ML2_TENANT_NETWORK_TYPE: vxlan"},{"line_number":537,"context_line":"          Q_ML2_PLUGIN_MECHANISM_DRIVERS: openvswitch"},{"line_number":538,"context_line":"          ML2_L3_PLUGIN: router"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b66a5c63_0cd61bd7","line":535,"updated":"2023-11-30 11:29:42.000000000","message":"the libvirt type should be set here too","commit_id":"ceb1874f6a3a8eef081d766fe7004a5b67d3722c"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"a7162e29acc5a602e2be84565e418820caa2c80f","unresolved":false,"context_lines":[{"line_number":532,"context_line":"    group-vars:"},{"line_number":533,"context_line":"      subnode:"},{"line_number":534,"context_line":"        devstack_localrc:"},{"line_number":535,"context_line":"          Q_AGENT: openvswitch"},{"line_number":536,"context_line":"          Q_ML2_TENANT_NETWORK_TYPE: vxlan"},{"line_number":537,"context_line":"          Q_ML2_PLUGIN_MECHANISM_DRIVERS: openvswitch"},{"line_number":538,"context_line":"          ML2_L3_PLUGIN: router"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"92994e39_968aec34","line":535,"in_reply_to":"b66a5c63_0cd61bd7","updated":"2023-11-30 18:36:34.000000000","message":"Done","commit_id":"ceb1874f6a3a8eef081d766fe7004a5b67d3722c"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a3a5f0533a4d8ee99ca9d66695e11b0131d457e8","unresolved":true,"context_lines":[{"line_number":808,"context_line":"        # the nova-next job."},{"line_number":809,"context_line":"        CIRROS_VERSION: 0.6.2"},{"line_number":810,"context_line":"        DEFAULT_IMAGE_NAME: cirros-0.6.2-x86_64-uec"},{"line_number":811,"context_line":"        DEFAULT_IMAGE_FILE_NAME: cirros-0.6.2-x86_64-uec.tar.gz"},{"line_number":812,"context_line":"    check:"},{"line_number":813,"context_line":"      jobs:"},{"line_number":814,"context_line":"        # We define our own irrelevant-files so we don\u0027t run the job"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"901265f4_f4628a89","line":811,"updated":"2023-11-30 11:29:42.000000000","message":"does this work i have never seen usign vars here but thats a nice feature","commit_id":"ceb1874f6a3a8eef081d766fe7004a5b67d3722c"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"30606440c9f2d18b6e7a858d6de2bef2332d3647","unresolved":false,"context_lines":[{"line_number":808,"context_line":"        # the nova-next job."},{"line_number":809,"context_line":"        CIRROS_VERSION: 0.6.2"},{"line_number":810,"context_line":"        DEFAULT_IMAGE_NAME: cirros-0.6.2-x86_64-uec"},{"line_number":811,"context_line":"        DEFAULT_IMAGE_FILE_NAME: cirros-0.6.2-x86_64-uec.tar.gz"},{"line_number":812,"context_line":"    check:"},{"line_number":813,"context_line":"      jobs:"},{"line_number":814,"context_line":"        # We define our own irrelevant-files so we don\u0027t run the job"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"6733a752_38a12712","line":811,"in_reply_to":"7fecae41_21d6fb98","updated":"2023-12-05 08:16:30.000000000","message":"Acknowledged","commit_id":"ceb1874f6a3a8eef081d766fe7004a5b67d3722c"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"a7162e29acc5a602e2be84565e418820caa2c80f","unresolved":true,"context_lines":[{"line_number":808,"context_line":"        # the nova-next job."},{"line_number":809,"context_line":"        CIRROS_VERSION: 0.6.2"},{"line_number":810,"context_line":"        DEFAULT_IMAGE_NAME: cirros-0.6.2-x86_64-uec"},{"line_number":811,"context_line":"        DEFAULT_IMAGE_FILE_NAME: cirros-0.6.2-x86_64-uec.tar.gz"},{"line_number":812,"context_line":"    check:"},{"line_number":813,"context_line":"      jobs:"},{"line_number":814,"context_line":"        # We define our own irrelevant-files so we don\u0027t run the job"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"df03510a_00f42097","line":811,"in_reply_to":"901265f4_f4628a89","updated":"2023-11-30 18:36:34.000000000","message":"I think it is working, yes. I tried it based on looking at this doc https://zuul-ci.org/docs/zuul/latest/job-content.html#project-variables","commit_id":"ceb1874f6a3a8eef081d766fe7004a5b67d3722c"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"af3b2c76fcc52ebb2bf0e50f05ef81ffe7ce67bd","unresolved":true,"context_lines":[{"line_number":808,"context_line":"        # the nova-next job."},{"line_number":809,"context_line":"        CIRROS_VERSION: 0.6.2"},{"line_number":810,"context_line":"        DEFAULT_IMAGE_NAME: cirros-0.6.2-x86_64-uec"},{"line_number":811,"context_line":"        DEFAULT_IMAGE_FILE_NAME: cirros-0.6.2-x86_64-uec.tar.gz"},{"line_number":812,"context_line":"    check:"},{"line_number":813,"context_line":"      jobs:"},{"line_number":814,"context_line":"        # We define our own irrelevant-files so we don\u0027t run the job"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7fecae41_21d6fb98","line":811,"in_reply_to":"df03510a_00f42097","updated":"2023-12-01 03:17:26.000000000","message":"OK, turns out it\u0027s not working on IMAGE_URLS for any jobs that inherit from devstack-tempest because devstack-tempest sets IMAGE_URLS [1] ... and apparently project vars don\u0027t override job vars, even if they are parent jobs.\n\n[1] https://opendev.org/openstack/tempest/src/commit/8def25cbb5885e91899793f8575d3919816a0d12/zuul.d/base.yaml#L17","commit_id":"ceb1874f6a3a8eef081d766fe7004a5b67d3722c"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"30606440c9f2d18b6e7a858d6de2bef2332d3647","unresolved":true,"context_lines":[{"line_number":183,"context_line":"      tempest_test_regex: (^tempest\\..*compute\\..*(migration|resize|reboot).*)"},{"line_number":184,"context_line":"      devstack_localrc:"},{"line_number":185,"context_line":"        \u003c\u003c: *uec_image_vars"},{"line_number":186,"context_line":"        Q_AGENT: openvswitch"},{"line_number":187,"context_line":"        Q_ML2_TENANT_NETWORK_TYPE: vxlan"},{"line_number":188,"context_line":"        Q_ML2_PLUGIN_MECHANISM_DRIVERS: openvswitch"},{"line_number":189,"context_line":"        ML2_L3_PLUGIN: router"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"c2c5150b_d6d6da7e","line":186,"updated":"2023-12-05 08:16:30.000000000","message":":) i like how you when straigt for the solution with the least duplication. i normally try to keep it simple until its working but this is a nice use of yaml anchors and apendinding.","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"1987e4796b5e6d71e90f13eb4dacfb617034f68e","unresolved":false,"context_lines":[{"line_number":183,"context_line":"      tempest_test_regex: (^tempest\\..*compute\\..*(migration|resize|reboot).*)"},{"line_number":184,"context_line":"      devstack_localrc:"},{"line_number":185,"context_line":"        \u003c\u003c: *uec_image_vars"},{"line_number":186,"context_line":"        Q_AGENT: openvswitch"},{"line_number":187,"context_line":"        Q_ML2_TENANT_NETWORK_TYPE: vxlan"},{"line_number":188,"context_line":"        Q_ML2_PLUGIN_MECHANISM_DRIVERS: openvswitch"},{"line_number":189,"context_line":"        ML2_L3_PLUGIN: router"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"35029b57_d0896a86","line":186,"in_reply_to":"0022f66f_805286cf","updated":"2023-12-05 18:02:21.000000000","message":"Acknowledged","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d7919415ecf7693b3b5518bdff8f6fc678dfbf18","unresolved":false,"context_lines":[{"line_number":183,"context_line":"      tempest_test_regex: (^tempest\\..*compute\\..*(migration|resize|reboot).*)"},{"line_number":184,"context_line":"      devstack_localrc:"},{"line_number":185,"context_line":"        \u003c\u003c: *uec_image_vars"},{"line_number":186,"context_line":"        Q_AGENT: openvswitch"},{"line_number":187,"context_line":"        Q_ML2_TENANT_NETWORK_TYPE: vxlan"},{"line_number":188,"context_line":"        Q_ML2_PLUGIN_MECHANISM_DRIVERS: openvswitch"},{"line_number":189,"context_line":"        ML2_L3_PLUGIN: router"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"df0d0360_44a3bb3c","line":186,"in_reply_to":"35029b57_d0896a86","updated":"2023-12-05 20:04:15.000000000","message":"yeah, i have used it in nova api ref and worked well :)","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"fe0a4af8a259c7dc13ccb9dc59f73d8e5e3a0cb3","unresolved":true,"context_lines":[{"line_number":183,"context_line":"      tempest_test_regex: (^tempest\\..*compute\\..*(migration|resize|reboot).*)"},{"line_number":184,"context_line":"      devstack_localrc:"},{"line_number":185,"context_line":"        \u003c\u003c: *uec_image_vars"},{"line_number":186,"context_line":"        Q_AGENT: openvswitch"},{"line_number":187,"context_line":"        Q_ML2_TENANT_NETWORK_TYPE: vxlan"},{"line_number":188,"context_line":"        Q_ML2_PLUGIN_MECHANISM_DRIVERS: openvswitch"},{"line_number":189,"context_line":"        ML2_L3_PLUGIN: router"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"0022f66f_805286cf","line":186,"in_reply_to":"c2c5150b_d6d6da7e","updated":"2023-12-05 09:42:41.000000000","message":"Haha yeah ... I found some good examples with codesearch first to copy, so I felt pretty confident it would be close to working 😆","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"30606440c9f2d18b6e7a858d6de2bef2332d3647","unresolved":true,"context_lines":[{"line_number":372,"context_line":"      tempest_exclude_regex: (^tempest\\.(api\\.compute\\.servers\\.(test_attach_interfaces.AttachInterfacesTestJSON.test_create_list_show_delete_interfaces_by_network_port|test_delete_server.DeleteServersTestJSON.test_delete_server_while_in_attached_volume.*|test_list_.*|test_disk_config|test_server_rescue.*|test_server_actions\\.ServerActionsTestJSON\\.test_resize.*|test_device_tag.*))|.*\\[.*\\bslow\\b.*\\])"},{"line_number":373,"context_line":"      devstack_localrc:"},{"line_number":374,"context_line":"        \u003c\u003c: *uec_image_vars"},{"line_number":375,"context_line":"        FORCE_CONFIG_DRIVE: False"},{"line_number":376,"context_line":"        ADMIN_PASSWORD: emulation"},{"line_number":377,"context_line":"        DATABASE_PASSWORD: $ADMIN_PASSWORD"},{"line_number":378,"context_line":"        RABBIT_PASSWORD: $ADMIN_PASSWORD"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"bd912239_af99b837","line":375,"updated":"2023-12-05 08:16:30.000000000","message":"wait is this why the emulation job has ben broken? \nthis should not have wokred before…","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"1987e4796b5e6d71e90f13eb4dacfb617034f68e","unresolved":true,"context_lines":[{"line_number":372,"context_line":"      tempest_exclude_regex: (^tempest\\.(api\\.compute\\.servers\\.(test_attach_interfaces.AttachInterfacesTestJSON.test_create_list_show_delete_interfaces_by_network_port|test_delete_server.DeleteServersTestJSON.test_delete_server_while_in_attached_volume.*|test_list_.*|test_disk_config|test_server_rescue.*|test_server_actions\\.ServerActionsTestJSON\\.test_resize.*|test_device_tag.*))|.*\\[.*\\bslow\\b.*\\])"},{"line_number":373,"context_line":"      devstack_localrc:"},{"line_number":374,"context_line":"        \u003c\u003c: *uec_image_vars"},{"line_number":375,"context_line":"        FORCE_CONFIG_DRIVE: False"},{"line_number":376,"context_line":"        ADMIN_PASSWORD: emulation"},{"line_number":377,"context_line":"        DATABASE_PASSWORD: $ADMIN_PASSWORD"},{"line_number":378,"context_line":"        RABBIT_PASSWORD: $ADMIN_PASSWORD"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"35ee2b9a_2aefc9e0","line":375,"in_reply_to":"0b66640d_8175f795","updated":"2023-12-05 18:02:21.000000000","message":"ya it was definitely wrong before\n\nthe format it was in before is what you would use in the local.conf but\nthis is transformed form yaml to generate that so this was an existing bug.","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d7919415ecf7693b3b5518bdff8f6fc678dfbf18","unresolved":true,"context_lines":[{"line_number":372,"context_line":"      tempest_exclude_regex: (^tempest\\.(api\\.compute\\.servers\\.(test_attach_interfaces.AttachInterfacesTestJSON.test_create_list_show_delete_interfaces_by_network_port|test_delete_server.DeleteServersTestJSON.test_delete_server_while_in_attached_volume.*|test_list_.*|test_disk_config|test_server_rescue.*|test_server_actions\\.ServerActionsTestJSON\\.test_resize.*|test_device_tag.*))|.*\\[.*\\bslow\\b.*\\])"},{"line_number":373,"context_line":"      devstack_localrc:"},{"line_number":374,"context_line":"        \u003c\u003c: *uec_image_vars"},{"line_number":375,"context_line":"        FORCE_CONFIG_DRIVE: False"},{"line_number":376,"context_line":"        ADMIN_PASSWORD: emulation"},{"line_number":377,"context_line":"        DATABASE_PASSWORD: $ADMIN_PASSWORD"},{"line_number":378,"context_line":"        RABBIT_PASSWORD: $ADMIN_PASSWORD"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"44c966b1_c928aed2","line":375,"in_reply_to":"35ee2b9a_2aefc9e0","updated":"2023-12-05 20:04:15.000000000","message":"old format also work, devstack do parse that as dict and extract key and value.\n\nhttps://github.com/openstack/devstack/blob/2211c778db0e18702c7177f7750571cba3697509/roles/write-devstack-local-conf/library/devstack_local_conf.py#L75","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"fe0a4af8a259c7dc13ccb9dc59f73d8e5e3a0cb3","unresolved":true,"context_lines":[{"line_number":372,"context_line":"      tempest_exclude_regex: (^tempest\\.(api\\.compute\\.servers\\.(test_attach_interfaces.AttachInterfacesTestJSON.test_create_list_show_delete_interfaces_by_network_port|test_delete_server.DeleteServersTestJSON.test_delete_server_while_in_attached_volume.*|test_list_.*|test_disk_config|test_server_rescue.*|test_server_actions\\.ServerActionsTestJSON\\.test_resize.*|test_device_tag.*))|.*\\[.*\\bslow\\b.*\\])"},{"line_number":373,"context_line":"      devstack_localrc:"},{"line_number":374,"context_line":"        \u003c\u003c: *uec_image_vars"},{"line_number":375,"context_line":"        FORCE_CONFIG_DRIVE: False"},{"line_number":376,"context_line":"        ADMIN_PASSWORD: emulation"},{"line_number":377,"context_line":"        DATABASE_PASSWORD: $ADMIN_PASSWORD"},{"line_number":378,"context_line":"        RABBIT_PASSWORD: $ADMIN_PASSWORD"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"0b66640d_8175f795","line":375,"in_reply_to":"bd912239_af99b837","updated":"2023-12-05 09:42:41.000000000","message":"Honestly I dunno ... it just wouldn\u0027t let me use my anchor here with it the way it was.","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d7919415ecf7693b3b5518bdff8f6fc678dfbf18","unresolved":true,"context_lines":[{"line_number":508,"context_line":"      devstack_localrc:"},{"line_number":509,"context_line":"        # For nested virt"},{"line_number":510,"context_line":"        LIBVIRT_TYPE: kvm"},{"line_number":511,"context_line":"        DEFAULT_IMAGE_NAME: cirros-0.6.2-x86_64-disk"},{"line_number":512,"context_line":"        DEFAULT_IMAGE_FILE_NAME: cirros-0.6.2-x86_64-disk.img"},{"line_number":513,"context_line":"        Q_AGENT: openvswitch"},{"line_number":514,"context_line":"        Q_ML2_TENANT_NETWORK_TYPE: vxlan"},{"line_number":515,"context_line":"        Q_ML2_PLUGIN_MECHANISM_DRIVERS: openvswitch"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"f180311d_71d7719c","line":512,"range":{"start_line":511,"start_character":0,"end_line":512,"end_character":61},"updated":"2023-12-05 20:04:15.000000000","message":"any specific reason to set the default image name here instead of replying what we have default in devstack? \n\nand if we are setting the DEFAULT_IMAGE_FILE_NAME then we should set the CIRROS_VERSION also (devstack use both to prepare the image url to download) otherwise it can end up invalid url if CIRROS_VERSION default change in devstack - https://github.com/openstack/devstack/blob/2211c778db0e18702c7177f7750571cba3697509/stackrc#L715","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6e40dcee3a650103264a07620ed3751f45a70fff","unresolved":true,"context_lines":[{"line_number":508,"context_line":"      devstack_localrc:"},{"line_number":509,"context_line":"        # For nested virt"},{"line_number":510,"context_line":"        LIBVIRT_TYPE: kvm"},{"line_number":511,"context_line":"        DEFAULT_IMAGE_NAME: cirros-0.6.2-x86_64-disk"},{"line_number":512,"context_line":"        DEFAULT_IMAGE_FILE_NAME: cirros-0.6.2-x86_64-disk.img"},{"line_number":513,"context_line":"        Q_AGENT: openvswitch"},{"line_number":514,"context_line":"        Q_ML2_TENANT_NETWORK_TYPE: vxlan"},{"line_number":515,"context_line":"        Q_ML2_PLUGIN_MECHANISM_DRIVERS: openvswitch"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"67306657_e767e833","line":512,"range":{"start_line":511,"start_character":0,"end_line":512,"end_character":61},"in_reply_to":"3b79cb79_b929d3d8","updated":"2023-12-05 22:30:53.000000000","message":"ok, as discussed in IRC, I am ok to update these in followup","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"64185ba0f2a36d629f8342cae7ffbe4b7fe2f56c","unresolved":true,"context_lines":[{"line_number":508,"context_line":"      devstack_localrc:"},{"line_number":509,"context_line":"        # For nested virt"},{"line_number":510,"context_line":"        LIBVIRT_TYPE: kvm"},{"line_number":511,"context_line":"        DEFAULT_IMAGE_NAME: cirros-0.6.2-x86_64-disk"},{"line_number":512,"context_line":"        DEFAULT_IMAGE_FILE_NAME: cirros-0.6.2-x86_64-disk.img"},{"line_number":513,"context_line":"        Q_AGENT: openvswitch"},{"line_number":514,"context_line":"        Q_ML2_TENANT_NETWORK_TYPE: vxlan"},{"line_number":515,"context_line":"        Q_ML2_PLUGIN_MECHANISM_DRIVERS: openvswitch"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"61a3c5b3_f03b5459","line":512,"range":{"start_line":511,"start_character":0,"end_line":512,"end_character":61},"in_reply_to":"67306657_e767e833","updated":"2023-12-05 22:36:48.000000000","message":"Follow up is at https://review.opendev.org/c/openstack/nova/+/902734","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"75505cca38d0f3f6c5be4e4d71c58781b894a588","unresolved":true,"context_lines":[{"line_number":508,"context_line":"      devstack_localrc:"},{"line_number":509,"context_line":"        # For nested virt"},{"line_number":510,"context_line":"        LIBVIRT_TYPE: kvm"},{"line_number":511,"context_line":"        DEFAULT_IMAGE_NAME: cirros-0.6.2-x86_64-disk"},{"line_number":512,"context_line":"        DEFAULT_IMAGE_FILE_NAME: cirros-0.6.2-x86_64-disk.img"},{"line_number":513,"context_line":"        Q_AGENT: openvswitch"},{"line_number":514,"context_line":"        Q_ML2_TENANT_NETWORK_TYPE: vxlan"},{"line_number":515,"context_line":"        Q_ML2_PLUGIN_MECHANISM_DRIVERS: openvswitch"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3b79cb79_b929d3d8","line":512,"range":{"start_line":511,"start_character":0,"end_line":512,"end_character":61},"in_reply_to":"f180311d_71d7719c","updated":"2023-12-05 22:24:49.000000000","message":"Oh, yeah, sorry ... this is from my original/earlier PS where I was setting the default image vars in project.vars and had intended for all jobs to inherit. That did not end up working out though bc the devstack-tempest ancestor job that some jobs inherit from eventually sets IMAGE_URLS and it overrides the project.vars I set. So I had to abandon that approach and instead use the yaml anchors.\n\nI\u0027ll remove these because in this PS, they are not needed.","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"}],"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d7919415ecf7693b3b5518bdff8f6fc678dfbf18","unresolved":true,"context_lines":[{"line_number":20,"context_line":"  https://review.opendev.org/c/openstack/neutron/+/843237"},{"line_number":21,"context_line":"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/821067"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Depends-On: https://review.opendev.org/c/openstack/tempest/+/902609"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Change-Id: I1a6468fbfa51eedec0ab91d73f313784a9a618a0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"5d902c1c_c15154c8","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":67},"updated":"2023-12-05 20:04:15.000000000","message":"I did not get how tempest is dependency of this change?","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"75505cca38d0f3f6c5be4e4d71c58781b894a588","unresolved":true,"context_lines":[{"line_number":20,"context_line":"  https://review.opendev.org/c/openstack/neutron/+/843237"},{"line_number":21,"context_line":"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/821067"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Depends-On: https://review.opendev.org/c/openstack/tempest/+/902609"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Change-Id: I1a6468fbfa51eedec0ab91d73f313784a9a618a0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"f4a84697_06141e79","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":67},"in_reply_to":"5d902c1c_c15154c8","updated":"2023-12-05 22:24:49.000000000","message":"There are a couple of places in tempest where it creates an image based on the default image but if the default image is UEC image, it can\u0027t handle it. When tempest creates images it needs to use the kernel and ramdisk image IDs when it creates the AMI image. In the scenario manager, tempest has a method that creates a fresh image from the default image file, but if that image file is a UEC image tarball, it can\u0027t handle that presently. It needs to extract the tarball and create the kernel and ramdisk images, then create the AMI image pointing at those.","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6e40dcee3a650103264a07620ed3751f45a70fff","unresolved":false,"context_lines":[{"line_number":20,"context_line":"  https://review.opendev.org/c/openstack/neutron/+/843237"},{"line_number":21,"context_line":"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/821067"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Depends-On: https://review.opendev.org/c/openstack/tempest/+/902609"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Change-Id: I1a6468fbfa51eedec0ab91d73f313784a9a618a0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"714489ca_721fbf49","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":67},"in_reply_to":"f4a84697_06141e79","updated":"2023-12-05 22:30:53.000000000","message":"thanks for explanation. got it.","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"cf572d32717a8d821caa40d88053307113c7ddad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3e4d22d6_ad056b9d","updated":"2023-11-30 20:52:37.000000000","message":"Some things aren\u0027t working right, looking into it.","commit_id":"44c56ce02ad6ed8ad02b5ac9e36c25f6a0420a22"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"1a0402040b19d835a78392246102aff136e2a5be","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"96377756_19aa2d87","updated":"2023-12-01 02:32:21.000000000","message":"IMAGE_URLS still isn\u0027t working right.","commit_id":"a995a86da5e150bd6f4a40ac6eff9f82aaef2963"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"a1271381cb6212fd4699aaf1d617783a8b91c857","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"8c31b71b_c61ca834","updated":"2023-12-05 19:12:45.000000000","message":"check experimental","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"1860e77e37baab583bc1e2b8552cc37bfe5c4644","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"30dad296_d6ed0a85","updated":"2023-12-05 19:08:53.000000000","message":"check experimental to try running nova-emulation","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d7919415ecf7693b3b5518bdff8f6fc678dfbf18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"2c13b5da_26b4e71c","updated":"2023-12-05 20:04:15.000000000","message":"overall lgtm except we should set CIRROS_VERSION  also if setting default image name.","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"fe0a4af8a259c7dc13ccb9dc59f73d8e5e3a0cb3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"bc307ba1_97d31860","updated":"2023-12-05 09:42:41.000000000","message":"recheck dep updated","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6e40dcee3a650103264a07620ed3751f45a70fff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"df8be8ca_a99ce645","updated":"2023-12-05 22:30:53.000000000","message":"thanks. lgtm","commit_id":"326279962ba787f845c6d79839df2e564e0f7419"}]}
