)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"ef8efd35277af6f084e0f96d13721990cf8ed280","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Enable sssd service on worker node"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change enabled sssd service on work node."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Test Plan on Debian:"},{"line_number":12,"context_line":"PASS: After worker node is unlocked, verify that sssd service"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9df29c8e_c6913cc5","line":9,"range":{"start_line":9,"start_character":36,"end_line":9,"end_character":40},"updated":"2022-08-31 19:24:50.000000000","message":"nit: worker?","commit_id":"fd874b5a5ee8a8a1d219b2d8e1cd04ee38b40592"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":31646,"name":"Carmen Rata","email":"carmen.rata@windriver.com","username":"crata"},"change_message_id":"e24250af13320c4abf991cc55d56d758b19786f8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8b6228fb_bcb2b900","updated":"2022-08-31 18:51:26.000000000","message":"LGTM","commit_id":"fd874b5a5ee8a8a1d219b2d8e1cd04ee38b40592"}],"puppet-manifests/src/modules/platform/manifests/sssd.pp":[{"author":{"_account_id":31646,"name":"Carmen Rata","email":"carmen.rata@windriver.com","username":"crata"},"change_message_id":"56e455164c040e30f4c1d1947ab506b749f94e44","unresolved":true,"context_lines":[{"line_number":27,"context_line":"class platform::sssd"},{"line_number":28,"context_line":"  inherits ::platform::sssd::params {"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"  if $::personality \u003d\u003d \u0027controller\u0027 {"},{"line_number":31,"context_line":"    Class[\u0027::platform::ldap::server\u0027] -\u003e Class[$name]"},{"line_number":32,"context_line":"  }"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":1,"id":"fad28a93_ed18424a","line":30,"updated":"2022-08-31 18:44:27.000000000","message":"Do we also need to check for Debian OS here?","commit_id":"fd874b5a5ee8a8a1d219b2d8e1cd04ee38b40592"},{"author":{"_account_id":31646,"name":"Carmen Rata","email":"carmen.rata@windriver.com","username":"crata"},"change_message_id":"e24250af13320c4abf991cc55d56d758b19786f8","unresolved":false,"context_lines":[{"line_number":27,"context_line":"class platform::sssd"},{"line_number":28,"context_line":"  inherits ::platform::sssd::params {"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"  if $::personality \u003d\u003d \u0027controller\u0027 {"},{"line_number":31,"context_line":"    Class[\u0027::platform::ldap::server\u0027] -\u003e Class[$name]"},{"line_number":32,"context_line":"  }"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":1,"id":"bfaf762e_b905cab7","line":30,"in_reply_to":"2373ae85_965ece28","updated":"2022-08-31 18:51:26.000000000","message":"Ack","commit_id":"fd874b5a5ee8a8a1d219b2d8e1cd04ee38b40592"},{"author":{"_account_id":28676,"name":"Andy Ning","email":"andy.ning@windriver.com","username":"andy.wrs"},"change_message_id":"2f4951b6f7a2fb28e181514a612a7c3e3f818f09","unresolved":true,"context_lines":[{"line_number":27,"context_line":"class platform::sssd"},{"line_number":28,"context_line":"  inherits ::platform::sssd::params {"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"  if $::personality \u003d\u003d \u0027controller\u0027 {"},{"line_number":31,"context_line":"    Class[\u0027::platform::ldap::server\u0027] -\u003e Class[$name]"},{"line_number":32,"context_line":"  }"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":1,"id":"2373ae85_965ece28","line":30,"in_reply_to":"fad28a93_ed18424a","updated":"2022-08-31 18:48:22.000000000","message":"No, this change is not OS specific. It just to ensure the following dependency only applies to controller node (since ldap server doesn\u0027t exist on worker node).","commit_id":"fd874b5a5ee8a8a1d219b2d8e1cd04ee38b40592"}]}