)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"f980d6545d1ac32dcc1156667dab01822951b7f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"30fff0d1_0f494689","updated":"2022-10-14 06:20:14.000000000","message":"recheck","commit_id":"82f63d1c5b45c44ed8397bd8c06eee99ff5730f4"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"5808281ed0a9de23781175ed264504b472e0d137","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ed0ae8ee_a26f184f","updated":"2022-10-16 23:39:57.000000000","message":"recheck","commit_id":"51fd74245a3e8b3cf8037176a24d729550d9e8a6"}],"playbooks/bootstrap-bridge.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"647f342513007baf7c65215a2e26d5b9bcb9201a","unresolved":true,"context_lines":[{"line_number":6,"context_line":"#    system-config-run jobs."},{"line_number":7,"context_line":"#"},{"line_number":8,"context_line":"# 2) Production -- here we actually run against the real bastion host."},{"line_number":9,"context_line":"#    The host is dynamically added in opendev/base-jobs before this"},{"line_number":10,"context_line":"#    runs, and put into a group called \"bastion\"."},{"line_number":11,"context_line":"#"},{"line_number":12,"context_line":"# In both cases, the \"bastion\" group has one entry, which is the"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"c635abf5_c24dc4a7","line":9,"updated":"2022-10-12 21:35:54.000000000","message":"I\u0027m not sure that the add_host with the group specification in base-jobs will apply here. The reason is that the base-jobs content runs in pre-run and this does not. The process boundary means the add_host doesn\u0027t make it across.\n\nThat said I think this either runs in testing where the job group specification covers this or via nested ansible in which case our nested ansible inventory takes effect. Basically I think this change is safe, but the comment may need updating.","commit_id":"eb7b148d0e98cb33b5d2a3979886d09cdb9e1ca0"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"a1451e27bc754a6f93e0b1a3c779709d831f61f0","unresolved":false,"context_lines":[{"line_number":6,"context_line":"#    system-config-run jobs."},{"line_number":7,"context_line":"#"},{"line_number":8,"context_line":"# 2) Production -- here we actually run against the real bastion host."},{"line_number":9,"context_line":"#    The host is dynamically added in opendev/base-jobs before this"},{"line_number":10,"context_line":"#    runs, and put into a group called \"bastion\"."},{"line_number":11,"context_line":"#"},{"line_number":12,"context_line":"# In both cases, the \"bastion\" group has one entry, which is the"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3d670e69_b63f6d02","line":9,"in_reply_to":"38c19a40_bfe03525","updated":"2022-10-14 02:14:22.000000000","message":"I think I\u0027ve worked through the issues in https://review.opendev.org/#/q/Iebaeed5028050d890ab541818f405978afd60124 and next revision will stack this on top.  So I think this comment is accurate in the overall stack.","commit_id":"eb7b148d0e98cb33b5d2a3979886d09cdb9e1ca0"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"d198cac7420bf3ad35f5a0fd915568bdcd8282b0","unresolved":true,"context_lines":[{"line_number":6,"context_line":"#    system-config-run jobs."},{"line_number":7,"context_line":"#"},{"line_number":8,"context_line":"# 2) Production -- here we actually run against the real bastion host."},{"line_number":9,"context_line":"#    The host is dynamically added in opendev/base-jobs before this"},{"line_number":10,"context_line":"#    runs, and put into a group called \"bastion\"."},{"line_number":11,"context_line":"#"},{"line_number":12,"context_line":"# In both cases, the \"bastion\" group has one entry, which is the"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"38c19a40_bfe03525","line":9,"in_reply_to":"c635abf5_c24dc4a7","updated":"2022-10-13 05:50:16.000000000","message":"I think I intended it to work more like the way described, but maybe got sidetracked?  We can discuss in Iebaeed5028050d890ab541818f405978afd60124","commit_id":"eb7b148d0e98cb33b5d2a3979886d09cdb9e1ca0"}]}