)]}'
{"tripleo_ansible/ansible_plugins/modules/tripleo_overcloud_network_extract.py":[{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"d70f5b6149d3afa0026768e12484575056e49e46","unresolved":false,"context_lines":[{"line_number":320,"context_line":"        net_resources \u003d get_overcloud_network_resources(conn, stack_name)"},{"line_number":321,"context_line":"        result[\u0027network_data\u0027] \u003d parse_net_resources(conn, net_resources)"},{"line_number":322,"context_line":""},{"line_number":323,"context_line":"        result[\u0027changed\u0027] \u003d True if result[\u0027network_data\u0027] else False"},{"line_number":324,"context_line":"        module.exit_json(**result)"},{"line_number":325,"context_line":"    except Exception as err:"},{"line_number":326,"context_line":"        result[\u0027error\u0027] \u003d str(err)"}],"source_content_type":"text/x-python","patch_set":16,"id":"9f560f44_220d068f","line":323,"range":{"start_line":323,"start_character":59,"end_line":323,"end_character":69},"updated":"2020-10-08 06:02:38.000000000","message":"redundant","commit_id":"230f648fbeb0580ccfe9b938f11f3babcc5ba61b"}],"tripleo_ansible/ansible_plugins/modules/tripleo_overcloud_network_extract_provisioned.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"4e4ef57fb6a01fb02b4630a13bf2fa4a7baab6d5","unresolved":false,"context_lines":[{"line_number":125,"context_line":""},{"line_number":126,"context_line":"    return network_resource_dict"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"def tripleo_resource_tags_to_dict(resource_tags):"},{"line_number":129,"context_line":"    tag_dict \u003d dict()"},{"line_number":130,"context_line":"    for tag in resource_tags:"},{"line_number":131,"context_line":"        if not tag.startswith(\u0027tripleo_\u0027):"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_8a4b2585","line":128,"updated":"2020-09-09 14:43:57.000000000","message":"linters: E302 expected 2 blank lines, found 1","commit_id":"82b4d9c24fc7218b90755385b41983c1a2e90812"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"4e4ef57fb6a01fb02b4630a13bf2fa4a7baab6d5","unresolved":false,"context_lines":[{"line_number":139,"context_line":""},{"line_number":140,"context_line":"    return tag_dict"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"def get_network_info(conn, network_id):"},{"line_number":143,"context_line":"    network \u003d conn.network.get_network(network_id)"},{"line_number":144,"context_line":"    tag_dict \u003d tripleo_resource_tags_to_dict(network.tags)"},{"line_number":145,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_6a505133","line":142,"updated":"2020-09-09 14:43:57.000000000","message":"linters: E302 expected 2 blank lines, found 1","commit_id":"82b4d9c24fc7218b90755385b41983c1a2e90812"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"4e4ef57fb6a01fb02b4630a13bf2fa4a7baab6d5","unresolved":false,"context_lines":[{"line_number":154,"context_line":""},{"line_number":155,"context_line":"    if \u0027tripleo_service_net_map_replace\u0027 in tag_dict:"},{"line_number":156,"context_line":"        net_dict.update({"},{"line_number":157,"context_line":"            service_net_map_replace:"},{"line_number":158,"context_line":"                tag_dict[\u0027tripleo_service_net_map_replace\u0027]"},{"line_number":159,"context_line":"        })"},{"line_number":160,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_ca419d65","line":157,"updated":"2020-09-09 14:43:57.000000000","message":"linters: F821 undefined name \u0027service_net_map_replace\u0027","commit_id":"82b4d9c24fc7218b90755385b41983c1a2e90812"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"0d5c6deca683feee521d2ed7a1a382179969a7ff","unresolved":false,"context_lines":[{"line_number":195,"context_line":"                net_entry.update(net_info)"},{"line_number":196,"context_line":"            if res_dict[\u0027resource_type\u0027] \u003d\u003d TYPE_SUBNET:"},{"line_number":197,"context_line":"                subnet_info \u003d get_subnet_info(conn, res_dict[RES_ID])"},{"line_number":198,"context_line":"                if res \u003d\u003d \u0027\u0027.join([name, \u0027Subnet\u0027]):"},{"line_number":199,"context_line":"                    # This is the base subnet"},{"line_number":200,"context_line":"                    net_entry.update(subnet_info)"},{"line_number":201,"context_line":"                else:"},{"line_number":202,"context_line":"                    # This is a \"leaf\" subnet, goes in subnets dict"},{"line_number":203,"context_line":"                    subnet_name \u003d res.rsplit(\u0027\u0027.join([name, \u0027Subnet_\u0027]))[-1]"},{"line_number":204,"context_line":"                    net_entry.update({\u0027subnets\u0027: {subnet_name: subnet_info}})"},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"        network_data.append(net_entry)"},{"line_number":207,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_ead621c3","line":204,"range":{"start_line":198,"start_character":16,"end_line":204,"end_character":77},"updated":"2020-09-16 17:39:52.000000000","message":"Should we remove this legacy, and store all subnet info as subnet items in the \u0027subnets\u0027 dictonary instead?","commit_id":"82b4d9c24fc7218b90755385b41983c1a2e90812"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"19fea862d2014a6e1e1ce64c14ad58ad5b2a9f2d","unresolved":false,"context_lines":[{"line_number":195,"context_line":"                net_entry.update(net_info)"},{"line_number":196,"context_line":"            if res_dict[\u0027resource_type\u0027] \u003d\u003d TYPE_SUBNET:"},{"line_number":197,"context_line":"                subnet_info \u003d get_subnet_info(conn, res_dict[RES_ID])"},{"line_number":198,"context_line":"                if res \u003d\u003d \u0027\u0027.join([name, \u0027Subnet\u0027]):"},{"line_number":199,"context_line":"                    # This is the base subnet"},{"line_number":200,"context_line":"                    net_entry.update(subnet_info)"},{"line_number":201,"context_line":"                else:"},{"line_number":202,"context_line":"                    # This is a \"leaf\" subnet, goes in subnets dict"},{"line_number":203,"context_line":"                    subnet_name \u003d res.rsplit(\u0027\u0027.join([name, \u0027Subnet_\u0027]))[-1]"},{"line_number":204,"context_line":"                    net_entry.update({\u0027subnets\u0027: {subnet_name: subnet_info}})"},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"        network_data.append(net_entry)"},{"line_number":207,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_a4497db8","line":204,"range":{"start_line":198,"start_character":16,"end_line":204,"end_character":77},"in_reply_to":"9f560f44_ead621c3","updated":"2020-09-17 08:52:46.000000000","message":"Done","commit_id":"82b4d9c24fc7218b90755385b41983c1a2e90812"}],"tripleo_ansible/playbooks/cli-overcloud-network-extract-networks.yaml":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"aa38c48e69aa476aba381d20a8a7a39546392d24","unresolved":false,"context_lines":[{"line_number":44,"context_line":"      tripleo_overcloud_network_extract_networks:"},{"line_number":45,"context_line":"        stack_name: \"{{ stack_name }}\""},{"line_number":46,"context_line":"      register: overcloud_network_data"},{"line_number":47,"context_line":"    - name: Write netowork data to output file"},{"line_number":48,"context_line":"      copy:"},{"line_number":49,"context_line":"        content: \"{{ overcloud_network_data.network_data | to_nice_yaml(indent\u003d2) }}\""},{"line_number":50,"context_line":"        dest: \"{{ output }}\""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"9f560f44_e2fcca0d","line":47,"range":{"start_line":47,"start_character":18,"end_line":47,"end_character":26},"updated":"2020-09-21 04:48:49.000000000","message":"nito, network","commit_id":"451ad9101d2e109eb3628c207bcdfc0d477ab1b7"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"8a425b9acddbb25a50d70404fbcb650f19d6baab","unresolved":false,"context_lines":[{"line_number":44,"context_line":"      tripleo_overcloud_network_extract_networks:"},{"line_number":45,"context_line":"        stack_name: \"{{ stack_name }}\""},{"line_number":46,"context_line":"      register: overcloud_network_data"},{"line_number":47,"context_line":"    - name: Write netowork data to output file"},{"line_number":48,"context_line":"      copy:"},{"line_number":49,"context_line":"        content: \"{{ overcloud_network_data.network_data | to_nice_yaml(indent\u003d2) }}\""},{"line_number":50,"context_line":"        dest: \"{{ output }}\""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"9f560f44_0ca3f001","line":47,"range":{"start_line":47,"start_character":18,"end_line":47,"end_character":26},"in_reply_to":"9f560f44_e2fcca0d","updated":"2020-09-21 09:10:49.000000000","message":"Done","commit_id":"451ad9101d2e109eb3628c207bcdfc0d477ab1b7"}],"tripleo_ansible/playbooks/cli-overcloud-network-extract-provisioned.yaml":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"445e1cd7bbbaba7107cc7e0508ccb07f07d0d747","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        - output is undefined"},{"line_number":32,"context_line":"    - name: Check if output file exists"},{"line_number":33,"context_line":"      stat:"},{"line_number":34,"context_line":"        path: output"},{"line_number":35,"context_line":"      register: stat_result"},{"line_number":36,"context_line":"    - fail:"},{"line_number":37,"context_line":"        msg: Output file exists"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_f132150f","line":34,"range":{"start_line":34,"start_character":14,"end_line":34,"end_character":20},"updated":"2020-09-16 16:58:02.000000000","message":"\"{{ output }}\"","commit_id":"070eb2c35fd955574cd6237ad2d9464ec4329265"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"0d5c6deca683feee521d2ed7a1a382179969a7ff","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        - output is undefined"},{"line_number":32,"context_line":"    - name: Check if output file exists"},{"line_number":33,"context_line":"      stat:"},{"line_number":34,"context_line":"        path: output"},{"line_number":35,"context_line":"      register: stat_result"},{"line_number":36,"context_line":"    - fail:"},{"line_number":37,"context_line":"        msg: Output file exists"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_045f8975","line":34,"range":{"start_line":34,"start_character":14,"end_line":34,"end_character":20},"in_reply_to":"9f560f44_f132150f","updated":"2020-09-16 17:39:52.000000000","message":"this was my mistake, causing the fail task to always be skipped.","commit_id":"070eb2c35fd955574cd6237ad2d9464ec4329265"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"d2fd2aaff154b8273fde8de2f4955ea1b245f145","unresolved":false,"context_lines":[{"line_number":35,"context_line":"      register: stat_result"},{"line_number":36,"context_line":"    - fail:"},{"line_number":37,"context_line":"        msg: Output file exists"},{"line_number":38,"context_line":"      when:"},{"line_number":39,"context_line":"        stat_result.stat.exists and prompt"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"  tasks:"},{"line_number":42,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_d1fa7141","line":39,"range":{"start_line":38,"start_character":0,"end_line":39,"end_character":42},"updated":"2020-09-16 16:52:55.000000000","message":"I don\u0027t know what the prompt var is (or where it comes from) but you need to make this a list\n\nwhen:\n  - prompt|bool\n  - stat_result.stat.exists","commit_id":"070eb2c35fd955574cd6237ad2d9464ec4329265"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"4ff6adfa916c7ed6264e33a2d2dc375982e6ff46","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    - fail:"},{"line_number":37,"context_line":"        msg: Output file exists"},{"line_number":38,"context_line":"      when:"},{"line_number":39,"context_line":"        stat_result.stat.exists and prompt"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"  tasks:"},{"line_number":42,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_016ae5eb","line":39,"range":{"start_line":39,"start_character":36,"end_line":39,"end_character":42},"updated":"2020-09-16 08:41:53.000000000","message":"how does this condition work?","commit_id":"070eb2c35fd955574cd6237ad2d9464ec4329265"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"0d5c6deca683feee521d2ed7a1a382179969a7ff","unresolved":false,"context_lines":[{"line_number":35,"context_line":"      register: stat_result"},{"line_number":36,"context_line":"    - fail:"},{"line_number":37,"context_line":"        msg: Output file exists"},{"line_number":38,"context_line":"      when:"},{"line_number":39,"context_line":"        stat_result.stat.exists and prompt"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"  tasks:"},{"line_number":42,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_448061c0","line":39,"range":{"start_line":38,"start_character":0,"end_line":39,"end_character":42},"in_reply_to":"9f560f44_31164d57","updated":"2020-09-16 17:39:52.000000000","message":"When doing the first iteration I figured I\u0027d try to do something elaborate with tripleoclient interaction prompting the user. But that\u0027s overkill, I changed the var to \u0027overwrite\u0027 and in the tripleoclient patch \u0027--overwrite\u0027 set\u0027s it true.","commit_id":"070eb2c35fd955574cd6237ad2d9464ec4329265"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"4428b269d97a11b3b2aaca20b76a0569b5e6be35","unresolved":false,"context_lines":[{"line_number":35,"context_line":"      register: stat_result"},{"line_number":36,"context_line":"    - fail:"},{"line_number":37,"context_line":"        msg: Output file exists"},{"line_number":38,"context_line":"      when:"},{"line_number":39,"context_line":"        stat_result.stat.exists and prompt"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"  tasks:"},{"line_number":42,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_31164d57","line":39,"range":{"start_line":38,"start_character":0,"end_line":39,"end_character":42},"in_reply_to":"9f560f44_d1fa7141","updated":"2020-09-16 17:01:56.000000000","message":"+1\n\nIs prompt supposed to be something passed in via the CLI or did you mean to use the prompt module?\n\nhttps://docs.ansible.com/ansible/latest/user_guide/playbooks_prompts.html","commit_id":"070eb2c35fd955574cd6237ad2d9464ec4329265"}]}