<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <tt><font size="-1">I forgot to add that the meeting was held in
        IRC. Logs are available if you're interested in following along
        [0].<br>
        <br>
        [0] </font></tt><tt><font size="-1"><a
href="http://eavesdrop.openstack.org/irclogs/%23openstack-keystone/%23openstack-keystone.2019-07-02.log.html#t2019-07-02T16:32:50">http://eavesdrop.openstack.org/irclogs/%23openstack-keystone/%23openstack-keystone.2019-07-02.log.html#t2019-07-02T16:32:50</a><br>
        <br>
      </font></tt>
    <div class="moz-cite-prefix">On 7/2/19 2:34 PM, Lance Bragstad
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:e1f1008c-27a6-2928-4b97-30e0e34cd806@gmail.com">
      <meta http-equiv="content-type" content="text/html; charset=UTF-8">
      <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"
            moz-do-not-send="true">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"
          moz-do-not-send="true">https://bugs.launchpad.net/oslo.limit/+bug/1835103</a><br>
        [2] <a
          href="https://bugs.launchpad.net/oslo.limit/+bug/1835104"
          moz-do-not-send="true">https://bugs.launchpad.net/oslo.limit/+bug/1835104</a><br>
        [3] <a
          href="https://bugs.launchpad.net/oslo.limit/+bug/1835106"
          moz-do-not-send="true">https://bugs.launchpad.net/oslo.limit/+bug/1835106</a><br>
        [4] <a
href="https://github.com/JohnGarbutt/oslo.limit/commit/a5b908046fd904c25b6cd15c65266c747774b5ab"
          moz-do-not-send="true">https://github.com/JohnGarbutt/oslo.limit/commit/a5b908046fd904c25b6cd15c65266c747774b5ab</a></font>
    </blockquote>
    <br>
  </body>
</html>