)]}'
{"bifrost/inventory.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"522ddac6ac805e264ad130b1b6764fc9807a8576","unresolved":false,"context_lines":[{"line_number":183,"context_line":"        return None"},{"line_number":184,"context_line":"    except IndexError:"},{"line_number":185,"context_line":"        LOG.debug((\"Out of range value encountered.  Requested \""},{"line_number":186,"context_line":"                  \"field %s Had: %s\", (location, array)))"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"def _process_baremetal_data(data_source, groups, hostvars):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f1a1f67_a21977d7","line":186,"updated":"2017-07-19 16:04:57.000000000","message":"you have to remove the brackets, otherwise it\u0027s going to be treated as a tuple. also there are redundant top-level brackets here","commit_id":"254c71da2fb18c26568d93456358b2433a5442df"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"522ddac6ac805e264ad130b1b6764fc9807a8576","unresolved":false,"context_lines":[{"line_number":409,"context_line":"                        hostvars)"},{"line_number":410,"context_line":"                except Exception as e:"},{"line_number":411,"context_line":"                    LOG.debug(\"CSV fallback processing failed, \""},{"line_number":412,"context_line":"                              \"received: \u0026s\", e)"},{"line_number":413,"context_line":"                    LOG.error(\"BIFROST_INVENTORY_SOURCE does not define \""},{"line_number":414,"context_line":"                              \"a file that could be processed: \""},{"line_number":415,"context_line":"                              \"Tried JSON, YAML, and CSV formats\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f1a1f67_820c3b94","line":412,"updated":"2017-07-19 16:04:57.000000000","message":"please fix \u0026 to % as well","commit_id":"254c71da2fb18c26568d93456358b2433a5442df"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"e10ca3e212bf5bca51ba973035dbeb6335c54e64","unresolved":false,"context_lines":[{"line_number":183,"context_line":"        return None"},{"line_number":184,"context_line":"    except IndexError:"},{"line_number":185,"context_line":"        LOG.debug((\"Out of range value encountered.  Requested \""},{"line_number":186,"context_line":"                  \"field %s Had: %s\", location, array))"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"def _process_baremetal_data(data_source, groups, hostvars):"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f1a1f67_d2aabcd0","line":186,"updated":"2017-07-21 08:22:36.000000000","message":"actually, I realized that using redundant brackets will make the whole thing a tuple. could you please remove one pair?","commit_id":"69ce25c4956bb5cf364ff5fa68e411a64f2a9415"},{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"06db96f4cb61aa9e14c84d6e4ba0e5ac4e4052b1","unresolved":false,"context_lines":[{"line_number":295,"context_line":"                            power[\u0027ipmi_transit_address\u0027]):"},{"line_number":296,"context_line":"                        power[\u0027ipmi_bridging\u0027] \u003d \u0027dual\u0027"},{"line_number":297,"context_line":""},{"line_number":298,"context_line":"            if \"ssh\" in host[\u0027driver\u0027]:"},{"line_number":299,"context_line":"                # Under another model, a user would define"},{"line_number":300,"context_line":"                # and value translations to load these"},{"line_number":301,"context_line":"                # values.  Since we\u0027re supporting the base"},{"line_number":302,"context_line":"                # model bifrost was developed with, then"},{"line_number":303,"context_line":"                # we need to make sure these are present as"},{"line_number":304,"context_line":"                # they are expected values."},{"line_number":305,"context_line":"                power[\u0027ssh_virt_type\u0027] \u003d \"virsh\""},{"line_number":306,"context_line":"                power[\u0027ssh_address\u0027] \u003d management_address"},{"line_number":307,"context_line":"                power[\u0027ssh_port\u0027] \u003d 22"},{"line_number":308,"context_line":"                # NOTE: The CSV format is desynced from the enrollment"},{"line_number":309,"context_line":"                # playbook at present, so we\u0027re hard coding ironic here"},{"line_number":310,"context_line":"                # as that is what the test is known to work with."},{"line_number":311,"context_line":"                power[\u0027ssh_username\u0027] \u003d \"ironic\""},{"line_number":312,"context_line":"                power[\u0027ssh_key_filename\u0027] \u003d \"/home/ironic/.ssh/id_rsa\""},{"line_number":313,"context_line":""},{"line_number":314,"context_line":"            # Group variables together under host."},{"line_number":315,"context_line":"            # NOTE(TheJulia): Given the split that this demonstrates, where"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f1a1f67_1de605ea","line":312,"range":{"start_line":298,"start_character":0,"end_line":312,"end_character":70},"updated":"2017-07-21 09:08:58.000000000","message":"wow, i just found this can be cleaned up too, Pavlo would glade to know","commit_id":"68bbf28c3959a2874d51dd16d94e62bc1e4a73c5"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"af7fa16d854a5f508dbf08b916b15b354c48ba8b","unresolved":false,"context_lines":[{"line_number":294,"context_line":"                    if (power[\u0027ipmi_transit_channel\u0027] and"},{"line_number":295,"context_line":"                            power[\u0027ipmi_transit_address\u0027]):"},{"line_number":296,"context_line":"                        power[\u0027ipmi_bridging\u0027] \u003d \u0027dual\u0027"},{"line_number":297,"context_line":""},{"line_number":298,"context_line":"            # Group variables together under host."},{"line_number":299,"context_line":"            # NOTE(TheJulia): Given the split that this demonstrates, where"},{"line_number":300,"context_line":"            # deploy details could possible be imported from a future"}],"source_content_type":"text/x-python","patch_set":4,"id":"ff346bd7_29187a1b","line":297,"updated":"2017-07-24 13:37:39.000000000","message":"Removal of the ssh support is unrelated\n\nPlease either restore it, or add a release note to that effect and note it in the commit message.","commit_id":"bff02e8d34e47f1ed02550998c13e008d8150278"}]}