)]}'
{"plugins/inventory/openstack.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"bb0796b7b5da47bb76ffc7dd517e5b34ad42db5a","unresolved":false,"context_lines":[{"line_number":255,"context_line":"            try:"},{"line_number":256,"context_line":"                source_data \u003d cloud_inventory.list_hosts("},{"line_number":257,"context_line":"                    expand\u003dexpand_hostvars, fail_on_cloud_config\u003dfail_on_errors,"},{"line_number":258,"context_line":"                    )"},{"line_number":259,"context_line":"                if isinstance(filters, dict) or isinstance(filters, string_types):"},{"line_number":260,"context_line":"                    source_data \u003d cloud_inventory.search_hosts("},{"line_number":261,"context_line":"                        expand\u003dexpand_hostvars, filters\u003dfilters)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_9069c58d","line":258,"updated":"2020-09-06 19:38:06.000000000","message":"linters-2.9: E123 closing bracket does not match indentation of opening bracket\u0027s line","commit_id":"c80c59992479b228f88a92a5901efdd0413cd9da"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"bb0796b7b5da47bb76ffc7dd517e5b34ad42db5a","unresolved":false,"context_lines":[{"line_number":255,"context_line":"            try:"},{"line_number":256,"context_line":"                source_data \u003d cloud_inventory.list_hosts("},{"line_number":257,"context_line":"                    expand\u003dexpand_hostvars, fail_on_cloud_config\u003dfail_on_errors,"},{"line_number":258,"context_line":"                    )"},{"line_number":259,"context_line":"                if isinstance(filters, dict) or isinstance(filters, string_types):"},{"line_number":260,"context_line":"                    source_data \u003d cloud_inventory.search_hosts("},{"line_number":261,"context_line":"                        expand\u003dexpand_hostvars, filters\u003dfilters)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_3070b972","line":258,"updated":"2020-09-06 19:38:06.000000000","message":"linters: E123 closing bracket does not match indentation of opening bracket\u0027s line","commit_id":"c80c59992479b228f88a92a5901efdd0413cd9da"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"bb0796b7b5da47bb76ffc7dd517e5b34ad42db5a","unresolved":false,"context_lines":[{"line_number":255,"context_line":"            try:"},{"line_number":256,"context_line":"                source_data \u003d cloud_inventory.list_hosts("},{"line_number":257,"context_line":"                    expand\u003dexpand_hostvars, fail_on_cloud_config\u003dfail_on_errors,"},{"line_number":258,"context_line":"                    )"},{"line_number":259,"context_line":"                if isinstance(filters, dict) or isinstance(filters, string_types):"},{"line_number":260,"context_line":"                    source_data \u003d cloud_inventory.search_hosts("},{"line_number":261,"context_line":"                        expand\u003dexpand_hostvars, filters\u003dfilters)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_7066b1bd","line":258,"updated":"2020-09-06 19:38:06.000000000","message":"linters: E123 closing bracket does not match indentation of opening bracket\u0027s line","commit_id":"c80c59992479b228f88a92a5901efdd0413cd9da"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"bb0796b7b5da47bb76ffc7dd517e5b34ad42db5a","unresolved":false,"context_lines":[{"line_number":255,"context_line":"            try:"},{"line_number":256,"context_line":"                source_data \u003d cloud_inventory.list_hosts("},{"line_number":257,"context_line":"                    expand\u003dexpand_hostvars, fail_on_cloud_config\u003dfail_on_errors,"},{"line_number":258,"context_line":"                    )"},{"line_number":259,"context_line":"                if isinstance(filters, dict) or isinstance(filters, string_types):"},{"line_number":260,"context_line":"                    source_data \u003d cloud_inventory.search_hosts("},{"line_number":261,"context_line":"                        expand\u003dexpand_hostvars, filters\u003dfilters)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_d004ddc8","line":258,"updated":"2020-09-06 19:38:06.000000000","message":"pep8: E123 closing bracket does not match indentation of opening bracket\u0027s line","commit_id":"c80c59992479b228f88a92a5901efdd0413cd9da"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"edb46a6a8d0976815823b5189325dbfbf945aab1","unresolved":false,"context_lines":[{"line_number":104,"context_line":"            Match hosts with metadata. Use dictionary or a string containing"},{"line_number":105,"context_line":"            a jmespath expression for further filtering."},{"line_number":106,"context_line":"            note: \u0027fail_on_errors\u0027 is always \u0027yes\u0027 when using filters and \u0027all_projects\u0027 is not used."},{"line_number":107,"context_line":"        type: dictionary or string"},{"line_number":108,"context_line":"        default: None"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_14539317","line":107,"range":{"start_line":107,"start_character":8,"end_line":107,"end_character":34},"updated":"2020-09-25 08:47:27.000000000","message":"we can\u0027t use 2 types, it should be only one chosen","commit_id":"ab18224fcd14a32898592166e19e3ea4cf78f30a"},{"author":{"_account_id":32390,"name":"Turo Soisenniemi","email":"turo.soisenniemi@outlook.com","username":"turmio"},"change_message_id":"435f9684fc541bee8111e228df1f42bf8fa7bb3b","unresolved":false,"context_lines":[{"line_number":104,"context_line":"            Match hosts with metadata. Use dictionary or a string containing"},{"line_number":105,"context_line":"            a jmespath expression for further filtering."},{"line_number":106,"context_line":"            note: \u0027fail_on_errors\u0027 is always \u0027yes\u0027 when using filters and \u0027all_projects\u0027 is not used."},{"line_number":107,"context_line":"        type: dictionary or string"},{"line_number":108,"context_line":"        default: None"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_60279ab7","line":107,"range":{"start_line":107,"start_character":8,"end_line":107,"end_character":34},"in_reply_to":"9f560f44_14539317","updated":"2020-09-26 10:59:40.000000000","message":"Hi!\n\nThese filters go just through from here to OpenStack SDK which takes either dictionary or jmespath expression. What is problem here for allowing both?","commit_id":"ab18224fcd14a32898592166e19e3ea4cf78f30a"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"034e446c0a10bc76621f717be93ab6dd043054db","unresolved":false,"context_lines":[{"line_number":104,"context_line":"            Match hosts with metadata. Use dictionary or a string containing"},{"line_number":105,"context_line":"            a jmespath expression for further filtering."},{"line_number":106,"context_line":"            note: \u0027fail_on_errors\u0027 is always \u0027yes\u0027 when using filters and \u0027all_projects\u0027 is not used."},{"line_number":107,"context_line":"        type: dictionary or string"},{"line_number":108,"context_line":"        default: None"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_e0728a7b","line":107,"range":{"start_line":107,"start_character":8,"end_line":107,"end_character":34},"in_reply_to":"9f560f44_60279ab7","updated":"2020-09-26 12:34:27.000000000","message":"This is ansible requirement, variable should be of one specific type. It\u0027s parsed later by argparser","commit_id":"ab18224fcd14a32898592166e19e3ea4cf78f30a"},{"author":{"_account_id":32390,"name":"Turo Soisenniemi","email":"turo.soisenniemi@outlook.com","username":"turmio"},"change_message_id":"9d9eda3eac2a20d4b6e54de83facbec0986d080c","unresolved":false,"context_lines":[{"line_number":104,"context_line":"            Match hosts with metadata. Use dictionary or a string containing"},{"line_number":105,"context_line":"            a jmespath expression for further filtering."},{"line_number":106,"context_line":"            note: \u0027fail_on_errors\u0027 is always \u0027yes\u0027 when using filters and \u0027all_projects\u0027 is not used."},{"line_number":107,"context_line":"        type: dictionary or string"},{"line_number":108,"context_line":"        default: None"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_72a8d577","line":107,"range":{"start_line":107,"start_character":8,"end_line":107,"end_character":34},"in_reply_to":"9f560f44_e0728a7b","updated":"2020-09-30 15:44:53.000000000","message":"For example in yum module description for `name` mentions this: \"To operate on several packages this can accept a comma separated string of packages or (as of 2.0) a list of packages.\"\n\nSo is collection of same type allowed but two different types are not? Or maybe different rules for inventory plugins and modules?\n\nI am just curious to know why isn\u0027t this work/allowed since we have been using filters in our testing environment for over a year without experiencing this limitation.","commit_id":"ab18224fcd14a32898592166e19e3ea4cf78f30a"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"edb46a6a8d0976815823b5189325dbfbf945aab1","unresolved":false,"context_lines":[{"line_number":105,"context_line":"            a jmespath expression for further filtering."},{"line_number":106,"context_line":"            note: \u0027fail_on_errors\u0027 is always \u0027yes\u0027 when using filters and \u0027all_projects\u0027 is not used."},{"line_number":107,"context_line":"        type: dictionary or string"},{"line_number":108,"context_line":"        default: None"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"extends_documentation_fragment:"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_74412f36","line":108,"range":{"start_line":108,"start_character":8,"end_line":108,"end_character":21},"updated":"2020-09-25 08:47:27.000000000","message":"it\u0027s None by default, if you want to specify a different, need to be default of type of option.","commit_id":"ab18224fcd14a32898592166e19e3ea4cf78f30a"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"034e446c0a10bc76621f717be93ab6dd043054db","unresolved":false,"context_lines":[{"line_number":105,"context_line":"            a jmespath expression for further filtering."},{"line_number":106,"context_line":"            note: \u0027fail_on_errors\u0027 is always \u0027yes\u0027 when using filters and \u0027all_projects\u0027 is not used."},{"line_number":107,"context_line":"        type: dictionary or string"},{"line_number":108,"context_line":"        default: None"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"extends_documentation_fragment:"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_c06dc699","line":108,"range":{"start_line":108,"start_character":8,"end_line":108,"end_character":21},"in_reply_to":"9f560f44_402a567d","updated":"2020-09-26 12:34:27.000000000","message":"all values are None if they aren\u0027t set, so no need to specify it","commit_id":"ab18224fcd14a32898592166e19e3ea4cf78f30a"},{"author":{"_account_id":32390,"name":"Turo Soisenniemi","email":"turo.soisenniemi@outlook.com","username":"turmio"},"change_message_id":"435f9684fc541bee8111e228df1f42bf8fa7bb3b","unresolved":false,"context_lines":[{"line_number":105,"context_line":"            a jmespath expression for further filtering."},{"line_number":106,"context_line":"            note: \u0027fail_on_errors\u0027 is always \u0027yes\u0027 when using filters and \u0027all_projects\u0027 is not used."},{"line_number":107,"context_line":"        type: dictionary or string"},{"line_number":108,"context_line":"        default: None"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"extends_documentation_fragment:"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_402a567d","line":108,"range":{"start_line":108,"start_character":8,"end_line":108,"end_character":21},"in_reply_to":"9f560f44_74412f36","updated":"2020-09-26 10:59:40.000000000","message":"I was just documenting default value. This can be removed if it is unnecessary.","commit_id":"ab18224fcd14a32898592166e19e3ea4cf78f30a"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"8526178c6ae8203fb8f1384eaabff4d28c5ef054","unresolved":false,"context_lines":[{"line_number":255,"context_line":"            try:"},{"line_number":256,"context_line":"                source_data \u003d cloud_inventory.list_hosts("},{"line_number":257,"context_line":"                    expand\u003dexpand_hostvars, fail_on_cloud_config\u003dfail_on_errors)"},{"line_number":258,"context_line":"                if isinstance(filters, dict) or isinstance(filters, string_types):"},{"line_number":259,"context_line":"                    source_data \u003d cloud_inventory.search_hosts("},{"line_number":260,"context_line":"                        expand\u003dexpand_hostvars, filters\u003dfilters)"},{"line_number":261,"context_line":"                else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_f4b6df88","line":258,"range":{"start_line":258,"start_character":16,"end_line":258,"end_character":82},"updated":"2020-09-25 09:23:00.000000000","message":"argument can be of one type only","commit_id":"ab18224fcd14a32898592166e19e3ea4cf78f30a"}]}