)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"30b333d2b9c3bbbfa0dee3d5be2a19f9dd67a5fe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6167f495_0c8873d4","updated":"2022-03-25 12:54:46.000000000","message":"makes sense, thanks","commit_id":"13e8db5a6f44a8f537988a102f9869e725bb97bb"}],"roles/setup-devstack-cache/tasks/main.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"dd9d2da1ece9c9a038974e611cf3d988a5bfc907","unresolved":true,"context_lines":[{"line_number":2,"context_line":"  # This uses hard links to avoid using extra space."},{"line_number":3,"context_line":"  command: \"find {{ devstack_cache_dir }}/files -mindepth 1 -maxdepth 1 -exec cp -l {} {{ devstack_base_dir }}/devstack/files/ ;\""},{"line_number":4,"context_line":"  become: true"},{"line_number":5,"context_line":"  ignore_errors: yes"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"- name: Set ownership of cached files"},{"line_number":8,"context_line":"  file:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"75dbdd72_c1ee8e51","line":5,"updated":"2022-03-09 20:31:19.000000000","message":"this is how it currently fails\n\n2022-03-09 20:25:18.056667 | TASK [setup-devstack-cache : Copy cached devstack files]\n2022-03-09 15:25:19.197499 | controller | find: ‘/opt/cache/files’: No such file or directory\n2022-03-09 20:25:19.114845 | controller | ERROR\n2022-03-09 20:25:19.115202 | controller | {\n2022-03-09 20:25:19.115284 | controller |   \"delta\": \"0:00:00.004500\",\n2022-03-09 20:25:19.115322 | controller |   \"end\": \"2022-03-09 15:25:19.197897\",\n2022-03-09 20:25:19.115353 | controller |   \"msg\": \"non-zero return code\",\n2022-03-09 20:25:19.115382 | controller |   \"rc\": 1,\n2022-03-09 20:25:19.115426 | controller |   \"start\": \"2022-03-09 15:25:19.193397\"\n2022-03-09 20:25:19.115463 | controller | }\n2022-03-09 20:25:19.148761 |","commit_id":"13e8db5a6f44a8f537988a102f9869e725bb97bb"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"30b333d2b9c3bbbfa0dee3d5be2a19f9dd67a5fe","unresolved":true,"context_lines":[{"line_number":2,"context_line":"  # This uses hard links to avoid using extra space."},{"line_number":3,"context_line":"  command: \"find {{ devstack_cache_dir }}/files -mindepth 1 -maxdepth 1 -exec cp -l {} {{ devstack_base_dir }}/devstack/files/ ;\""},{"line_number":4,"context_line":"  become: true"},{"line_number":5,"context_line":"  ignore_errors: yes"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"- name: Set ownership of cached files"},{"line_number":8,"context_line":"  file:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d05a6a40_d1d74568","line":5,"in_reply_to":"75dbdd72_c1ee8e51","updated":"2022-03-25 12:54:46.000000000","message":"thanks for pointing it out .. in case you\u0027ll be proposing another patch set, it would be worth of adding a very brief comment why ignore_errors is used here","commit_id":"13e8db5a6f44a8f537988a102f9869e725bb97bb"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"e0c26621b73f4abbe2277294a550de0311b9ec07","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  # This uses hard links to avoid using extra space."},{"line_number":3,"context_line":"  command: \"find {{ devstack_cache_dir }}/files -mindepth 1 -maxdepth 1 -exec cp -l {} {{ devstack_base_dir }}/devstack/files/ ;\""},{"line_number":4,"context_line":"  become: true"},{"line_number":5,"context_line":"  ignore_errors: yes"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"- name: Set ownership of cached files"},{"line_number":8,"context_line":"  file:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7b53a9dd_0bbb5765","line":5,"in_reply_to":"d05a6a40_d1d74568","updated":"2022-03-27 07:31:12.000000000","message":"For a clean solution, we should check whether to directory exists before trying to copy things from it. ignore_errors is frowned upon by ansible-lint, but we don\u0027t lint our playbooks yet, so I\u0027m fine with this for now","commit_id":"13e8db5a6f44a8f537988a102f9869e725bb97bb"}]}