<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; color: rgb(0, 0, 0); font-size: 14px; font-family: Calibri, sans-serif;">
<div>Hi Manila community, </div>
<div><br>
</div>
<div>This is regarding the "bug": <a href="https://bugs.launchpad.net/python-manilaclient/+bug/1457155">https://bugs.launchpad.net/python-manilaclient/+bug/1457155</a> in the python-manilaclient. </div>
<div>A commit was made for this and it merged yesterday: <a href="https://github.com/openstack/python-manilaclient/commit/37f2e50bd433149b893d30a478947f3e17f928e9">https://github.com/openstack/python-manilaclient/commit/37f2e50bd433149b893d30a478947f3e17f928e9</a> (<a rel="nofollow" href="https://review.openstack.org/264110" style="color: rgb(0, 51, 170); text-decoration: none; font-family: monospace; font-size: 12px;"></a><a href="https://review.openstack.org/264110">https:/</a><wbr><a href="https://review.openstack.org/264110">/review.</a><wbr><a href="https://review.openstack.org/264110">openstack.</a><wbr><a href="https://review.openstack.org/264110">org/264110</a>)</div>
<div><br>
</div>
<div>I disagree with the approach in this patch. I feel this bug is invalid. Deployers have a way to modify policies in "policy.json" as with any other OpenStack project. It would be extremely confusing to see this "Admin Only" added to certain commands that
<b>we</b> think will be "admin only" (as defined in the "default" policy.json). Essentially, ANY API we build can be exposed to the user (or some users); or administrators; as determined by the deployer.</div>
<div><br>
</div>
<div>IMHO, assuming that policies can change, we shouldn't hard code "admin only" as help text. Allow the manila-api service to respond to a request with a 403 if it deems fit; it can see the policy file and works with it. That's correct behavior, as is.</div>
<div><br>
</div>
<div>I feel we should revert this change in Mitaka before the feature freeze. </div>
<div><br>
</div>
<div>Thoughts?</div>
<div><br>
</div>
<div>Thanks, </div>
<div>Goutham</div>
<div><br>
</div>
<div>
<div id="MAC_OUTLOOK_SIGNATURE"></div>
</div>
</body>
</html>