)]}'
{"tests/run.yml":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"fc1d5559b0d552f9d6a17b13524be01a03baaba5","unresolved":false,"context_lines":[{"line_number":182,"context_line":"        - src: \"tests/templates/tenks-deploy-config.yml.j2\""},{"line_number":183,"context_line":"          dest: \"{{ ansible_env.HOME }}/tenks.yml\""},{"line_number":184,"context_line":"          when: \"{{ scenario \u003d\u003d \u0027ironic\u0027 }}\""},{"line_number":185,"context_line":"        # cinder.conf"},{"line_number":186,"context_line":"        - src: \"tests/templates/cinder-overrides.j2\""},{"line_number":187,"context_line":"          dest: /etc/kolla/config/cinder/cinder-volume.conf"},{"line_number":188,"context_line":"          when: \"{{ scenario \u003d\u003d \u0027cephadm\u0027 }}\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"95bafec2_c341edd0","line":185,"range":{"start_line":185,"start_character":7,"end_line":185,"end_character":21},"updated":"2023-05-26 13:44:41.000000000","message":"I know it\u0027s in the spirit of the code before, but what is the point of these comments exactly? I\u0027m all for meaningful comments which help reason about code.\n\nbut these comments just state which config file get\u0027s copied, which is just a reiteration of the config files name? so maybe we could drop these?\n\nmarking as resolved, as it\u0027s not really the point of this review, will provide a patch in a different changeset.\n\n:edit: wip: https://review.opendev.org/c/openstack/kolla-ansible/+/884503","commit_id":"cca21742377d49d25527c7b97e8a45c59edf347b"}]}