)]}'
{".zuul.yaml":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7863a5584c8fdeda3d64d617243ed62bf5e72472","unresolved":false,"context_lines":[{"line_number":238,"context_line":"            oslo_policy:"},{"line_number":239,"context_line":"              # The default policy file is policy.json but the"},{"line_number":240,"context_line":"              # setup-multi-cell-policy role will write to policy.yaml."},{"line_number":241,"context_line":"              policy_file: policy.yaml"},{"line_number":242,"context_line":"        test-config:"},{"line_number":243,"context_line":"          $TEMPEST_CONFIG:"},{"line_number":244,"context_line":"            compute-feature-enabled:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ffb9cba7_3a961495","line":241,"updated":"2019-05-01 21:25:12.000000000","message":"This works:\n\nhttp://logs.openstack.org/56/656656/1/check/nova-multi-cell/0a40a97/controller/logs/etc/nova/nova_conf.txt.gz","commit_id":"fa45babd6cb54734db5dc00ddfc4a968f5e4dd76"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7863a5584c8fdeda3d64d617243ed62bf5e72472","unresolved":false,"context_lines":[{"line_number":245,"context_line":"              # Enable cold migration for migrating across cells. Note that"},{"line_number":246,"context_line":"              # because NOVA_ALLOW_MOVE_TO_SAME_HOST\u003dfalse, all cold migrations"},{"line_number":247,"context_line":"              # will move across cells."},{"line_number":248,"context_line":"              cold_migration: true"},{"line_number":249,"context_line":"      devstack_services:"},{"line_number":250,"context_line":"        # Disable n-novnc until bug 1822873 is fixed."},{"line_number":251,"context_line":"        n-novnc: false"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ffb9cba7_fae07c28","line":248,"updated":"2019-05-01 21:25:12.000000000","message":"This is fine:\n\nhttp://logs.openstack.org/56/656656/1/check/nova-multi-cell/0a40a97/controller/logs/tempest_conf.txt.gz\n\nIt\u0027s also the default in tempest so we don\u0027t really need this, but it\u0027s nice to be explicit.","commit_id":"fa45babd6cb54734db5dc00ddfc4a968f5e4dd76"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7863a5584c8fdeda3d64d617243ed62bf5e72472","unresolved":false,"context_lines":[{"line_number":256,"context_line":"        NOVA_NUM_CELLS: 2"},{"line_number":257,"context_line":"        # Disable resize to the same host so all resizes will move across"},{"line_number":258,"context_line":"        # cells."},{"line_number":259,"context_line":"        NOVA_ALLOW_MOVE_TO_SAME_HOST: false"},{"line_number":260,"context_line":"        # We only have two computes and we don\u0027t yet support cross-cell live"},{"line_number":261,"context_line":"        # migration."},{"line_number":262,"context_line":"        LIVE_MIGRATION_AVAILABLE: false"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ffb9cba7_7af56cec","line":259,"updated":"2019-05-01 21:25:12.000000000","message":"This doesn\u0027t set anything:\n\nhttp://logs.openstack.org/56/656656/1/check/nova-multi-cell/0a40a97/controller/logs/etc/nova/nova_conf.txt.gz\n\nBut it\u0027s also the default in the code so maybe doesn\u0027t matter.\n\nAh, because devstack does this:\n\n    if [ \"$NOVA_ALLOW_MOVE_TO_SAME_HOST\" \u003d\u003d \"True\" ]; then\n        iniset $NOVA_CONF DEFAULT allow_resize_to_same_host \"True\"\n    fi","commit_id":"fa45babd6cb54734db5dc00ddfc4a968f5e4dd76"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b0247bb4770bb60c62770f883f64c5b32a30325b","unresolved":false,"context_lines":[{"line_number":240,"context_line":"      # Run compute API and scenario tests."},{"line_number":241,"context_line":"#      tempest_test_regex: ^tempest\\.(scenario|(api\\.compute))"},{"line_number":242,"context_line":"      # TODO(mriedem): Unrestrict the tests after debugging."},{"line_number":243,"context_line":"      tempest_test_regex: test_cold_migration|test_revert_cold_migration|test_resize_volume_backed_server_confirm"},{"line_number":244,"context_line":"      tempest_test_blacklist: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/nova\"].src_dir }}/devstack/nova-multi-cell-blacklist.txt\u0027"},{"line_number":245,"context_line":"      devstack_local_conf:"},{"line_number":246,"context_line":"        post-config:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"dfbec78f_9ba15ffb","line":243,"range":{"start_line":243,"start_character":73,"end_line":243,"end_character":113},"updated":"2019-05-13 16:06:21.000000000","message":"just focus on this one next since it\u0027s failing early","commit_id":"78a3d2d4e63c4d63a177ce71594847ca9515f49b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b0247bb4770bb60c62770f883f64c5b32a30325b","unresolved":false,"context_lines":[{"line_number":240,"context_line":"      # Run compute API and scenario tests."},{"line_number":241,"context_line":"#      tempest_test_regex: ^tempest\\.(scenario|(api\\.compute))"},{"line_number":242,"context_line":"      # TODO(mriedem): Unrestrict the tests after debugging."},{"line_number":243,"context_line":"      tempest_test_regex: test_cold_migration|test_revert_cold_migration|test_resize_volume_backed_server_confirm"},{"line_number":244,"context_line":"      tempest_test_blacklist: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/nova\"].src_dir }}/devstack/nova-multi-cell-blacklist.txt\u0027"},{"line_number":245,"context_line":"      devstack_local_conf:"},{"line_number":246,"context_line":"        post-config:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"dfbec78f_1b680fb7","line":243,"range":{"start_line":243,"start_character":46,"end_line":243,"end_character":72},"updated":"2019-05-13 16:06:21.000000000","message":"nix this for now","commit_id":"78a3d2d4e63c4d63a177ce71594847ca9515f49b"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5d60d80ed63d188010c59ca13a4c23ca1444fe09","unresolved":false,"context_lines":[{"line_number":299,"context_line":"        post-config:"},{"line_number":300,"context_line":"          $NOVA_CONF:"},{"line_number":301,"context_line":"            oslo_policy:"},{"line_number":302,"context_line":"              # The default policy file is policy.json but the"},{"line_number":303,"context_line":"              # setup-multi-cell-policy role will write to policy.yaml."},{"line_number":304,"context_line":"              policy_file: policy.yaml"},{"line_number":305,"context_line":"        test-config:"},{"line_number":306,"context_line":"          $TEMPEST_CONFIG:"}],"source_content_type":"text/x-yaml","patch_set":51,"id":"3fa7e38b_e723c6b9","line":303,"range":{"start_line":302,"start_character":0,"end_line":303,"end_character":71},"updated":"2019-12-13 12:23:13.000000000","message":"why?","commit_id":"b3dcf86efba76cf421017d709ad67b390460d5ac"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"0e023d8b6f90d07d509281aa1a2d4d8dc7b4a1dc","unresolved":false,"context_lines":[{"line_number":299,"context_line":"        post-config:"},{"line_number":300,"context_line":"          $NOVA_CONF:"},{"line_number":301,"context_line":"            oslo_policy:"},{"line_number":302,"context_line":"              # The default policy file is policy.json but the"},{"line_number":303,"context_line":"              # setup-multi-cell-policy role will write to policy.yaml."},{"line_number":304,"context_line":"              policy_file: policy.yaml"},{"line_number":305,"context_line":"        test-config:"},{"line_number":306,"context_line":"          $TEMPEST_CONFIG:"}],"source_content_type":"text/x-yaml","patch_set":51,"id":"3fa7e38b_afd4c4ea","line":303,"range":{"start_line":302,"start_character":0,"end_line":303,"end_character":71},"in_reply_to":"3fa7e38b_8f4ea86b","updated":"2019-12-13 15:07:06.000000000","message":"Ah. Stupid JSON.","commit_id":"b3dcf86efba76cf421017d709ad67b390460d5ac"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"3b050528fa9fc54c596f0426e808f2ad73d6dc36","unresolved":false,"context_lines":[{"line_number":299,"context_line":"        post-config:"},{"line_number":300,"context_line":"          $NOVA_CONF:"},{"line_number":301,"context_line":"            oslo_policy:"},{"line_number":302,"context_line":"              # The default policy file is policy.json but the"},{"line_number":303,"context_line":"              # setup-multi-cell-policy role will write to policy.yaml."},{"line_number":304,"context_line":"              policy_file: policy.yaml"},{"line_number":305,"context_line":"        test-config:"},{"line_number":306,"context_line":"          $TEMPEST_CONFIG:"}],"source_content_type":"text/x-yaml","patch_set":51,"id":"3fa7e38b_8f4ea86b","line":303,"range":{"start_line":302,"start_character":0,"end_line":303,"end_character":71},"in_reply_to":"3fa7e38b_e723c6b9","updated":"2019-12-13 14:45:26.000000000","message":"So it can have comments:\n\nhttps://review.opendev.org/#/c/656656/51/roles/setup-multi-cell-policy/tasks/main.yaml@14","commit_id":"b3dcf86efba76cf421017d709ad67b390460d5ac"}],"playbooks/nova-multi-cell/pre.yaml":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b0515ba62689251cae3629eb011eb5037a4e10c5","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Changes that run through nova-multi-cell are likely to have an impact on"},{"line_number":2,"context_line":"# the devstack part of the job, so we keep devstack in the main play to"},{"line_number":3,"context_line":"# avoid zuul retrying on legitimate failures."},{"line_number":4,"context_line":"- hosts: all"},{"line_number":5,"context_line":"  roles:"},{"line_number":6,"context_line":"    - orchestrate-devstack"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"- hosts: controller"},{"line_number":9,"context_line":"  roles:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ffb9cba7_e220ce9d","line":6,"range":{"start_line":1,"start_character":0,"end_line":6,"end_character":26},"updated":"2019-05-02 03:07:32.000000000","message":"I\u0027m not totally sure about this, but I copied it from the devstack-tempest job, and maybe it\u0027s contributing to the job failure now because it looks like devstack is running twice and g-api is getting started multiple times with the wrong proxy, i.e. http://status.openstack.org/elastic-recheck/#1806912","commit_id":"fa45babd6cb54734db5dc00ddfc4a968f5e4dd76"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"81e2265a85cb9c08010f23ec88fb40f2874c7b6c","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# avoid zuul retrying on legitimate failures."},{"line_number":4,"context_line":"- hosts: all"},{"line_number":5,"context_line":"  roles:"},{"line_number":6,"context_line":"    - orchestrate-devstack"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"- hosts: controller"},{"line_number":9,"context_line":"  roles:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"ffb9cba7_f1e89a7c","line":6,"updated":"2019-05-02 14:53:07.000000000","message":"I think this is screwing stuff up somehow but not sure what it\u0027s doing, is it re-stacking devstack?\n\nhttp://logs.openstack.org/56/656656/2/check/nova-multi-cell/347dd5d/controller/logs/devstacklog.txt.gz#_2019-05-02_05_12_23_639\n\nhttp://logs.openstack.org/56/656656/2/check/nova-multi-cell/347dd5d/ara-report/result/a01072aa-7728-4426-8ebd-b53d2e45bf0c/\n\nhttp://logs.openstack.org/56/656656/2/check/nova-multi-cell/347dd5d/ara-report/result/f4d8dcf9-0e10-4f86-b267-c3bd54a3b5b1/","commit_id":"459d1a1613e5f47f2f160aa597457693dd671242"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"9a7a41ac7c687e030168b5f0ce31835432748732","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# avoid zuul retrying on legitimate failures."},{"line_number":4,"context_line":"- hosts: all"},{"line_number":5,"context_line":"  roles:"},{"line_number":6,"context_line":"    - orchestrate-devstack"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"- hosts: controller"},{"line_number":9,"context_line":"  roles:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"dfbec78f_7230e908","line":6,"in_reply_to":"ffb9cba7_f19dfa09","updated":"2019-05-02 22:57:12.000000000","message":"Dropping this fails:\n\nhttp://logs.openstack.org/56/656656/3/check/nova-multi-cell/6fd8226/ara-report/result/8be66d2c-58c7-48e0-b979-b34acf4e05e6/\n\nFailed to set permissions on the temporary files Ansible needs to create when becoming an unprivileged user (rc: 1, err: chown: changing ownership of \u0027/var/tmp/ansible-tmp-1556815320.9190807-234696689231583/\u0027: Operation not permitted\nchown: changing ownership of \u0027/var/tmp/ansible-tmp-1556815320.9190807-234696689231583/source\u0027: Operation not permitted\n}). For information on working around this, see https://docs.ansible.com/ansible/become.html#becoming-an-unprivileged-user","commit_id":"459d1a1613e5f47f2f160aa597457693dd671242"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a4b786e0086892a5ecd0f7a227d9d6e5925328d6","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# avoid zuul retrying on legitimate failures."},{"line_number":4,"context_line":"- hosts: all"},{"line_number":5,"context_line":"  roles:"},{"line_number":6,"context_line":"    - orchestrate-devstack"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"- hosts: controller"},{"line_number":9,"context_line":"  roles:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"ffb9cba7_f19dfa09","line":6,"in_reply_to":"ffb9cba7_f1e89a7c","updated":"2019-05-02 14:54:53.000000000","message":"I copied it from here https://github.com/openstack/tempest/blob/master/playbooks/devstack-tempest.yaml but I\u0027m not sure it\u0027s needed. I could probably also just make this a lot simpler but not using a new role and just adding a pre-run play with the host and task in there.","commit_id":"459d1a1613e5f47f2f160aa597457693dd671242"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"d4d3b701104f2b36103b2da50fccf6009dc55e18","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  roles:"},{"line_number":3,"context_line":"    # /etc/nova/policy.yaml is going to be owned by the stack user so we need"},{"line_number":4,"context_line":"    # to make sure that user exists on the controller host first."},{"line_number":5,"context_line":"    - setup-stack-user"},{"line_number":6,"context_line":"    # Write rules to the /etc/nova/policy.yaml file."},{"line_number":7,"context_line":"    - setup-multi-cell-policy"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dfbec78f_b72f2af5","line":5,"updated":"2019-05-03 15:41:27.000000000","message":"This is defined in devstack.","commit_id":"0e01175717ab6ccce197b033ad6fdcdcf355d58c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"2b8455b7f2d248a207c8727307d8bed93a729c05","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- hosts: controller"},{"line_number":2,"context_line":"  roles:"},{"line_number":3,"context_line":"    # /etc/nova/policy.yaml is going to be owned by the stack user so we need"},{"line_number":4,"context_line":"    # to make sure that user exists on the controller host first."},{"line_number":5,"context_line":"    - setup-stack-user"},{"line_number":6,"context_line":"    # Write rules to the /etc/nova/policy.yaml file."},{"line_number":7,"context_line":"    - setup-multi-cell-policy"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dfbec78f_2c70c572","line":7,"range":{"start_line":1,"start_character":0,"end_line":7,"end_character":29},"updated":"2019-05-03 21:02:07.000000000","message":"I think we need to override the complete pre-yaml of base job which setup the log dir etc on all host[1].\n\n[1] https://github.com/openstack/devstack/blob/58de715453fd9c55d57ee18e8f915685e8ebb6b7/.zuul.yaml#L291\n\nand then you extend it for controller host to roles - setup-multi-cell-policy","commit_id":"0e01175717ab6ccce197b033ad6fdcdcf355d58c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c671cdb3f9d742fba5271cfc87ee93c66a790799","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- hosts: controller"},{"line_number":2,"context_line":"  roles:"},{"line_number":3,"context_line":"    # /etc/nova/policy.yaml is going to be owned by the stack user so we need"},{"line_number":4,"context_line":"    # to make sure that user exists on the controller host first."},{"line_number":5,"context_line":"    - setup-stack-user"},{"line_number":6,"context_line":"    # Write rules to the /etc/nova/policy.yaml file."},{"line_number":7,"context_line":"    - setup-multi-cell-policy"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dfbec78f_a3604334","line":7,"range":{"start_line":1,"start_character":0,"end_line":7,"end_character":29},"in_reply_to":"dfbec78f_2c70c572","updated":"2019-05-03 23:06:07.000000000","message":"sorry, override is not required. this should work.\n\npre-run and post-run are overridable: order is like below-  \n\nparent pre-run playbook\nchild pre-run playbook\nchild playbook","commit_id":"0e01175717ab6ccce197b033ad6fdcdcf355d58c"}],"roles/setup-multi-cell-policy/tasks/main.yaml":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7863a5584c8fdeda3d64d617243ed62bf5e72472","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- name: Setup multi-cell policy"},{"line_number":2,"context_line":"  copy:"},{"line_number":3,"context_line":"    content: |"},{"line_number":4,"context_line":"      # Enable cross-cell resize."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ffb9cba7_5a7bc879","line":1,"updated":"2019-05-01 21:25:12.000000000","message":"It works:\n\nhttp://logs.openstack.org/56/656656/1/check/nova-multi-cell/0a40a97/controller/logs/etc/nova/policy_yaml.txt.gz","commit_id":"fa45babd6cb54734db5dc00ddfc4a968f5e4dd76"}]}