)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"1035e866b72ab99006cf289076154340906c2287","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"74ad1444_20e02dfa","updated":"2022-04-19 12:29:45.000000000","message":"recheck","commit_id":"39ea8939c54b3c399af6cccd640ebeddab1194e1"},{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"aa31e90ffed22f8f8fdb9254922ef280645c4663","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"427ec25b_858d8862","updated":"2022-05-06 18:57:54.000000000","message":"recheck","commit_id":"507d9d8ff7ddcceb04cc2df226b8e08f4183f93b"},{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"7bbe863a20cd4099f11fb95a60ec4e495cd75e00","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"24377e4a_10c0ed17","updated":"2022-05-11 00:11:54.000000000","message":"recheck","commit_id":"7b41bf14511e7786dbde6ac7ea96f461b6abcec9"},{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"c850c75213cce7178e32b0134f31eb88e4eb3521","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5bf01ab1_13bd504a","updated":"2022-05-10 01:32:19.000000000","message":"recheck","commit_id":"7b41bf14511e7786dbde6ac7ea96f461b6abcec9"}],"deployment/keystone/keystone-container-puppet.yaml":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"e9e66ba0ae1fb6463476a1d9c22d0504618d621d","unresolved":true,"context_lines":[{"line_number":914,"context_line":"                name: tripleo_keystone"},{"line_number":915,"context_line":"                tasks_from: keystone-db-sync.yaml"},{"line_number":916,"context_line":"              when:"},{"line_number":917,"context_line":"                - step|int \u003d\u003d 3"},{"line_number":918,"context_line":"            - name: Keystone containers"},{"line_number":919,"context_line":"              include_role:"},{"line_number":920,"context_line":"                name: tripleo_keystone"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5459aa93_61fdab20","line":917,"updated":"2022-04-06 11:21:51.000000000","message":"what about maintaining the ordering within a step?\nframework rely on this quite tightly...","commit_id":"39ea8939c54b3c399af6cccd640ebeddab1194e1"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"ab90371f5aec1616e1fcdb8d284077a999156fbb","unresolved":true,"context_lines":[{"line_number":914,"context_line":"                name: tripleo_keystone"},{"line_number":915,"context_line":"                tasks_from: keystone-db-sync.yaml"},{"line_number":916,"context_line":"              when:"},{"line_number":917,"context_line":"                - step|int \u003d\u003d 3"},{"line_number":918,"context_line":"            - name: Keystone containers"},{"line_number":919,"context_line":"              include_role:"},{"line_number":920,"context_line":"                name: tripleo_keystone"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"a95d9fa6_7f3d69da","line":917,"in_reply_to":"5459aa93_61fdab20","updated":"2022-04-06 11:23:33.000000000","message":"I\u0027m referring to start_order","commit_id":"39ea8939c54b3c399af6cccd640ebeddab1194e1"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"dec24d20bdf63dfc61fa5fd31de14c742902c9f0","unresolved":true,"context_lines":[{"line_number":914,"context_line":"                name: tripleo_keystone"},{"line_number":915,"context_line":"                tasks_from: keystone-db-sync.yaml"},{"line_number":916,"context_line":"              when:"},{"line_number":917,"context_line":"                - step|int \u003d\u003d 3"},{"line_number":918,"context_line":"            - name: Keystone containers"},{"line_number":919,"context_line":"              include_role:"},{"line_number":920,"context_line":"                name: tripleo_keystone"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"a686c4b3_c992fbd1","line":917,"in_reply_to":"642f4e3f_f745e6bc","updated":"2022-04-21 08:03:19.000000000","message":"I don\u0027t mind to have this change, just please let\u0027s figure out some testing coverage for not breaking inter-services start_order","commit_id":"39ea8939c54b3c399af6cccd640ebeddab1194e1"},{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"68d161d86f6199689069f2f637effca528ff1427","unresolved":true,"context_lines":[{"line_number":914,"context_line":"                name: tripleo_keystone"},{"line_number":915,"context_line":"                tasks_from: keystone-db-sync.yaml"},{"line_number":916,"context_line":"              when:"},{"line_number":917,"context_line":"                - step|int \u003d\u003d 3"},{"line_number":918,"context_line":"            - name: Keystone containers"},{"line_number":919,"context_line":"              include_role:"},{"line_number":920,"context_line":"                name: tripleo_keystone"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"35bc681c_573f8369","line":917,"in_reply_to":"a686c4b3_c992fbd1","updated":"2022-04-21 13:46:25.000000000","message":"ah, yea i see. that would be an issue. i\u0027ll need to find a way to handle start_order. I\u0027d like to keep that isolated to tripleo-heat-templates if possible, so we don\u0027t have that legacy debt in the new ansible roles (where it\u0027s much easier to just control ordering the roles/tasks in a playbook).","commit_id":"39ea8939c54b3c399af6cccd640ebeddab1194e1"},{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"aa33fe33a4f216a7696b25d86d891900438a8fc9","unresolved":true,"context_lines":[{"line_number":914,"context_line":"                name: tripleo_keystone"},{"line_number":915,"context_line":"                tasks_from: keystone-db-sync.yaml"},{"line_number":916,"context_line":"              when:"},{"line_number":917,"context_line":"                - step|int \u003d\u003d 3"},{"line_number":918,"context_line":"            - name: Keystone containers"},{"line_number":919,"context_line":"              include_role:"},{"line_number":920,"context_line":"                name: tripleo_keystone"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d9fc0af5_8d524ed7","line":917,"in_reply_to":"a95d9fa6_7f3d69da","updated":"2022-04-19 12:29:42.000000000","message":"the start_order is maintained b/c this is an ordered task list. they all will run when step\u003d\u003d3, but in the listed order.\n\nI don\u0027t *think* we have any inter-service dependencies relying on start_order. If we did, then yes that might be an issue, but I think we rely solely on step for inter-service stuff.","commit_id":"39ea8939c54b3c399af6cccd640ebeddab1194e1"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"b12b6010ad89929c75f7a2859f112f37459259cb","unresolved":true,"context_lines":[{"line_number":914,"context_line":"                name: tripleo_keystone"},{"line_number":915,"context_line":"                tasks_from: keystone-db-sync.yaml"},{"line_number":916,"context_line":"              when:"},{"line_number":917,"context_line":"                - step|int \u003d\u003d 3"},{"line_number":918,"context_line":"            - name: Keystone containers"},{"line_number":919,"context_line":"              include_role:"},{"line_number":920,"context_line":"                name: tripleo_keystone"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"642f4e3f_f745e6bc","line":917,"in_reply_to":"d9fc0af5_8d524ed7","updated":"2022-04-21 08:02:23.000000000","message":"Unfortunately, we have a whole ensemble of inter-services start-order symphony :)\nSee an example https://review.opendev.org/c/openstack/tripleo-heat-templates/+/826041/8//COMMIT_MSG","commit_id":"39ea8939c54b3c399af6cccd640ebeddab1194e1"}]}