<html>
  <head>

    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <font size="-1"><tt>Today in keystone's office hours, we went
        through a group code review of what's currently proposed for the
        oslo.limit library [0]. This is a summary of the action items
        that came out of that meeting.<br>
        <br>
        * We should implement a basic functional testing framework that
        exercises keystoneauth connections (used for pulling limit
        information for keystone). Otherwise, we'll be mocking things
        left and right in unit tests to get decent test coverage with
        the current keystoneauth code.<br>
        * Investigate alternatives to globals for keystoneauth
        connections [1].<br>
        * Investigate adopting a keystoneauth-like way of loading
        enforcement models (similar to how ksa loads authentication
        plugins) [2].<br>
        * Figure out if we want to use endpoint_id or service name +
        region name for service configuration [3].<br>
        * Build out functional testing for flat enforcement<br>
        * Implement strict-two-level enforcement model<br>
        <br>
        This existing rewrite was mostly stolen from John's patches to
        his fork oslo.limit [4]. Hopefully the current series moves
        things in that direction.<br>
        <br>
        Feel free to chime in if you have additional notes or comments.<br>
        <br>
        Lance<br>
        <br>
        [0] </tt><tt><a
href="https://review.opendev.org/#/q/topic:rewrite+(status:open+OR+status:merged)+project:openstack/oslo.limit">https://review.opendev.org/#/q/topic:rewrite+(status:open+OR+status:merged)+project:openstack/oslo.limit</a><br>
        [1] </tt><a
        href="https://bugs.launchpad.net/oslo.limit/+bug/1835103">https://bugs.launchpad.net/oslo.limit/+bug/1835103</a><br>
      [2] <a href="https://bugs.launchpad.net/oslo.limit/+bug/1835104">https://bugs.launchpad.net/oslo.limit/+bug/1835104</a><br>
      [3] <a href="https://bugs.launchpad.net/oslo.limit/+bug/1835106">https://bugs.launchpad.net/oslo.limit/+bug/1835106</a><br>
      [4] <a
href="https://github.com/JohnGarbutt/oslo.limit/commit/a5b908046fd904c25b6cd15c65266c747774b5ab">https://github.com/JohnGarbutt/oslo.limit/commit/a5b908046fd904c25b6cd15c65266c747774b5ab</a></font>
  </body>
</html>