<html dir="ltr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
<style>
<!--
@font-face
        {font-family:Wingdings}
@font-face
        {font-family:SimSun}
@font-face
        {font-family:"Cambria Math"}
@font-face
        {font-family:Calibri}
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif}
a:link, span.MsoHyperlink
        {color:#0563C1;
        text-decoration:underline}
a:visited, span.MsoHyperlinkFollowed
        {color:#954F72;
        text-decoration:underline}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
        {margin-top:0in;
        margin-right:0in;
        margin-bottom:0in;
        margin-left:.5in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif}
span.EmailStyle17
        {font-family:"Calibri",sans-serif;
        color:windowtext}
.MsoChpDefault
        {font-family:"Calibri",sans-serif}
@page WordSection1
        {margin:1.0in 1.0in 1.0in 1.0in}
ol
        {margin-bottom:0in}
ul
        {margin-bottom:0in}
-->
</style><style type="text/css" id="owaParaStyle"></style>
</head>
<body lang="EN-US" link="#0563C1" vlink="#954F72" fpstyle="1" ocsi="0">
<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">Hello,
<div><br>
</div>
<div>I havent' tested that part of the code in some time now, but from what I remember it was part of a larger set of changes to prevent ceph from hanging if not enough space was available.</div>
<div><br>
</div>
<div>How it should work (but as I said, I haven't tested this in some time):</div>
<div>-  decrease the glance image pool quota (this is just so it's easier to test)</div>
<div>-  upload images to glance until space if almost full</div>
<div>- the image that would have filled the pool should fail to upload entirely.</div>
<div><br>
</div>
<div>It seems you are doing the correct procedure from what you described, so this probably requires a more in-depth investigation to see why it fails.<br>
<div style="font-family: Times New Roman; color: #000000; font-size: 16px">
<hr tabindex="-1">
<div id="divRpF493901" style="direction: ltr;"><font face="Tahoma" size="2" color="#000000"><b>From:</b> Fang, Liang A [liang.a.fang@intel.com]<br>
<b>Sent:</b> Saturday, November 10, 2018 11:23 AM<br>
<b>To:</b> Badea, Daniel; Dinescu, Stefan<br>
<b>Cc:</b> starlingx-discuss@lists.starlingx.io<br>
<b>Subject:</b> [Starlingx-discuss] [glance] Analysis of patch c73ab48 (Enhanced-error-messages) for StartlingX upstreaming<br>
</font><br>
</div>
<div></div>
<div>
<div class="WordSection1">
<p class="MsoNormal">Hi Daniel and Stefan</p>
<p class="MsoNormal"> </p>
<p class="MsoNormal">Please refer to <a href="https://review.openstack.org/#/c/616880/" target="_blank" rel="noopener noreferrer">
https://review.openstack.org/#/c/616880/</a></p>
<p class="MsoNormal">This patch is to provide detailed exception info when no space in rbd. But in my environment, if rbd has no space, the function call to rbd will not return, there’s no exception raised.</p>
<p class="MsoNormal"> </p>
<p class="MsoNormal">Details:</p>
<p class="MsoNormal">In /usr/local/lib/python2.7/dist-packages/glance_store/_drivers/rbd.py</p>
<p class="MsoListParagraph" style="text-indent:-.25in"><span style="">-<span style="font:7.0pt "Times New Roman"">         
</span></span>Function call will hang at line 484 (_create_image) if no space to create image at the beginning</p>
<p class="MsoNormal" style="text-indent:.5in"><img id="Picture_x0020_1" src="cid:image001.png@01D47916.203BFA30" height="27" width="599" border="0"></p>
<p class="MsoListParagraph" style="text-indent:-.25in"><span style="">-<span style="font:7.0pt "Times New Roman"">         
</span></span>Function call will hang at line 506 (image.resize) if no space at the middle of uploading.</p>
<p class="MsoListParagraph"><img id="Picture_x0020_2" src="cid:image002.png@01D47917.1BBFA460" height="17" width="437" border="0"></p>
<p class="MsoNormal"> </p>
<p class="MsoNormal">Could you please give some clue to reproduce this patch (raise rbd.NoSpace)?</p>
<p class="MsoNormal">Couple of other thought:</p>
<p class="MsoListParagraph" style="text-indent:-.25in"><span style="">1.<span style="font:7.0pt "Times New Roman"">      
</span></span>This patch is to catch exception at v1 api, but v1 is about to be removed by community.</p>
<p class="MsoListParagraph" style="text-indent:-.25in"><span style="">2.<span style="font:7.0pt "Times New Roman"">      
</span></span>rbd is just one type of the multiple types of backend, so I think catch specific backend’s exception at a common code is not so good. glance_store/rbd.py may be the better place to catch this exception. In fact rbd.py already did similar thing,
 see:</p>
<p class="MsoListParagraph"><img id="Picture_x0020_3" src="cid:image003.png@01D47919.B20AE9F0" height="90" width="565" border="0"></p>
<p class="MsoNormal"> </p>
<p class="MsoNormal">Thanks</p>
<p class="MsoNormal">Liang</p>
</div>
</div>
</div>
</div>
</div>
</body>
</html>