)]}'
{"accessbot/channels.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"72666bd9b1a4623e3b4a5c74595cefd6dd8b9d31","unresolved":true,"context_lines":[{"line_number":88,"context_line":"  mode: \u0027+cnrtf #openstack-unregistered\u0027"},{"line_number":89,"context_line":"  masters:"},{"line_number":90,"context_line":"    - opendevaccess"},{"line_number":91,"context_line":"  # TODO (fungi): we need to double-check these nicks before we merge this"},{"line_number":92,"context_line":"  operators:"},{"line_number":93,"context_line":"    - AJaeger"},{"line_number":94,"context_line":"    - clarkb"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b7a46fdf_4d575759","line":91,"updated":"2021-05-24 19:28:10.000000000","message":"Note this config file is also consumed and applied by accessbot itself, not just the check script. We need to ensure that we update accessbot to configure oftc at the same time (or disable it).","commit_id":"9a9b42636369127a6e583d91a3aa356babc9fa95"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"8c48c9b76d0093fd170112328b8dfdb064346fd4","unresolved":false,"context_lines":[{"line_number":88,"context_line":"  mode: \u0027+cnrtf #openstack-unregistered\u0027"},{"line_number":89,"context_line":"  masters:"},{"line_number":90,"context_line":"    - opendevaccess"},{"line_number":91,"context_line":"  # TODO (fungi): we need to double-check these nicks before we merge this"},{"line_number":92,"context_line":"  operators:"},{"line_number":93,"context_line":"    - AJaeger"},{"line_number":94,"context_line":"    - clarkb"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"25eb2a9a_7842e83e","line":91,"updated":"2021-05-24 21:29:53.000000000","message":"Yep, that\u0027s part of why this change is still WIP, there will be a companion change to https://opendev.org/opendev/system-config/src/branch/master/docker/accessbot/accessbot.py but the overhaul there will likely be more significant since we need to also cover any changes in identifying to NickServ.","commit_id":"9a9b42636369127a6e583d91a3aa356babc9fa95"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"25e7598d3d517cb03d61619879d4e9e97638449d","unresolved":false,"context_lines":[{"line_number":82,"context_line":"  #  r : registered users only"},{"line_number":83,"context_line":"  #  t : op to modify topic"},{"line_number":84,"context_line":"  #  f : forward unregistered users to #openstack-unregistered"},{"line_number":85,"context_line":"  mode: \u0027+cnrtf #openstack-unregistered\u0027"},{"line_number":86,"context_line":"  masters:"},{"line_number":87,"context_line":"    - openstackinfra"},{"line_number":88,"context_line":"  operators:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"c7fd7783_f36b66ae","side":"PARENT","line":85,"updated":"2021-05-25 23:57:40.000000000","message":"As Clark pointed out in IRC, I really only meant to get rid of the f flag to forward channels and the r flag for registered users only. I need to dig up the mlock info for OFTC and figure out if these others have analogs there.","commit_id":"2180224020d740c968a745bb06448d6a24c94594"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"e6518f7012289e7f94f137f0bb27ec9e0fe12cf3","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"# Access levels (map names in this file to chanserv access levels):"},{"line_number":46,"context_line":"access:"},{"line_number":47,"context_line":"  full: MASTER"},{"line_number":48,"context_line":"  ops: CHANOP"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"# Define configuration that should apply to all channels."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"18580969_c3c08d43","line":47,"updated":"2021-05-25 21:14:34.000000000","message":"What do folks think about maybe using admin instead of full as our level name here?","commit_id":"ba004a959480f5807d8dbeda7f4b233dc32e6a05"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"af54ac04649b260a45e11e100fdd4c4f09a0d5de","unresolved":false,"context_lines":[{"line_number":80,"context_line":"# Individual channel configuration:"},{"line_number":81,"context_line":"channels:"},{"line_number":82,"context_line":"  - name: airshipit"},{"line_number":83,"context_line":"    allowed: full"},{"line_number":84,"context_line":"  - name: airshipit-gerritbot"},{"line_number":85,"context_line":"    allowed: full"},{"line_number":86,"context_line":"  - name: cloudkitty"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"52e5aff2_64cdda93","line":83,"updated":"2021-05-25 21:27:32.000000000","message":"I\u0027m not sure that this really makes sense as an attempt at translating the old per-channel op mask mechanism we were using before. How about adding keys like ops or full/admin here which contain lists of nicks, in order to be clear about the access levels of channel-specific ops managed via code review? I also think accessbot could probably get out of the business of removing access list entries which don\u0027t exist in our lists.","commit_id":"ba004a959480f5807d8dbeda7f4b233dc32e6a05"}],"tools/check_irc_access.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"72666bd9b1a4623e3b4a5c74595cefd6dd8b9d31","unresolved":true,"context_lines":[{"line_number":51,"context_line":"        self.advance()"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    def on_privnotice(self, c, e):"},{"line_number":54,"context_line":"        msg \u003d e.arguments[0]"},{"line_number":55,"context_line":"        self.advance(msg)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def advance(self, msg\u003dNone):"}],"source_content_type":"text/x-python","patch_set":1,"id":"f7ae21ec_eb41d293","line":54,"updated":"2021-05-24 19:28:10.000000000","message":"Previously msg was defined as:\n\n  msg \u003d e.arguments[0][1:]\n\nI don\u0027t expect the format of the messages changes between freenode and oftc. Do we need to take the later slice here? Or do we want to include the auth info in the msg?","commit_id":"9a9b42636369127a6e583d91a3aa356babc9fa95"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"8c48c9b76d0093fd170112328b8dfdb064346fd4","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        self.advance()"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    def on_privnotice(self, c, e):"},{"line_number":54,"context_line":"        msg \u003d e.arguments[0]"},{"line_number":55,"context_line":"        self.advance(msg)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def advance(self, msg\u003dNone):"}],"source_content_type":"text/x-python","patch_set":1,"id":"08ee98a6_c36372c8","line":54,"updated":"2021-05-24 21:29:53.000000000","message":"The format of the messages changes between freenode and oftc. On Freenode we rely on the identify-msg capability which prefixes identified nicks with a \"+\", and the [1:] was subsequently stripping that from each nick prior to comparison. That is not consistent behavior across networks.","commit_id":"9a9b42636369127a6e583d91a3aa356babc9fa95"}]}