)]}'
{"tripleo_ansible/playbooks/cli-container-image-prepare.yaml":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"e850d767bb39af80477379c4a3c70ec3f2e57366","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        debug: \"{{ debug }}\""},{"line_number":62,"context_line":"      register: prepare_result"},{"line_number":63,"context_line":"    - name: Write output to file"},{"line_number":64,"context_line":"      copy:"},{"line_number":65,"context_line":"        content: \"{{ prepare_result.params | to_nice_yaml }}\""},{"line_number":66,"context_line":"        dest: \"{{ output_env_file }}\""},{"line_number":67,"context_line":"      when: output_env_file is defined"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_8537414e","line":64,"updated":"2020-08-28 16:13:17.000000000","message":"Is there a backup file rotate we can use? that\u0027s something the cli currently does.","commit_id":"e410e8546c0a9de0ef3bc78b72389693dcb96dee"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"b209704b224f00208a80160af6f66cc92b41288b","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        debug: \"{{ debug }}\""},{"line_number":62,"context_line":"      register: prepare_result"},{"line_number":63,"context_line":"    - name: Write output to file"},{"line_number":64,"context_line":"      copy:"},{"line_number":65,"context_line":"        content: \"{{ prepare_result.params | to_nice_yaml }}\""},{"line_number":66,"context_line":"        dest: \"{{ output_env_file }}\""},{"line_number":67,"context_line":"      when: output_env_file is defined"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_898c7b2f","line":64,"in_reply_to":"9f560f44_8537414e","updated":"2020-08-28 16:52:32.000000000","message":"I thought that was kinda of redundant, but can be added if we want. If someone specifies a file explicitly that would surely be overwritten.\n\nWe don\u0027t do it for other generated env files[1] too.\n\n[1] https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/playbooks/cli-overcloud-node-provision.yaml#L140","commit_id":"e410e8546c0a9de0ef3bc78b72389693dcb96dee"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"f32d0ee14304b600d4fb5e5c3a13d3f4a61fa3ec","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        debug: \"{{ debug }}\""},{"line_number":62,"context_line":"      register: prepare_result"},{"line_number":63,"context_line":"    - name: Write output to file"},{"line_number":64,"context_line":"      copy:"},{"line_number":65,"context_line":"        content: \"{{ prepare_result.params | to_nice_yaml }}\""},{"line_number":66,"context_line":"        dest: \"{{ output_env_file }}\""},{"line_number":67,"context_line":"      when: output_env_file is defined"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_298d6fd3","line":64,"in_reply_to":"9f560f44_898c7b2f","updated":"2020-08-28 17:05:39.000000000","message":"It\u0027s specific to the container image prepare, it spits a warning and backs up the previous version.","commit_id":"e410e8546c0a9de0ef3bc78b72389693dcb96dee"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"e850d767bb39af80477379c4a3c70ec3f2e57366","unresolved":false,"context_lines":[{"line_number":64,"context_line":"      copy:"},{"line_number":65,"context_line":"        content: \"{{ prepare_result.params | to_nice_yaml }}\""},{"line_number":66,"context_line":"        dest: \"{{ output_env_file }}\""},{"line_number":67,"context_line":"      when: output_env_file is defined"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_e53b9d36","line":67,"updated":"2020-08-28 16:13:17.000000000","message":"We\u0027re also missing the output images file for this as well","commit_id":"e410e8546c0a9de0ef3bc78b72389693dcb96dee"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"f32d0ee14304b600d4fb5e5c3a13d3f4a61fa3ec","unresolved":false,"context_lines":[{"line_number":64,"context_line":"      copy:"},{"line_number":65,"context_line":"        content: \"{{ prepare_result.params | to_nice_yaml }}\""},{"line_number":66,"context_line":"        dest: \"{{ output_env_file }}\""},{"line_number":67,"context_line":"      when: output_env_file is defined"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_89bbdb73","line":67,"in_reply_to":"9f560f44_29ebafdc","updated":"2020-08-28 17:05:39.000000000","message":"There are two files generated by this command and Env file and an Images file. Check the cli command. --output-images-file and --output-env-file","commit_id":"e410e8546c0a9de0ef3bc78b72389693dcb96dee"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"4bbec872d014285fc7343f739c43bea949f431f9","unresolved":false,"context_lines":[{"line_number":64,"context_line":"      copy:"},{"line_number":65,"context_line":"        content: \"{{ prepare_result.params | to_nice_yaml }}\""},{"line_number":66,"context_line":"        dest: \"{{ output_env_file }}\""},{"line_number":67,"context_line":"      when: output_env_file is defined"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_a9485f87","line":67,"in_reply_to":"9f560f44_89bbdb73","updated":"2020-08-28 17:14:18.000000000","message":"I don\u0027t see it unless I am blind. I guess you are talking about the deprecated command which had that.\n\nusage: openstack tripleo container image prepare [-h]\n                                                 [--environment-file \u003cfile path\u003e]\n                                                 [--environment-directory \u003cHEAT ENVIRONMENT DIRECTORY\u003e]\n                                                 [--roles-file ROLES_FILE]\n                                                 [--output-env-file \u003cfile path\u003e]\n                                                 [--dry-run]\n                                                 [--cleanup \u003cfull, partial, none\u003e]\n\nPrepare and upload containers from a single command.\n\noptional arguments:\n  -h, --help            show this help message and exit\n  --environment-file \u003cfile path\u003e, -e \u003cfile path\u003e\n                        Environment file containing the ContainerImagePrepare\n                        parameter which specifies all prepare actions. Also,\n                        environment files specifying which services are\n                        containerized. Entries will be filtered to only\n                        contain images used by containerized services. (Can be\n                        specified more than once.)\n  --environment-directory \u003cHEAT ENVIRONMENT DIRECTORY\u003e\n                        Environment file directories that are automatically\n                        added to the environment. Can be specified more than\n                        once. Files in directories are loaded in ascending\n                        sort order.\n  --roles-file ROLES_FILE, -r ROLES_FILE\n                        Roles file, overrides the default roles_data.yaml in\n                        the t-h-t templates directory used for deployment. May\n                        be an absolute path or the path relative to the\n                        templates dir.\n  --output-env-file \u003cfile path\u003e\n                        File to write heat environment file which specifies\n                        all image parameters. Any existing file will be\n                        overwritten.\n  --dry-run             Do not perform any pull, modify, or push operations.\n                        The environment file will still be populated as if\n                        these operations were performed.\n  --cleanup \u003cfull, partial, none\u003e\n                        Cleanup behavior for local images left after upload.\n                        The default \u0027full\u0027 will attempt to delete all local\n                        images. \u0027partial\u0027 will leave images required for\n                        deployment on this host. \u0027none\u0027 will do no cleanup.\n\nThis command is provided by the python-tripleoclient plugin.","commit_id":"e410e8546c0a9de0ef3bc78b72389693dcb96dee"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"55ecd4561f440aecdca15aa32802676e197e6c34","unresolved":false,"context_lines":[{"line_number":64,"context_line":"      copy:"},{"line_number":65,"context_line":"        content: \"{{ prepare_result.params | to_nice_yaml }}\""},{"line_number":66,"context_line":"        dest: \"{{ output_env_file }}\""},{"line_number":67,"context_line":"      when: output_env_file is defined"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_af24dfb0","line":67,"in_reply_to":"9f560f44_a9485f87","updated":"2020-08-28 18:17:06.000000000","message":"You are correct, sorry was looking in the wrong spot for this part. nevermind here. But I would still do the backup bits if possible","commit_id":"e410e8546c0a9de0ef3bc78b72389693dcb96dee"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"b209704b224f00208a80160af6f66cc92b41288b","unresolved":false,"context_lines":[{"line_number":64,"context_line":"      copy:"},{"line_number":65,"context_line":"        content: \"{{ prepare_result.params | to_nice_yaml }}\""},{"line_number":66,"context_line":"        dest: \"{{ output_env_file }}\""},{"line_number":67,"context_line":"      when: output_env_file is defined"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_29ebafdc","line":67,"in_reply_to":"9f560f44_e53b9d36","updated":"2020-08-28 16:52:32.000000000","message":"Not sure what you mean?  Image parameters would be written to output_env_file. Which other file you mean?","commit_id":"e410e8546c0a9de0ef3bc78b72389693dcb96dee"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"d2506f329a2ad1b17fa275a84190d9024e4c7bca","unresolved":false,"context_lines":[{"line_number":50,"context_line":"      tripleo_build_heat_environment:"},{"line_number":51,"context_line":"        env_files: \"{{ env_files }}\""},{"line_number":52,"context_line":"      register: env_result"},{"line_number":53,"context_line":"    - name: Prepare cntainer images"},{"line_number":54,"context_line":"      become: true"},{"line_number":55,"context_line":"      tripleo_container_image_prepare:"},{"line_number":56,"context_line":"        roles_data: \"{{ roles_data[\u0027content\u0027] | b64decode | from_yaml}}\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f560f44_c84f7768","line":53,"range":{"start_line":53,"start_character":20,"end_line":53,"end_character":28},"updated":"2020-08-31 13:06:19.000000000","message":"container","commit_id":"abffbdd4b69159152b03bbfcb0cc184b168757e4"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"d2506f329a2ad1b17fa275a84190d9024e4c7bca","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        environment: \"{{ env_result.environment }}\""},{"line_number":58,"context_line":"        cleanup: \"{{ cleanup }}\""},{"line_number":59,"context_line":"        dry_run: \"{{ dry_run }}\""},{"line_number":60,"context_line":"        log_file: \"{{ cip_log_file }}\""},{"line_number":61,"context_line":"        debug: \"{{ debug }}\""},{"line_number":62,"context_line":"      register: prepare_result"},{"line_number":63,"context_line":"    - name: Write output to file"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f560f44_683e2bba","line":60,"range":{"start_line":60,"start_character":18,"end_line":60,"end_character":38},"updated":"2020-08-31 13:06:19.000000000","message":"Might be nice to indicate that we\u0027re logging to a file while this is executing in the name or something","commit_id":"abffbdd4b69159152b03bbfcb0cc184b168757e4"}]}