)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},"change_message_id":"12117501deb28b72d26770d6c1043f375de3004e","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Tighten up how we add and remove kernel modules"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change adjusts our two roles that configure kernel mofules."},{"line_number":10,"context_line":"These two roles serve slightly different purposes so they both"},{"line_number":11,"context_line":"must exist at this time. The change being made will ensure that"},{"line_number":12,"context_line":"the system module-load service is in reloaded anytime modules"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_e99a3cc3","line":9,"updated":"2019-10-08 21:30:36.000000000","message":"nit: \"modules\" vs \"mofules\"","commit_id":"14ee74846665dcbd4350c1259bd60c4241e4987a"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"392e581cf6a490dc23359a8c4e385be765d5cf95","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Tighten up how we add and remove kernel modules"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change adjusts our two roles that configure kernel mofules."},{"line_number":10,"context_line":"These two roles serve slightly different purposes so they both"},{"line_number":11,"context_line":"must exist at this time. The change being made will ensure that"},{"line_number":12,"context_line":"the system module-load service is in reloaded anytime modules"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_694acc27","line":9,"in_reply_to":"3fa7e38b_e99a3cc3","updated":"2019-10-08 21:38:05.000000000","message":"Done","commit_id":"14ee74846665dcbd4350c1259bd60c4241e4987a"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"bc327465961bec7909430f9892332903e6c3f969","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"\u003e Molecule configuration for the tripleo-module-load role has"},{"line_number":17,"context_line":"  been updated to ensure the role is functionally testing the"},{"line_number":18,"context_line":"  code paths with real system values."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Closes-Bug: #1847282"},{"line_number":21,"context_line":"Change-Id: Id20d7415e8430c66f3afc8f86e0e813a54ee3ca2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3fa7e38b_23a9447b","line":18,"updated":"2019-10-09 15:30:58.000000000","message":"It may be worth noting that this expects to manage the host on which the test runs, whereas before it did everything in a container. This test therefore expects a single-use VM.","commit_id":"be2e9e046eec2c5ac6e0b17e520df35fe6399776"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"6a416e066986e82406b1383c8d7e0fbc06daac8e","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"\u003e Molecule configuration for the tripleo-module-load role has"},{"line_number":17,"context_line":"  been updated to ensure the role is functionally testing the"},{"line_number":18,"context_line":"  code paths with real system values."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Closes-Bug: #1847282"},{"line_number":21,"context_line":"Change-Id: Id20d7415e8430c66f3afc8f86e0e813a54ee3ca2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"3fa7e38b_cd8e3dd0","line":18,"updated":"2019-10-11 07:24:12.000000000","message":"wondering if this is really two reviews. the fix for the bug is @ https://review.opendev.org/#/c/687419/6/tripleo_ansible/roles/tripleo-kernel/tasks/main.yml \u0026 https://review.opendev.org/#/c/687419/6/tripleo_ansible/roles/tripleo-module-load/tasks/main.yml\n\nthe rest is about moving the 3 scenarios to delegated driver and giving them better input values maybe that part needs to be moved","commit_id":"c833b44ccc5a66c08c07816732e43a100fdf49ff"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"0fad50ae345fd7a070318c3586a16bad8d0a8055","unresolved":false,"context_lines":[{"line_number":18,"context_line":"  code paths with real system values."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Closes-Bug: #1847282"},{"line_number":21,"context_line":"Depends-On: https://review.opendev.org/#/c/688526/"},{"line_number":22,"context_line":"Change-Id: Id20d7415e8430c66f3afc8f86e0e813a54ee3ca2"},{"line_number":23,"context_line":"Signed-off-by: Kevin Carter \u003ckecarter@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"3fa7e38b_773c3630","line":21,"range":{"start_line":21,"start_character":12,"end_line":21,"end_character":49},"updated":"2019-10-21 08:38:58.000000000","message":"that\u0027s a DNM we still debugging here?","commit_id":"9639fd09e0478d46009f10a2905ce60fb60ea8fc"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"82e4b886776147d7e99e8dd333a348ead1619218","unresolved":false,"context_lines":[{"line_number":18,"context_line":"  code paths with real system values."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Closes-Bug: #1847282"},{"line_number":21,"context_line":"Depends-On: https://review.opendev.org/#/c/688526/"},{"line_number":22,"context_line":"Change-Id: Id20d7415e8430c66f3afc8f86e0e813a54ee3ca2"},{"line_number":23,"context_line":"Signed-off-by: Kevin Carter \u003ckecarter@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"3fa7e38b_9d29ac38","line":21,"range":{"start_line":21,"start_character":12,"end_line":21,"end_character":49},"in_reply_to":"3fa7e38b_773c3630","updated":"2019-10-23 07:55:31.000000000","message":"bump","commit_id":"9639fd09e0478d46009f10a2905ce60fb60ea8fc"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"4250adb740351dd65ddd160e5c526c5972c15a8e","unresolved":false,"context_lines":[{"line_number":18,"context_line":"  code paths with real system values."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Closes-Bug: #1847282"},{"line_number":21,"context_line":"Depends-On: https://review.opendev.org/#/c/688526/"},{"line_number":22,"context_line":"Change-Id: Id20d7415e8430c66f3afc8f86e0e813a54ee3ca2"},{"line_number":23,"context_line":"Signed-off-by: Kevin Carter \u003ckecarter@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"3fa7e38b_fd19c021","line":21,"range":{"start_line":21,"start_character":12,"end_line":21,"end_character":49},"in_reply_to":"3fa7e38b_9d29ac38","updated":"2019-10-23 07:56:31.000000000","message":"this will not merge. do we need that depends on? then we should probably merge it first.\notherwise remove it","commit_id":"9639fd09e0478d46009f10a2905ce60fb60ea8fc"}]}