)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":32870,"name":"Adriano Oliveira","email":"adriano.oliveira@windriver.com","username":"aoliveir"},"change_message_id":"afe8c8d15b29bea198e2963bbcf340b51ff684ee","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Heron Vieira \u003cheron.vieira@windriver.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-10-13 16:28:52 -0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"K8s Coredump - Authentication and configuration as default core handler"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Create and configure kubectl token for the k8s coredump handler"},{"line_number":10,"context_line":"in the path /etc/k8s-coredump-conf.json"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff15cec1_0811ced6","line":7,"updated":"2022-10-14 17:53:59.000000000","message":"First line exceeding 50 characters","commit_id":"ab36d8d18492f0750f236aaa20e29e9d048ee69e"},{"author":{"_account_id":32870,"name":"Adriano Oliveira","email":"adriano.oliveira@windriver.com","username":"aoliveir"},"change_message_id":"afe8c8d15b29bea198e2963bbcf340b51ff684ee","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2022-10-13 16:28:52 -0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"K8s Coredump - Authentication and configuration as default core handler"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Create and configure kubectl token for the k8s coredump handler"},{"line_number":10,"context_line":"in the path /etc/k8s-coredump-conf.json"},{"line_number":11,"context_line":"Also configure k8s-coredump-handler as default handler by modifying"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"cd3e8d65_be9baa60","line":8,"updated":"2022-10-14 17:53:59.000000000","message":"Also, this should be more related to the change on this repo, maybe something like \"K8s Coredump: Authentication (puppet)\", else same subject as other reviews.","commit_id":"ab36d8d18492f0750f236aaa20e29e9d048ee69e"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":32870,"name":"Adriano Oliveira","email":"adriano.oliveira@windriver.com","username":"aoliveir"},"change_message_id":"e89095359a17b3729f61af278fd73bcb95adc9d6","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"95888366_1e226458","updated":"2022-10-17 15:37:02.000000000","message":"This is the last change on the feature, which means with this change the default coredump handler on the system will be replaced, as such, this should already contain all manual tests that show that the k8s applications that does not have annotations defined, as well the non k8s applications continue generating coredump as before (so more testing shall be defined).\nElse, I would recommend to not replace the default coredump handler yet and only do it when all the manual test is done.","commit_id":"ab36d8d18492f0750f236aaa20e29e9d048ee69e"}],"puppet-manifests/src/modules/platform/manifests/coredump.pp":[{"author":{"_account_id":33620,"name":"Samuel Presa Toledo","email":"samuel.presatoledo@windriver.com","username":"spresato"},"change_message_id":"a4d41dcda6e5bbc5f5f79d7b8909a2a834520af4","unresolved":true,"context_lines":[{"line_number":26,"context_line":"  exec {\u0027restart-coredump-service\u0027:"},{"line_number":27,"context_line":"    command \u003d\u003e \u0027sysctl -p /etc/sysctl.d/50-coredump.conf\u0027"},{"line_number":28,"context_line":"  }"},{"line_number":29,"context_line":"  "},{"line_number":30,"context_line":"  exec {\u0027configure-k8s-coredump-handler\u0027:"},{"line_number":31,"context_line":"    command \u003d\u003e \u0027sysctl -w kernel.core_pattern\u003d\"|/usr/bin/k8s-coredump %P %u %g %s %t 9223372036854775808 %h %e\"\u0027"},{"line_number":32,"context_line":"  }"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"c744eb8a_a13f2375","line":29,"range":{"start_line":29,"start_character":0,"end_line":29,"end_character":2},"updated":"2022-10-17 16:46:31.000000000","message":"tox raised these extra spaces as an error","commit_id":"ab36d8d18492f0750f236aaa20e29e9d048ee69e"}]}
