)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"41376380968b4fdb6a33efb3a6826b2f2fc4c8ec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c662f787_0baccd37","updated":"2021-11-25 09:33:07.000000000","message":"Could you add a couple of lines in doc/source/reference/compute/nova-guide.rst?","commit_id":"a10385bd82f73c932abed4760fa40b470470f8d9"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"827ebe12e717763cfe6ef2afef17a93610a18471","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fc588ad4_589c2550","updated":"2021-11-25 11:57:57.000000000","message":"Hopefully everyone is happy now!","commit_id":"06155ef44bc36e315aacc09235319ee9b5cdf371"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"1d49d3159de55a450f070eec4851360295b82752","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3f9a43a4_87a8584b","updated":"2021-11-25 12:04:14.000000000","message":"Thanks!","commit_id":"06155ef44bc36e315aacc09235319ee9b5cdf371"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"a9b6c7655dca44beaf181277124dba7ae1db4d1b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c73dd204_656c8ecf","updated":"2021-11-25 14:56:38.000000000","message":"I have restored the set_fact so the linter doesn\u0027t complain about `vendordata_file` not being defined in the template. Unless anyone else has a better suggestion than adding more `is defined`s in the `if` condition especially for the linter.","commit_id":"8645b401472628ddb3f1f314379c432241110a72"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6b7905ce378397f1dfc0447b6fe5882d05b3c292","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3a1b6bcf_e6ca8ea9","in_reply_to":"c73dd204_656c8ecf","updated":"2021-11-26 09:05:46.000000000","message":"There is a way to pass in variables in the validator, but it\u0027s probably not worth it.","commit_id":"8645b401472628ddb3f1f314379c432241110a72"}],"ansible/roles/nova/tasks/config.yml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"a363705496b05f1bbc8906425b0160001d4b961d","unresolved":true,"context_lines":[{"line_number":38,"context_line":"  run_once: True"},{"line_number":39,"context_line":"  register: vendordata_file"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"- name: Set vendordata file path"},{"line_number":42,"context_line":"  set_fact:"},{"line_number":43,"context_line":"    vendordata_file_path: \"{{ vendordata_file.stat.path }}\""},{"line_number":44,"context_line":"  when:"},{"line_number":45,"context_line":"    - vendordata_file.stat.exists"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"- include_tasks: copy-certs.yml"},{"line_number":48,"context_line":"  when:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"c7c9fedb_bd99256c","line":45,"range":{"start_line":41,"start_character":0,"end_line":45,"end_character":33},"updated":"2021-11-24 16:17:38.000000000","message":"can\u0027t we just use it as a var in \"Copying over vendordata file\" task?","commit_id":"a10385bd82f73c932abed4760fa40b470470f8d9"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"41376380968b4fdb6a33efb3a6826b2f2fc4c8ec","unresolved":true,"context_lines":[{"line_number":38,"context_line":"  run_once: True"},{"line_number":39,"context_line":"  register: vendordata_file"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"- name: Set vendordata file path"},{"line_number":42,"context_line":"  set_fact:"},{"line_number":43,"context_line":"    vendordata_file_path: \"{{ vendordata_file.stat.path }}\""},{"line_number":44,"context_line":"  when:"},{"line_number":45,"context_line":"    - vendordata_file.stat.exists"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"- include_tasks: copy-certs.yml"},{"line_number":48,"context_line":"  when:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"026787ce_238a0bbe","line":45,"range":{"start_line":41,"start_character":0,"end_line":45,"end_character":33},"in_reply_to":"c7c9fedb_bd99256c","updated":"2021-11-25 09:33:07.000000000","message":"I\u0027m ok either way - we do use this pattern for policy (although that is a bit more complicated). At least it\u0027s only on controllers, so won\u0027t impact performance hugely.","commit_id":"a10385bd82f73c932abed4760fa40b470470f8d9"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f09342fe5b501b6de82500970afe741dd47a5d44","unresolved":true,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"- name: Check for vendordata file"},{"line_number":35,"context_line":"  stat:"},{"line_number":36,"context_line":"    path: \"{{ node_custom_config }}/nova/vendordata.json\""},{"line_number":37,"context_line":"  delegate_to: localhost"},{"line_number":38,"context_line":"  run_once: True"},{"line_number":39,"context_line":"  register: vendordata_file"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"75399b36_2db75cbf","line":36,"updated":"2021-12-22 19:13:02.000000000","message":"I guess it would be nice to support this per host or even hostgroup","commit_id":"9af42fcebc785bf8a792ce61c8e88cce55837cb7"}]}