)]}'
{"playbooks/oslo-config-validator.yaml":[{"author":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"change_message_id":"0e772299db0f55597ad3e9019687c1f2e1905c42","unresolved":true,"context_lines":[{"line_number":17,"context_line":"        - post-deployment"},{"line_number":18,"context_line":"        - post-system-upgrade"},{"line_number":19,"context_line":"        - post-update"},{"line_number":20,"context_line":"        - post-upgrade"},{"line_number":21,"context_line":"    oslo_config_validator_debug: false"},{"line_number":22,"context_line":"    oslo_config_validator_report: false"},{"line_number":23,"context_line":"    oslo_config_validator_validation: true"}],"source_content_type":"text/x-yaml","patch_set":21,"id":"4a8037f6_cdf0f925","line":20,"updated":"2021-06-02 20:25:10.000000000","message":"Did you plan to remove the update/upgrade groups?","commit_id":"fa456e1031b395cadd1ecc3bb97dd1ad219b65aa"}],"roles/oslo_config_validator/README.md":[{"author":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"change_message_id":"25a1158f0063dfe85f2bce3b9cb511bdc5ef564f","unresolved":true,"context_lines":[{"line_number":24,"context_line":"- oslo_config_validator_global_ignored_messages: List of regular expressions that will filter out messages globally, across all namespaces"},{"line_number":25,"context_line":"- oslo_config_validator_namespaces_config: Specific namespace configurations. It contains namespace-specific ignored patterns as well as invalid settings configuration."},{"line_number":26,"context_line":"- oslo_config_validator_service_configs: Mapping of known Openstack services with their namespace configuration."},{"line_number":27,"context_line":"- oslo_config_validator_checked_services: List of services being validated."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Dependencies"},{"line_number":30,"context_line":"------------"}],"source_content_type":"text/x-gfm","patch_set":16,"id":"0756f4d9_e49cd0b9","line":27,"updated":"2021-05-31 19:51:42.000000000","message":"s/Wether/Whether/g 😄","commit_id":"f45d15d8f487bec24b9cdda5dfa907dd956c8a03"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"dc392cc76fd81023862e493cc0c66923fef2f18a","unresolved":false,"context_lines":[{"line_number":24,"context_line":"- oslo_config_validator_global_ignored_messages: List of regular expressions that will filter out messages globally, across all namespaces"},{"line_number":25,"context_line":"- oslo_config_validator_namespaces_config: Specific namespace configurations. It contains namespace-specific ignored patterns as well as invalid settings configuration."},{"line_number":26,"context_line":"- oslo_config_validator_service_configs: Mapping of known Openstack services with their namespace configuration."},{"line_number":27,"context_line":"- oslo_config_validator_checked_services: List of services being validated."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Dependencies"},{"line_number":30,"context_line":"------------"}],"source_content_type":"text/x-gfm","patch_set":16,"id":"878cccaf_4323639c","line":27,"in_reply_to":"0756f4d9_e49cd0b9","updated":"2021-05-31 22:02:33.000000000","message":"Done","commit_id":"f45d15d8f487bec24b9cdda5dfa907dd956c8a03"}],"roles/oslo_config_validator/defaults/main.yml":[{"author":{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"},"change_message_id":"4c796fa09eb3324f0a29ec9d2eebb40b2eb90efb","unresolved":true,"context_lines":[{"line_number":52,"context_line":"oslo_config_validator_invalid_settings: true"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"# These messages are globbally ignored and will not trigger a validation failure"},{"line_number":56,"context_line":"oslo_config_validator_global_ignored_messages:"},{"line_number":57,"context_line":"  - \u0027INFO:keyring.backend:Loading.*\u0027"},{"line_number":58,"context_line":"  - \u0027WARNING:oslo_config.generator:normalizing group name.*\u0027"}],"source_content_type":"text/x-yaml","patch_set":25,"id":"45492886_c8714906","line":55,"updated":"2021-06-04 17:29:39.000000000","message":"nit - typo on globally","commit_id":"c6c2d2a5fc2b4f0830aec06c440d8c8ed4019900"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"64810d51d341a32427cc5ae321a4ac0c3be75064","unresolved":false,"context_lines":[{"line_number":52,"context_line":"oslo_config_validator_invalid_settings: true"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"# These messages are globbally ignored and will not trigger a validation failure"},{"line_number":56,"context_line":"oslo_config_validator_global_ignored_messages:"},{"line_number":57,"context_line":"  - \u0027INFO:keyring.backend:Loading.*\u0027"},{"line_number":58,"context_line":"  - \u0027WARNING:oslo_config.generator:normalizing group name.*\u0027"}],"source_content_type":"text/x-yaml","patch_set":25,"id":"35eeb1ef_c119da30","line":55,"in_reply_to":"45492886_c8714906","updated":"2021-06-04 19:48:03.000000000","message":"Done","commit_id":"c6c2d2a5fc2b4f0830aec06c440d8c8ed4019900"},{"author":{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"},"change_message_id":"4c796fa09eb3324f0a29ec9d2eebb40b2eb90efb","unresolved":true,"context_lines":[{"line_number":78,"context_line":"#                   - not: current value should not match exactly any element in value_list"},{"line_number":79,"context_line":"#                   - lt: current value should be lesser than first element of value_list"},{"line_number":80,"context_line":"#                   - gt: current value should be greater than first element of value_list"},{"line_number":81,"context_line":"#                   - eq: current value, if defined, should be equal to one of the element in value_list"},{"line_number":82,"context_line":"#"},{"line_number":83,"context_line":"oslo_config_validator_namespaces_config:"},{"line_number":84,"context_line":"  - namespace: glance.store"}],"source_content_type":"text/x-yaml","patch_set":25,"id":"4cbce4d6_783d33cc","line":81,"updated":"2021-06-04 17:29:39.000000000","message":"nit - wording suggestion: \"should be equal to one element in value_list\"","commit_id":"c6c2d2a5fc2b4f0830aec06c440d8c8ed4019900"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"64810d51d341a32427cc5ae321a4ac0c3be75064","unresolved":false,"context_lines":[{"line_number":78,"context_line":"#                   - not: current value should not match exactly any element in value_list"},{"line_number":79,"context_line":"#                   - lt: current value should be lesser than first element of value_list"},{"line_number":80,"context_line":"#                   - gt: current value should be greater than first element of value_list"},{"line_number":81,"context_line":"#                   - eq: current value, if defined, should be equal to one of the element in value_list"},{"line_number":82,"context_line":"#"},{"line_number":83,"context_line":"oslo_config_validator_namespaces_config:"},{"line_number":84,"context_line":"  - namespace: glance.store"}],"source_content_type":"text/x-yaml","patch_set":25,"id":"3fa8b1ac_cbd86705","line":81,"in_reply_to":"4cbce4d6_783d33cc","updated":"2021-06-04 19:48:03.000000000","message":"Done","commit_id":"c6c2d2a5fc2b4f0830aec06c440d8c8ed4019900"},{"author":{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"},"change_message_id":"4c796fa09eb3324f0a29ec9d2eebb40b2eb90efb","unresolved":true,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":"  - namespace: heat.common.config"},{"line_number":91,"context_line":"    ignored_messages:"},{"line_number":92,"context_line":"      # NOTE(dvd): openstack/heat fix: https://review.opendev.org/789680"},{"line_number":93,"context_line":"      - \u0027ERROR:root:(yaql|(resource_finder_)*cache)/[^\\s]+ {{ invalid_setting_regex }}\u0027"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"  - namespace: ironic"}],"source_content_type":"text/x-yaml","patch_set":25,"id":"2af20f2d_d3af835c","line":92,"updated":"2021-06-04 17:29:39.000000000","message":"these comments should likely have some more context for the uninitiated","commit_id":"c6c2d2a5fc2b4f0830aec06c440d8c8ed4019900"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"64810d51d341a32427cc5ae321a4ac0c3be75064","unresolved":false,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":"  - namespace: heat.common.config"},{"line_number":91,"context_line":"    ignored_messages:"},{"line_number":92,"context_line":"      # NOTE(dvd): openstack/heat fix: https://review.opendev.org/789680"},{"line_number":93,"context_line":"      - \u0027ERROR:root:(yaql|(resource_finder_)*cache)/[^\\s]+ {{ invalid_setting_regex }}\u0027"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"  - namespace: ironic"}],"source_content_type":"text/x-yaml","patch_set":25,"id":"b5cf7dec_d0a01856","line":92,"in_reply_to":"2af20f2d_d3af835c","updated":"2021-06-04 19:48:03.000000000","message":"Done","commit_id":"c6c2d2a5fc2b4f0830aec06c440d8c8ed4019900"}],"roles/oslo_config_validator/resources/library/podman_container_info.py":[{"author":{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"},"change_message_id":"4c796fa09eb3324f0a29ec9d2eebb40b2eb90efb","unresolved":true,"context_lines":[{"line_number":49,"context_line":"      }"},{"line_number":50,"context_line":"\"\"\""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"from ansible.module_utils.basic import AnsibleModule  # noqa: F402"},{"line_number":53,"context_line":"from yaml import safe_load as yaml_safe_load"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":""}],"source_content_type":"text/x-python","patch_set":25,"id":"43dda620_6932ea9e","line":52,"updated":"2021-06-04 17:29:39.000000000","message":"What does noqa F402 mean?","commit_id":"c6c2d2a5fc2b4f0830aec06c440d8c8ed4019900"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"64810d51d341a32427cc5ae321a4ac0c3be75064","unresolved":true,"context_lines":[{"line_number":49,"context_line":"      }"},{"line_number":50,"context_line":"\"\"\""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"from ansible.module_utils.basic import AnsibleModule  # noqa: F402"},{"line_number":53,"context_line":"from yaml import safe_load as yaml_safe_load"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":""}],"source_content_type":"text/x-python","patch_set":25,"id":"a965891f_867ee43c","line":52,"in_reply_to":"43dda620_6932ea9e","updated":"2021-06-04 19:48:03.000000000","message":"This came from podman_container_info.py (which was recently removed) [1]\n\n    import module from line N shadowed by loop variable\n\nWe still need this script to mock podman data.\n\n[1] https://github.com/openstack/tripleo-ansible/commit/7a2a75267cbef6661ba453732e9dbfd22934cf36#diff-0c51384bf8b0798a30cea3a44b329be7ad8b5456937d1c2b19c96c8f71d8086fL329","commit_id":"c6c2d2a5fc2b4f0830aec06c440d8c8ed4019900"}],"roles/oslo_config_validator/tasks/build_validation_config.yml":[{"author":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"change_message_id":"25a1158f0063dfe85f2bce3b9cb511bdc5ef564f","unresolved":true,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"- name: Basic usage"},{"line_number":48,"context_line":"  debug:"},{"line_number":49,"context_line":"    msg: \"\u0027{{ lookup(\u0027env\u0027, \u0027ANSIBLE_LIBRARY\u0027) }}\u0027 is the HOME environment variable.\""},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"- name: Gather facts for all containers"},{"line_number":52,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"37a0946a_80fec036","line":49,"updated":"2021-05-31 19:51:42.000000000","message":"Should be conditioned.\n  \n  when:\n    - oslo_config_validator_debug | bool","commit_id":"a942855f38872c6446f48fc881518669c95d3288"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"dc392cc76fd81023862e493cc0c66923fef2f18a","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"- name: Basic usage"},{"line_number":48,"context_line":"  debug:"},{"line_number":49,"context_line":"    msg: \"\u0027{{ lookup(\u0027env\u0027, \u0027ANSIBLE_LIBRARY\u0027) }}\u0027 is the HOME environment variable.\""},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"- name: Gather facts for all containers"},{"line_number":52,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"e2749dcc_53e64b5c","line":49,"in_reply_to":"37a0946a_80fec036","updated":"2021-05-31 22:02:33.000000000","message":"this shouldn\u0027t even be there, removing.","commit_id":"a942855f38872c6446f48fc881518669c95d3288"}],"roles/oslo_config_validator/tasks/container_run.yml":[{"author":{"_account_id":32926,"name":"Jiri Podivin","display_name":"jpodivin","email":"jpodivin@redhat.com","username":"jpodivin"},"change_message_id":"2f98209420ecf17ee07349f9beebfb99d284f4d2","unresolved":true,"context_lines":[{"line_number":19,"context_line":"# this shouldn\u0027t be necessary once this proposed change has landed"},{"line_number":20,"context_line":"# and propagated in images"},{"line_number":21,"context_line":"# https://review.opendev.org/c/openstack/oslo.config/+/790883"},{"line_number":22,"context_line":"- name: Applying the most dirty patch in tripleo-validation history"},{"line_number":23,"context_line":"  when:"},{"line_number":24,"context_line":"    - \"\u0027oslo-config-generator\u0027 in oslo_command\""},{"line_number":25,"context_line":"  vars:"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"ebd89765_07098f19","line":22,"updated":"2021-05-31 15:01:39.000000000","message":"Notification about this temporary construct should be raised at some point. Ideally as a debug message. Some info should also be included in docs, possibly in a `warning` clause. Finally a commit msg should mention the workaround.","commit_id":"6884f12c96732cf86181e79eb238429bbda49dfc"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"336a2f552c807c6f80a08ca0f3f49ef525ff2012","unresolved":false,"context_lines":[{"line_number":19,"context_line":"# this shouldn\u0027t be necessary once this proposed change has landed"},{"line_number":20,"context_line":"# and propagated in images"},{"line_number":21,"context_line":"# https://review.opendev.org/c/openstack/oslo.config/+/790883"},{"line_number":22,"context_line":"- name: Applying the most dirty patch in tripleo-validation history"},{"line_number":23,"context_line":"  when:"},{"line_number":24,"context_line":"    - \"\u0027oslo-config-generator\u0027 in oslo_command\""},{"line_number":25,"context_line":"  vars:"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"83643c8b_55740382","line":22,"in_reply_to":"ebd89765_07098f19","updated":"2021-05-31 15:21:38.000000000","message":"You\u0027re totally right, Added to commit and added a whole \"Exceptions\" section to the README.","commit_id":"6884f12c96732cf86181e79eb238429bbda49dfc"}],"roles/oslo_config_validator/tasks/main.yml":[{"author":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"change_message_id":"25a1158f0063dfe85f2bce3b9cb511bdc5ef564f","unresolved":true,"context_lines":[{"line_number":24,"context_line":"  file:"},{"line_number":25,"context_line":"    mode: 0777"},{"line_number":26,"context_line":"    path: \"{{ oslo_config_validator_work_path }}\""},{"line_number":27,"context_line":"    state: directory"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"- name: Validate configuration"},{"line_number":30,"context_line":"  include_tasks: validate_config.yml"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"88160186_f6a66164","line":27,"updated":"2021-05-31 19:51:42.000000000","message":"Please use:\n\n   changed_when: false","commit_id":"a942855f38872c6446f48fc881518669c95d3288"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"dc392cc76fd81023862e493cc0c66923fef2f18a","unresolved":false,"context_lines":[{"line_number":24,"context_line":"  file:"},{"line_number":25,"context_line":"    mode: 0777"},{"line_number":26,"context_line":"    path: \"{{ oslo_config_validator_work_path }}\""},{"line_number":27,"context_line":"    state: directory"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"- name: Validate configuration"},{"line_number":30,"context_line":"  include_tasks: validate_config.yml"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"cf7f28a8_8575ebe3","line":27,"in_reply_to":"88160186_f6a66164","updated":"2021-05-31 22:02:33.000000000","message":"Done","commit_id":"a942855f38872c6446f48fc881518669c95d3288"},{"author":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"change_message_id":"25a1158f0063dfe85f2bce3b9cb511bdc5ef564f","unresolved":true,"context_lines":[{"line_number":37,"context_line":"  become: true"},{"line_number":38,"context_line":"  file:"},{"line_number":39,"context_line":"    path: \"{{ oslo_config_validator_work_path }}\""},{"line_number":40,"context_line":"    state: absent"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"- name: Preparing and filtering output datastructure"},{"line_number":43,"context_line":"  when:"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"dee447bd_56307d9e","line":40,"updated":"2021-05-31 19:51:42.000000000","message":"Please use:\n\n   changed_when: false","commit_id":"a942855f38872c6446f48fc881518669c95d3288"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"dc392cc76fd81023862e493cc0c66923fef2f18a","unresolved":false,"context_lines":[{"line_number":37,"context_line":"  become: true"},{"line_number":38,"context_line":"  file:"},{"line_number":39,"context_line":"    path: \"{{ oslo_config_validator_work_path }}\""},{"line_number":40,"context_line":"    state: absent"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"- name: Preparing and filtering output datastructure"},{"line_number":43,"context_line":"  when:"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"79620f87_36a736ba","line":40,"in_reply_to":"dee447bd_56307d9e","updated":"2021-05-31 22:02:33.000000000","message":"Done","commit_id":"a942855f38872c6446f48fc881518669c95d3288"},{"author":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"change_message_id":"25a1158f0063dfe85f2bce3b9cb511bdc5ef564f","unresolved":true,"context_lines":[{"line_number":85,"context_line":"      file:"},{"line_number":86,"context_line":"        path: \"{{ local_folder_name }}\""},{"line_number":87,"context_line":"        state: directory"},{"line_number":88,"context_line":"        recurse: true"},{"line_number":89,"context_line":"      delegate_to: localhost"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    - name: Saving filtered outputs to files"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"3bfbce82_547054fb","line":88,"updated":"2021-05-31 19:51:42.000000000","message":"Please use:\n\n   changed_when: false","commit_id":"a942855f38872c6446f48fc881518669c95d3288"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"dc392cc76fd81023862e493cc0c66923fef2f18a","unresolved":false,"context_lines":[{"line_number":85,"context_line":"      file:"},{"line_number":86,"context_line":"        path: \"{{ local_folder_name }}\""},{"line_number":87,"context_line":"        state: directory"},{"line_number":88,"context_line":"        recurse: true"},{"line_number":89,"context_line":"      delegate_to: localhost"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    - name: Saving filtered outputs to files"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"7693e08e_f4344b50","line":88,"in_reply_to":"3bfbce82_547054fb","updated":"2021-05-31 22:02:33.000000000","message":"Done","commit_id":"a942855f38872c6446f48fc881518669c95d3288"},{"author":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"change_message_id":"25a1158f0063dfe85f2bce3b9cb511bdc5ef564f","unresolved":true,"context_lines":[{"line_number":112,"context_line":"    - name: Delete local files"},{"line_number":113,"context_line":"      file:"},{"line_number":114,"context_line":"        path: \"{{ oslo_config_validator_report_path }}\""},{"line_number":115,"context_line":"        state: absent"},{"line_number":116,"context_line":"      run_once: true"},{"line_number":117,"context_line":"      delegate_to: localhost"},{"line_number":118,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":20,"id":"a2d213ee_f7fb078d","line":115,"updated":"2021-05-31 19:51:42.000000000","message":"Please use:\n\n   changed_when: false\n\nWell wherever you create files/dir and delete something, please don\u0027t report the \u0027changed\u0027 status. Validations should be as non-intrusive as possible.","commit_id":"a942855f38872c6446f48fc881518669c95d3288"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"dc392cc76fd81023862e493cc0c66923fef2f18a","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    - name: Delete local files"},{"line_number":113,"context_line":"      file:"},{"line_number":114,"context_line":"        path: \"{{ oslo_config_validator_report_path }}\""},{"line_number":115,"context_line":"        state: absent"},{"line_number":116,"context_line":"      run_once: true"},{"line_number":117,"context_line":"      delegate_to: localhost"},{"line_number":118,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":20,"id":"fda52388_da0f46cc","line":115,"in_reply_to":"a2d213ee_f7fb078d","updated":"2021-05-31 22:02:33.000000000","message":"Fair enough, I tried to remove everything I could whenever I could.","commit_id":"a942855f38872c6446f48fc881518669c95d3288"},{"author":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"change_message_id":"25a1158f0063dfe85f2bce3b9cb511bdc5ef564f","unresolved":true,"context_lines":[{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    - name: Report path"},{"line_number":120,"context_line":"      run_once: true"},{"line_number":121,"context_line":"      debug:"},{"line_number":122,"context_line":"        msg: |-"},{"line_number":123,"context_line":"          Reports are available here: {{ oslo_config_validator_report_path }}.tar.xz"},{"line_number":124,"context_line":"          {{ archive_out.archived.keys()| default([]) }}"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"d31c33d6_901b8eef","line":121,"updated":"2021-05-31 19:51:42.000000000","message":"You should use the \u0027warn\u0027 custom module (from validations-common) instead of \u0027debug\u0027 here. Debug message will be lost in the middle of the logs while the warn message will be highlighted in the validation_output dict of the log file.","commit_id":"a942855f38872c6446f48fc881518669c95d3288"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"dc392cc76fd81023862e493cc0c66923fef2f18a","unresolved":false,"context_lines":[{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    - name: Report path"},{"line_number":120,"context_line":"      run_once: true"},{"line_number":121,"context_line":"      debug:"},{"line_number":122,"context_line":"        msg: |-"},{"line_number":123,"context_line":"          Reports are available here: {{ oslo_config_validator_report_path }}.tar.xz"},{"line_number":124,"context_line":"          {{ archive_out.archived.keys()| default([]) }}"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"1d417c70_dbd52803","line":121,"in_reply_to":"d31c33d6_901b8eef","updated":"2021-05-31 22:02:33.000000000","message":"Done","commit_id":"a942855f38872c6446f48fc881518669c95d3288"},{"author":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"change_message_id":"25a1158f0063dfe85f2bce3b9cb511bdc5ef564f","unresolved":true,"context_lines":[{"line_number":121,"context_line":"      debug:"},{"line_number":122,"context_line":"        msg: |-"},{"line_number":123,"context_line":"          Reports are available here: {{ oslo_config_validator_report_path }}.tar.xz"},{"line_number":124,"context_line":"          {{ archive_out.archived.keys()| default([]) }}"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"    - name: Role terminated after reports are ready."},{"line_number":127,"context_line":"      meta: end_play"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"1abaaf09_5a1d6514","line":124,"updated":"2021-05-31 19:51:42.000000000","message":"When running it with report creation set to true, I\u0027ve got:\n\n    The error was: \u0027list object\u0027 has no attribute \u0027keys\u0027\n\nSee https://seashells.io/v/ctUh4cTz\n\nI think it should be:\n\n    {{ archived_out.archived | list | default([]) }}","commit_id":"a942855f38872c6446f48fc881518669c95d3288"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"dc392cc76fd81023862e493cc0c66923fef2f18a","unresolved":true,"context_lines":[{"line_number":121,"context_line":"      debug:"},{"line_number":122,"context_line":"        msg: |-"},{"line_number":123,"context_line":"          Reports are available here: {{ oslo_config_validator_report_path }}.tar.xz"},{"line_number":124,"context_line":"          {{ archive_out.archived.keys()| default([]) }}"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"    - name: Role terminated after reports are ready."},{"line_number":127,"context_line":"      meta: end_play"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"1d589c2d_08f9e4f7","line":124,"in_reply_to":"1abaaf09_5a1d6514","updated":"2021-05-31 22:02:33.000000000","message":"I just killed my lab this morning but I\u0027ll try that soon.","commit_id":"a942855f38872c6446f48fc881518669c95d3288"}]}