)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"acbed9977981189defbe5365e14d62c9494fa116","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":31,"id":"7135e43b_c14f10fe","updated":"2022-09-02 07:53:04.000000000","message":"recheck","commit_id":"35b1dbee4fb865b427c10e98186d087fc1328e96"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"55fa038252db6cc38ba33ca4990442cc5c82cdb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"33b695e2_410f7549","updated":"2022-09-07 00:45:51.000000000","message":"recheck","commit_id":"c286f870cc46c0a89ff82b0ebfe9f5b6db9fe544"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"631dd8834a20254674ee3d17acc4c881c5b07fe4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":33,"id":"047d590b_5c1e458e","updated":"2022-09-08 06:08:21.000000000","message":"recheck","commit_id":"34543b8ac55348bc7d55f465231d6f9ca7aded3d"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"b3349b89a0dd56bc81ec6a7a63b41afe8e86f337","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":33,"id":"bfdac46d_f4a3cc47","updated":"2022-09-08 03:00:24.000000000","message":"recheck","commit_id":"34543b8ac55348bc7d55f465231d6f9ca7aded3d"}],"playbooks/zuul-stream/create-inventory.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"3bfcef7ffee08b45c68c1ba12a279b2e455d661f","unresolved":true,"context_lines":[{"line_number":11,"context_line":"- name: Extract inventory"},{"line_number":12,"context_line":"  set_fact:"},{"line_number":13,"context_line":"    _new_inventory: \"{{ _inventory_yaml[\u0027content\u0027] | b64decode | from_yaml }}\""},{"line_number":14,"context_line":"  register: _inventory"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: Setup new facts"},{"line_number":17,"context_line":"  set_fact:"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"804341f4_537f292c","line":14,"updated":"2022-08-18 22:23:57.000000000","message":"Is this register necessary? It doesn\u0027t seem to be used later.","commit_id":"b72539cf8612399c77eee6b4b60f88f203b69f8b"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"dc4ba6371f8951c4874f82c41fab8beefbd927a1","unresolved":false,"context_lines":[{"line_number":11,"context_line":"- name: Extract inventory"},{"line_number":12,"context_line":"  set_fact:"},{"line_number":13,"context_line":"    _new_inventory: \"{{ _inventory_yaml[\u0027content\u0027] | b64decode | from_yaml }}\""},{"line_number":14,"context_line":"  register: _inventory"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: Setup new facts"},{"line_number":17,"context_line":"  set_fact:"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"9e03dffe_78a8a37a","line":14,"in_reply_to":"804341f4_537f292c","updated":"2022-08-18 23:53:20.000000000","message":"Yeah not necessary; I was using it to debug.  Will remove for clarity","commit_id":"b72539cf8612399c77eee6b4b60f88f203b69f8b"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"5c8cdf61a5a859b21359bae6f76a33d800e0b45e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"- name: Extract inventory"},{"line_number":12,"context_line":"  set_fact:"},{"line_number":13,"context_line":"    _new_inventory: \"{{ _inventory_yaml[\u0027content\u0027] | b64decode | from_yaml }}\""},{"line_number":14,"context_line":"  register: _inventory"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: Setup new facts"},{"line_number":17,"context_line":"  set_fact:"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"54f35726_98684a36","line":14,"in_reply_to":"9e03dffe_78a8a37a","updated":"2022-09-06 20:40:09.000000000","message":"actually done in ps32, thanks 😊","commit_id":"b72539cf8612399c77eee6b4b60f88f203b69f8b"}],"zuul/ansible/base/callback/zuul_stream.py":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"35b0187e24cd0bfb43ef810526c4c3f46c07f21b","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"LOG_STREAM_PORT \u003d int(os.environ.get(\"ZUUL_CONSOLE_PORT\", 19885))"},{"line_number":51,"context_line":"LOG_STREAM_VERSION \u003d 0"},{"line_number":52,"context_line":"#"},{"line_number":53,"context_line":"LOG_STREAM_LOCALHOST \u003d int(os.environ.get(\"ZUUL_CONSOLE_STREAM_LOCALHOST\", 0))"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":""}],"source_content_type":"text/x-python","patch_set":32,"id":"ecfd9471_aaaeaa3b","line":52,"updated":"2022-09-06 22:19:45.000000000","message":"-1ish: It might be worth putting a comment here indicating that this is only used for testing.  The whole \"we don\u0027t stream on localhost\" is already a bit counter-intuitive.","commit_id":"c286f870cc46c0a89ff82b0ebfe9f5b6db9fe544"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"acf3c4213cc6c03cec50fee8251774fa6ac08478","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"LOG_STREAM_PORT \u003d int(os.environ.get(\"ZUUL_CONSOLE_PORT\", 19885))"},{"line_number":51,"context_line":"LOG_STREAM_VERSION \u003d 0"},{"line_number":52,"context_line":"#"},{"line_number":53,"context_line":"LOG_STREAM_LOCALHOST \u003d int(os.environ.get(\"ZUUL_CONSOLE_STREAM_LOCALHOST\", 0))"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":""}],"source_content_type":"text/x-python","patch_set":32,"id":"46ec7ae6_cab9ce38","line":52,"in_reply_to":"ecfd9471_aaaeaa3b","updated":"2022-09-06 22:54:19.000000000","message":"Sorry, I think that # indicates I meant to write something.  I\u0027ve expanded this with I75a81093283b1dfc3104f171249ed8903c79e62e","commit_id":"c286f870cc46c0a89ff82b0ebfe9f5b6db9fe544"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"35b0187e24cd0bfb43ef810526c4c3f46c07f21b","unresolved":false,"context_lines":[{"line_number":329,"context_line":"                    \u0027ansible_host\u0027, play_vars[host].get("},{"line_number":330,"context_line":"                        \u0027ansible_inventory_host\u0027))"},{"line_number":331,"context_line":"                if (ip in (\u0027localhost\u0027, \u0027127.0.0.1\u0027) and"},{"line_number":332,"context_line":"                    not LOG_STREAM_LOCALHOST):"},{"line_number":333,"context_line":"                    # Don\u0027t try to stream from localhost"},{"line_number":334,"context_line":"                    continue"},{"line_number":335,"context_line":"                if play_vars[host].get(\u0027ansible_connection\u0027) in (\u0027winrm\u0027,):"}],"source_content_type":"text/x-python","patch_set":32,"id":"29d6b6e1_32021b8f","line":332,"updated":"2022-09-06 22:19:45.000000000","message":"Just an aside: this does make me uncomfortable because a big part of why we created this job in the first place was that our test environment wasn\u0027t mirroring the real environment, and now we\u0027re poking a (small) hole in that.  I can\u0027t think of a better way right now though....\n\nMaybe we could run the docker container on a second node?  I know it\u0027s a little more ansible to set up... but probably not much more than what we have now.","commit_id":"c286f870cc46c0a89ff82b0ebfe9f5b6db9fe544"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"acf3c4213cc6c03cec50fee8251774fa6ac08478","unresolved":true,"context_lines":[{"line_number":329,"context_line":"                    \u0027ansible_host\u0027, play_vars[host].get("},{"line_number":330,"context_line":"                        \u0027ansible_inventory_host\u0027))"},{"line_number":331,"context_line":"                if (ip in (\u0027localhost\u0027, \u0027127.0.0.1\u0027) and"},{"line_number":332,"context_line":"                    not LOG_STREAM_LOCALHOST):"},{"line_number":333,"context_line":"                    # Don\u0027t try to stream from localhost"},{"line_number":334,"context_line":"                    continue"},{"line_number":335,"context_line":"                if play_vars[host].get(\u0027ansible_connection\u0027) in (\u0027winrm\u0027,):"}],"source_content_type":"text/x-python","patch_set":32,"id":"08d06445_d85d0daa","line":332,"in_reply_to":"29d6b6e1_32021b8f","updated":"2022-09-06 22:54:19.000000000","message":"It took me quite a while to get this far (see 32 changesets 😊) so I\u0027m hesitant to add another layer of complexity to it here.  Hopefully the comment added with above helps with some of this.\n\nI do think it might be better to have this test running against separate individual containers from 2.7 -\u003e 3.\u003cX\u003e to individually and explicitly test each supported target python version.  Perhaps as part of that we could have them in a separate docker network?  Then they would all appear in 172.X?","commit_id":"c286f870cc46c0a89ff82b0ebfe9f5b6db9fe544"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"3a899c718b6c94fe50a59def9f847221916286f6","unresolved":false,"context_lines":[{"line_number":329,"context_line":"                    \u0027ansible_host\u0027, play_vars[host].get("},{"line_number":330,"context_line":"                        \u0027ansible_inventory_host\u0027))"},{"line_number":331,"context_line":"                if (ip in (\u0027localhost\u0027, \u0027127.0.0.1\u0027) and"},{"line_number":332,"context_line":"                    not LOG_STREAM_LOCALHOST):"},{"line_number":333,"context_line":"                    # Don\u0027t try to stream from localhost"},{"line_number":334,"context_line":"                    continue"},{"line_number":335,"context_line":"                if play_vars[host].get(\u0027ansible_connection\u0027) in (\u0027winrm\u0027,):"}],"source_content_type":"text/x-python","patch_set":32,"id":"69d8cb32_1af951f9","line":332,"in_reply_to":"29d6b6e1_32021b8f","updated":"2022-09-06 23:09:13.000000000","message":"That sounds great!","commit_id":"c286f870cc46c0a89ff82b0ebfe9f5b6db9fe544"}]}