)]}'
{"defaults/main.yml":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"71f8b9029507d5cc804ae67b89525edeecd2d490","unresolved":false,"context_lines":[{"line_number":184,"context_line":"# the OBS repositories. If you want to use the same mirror in both cases, then leave the"},{"line_number":185,"context_line":"# \u0027lxc_hosts_opensuse_mirror_obs_url\u0027 to its default value."},{"line_number":186,"context_line":"lxc_hosts_opensuse_mirror_url: \u0027http://download.opensuse.org\u0027"},{"line_number":187,"context_line":"lxc_hosts_opensuse_mirror_obs_url: \"{{ lxc_hosts_opensuse_mirror_url | default(\u0027http://download.opensuse.org\u0027) }}\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7f287b81_b6733716","line":187,"range":{"start_line":187,"start_character":69,"end_line":187,"end_character":110},"updated":"2017-09-07 10:48:56.000000000","message":"| default not needed here, because it\u0027s the same value.\nKeeping the two variables makes sense though.","commit_id":"06fb16050c9f7a818e2bb51ec0a242d9b4c3d24d"},{"author":{"_account_id":23163,"name":"Markos Chandras (hwoarang)","email":"markos@chandras.me","username":"mchandras"},"change_message_id":"6f2308f64f765f5dbcf463fd333b0d9b3769bfc1","unresolved":false,"context_lines":[{"line_number":184,"context_line":"# the OBS repositories. If you want to use the same mirror in both cases, then leave the"},{"line_number":185,"context_line":"# \u0027lxc_hosts_opensuse_mirror_obs_url\u0027 to its default value."},{"line_number":186,"context_line":"lxc_hosts_opensuse_mirror_url: \u0027http://download.opensuse.org\u0027"},{"line_number":187,"context_line":"lxc_hosts_opensuse_mirror_obs_url: \"{{ lxc_hosts_opensuse_mirror_url | default(\u0027http://download.opensuse.org\u0027) }}\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7f287b81_7664af22","line":187,"range":{"start_line":187,"start_character":69,"end_line":187,"end_character":110},"in_reply_to":"7f287b81_b6733716","updated":"2017-09-07 10:54:24.000000000","message":"Done","commit_id":"06fb16050c9f7a818e2bb51ec0a242d9b4c3d24d"}],"vars/suse-42.yml":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"5973dba9e30e49ac2282dc59302e4de3aadbeb48","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"lxc_hosts_external_repo:"},{"line_number":18,"context_line":"  - name: \"OBS:Virtualization:containers\""},{"line_number":19,"context_line":"    uri: \"{{ pip_install_opensuse_mirror_obs_url }}/repositories/Virtualization:/containers/openSUSE_Leap_{{ ansible_distribution_version }}/\""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"system_config_dir: \"/etc/sysconfig\""},{"line_number":22,"context_line":"systemd_utils_prefix: \"/usr/lib/systemd\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7f287b81_b3734916","line":19,"range":{"start_line":19,"start_character":13,"end_line":19,"end_character":49},"updated":"2017-09-07 09:57:45.000000000","message":"We should probably make the URI a variable in defaults, and have a way to override.\n\nOn top of that, using something that\u0027s outside the scope of the role is a bad practice.","commit_id":"bea20dc5aa3aa9f28f6209fe95c0bb1883fa6324"},{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"71f8b9029507d5cc804ae67b89525edeecd2d490","unresolved":false,"context_lines":[{"line_number":45,"context_line":"  copy_from_host:"},{"line_number":46,"context_line":"    - /etc/environment"},{"line_number":47,"context_line":"    - /etc/localtime"},{"line_number":48,"context_line":"    - /etc/zypp/repos.d/repo-oss"},{"line_number":49,"context_line":"    - /etc/zypp/repos.d/repo-update"},{"line_number":50,"context_line":"  cache_prep_commands: |"},{"line_number":51,"context_line":"    {{ lxc_cache_prep_pre_commands }}"},{"line_number":52,"context_line":"    mkdir -p /etc/ansible/facts.d/"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7f287b81_76c82f21","line":49,"range":{"start_line":48,"start_character":0,"end_line":49,"end_character":35},"updated":"2017-09-07 10:48:56.000000000","message":"Can\u0027t judge if that would be impactful or not if they are not present, or anything... Looks good at first sight.","commit_id":"06fb16050c9f7a818e2bb51ec0a242d9b4c3d24d"},{"author":{"_account_id":23163,"name":"Markos Chandras (hwoarang)","email":"markos@chandras.me","username":"mchandras"},"change_message_id":"6f2308f64f765f5dbcf463fd333b0d9b3769bfc1","unresolved":false,"context_lines":[{"line_number":45,"context_line":"  copy_from_host:"},{"line_number":46,"context_line":"    - /etc/environment"},{"line_number":47,"context_line":"    - /etc/localtime"},{"line_number":48,"context_line":"    - /etc/zypp/repos.d/repo-oss"},{"line_number":49,"context_line":"    - /etc/zypp/repos.d/repo-update"},{"line_number":50,"context_line":"  cache_prep_commands: |"},{"line_number":51,"context_line":"    {{ lxc_cache_prep_pre_commands }}"},{"line_number":52,"context_line":"    mkdir -p /etc/ansible/facts.d/"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7f287b81_3643c76e","line":49,"range":{"start_line":48,"start_character":0,"end_line":49,"end_character":35},"in_reply_to":"7f287b81_76c82f21","updated":"2017-09-07 10:54:24.000000000","message":"The tasks/lxc_cache_preparation.yml \"Rsyncing files from the LXC host to the container cache\" tasks seems to be checking if the file exists before copying it so we should be ok","commit_id":"06fb16050c9f7a818e2bb51ec0a242d9b4c3d24d"}]}