)]}'
{"playbooks/inventory/dynamic_inventory.py":[{"author":{"_account_id":22042,"name":"Chris Smart","email":"distroguy@gmail.com","username":"csmart"},"change_message_id":"e674586229b350d75f80eaf2d0e8a31590cb140c","unresolved":false,"context_lines":[{"line_number":1146,"context_line":""},{"line_number":1147,"context_line":"    check \u003d all_args.get(\u0027check\u0027)"},{"line_number":1148,"context_line":"    if check:"},{"line_number":1149,"context_line":"        return \u0027Configuration ok!\u0027"},{"line_number":1150,"context_line":""},{"line_number":1151,"context_line":"    # Generate a list of all hosts and their used IP addresses"},{"line_number":1152,"context_line":"    hostnames_ips \u003d {}"}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad45d7e_8baeffe3","line":1149,"updated":"2016-08-11 03:58:51.000000000","message":"so if I\u0027m reading this correctly, we currently just return that everything is OK without actually doing any dry-run tests? Later we plan to build this out? Or am I missing something (entirely possible). Thanks!","commit_id":"5fd4cd20c5e792a008de3dfe754fb67baf16d6c1"},{"author":{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},"change_message_id":"0765eaf0127507332665c5eba397b78ef64dbed9","unresolved":false,"context_lines":[{"line_number":1146,"context_line":""},{"line_number":1147,"context_line":"    check \u003d all_args.get(\u0027check\u0027)"},{"line_number":1148,"context_line":"    if check:"},{"line_number":1149,"context_line":"        return \u0027Configuration ok!\u0027"},{"line_number":1150,"context_line":""},{"line_number":1151,"context_line":"    # Generate a list of all hosts and their used IP addresses"},{"line_number":1152,"context_line":"    hostnames_ips \u003d {}"}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad45d7e_f00f7f34","line":1149,"in_reply_to":"9ad45d7e_8baeffe3","updated":"2016-08-11 16:58:10.000000000","message":"By the time we reach this point in the code, we\u0027ve executed the whole dry-run. The rest of the `main` function here (L1152-1175) is simply outputting the results.\n\nThe check is essentially looking to see if we raised any known errors or any exceptions.","commit_id":"5fd4cd20c5e792a008de3dfe754fb67baf16d6c1"},{"author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"change_message_id":"74391fe3c33af2e6dc511bb674e4595c777e57f7","unresolved":false,"context_lines":[{"line_number":1146,"context_line":""},{"line_number":1147,"context_line":"    check \u003d all_args.get(\u0027check\u0027)"},{"line_number":1148,"context_line":"    if check:"},{"line_number":1149,"context_line":"        return \u0027Configuration ok!\u0027"},{"line_number":1150,"context_line":""},{"line_number":1151,"context_line":"    # Generate a list of all hosts and their used IP addresses"},{"line_number":1152,"context_line":"    hostnames_ips \u003d {}"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ac06dbe_15401256","line":1149,"updated":"2016-08-19 14:02:04.000000000","message":"Got it -- thanks!","commit_id":"916a3d38a1d22dce7930293ee5a4ea2ea436740c"},{"author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"change_message_id":"7dcff3649c25817c359df9e6557cb5124e514fd6","unresolved":false,"context_lines":[{"line_number":1146,"context_line":""},{"line_number":1147,"context_line":"    check \u003d all_args.get(\u0027check\u0027)"},{"line_number":1148,"context_line":"    if check:"},{"line_number":1149,"context_line":"        return \u0027Configuration ok!\u0027"},{"line_number":1150,"context_line":""},{"line_number":1151,"context_line":"    # Generate a list of all hosts and their used IP addresses"},{"line_number":1152,"context_line":"    hostnames_ips \u003d {}"}],"source_content_type":"text/x-python","patch_set":4,"id":"3ac371cc_d7f9f881","line":1149,"updated":"2016-08-16 19:26:47.000000000","message":"Should this return something to the effect of \"not yet implemented\" since it short circuits here?","commit_id":"916a3d38a1d22dce7930293ee5a4ea2ea436740c"},{"author":{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},"change_message_id":"6834eddd014c82bdfeec0f1296739eaa6a356876","unresolved":false,"context_lines":[{"line_number":1146,"context_line":""},{"line_number":1147,"context_line":"    check \u003d all_args.get(\u0027check\u0027)"},{"line_number":1148,"context_line":"    if check:"},{"line_number":1149,"context_line":"        return \u0027Configuration ok!\u0027"},{"line_number":1150,"context_line":""},{"line_number":1151,"context_line":"    # Generate a list of all hosts and their used IP addresses"},{"line_number":1152,"context_line":"    hostnames_ips \u003d {}"}],"source_content_type":"text/x-python","patch_set":4,"id":"3ac371cc_5e30f5e8","line":1149,"in_reply_to":"3ac371cc_d7f9f881","updated":"2016-08-18 18:38:47.000000000","message":"All the proceeding lines in main do the actual work of building the inventory - the remaining lines are about generating the output. My reasoning was that if main managed to reach this point in the code, all known issues would have been encountered. Thus, we can return out of the function but not write any of the generated inventory information to the filesystem or stdout.","commit_id":"916a3d38a1d22dce7930293ee5a4ea2ea436740c"}]}