)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"faa91389736ccfd13ae7c090a25e836133f729e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e471381a_4e867401","updated":"2024-01-31 11:28:03.000000000","message":"I think the deprecation warning is not getting displayed. I will amend the change.","commit_id":"4326762506ecfe5dfea3fffa4093166b1d74f19e"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"bc980ff94ea6551f9238891ed883def79870c3b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d7bf62f0_c0ffb413","updated":"2024-01-31 14:47:30.000000000","message":"Good catch, I see the message now, e.g: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_a60/907064/5/check/kayobe-overcloud-host-configure-rocky9/a601bb2/primary/ansible/overcloud-deploy","commit_id":"f018e0a8ba97a998fad977d16cb1ea3fbb4e8335"}],"ansible/docker-devicemapper.yml":[{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"42cc83e3bbbd915358e1b1efe99dd2942cb35805","unresolved":true,"context_lines":[{"line_number":15,"context_line":"      # NOTE(priteau): We want this to print a nice big red warning and not to"},{"line_number":16,"context_line":"      # fail the run."},{"line_number":17,"context_line":"      ignore_errors: yes"},{"line_number":18,"context_line":"      run_once: true"},{"line_number":19,"context_line":"      when: docker_storage_driver \u003d\u003d \u0027devicemapper\u0027"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    - name: Check for existing Docker configuration using devicemapper"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"d3a48883_61614e2e","line":18,"updated":"2024-02-01 09:06:21.000000000","message":"I guess we actually want to run this against every host? In case, there is a mix of storage drivers?","commit_id":"f018e0a8ba97a998fad977d16cb1ea3fbb4e8335"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"5586203d837d1a3dd99b5c79977c5f8cfb8a8eba","unresolved":false,"context_lines":[{"line_number":15,"context_line":"      # NOTE(priteau): We want this to print a nice big red warning and not to"},{"line_number":16,"context_line":"      # fail the run."},{"line_number":17,"context_line":"      ignore_errors: yes"},{"line_number":18,"context_line":"      run_once: true"},{"line_number":19,"context_line":"      when: docker_storage_driver \u003d\u003d \u0027devicemapper\u0027"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    - name: Check for existing Docker configuration using devicemapper"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"2f1fc55b_5b3dfd11","line":18,"in_reply_to":"7a9522b1_e92bb110","updated":"2024-02-01 12:13:00.000000000","message":"I had not considered this scenario, thanks.\n\nAlso I thought run_once would still work because it is inside a task controlled with \"when:\", but it appears to take priority over it, so if the first host is overlay2 we would just get:\n\nskipping: [host]","commit_id":"f018e0a8ba97a998fad977d16cb1ea3fbb4e8335"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"43a67f10faebd4d9aaaee10ca2b90447f6cd4c86","unresolved":false,"context_lines":[{"line_number":15,"context_line":"      # NOTE(priteau): We want this to print a nice big red warning and not to"},{"line_number":16,"context_line":"      # fail the run."},{"line_number":17,"context_line":"      ignore_errors: yes"},{"line_number":18,"context_line":"      run_once: true"},{"line_number":19,"context_line":"      when: docker_storage_driver \u003d\u003d \u0027devicemapper\u0027"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    - name: Check for existing Docker configuration using devicemapper"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7a9522b1_e92bb110","line":18,"in_reply_to":"d3a48883_61614e2e","updated":"2024-02-01 09:33:11.000000000","message":"Done","commit_id":"f018e0a8ba97a998fad977d16cb1ea3fbb4e8335"}]}