)]}'
{"doc/source/reference/job_def.rst":[{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"950d9fb9374e8a27a7ab25c37a6681e4362cc260","unresolved":true,"context_lines":[{"line_number":514,"context_line":""},{"line_number":515,"context_line":"      .. program-output:: zuul-manage-ansible -l"},{"line_number":516,"context_line":""},{"line_number":517,"context_line":"   .. attr:: mitogen"},{"line_number":518,"context_line":"      :default: False"},{"line_number":519,"context_line":""},{"line_number":520,"context_line":"      Use Mitogen_ when running Ansible. This can potentially speed up the"}],"source_content_type":"text/x-rst","patch_set":28,"id":"e3fe8b5a_3ab655f4","line":517,"updated":"2021-09-09 13:38:22.000000000","message":"Instead of using a new job attribute, could this be activated using the existing ansible-version attribute instead? e.g. `ansible-version: 3.10-mitogen`","commit_id":"ea9ee08b9976e09ec88475133707862a1304fb30"},{"author":{"_account_id":31083,"name":"Benjamin Schanzel","email":"benjamin.schanzel@bmw.de","username":"benjamin.schanzel"},"change_message_id":"1772dcc0322ad70cca30eca09b58832a68fe7510","unresolved":true,"context_lines":[{"line_number":514,"context_line":""},{"line_number":515,"context_line":"      .. program-output:: zuul-manage-ansible -l"},{"line_number":516,"context_line":""},{"line_number":517,"context_line":"   .. attr:: mitogen"},{"line_number":518,"context_line":"      :default: False"},{"line_number":519,"context_line":""},{"line_number":520,"context_line":"      Use Mitogen_ when running Ansible. This can potentially speed up the"}],"source_content_type":"text/x-rst","patch_set":28,"id":"513ff437_82bb7824","line":517,"in_reply_to":"e3fe8b5a_3ab655f4","updated":"2021-09-13 07:29:29.000000000","message":"My first thought is that I\u0027d rather not mix up such optional things with the Ansible version, as this is more or less independent from that. Also, I find a boolean flag catchier than encoding this into a version string. But that\u0027s just my five cents, I\u0027m open to other opinions there.\n\nAnyhow, there\u0027s still some problems with the zuul_console log streaming and the daemonizing/forking mechanism which needs to be addressed.","commit_id":"ea9ee08b9976e09ec88475133707862a1304fb30"}],"zuul/executor/server.py":[{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"95dabb484aea48b8a400012195d4ce19fdc4d2ab","unresolved":false,"context_lines":[{"line_number":1399,"context_line":""},{"line_number":1400,"context_line":"                    # TODO: Mitogen does not yet support \u0027auto\u0027. Remove this"},{"line_number":1401,"context_line":"                    #       when it does support it."},{"line_number":1402,"context_line":"                    compat \u003d compat or self.arguments.get(\u0027mitogen\u0027, False)"},{"line_number":1403,"context_line":""},{"line_number":1404,"context_line":"                    if python \u003d\u003d \"auto\" and compat:"},{"line_number":1405,"context_line":"                        self.log.debug("}],"source_content_type":"text/x-python","patch_set":16,"id":"3fa7e38b_f1791dc6","line":1402,"range":{"start_line":1402,"start_character":36,"end_line":1402,"end_character":38},"updated":"2020-01-14 12:35:04.000000000","message":"shouldn\u0027t this be \u0027and\u0027?","commit_id":"c64020dbac7bfcece5986ba86276d0e5120b7b2f"},{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"b1844f063581555336627910eb8578868b1e76d9","unresolved":false,"context_lines":[{"line_number":1399,"context_line":""},{"line_number":1400,"context_line":"                    # TODO: Mitogen does not yet support \u0027auto\u0027. Remove this"},{"line_number":1401,"context_line":"                    #       when it does support it."},{"line_number":1402,"context_line":"                    compat \u003d compat or self.arguments.get(\u0027mitogen\u0027, False)"},{"line_number":1403,"context_line":""},{"line_number":1404,"context_line":"                    if python \u003d\u003d \"auto\" and compat:"},{"line_number":1405,"context_line":"                        self.log.debug("}],"source_content_type":"text/x-python","patch_set":16,"id":"3fa7e38b_ace90e83","line":1402,"range":{"start_line":1402,"start_character":36,"end_line":1402,"end_character":38},"in_reply_to":"3fa7e38b_8c269267","updated":"2020-01-14 13:21:24.000000000","message":"oops, i read it the other way. Thanks for the explanation.","commit_id":"c64020dbac7bfcece5986ba86276d0e5120b7b2f"},{"author":{"_account_id":16068,"name":"Tobias Henkel","email":"tobias.henkel@bmw.de","username":"tobias.henkel"},"change_message_id":"a2003de32359fb53e5eee1a449b950eb24e1e151","unresolved":false,"context_lines":[{"line_number":1399,"context_line":""},{"line_number":1400,"context_line":"                    # TODO: Mitogen does not yet support \u0027auto\u0027. Remove this"},{"line_number":1401,"context_line":"                    #       when it does support it."},{"line_number":1402,"context_line":"                    compat \u003d compat or self.arguments.get(\u0027mitogen\u0027, False)"},{"line_number":1403,"context_line":""},{"line_number":1404,"context_line":"                    if python \u003d\u003d \"auto\" and compat:"},{"line_number":1405,"context_line":"                        self.log.debug("}],"source_content_type":"text/x-python","patch_set":16,"id":"3fa7e38b_8c269267","line":1402,"range":{"start_line":1402,"start_character":36,"end_line":1402,"end_character":38},"in_reply_to":"3fa7e38b_f1791dc6","updated":"2020-01-14 12:58:46.000000000","message":"no, if compat is already true we need to keep it true to disable use of auto plus set it to true if mitogen was requested.","commit_id":"c64020dbac7bfcece5986ba86276d0e5120b7b2f"},{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"b1844f063581555336627910eb8578868b1e76d9","unresolved":false,"context_lines":[{"line_number":1404,"context_line":"                    if python \u003d\u003d \"auto\" and compat:"},{"line_number":1405,"context_line":"                        self.log.debug("},{"line_number":1406,"context_line":"                            \"ansible_version set to auto but \""},{"line_number":1407,"context_line":"                            \"overriding to python2 for Ansible \u003c2.8\")"},{"line_number":1408,"context_line":"                        python \u003d \u0027/usr/bin/python2\u0027"},{"line_number":1409,"context_line":"                    host_vars.setdefault(api, python)"},{"line_number":1410,"context_line":""}],"source_content_type":"text/x-python","patch_set":16,"id":"3fa7e38b_4cfada44","line":1407,"range":{"start_line":1407,"start_character":28,"end_line":1407,"end_character":68},"updated":"2020-01-14 13:21:24.000000000","message":"It seems like this should be updated. Though, since that\u0027s a new default behavior, could we make it that with mitogen, the default is python3 instead?","commit_id":"c64020dbac7bfcece5986ba86276d0e5120b7b2f"},{"author":{"_account_id":16068,"name":"Tobias Henkel","email":"tobias.henkel@bmw.de","username":"tobias.henkel"},"change_message_id":"92e0ad13187fd675e40a90d4727cb2b06d250737","unresolved":false,"context_lines":[{"line_number":1404,"context_line":"                    if python \u003d\u003d \"auto\" and compat:"},{"line_number":1405,"context_line":"                        self.log.debug("},{"line_number":1406,"context_line":"                            \"ansible_version set to auto but \""},{"line_number":1407,"context_line":"                            \"overriding to python2 for Ansible \u003c2.8\")"},{"line_number":1408,"context_line":"                        python \u003d \u0027/usr/bin/python2\u0027"},{"line_number":1409,"context_line":"                    host_vars.setdefault(api, python)"},{"line_number":1410,"context_line":""}],"source_content_type":"text/x-python","patch_set":16,"id":"3fa7e38b_07c03b4a","line":1407,"range":{"start_line":1407,"start_character":28,"end_line":1407,"end_character":68},"in_reply_to":"3fa7e38b_4cfada44","updated":"2020-01-14 13:48:02.000000000","message":"Good idea, that sounds useful.","commit_id":"c64020dbac7bfcece5986ba86276d0e5120b7b2f"}]}
