<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>