)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"39c8208c88edc7a29d6f9ece15d10ecebdfcc626","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Dmitriy Rabotyagov \u003cnoonedeadpunk@ya.ru\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-09-03 09:33:26 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Define condition for \"run_once\" one time"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We use the same condition, which defines against what host some \"service\""},{"line_number":10,"context_line":"tasks should run against, several times. It\u0027s hard to keep it the same"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f560f44_ae3938f3","line":7,"range":{"start_line":7,"start_character":22,"end_line":7,"end_character":30},"updated":"2020-09-03 07:23:54.000000000","message":"this is a bit confusing because i looked through the patch expecting to see actual ansible \u0027run_once\u0027, which of course we don\u0027t use.\n\n\"Single definition for \u0027one time\u0027 service setup conditional\"","commit_id":"bc99b632d023a56b8d8d8c1ebe8f17216f49ac6e"},{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"39c8208c88edc7a29d6f9ece15d10ecebdfcc626","unresolved":false,"context_lines":[{"line_number":10,"context_line":"tasks should run against, several times. It\u0027s hard to keep it the same"},{"line_number":11,"context_line":"across the role and ansible spending additional resources to evaluate"},{"line_number":12,"context_line":"it each time, so it\u0027s simpler and better for the maintenance to set"},{"line_number":13,"context_line":"a boolean variable which will say for all tasks if they should run or not"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Iac06d3f02b1c9ee5e3bfbd28043fbb70d8b1d328"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f560f44_4e1b3c7f","line":13,"range":{"start_line":13,"start_character":38,"end_line":13,"end_character":47},"updated":"2020-09-03 07:23:54.000000000","message":"all service setup tasks","commit_id":"bc99b632d023a56b8d8d8c1ebe8f17216f49ac6e"}],"vars/main.yml":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"39c8208c88edc7a29d6f9ece15d10ecebdfcc626","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"keystone_core_files: []"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"_keystone_single_run_condition: \"{{ (keystone_services[\u0027keystone-wsgi-public\u0027][\u0027group\u0027] in group_names and inventory_hostname \u003d\u003d ((groups[keystone_services[\u0027keystone-wsgi-public\u0027][\u0027group\u0027]] | intersect(ansible_play_hosts)) | list)[0]) | bool }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f560f44_ae12d860","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":30},"updated":"2020-09-03 07:23:54.000000000","message":"nit _keystone_service_setup_run_condition ?\n\nThats a bit more descriptive of what the var is for?","commit_id":"bc99b632d023a56b8d8d8c1ebe8f17216f49ac6e"}]}