)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"e23e21af965e1f13b51254c3f9721fb75fe8d5e8","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ian Wienand \u003ciwienand@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-09-24 19:03:33 +1000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Allow debootstrap to clenaup without a kernel"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently there\u0027s more-or-less an assumption that a kernel is"},{"line_number":10,"context_line":"installed, so module blacklists are simply echoed into the modprobe"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_49764742","line":7,"range":{"start_line":7,"start_character":21,"end_line":7,"end_character":28},"updated":"2018-09-24 09:11:56.000000000","message":"nit: should be \u0027cleanup\u0027\n\nDon\u0027t bother changing it if there are no material changes to be made in the patch.","commit_id":"a84a25e591db2b5c44a017edba259739b6a04331"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"55586eb1720b993d9be3ddd2c3504d81d1dd0f73","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ian Wienand \u003ciwienand@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-09-24 19:03:33 +1000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Allow debootstrap to clenaup without a kernel"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently there\u0027s more-or-less an assumption that a kernel is"},{"line_number":10,"context_line":"installed, so module blacklists are simply echoed into the modprobe"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_1bb866e6","line":7,"range":{"start_line":7,"start_character":21,"end_line":7,"end_character":28},"in_reply_to":"3f79a3b5_49764742","updated":"2018-09-26 00:40:37.000000000","message":"done ... ci needs to rerun anyway","commit_id":"a84a25e591db2b5c44a017edba259739b6a04331"}],"diskimage_builder/elements/modprobe/extra-data.d/50-modprobe-blacklist":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"2a631744694596cf6fc604ffe12d5dce01e045b1","unresolved":false,"context_lines":[{"line_number":12,"context_line":"# this to work can allow modules that do other things that *are*"},{"line_number":13,"context_line":"# useful for a container to \"just work\" without a whole bunch of"},{"line_number":14,"context_line":"# refactoring."},{"line_number":15,"context_line":"if [ -d /etc/modprobe.d ]; then"},{"line_number":16,"context_line":"    # ^ so we can see in the logs if we took this path ..."},{"line_number":17,"context_line":"    mkdir -p /etc/modprobe.d"},{"line_number":18,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"3f79a3b5_06d35c03","line":15,"updated":"2018-09-24 08:18:58.000000000","message":"I think this is missing a ! in the conditional.","commit_id":"41f5320208ba5244273213749c7cf930bdcf8e96"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"0a4e4767b4adc606b2b2a747c7211bbc51899907","unresolved":false,"context_lines":[{"line_number":12,"context_line":"# this to work can allow modules that do other things that *are*"},{"line_number":13,"context_line":"# useful for a container to \"just work\" without a whole bunch of"},{"line_number":14,"context_line":"# refactoring."},{"line_number":15,"context_line":"if [ -d /etc/modprobe.d ]; then"},{"line_number":16,"context_line":"    # ^ so we can see in the logs if we took this path ..."},{"line_number":17,"context_line":"    mkdir -p /etc/modprobe.d"},{"line_number":18,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"3f79a3b5_695543c2","line":15,"in_reply_to":"3f79a3b5_06d35c03","updated":"2018-09-24 09:06:26.000000000","message":"thanks, yep that is the intent","commit_id":"41f5320208ba5244273213749c7cf930bdcf8e96"}]}