)]}'
{"playbooks/roles/bifrost-ironic-install/README.md":[{"author":{"_account_id":12459,"name":"Stephanie Miller","email":"stephane@alum.mit.edu","username":"cinerama"},"change_message_id":"fbcd24c6fcd46d5cdcfe53e9550937fed3fe4f78","unresolved":false,"context_lines":[{"line_number":100,"context_line":"- If a given default applies to multiple versions of a distribution, that"},{"line_number":101,"context_line":"  variable needs to be specified for each version which it affects."},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"If you wish to enable CORS, such as to connect a javascript based web client,"},{"line_number":104,"context_line":"options have been added to allow a user to enable the integrated support."},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"By default, this support is disabled, but the configuration options are below:"}],"source_content_type":"text/x-gfm","patch_set":1,"id":"ba15a1d1_dc3b219b","line":103,"updated":"2015-09-15 16:43:34.000000000","message":"I think it is worth expanding CORS as \"cross-origin resource sharing (CORS)\" for the first mention. We can leave the others in abbreviated form. Also, all you are really saying here is that we have CORS support. This could be folded into the next paragraph where we say that it\u0027s disabled by default.","commit_id":"043fbc3c161d3843de97abd5b46184db9ac9aec6"},{"author":{"_account_id":12459,"name":"Stephanie Miller","email":"stephane@alum.mit.edu","username":"cinerama"},"change_message_id":"bbe0e35d8789ae3d783db1a9ba06063436275503","unresolved":false,"context_lines":[{"line_number":100,"context_line":"- If a given default applies to multiple versions of a distribution, that"},{"line_number":101,"context_line":"  variable needs to be specified for each version which it affects."},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"If you wish to enable CORS, such as to connect a javascript based web client,"},{"line_number":104,"context_line":"options have been added to allow a user to enable the integrated support."},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"By default, this support is disabled, but the configuration options are below:"}],"source_content_type":"text/x-gfm","patch_set":1,"id":"ba15a1d1_baa87f8e","line":103,"in_reply_to":"ba15a1d1_b9d43447","updated":"2015-09-16 15:41:59.000000000","message":"Thanks.","commit_id":"043fbc3c161d3843de97abd5b46184db9ac9aec6"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"ef71eb8f63dc85638be5d1838aca49c8e8a31f9e","unresolved":false,"context_lines":[{"line_number":100,"context_line":"- If a given default applies to multiple versions of a distribution, that"},{"line_number":101,"context_line":"  variable needs to be specified for each version which it affects."},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"If you wish to enable CORS, such as to connect a javascript based web client,"},{"line_number":104,"context_line":"options have been added to allow a user to enable the integrated support."},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"By default, this support is disabled, but the configuration options are below:"}],"source_content_type":"text/x-gfm","patch_set":1,"id":"9a1a9d01_3a04abbe","line":103,"in_reply_to":"ba15a1d1_dc3b219b","updated":"2015-10-05 12:27:25.000000000","message":"Done","commit_id":"043fbc3c161d3843de97abd5b46184db9ac9aec6"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"dcc1d3359ddb46a7a772e95868a290d8178770ac","unresolved":false,"context_lines":[{"line_number":100,"context_line":"- If a given default applies to multiple versions of a distribution, that"},{"line_number":101,"context_line":"  variable needs to be specified for each version which it affects."},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"If you wish to enable CORS, such as to connect a javascript based web client,"},{"line_number":104,"context_line":"options have been added to allow a user to enable the integrated support."},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"By default, this support is disabled, but the configuration options are below:"}],"source_content_type":"text/x-gfm","patch_set":1,"id":"ba15a1d1_b9d43447","line":103,"in_reply_to":"ba15a1d1_dc3b219b","updated":"2015-09-15 20:36:38.000000000","message":"Good point, I was thinking that after I typed git review out, I\u0027ll update.","commit_id":"043fbc3c161d3843de97abd5b46184db9ac9aec6"},{"author":{"_account_id":12459,"name":"Stephanie Miller","email":"stephane@alum.mit.edu","username":"cinerama"},"change_message_id":"fbcd24c6fcd46d5cdcfe53e9550937fed3fe4f78","unresolved":false,"context_lines":[{"line_number":109,"context_line":"cors_allowed_origin: A URL string that represents the origin sent by the"},{"line_number":110,"context_line":"                     client web browser. If CORS is enabled, and this is"},{"line_number":111,"context_line":"                     not set, it will default to http://localhost:8000/."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Dependencies"},{"line_number":114,"context_line":"------------"},{"line_number":115,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":1,"id":"ba15a1d1_1c52f9dd","line":112,"updated":"2015-09-15 16:43:34.000000000","message":"enable_cors_credential_support is not documented here.","commit_id":"043fbc3c161d3843de97abd5b46184db9ac9aec6"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"ef71eb8f63dc85638be5d1838aca49c8e8a31f9e","unresolved":false,"context_lines":[{"line_number":109,"context_line":"cors_allowed_origin: A URL string that represents the origin sent by the"},{"line_number":110,"context_line":"                     client web browser. If CORS is enabled, and this is"},{"line_number":111,"context_line":"                     not set, it will default to http://localhost:8000/."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Dependencies"},{"line_number":114,"context_line":"------------"},{"line_number":115,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":1,"id":"9a1a9d01_5aff1faa","line":112,"in_reply_to":"ba15a1d1_1c52f9dd","updated":"2015-10-05 12:27:25.000000000","message":"Done","commit_id":"043fbc3c161d3843de97abd5b46184db9ac9aec6"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"dcc1d3359ddb46a7a772e95868a290d8178770ac","unresolved":false,"context_lines":[{"line_number":109,"context_line":"cors_allowed_origin: A URL string that represents the origin sent by the"},{"line_number":110,"context_line":"                     client web browser. If CORS is enabled, and this is"},{"line_number":111,"context_line":"                     not set, it will default to http://localhost:8000/."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Dependencies"},{"line_number":114,"context_line":"------------"},{"line_number":115,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":1,"id":"ba15a1d1_3948242d","line":112,"in_reply_to":"ba15a1d1_1c52f9dd","updated":"2015-09-15 20:36:38.000000000","message":"Even though it really shouldn\u0027t be toggled at this time, do you think it is worth detailing in the README?","commit_id":"043fbc3c161d3843de97abd5b46184db9ac9aec6"},{"author":{"_account_id":12459,"name":"Stephanie Miller","email":"stephane@alum.mit.edu","username":"cinerama"},"change_message_id":"bbe0e35d8789ae3d783db1a9ba06063436275503","unresolved":false,"context_lines":[{"line_number":109,"context_line":"cors_allowed_origin: A URL string that represents the origin sent by the"},{"line_number":110,"context_line":"                     client web browser. If CORS is enabled, and this is"},{"line_number":111,"context_line":"                     not set, it will default to http://localhost:8000/."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Dependencies"},{"line_number":114,"context_line":"------------"},{"line_number":115,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":1,"id":"ba15a1d1_faecb7f4","line":112,"in_reply_to":"ba15a1d1_3948242d","updated":"2015-09-16 15:41:59.000000000","message":"It seems like we usually doc all the parameters for a role. Might be worth mentioning if only to tell people not to mess with it right now (even though we do that in the role defaults).","commit_id":"043fbc3c161d3843de97abd5b46184db9ac9aec6"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"ef71eb8f63dc85638be5d1838aca49c8e8a31f9e","unresolved":false,"context_lines":[{"line_number":109,"context_line":"cors_allowed_origin: A URL string that represents the origin sent by the"},{"line_number":110,"context_line":"                     client web browser. If CORS is enabled, and this is"},{"line_number":111,"context_line":"                     not set, it will default to http://localhost:8000/."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Dependencies"},{"line_number":114,"context_line":"------------"},{"line_number":115,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":1,"id":"9a1a9d01_1a09a7dd","line":112,"in_reply_to":"ba15a1d1_faecb7f4","updated":"2015-10-05 12:27:25.000000000","message":"The install is a real odd-ball of a role.  I\u0027ll go ahead and doc it in there as well.","commit_id":"043fbc3c161d3843de97abd5b46184db9ac9aec6"},{"author":{"_account_id":12459,"name":"Stephanie Miller","email":"stephane@alum.mit.edu","username":"cinerama"},"change_message_id":"87c668e94fca3c33cfc0f5badfb96e115d95abab","unresolved":false,"context_lines":[{"line_number":100,"context_line":"- If a given default applies to multiple versions of a distribution, that"},{"line_number":101,"context_line":"  variable needs to be specified for each version which it affects."},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"If you wish to enable Cross-Orogin Resource Sharing (CORS), such as to"},{"line_number":104,"context_line":"connect a javascript based web client, options have been added to allow"},{"line_number":105,"context_line":"a user to enable the integrated support."},{"line_number":106,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":2,"id":"9a1a9d01_0c3710e4","line":103,"updated":"2015-09-29 19:29:20.000000000","message":"nit: should be origin","commit_id":"0bcdeaf0e725ab044bb6e1fb56b0c49516dc9345"},{"author":{"_account_id":5805,"name":"Chris Krelle","email":"nobodycam@gmail.com","username":"nobodycam"},"change_message_id":"74feed4fddf2c16637939978f6e2238cf6c95a5a","unresolved":false,"context_lines":[{"line_number":100,"context_line":"- If a given default applies to multiple versions of a distribution, that"},{"line_number":101,"context_line":"  variable needs to be specified for each version which it affects."},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"If you wish to enable Cross-Orogin Resource Sharing (CORS), such as to"},{"line_number":104,"context_line":"connect a javascript based web client, options have been added to allow"},{"line_number":105,"context_line":"a user to enable the integrated support."},{"line_number":106,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":2,"id":"7a2fa921_b81d6311","line":103,"in_reply_to":"9a1a9d01_0c3710e4","updated":"2015-10-05 22:43:39.000000000","message":"this should be fixed with a follow up.","commit_id":"0bcdeaf0e725ab044bb6e1fb56b0c49516dc9345"}]}