<div dir="ltr">We have more generic ticket: <a href="https://bugs.launchpad.net/fuel/+bug/1354803">https://bugs.launchpad.net/fuel/+bug/1354803</a> and corresponding CR: <a href="https://review.openstack.org/#/c/245941/">https://review.openstack.org/#/c/245941/</a><br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr">Aleksey Kasatkin
<br><br></div></div></div>
<br><div class="gmail_quote">On Fri, Nov 20, 2015 at 11:24 AM, Aleksey Kasatkin <span dir="ltr"><<a href="mailto:akasatkin@mirantis.com" target="_blank">akasatkin@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>It's not about Public networks only. There can be the same problem with other networks as well.<br></div>It's required to check all the networks (across all node groups).<br></div>But it is done just for Public network now (and VIPs for plugins are not taken into account).<span class="HOEnZb"><font color="#888888"><br><div><div><br></div></div></font></span></div><div class="gmail_extra"><span class="HOEnZb"><font color="#888888"><br clear="all"><div><div><div dir="ltr">Aleksey Kasatkin
<br><br></div></div></div></font></span><div><div class="h5">
<br><div class="gmail_quote">On Fri, Nov 20, 2015 at 12:04 AM, Andrew Woodward <span dir="ltr"><<a href="mailto:xarses@gmail.com" target="_blank">xarses@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">The high value of the bug here reflects that the error message is wrong. From a UX side we could maybe even justify this as Critical. The error message must reflect the correct quantity of addresses required.<div><br></div><br><div class="gmail_quote"><span><div dir="ltr">On Tue, Nov 17, 2015 at 1:31 PM Roman Prykhodchenko <<a href="mailto:me@romcheg.me" target="_blank">me@romcheg.me</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Folks, we should resurrect this thread and find a consensus.<div><br><div><blockquote type="cite"><div>1 вер. 2015 р. о 15:00 Andrey Danin <<a href="mailto:adanin@mirantis.com" target="_blank">adanin@mirantis.com</a>> написав(ла):</div></blockquote></div></div></div><div style="word-wrap:break-word"><div><div><blockquote type="cite"><br><div><div dir="ltr"><div>+1 to Igor.<br><br></div>It's definitely not a High bug. The biggest problem I see here is a confusing error message with a wrong number of required IPs. AFAIU we cannot fix it easily now so let's postpone it to 8.0 but change a message itself [0] in 7.0.<br></div></div></blockquote></div></div></div></blockquote></span><div>We managed to create an error that returns '7', when there are 8 available, but 9 are required, at some level we knew that we came up short or we'd just have some lower level error caught here.</div><div><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div><blockquote type="cite"><div><div dir="ltr"><br>[0] <a href="https://github.com/stackforge/fuel-web/blob/master/nailgun/nailgun/task/task.py#L1160-L1163" target="_blank">https://github.com/stackforge/fuel-web/blob/master/nailgun/nailgun/task/task.py#L1160-L1163</a><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 1, 2015 at 1:39 PM, Igor Kalnitsky <span dir="ltr"><<a href="mailto:ikalnitsky@mirantis.com" target="_blank">ikalnitsky@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
My 5 cents on it.<br>
<br>
I don't think it's really a High or Critical bug for 7.0. If there's<br>
not enough IPs the CheckBeforeDeploymentTask will fail. And that's<br>
actually Ok, it may fail by different reason without starting actual<br>
deployment (sending message to Astute).<br>
<br>
But I agree it's kinda strange that we don't check IPs during network<br>
verification step. The good fix in my opinion is to move this check<br>
into network checker (perhaps keep it here either), but that<br>
definitely shouldn't be done in 7.0.<br>
<br>
Thanks,<br>
Igor<br>
<div><div><br>
<br>
On Mon, Aug 31, 2015 at 2:54 PM, Roman Prykhodchenko <<a href="mailto:me@romcheg.me" target="_blank">me@romcheg.me</a>> wrote:<br>
> Hi folks!<br>
><br>
> Recently a problem that network check does not tell whether there’s enough IP addresses in a public network [1] was reported. That check is performed by CheckBeforeDeployment task, but there is two problems that happen because this verification is done that late:<br>
><br>
>  - A deployment fails, if there’s not enough addresses in specified ranges<br>
>  - If a user wants to get network configuration they will get an error<br>
><br>
> The solution for this problems seems to be easy and a straightforward patch [2] was proposed. However, there is a hidden problem which is that patch does not address which is that installed plugins may reserve VIPs for their needs. The issue is that they do it just before deployment and so it’s not possible to get those reservations when a user wants to check their network set up.<br>
><br>
> The important issue we have to address here is that network configuration generator will fail, if specified ranges don’t fit all VIPs. There were several proposals to fix that, I’d like to highlight two of them:<br>
><br>
>  a) Allow VIPs to not have an IP address assigned, if network config generator works for API output.<br>
>      That will prevent GET requests from failure, but since IP addresses for VIPs are required, generator will have to fail, if it generates a configuration for the orchestrator.<br>
>  b) Add a release note that users have to calculate IP addresses manually and put sane ranges in order to not shoot their own legs. Then it’s also possible to change network verification output to remind users to check the ranges before starting a deployment.<br>
><br>
> In my opinion we cannot follow (a) because it only masks a problem instead of providing a fix. Also it requires to change the API which is not a good thing to do after the SCF. If we choose (b), then we can work on a firm solution in 8.0 and fix the problem for real.<br>
><br>
><br>
> P. S. We can still merge [2], because it checks, if IP ranges can at least fit the basic configuration. If you agree, I will update it soon.<br>
><br>
> [1] <a href="https://bugs.launchpad.net/fuel/+bug/1487996" rel="noreferrer" target="_blank">https://bugs.launchpad.net/fuel/+bug/1487996</a><br>
> [2] <a href="https://review.openstack.org/#/c/217267/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/217267/</a><br>
><br>
><br>
><br>
> - romcheg<br>
><br>
><br>
><br>
><br>
</div></div>> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org/?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org/?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br><br clear="all"><br>-- <br><div>Andrey Danin<br><a href="mailto:adanin@mirantis.com" target="_blank">adanin@mirantis.com</a><br>skype: gcon.monolake<br></div>
</div>
__________________________________________________________________________<br>OpenStack Development Mailing List (not for usage questions)<br>Unsubscribe: <a href="mailto:OpenStack-dev-request@lists.openstack.org" target="_blank">OpenStack-dev-request@lists.openstack.org</a>?subject:unsubscribe<br><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br></div></blockquote></div></div></div>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div></div></div></div><span><font color="#888888"><div dir="ltr">-- <br></div><div dir="ltr"><p dir="ltr">--</p><p dir="ltr"><span style="font-size:13.1999998092651px">Andrew Woodward</span></p><p dir="ltr"><span style="font-size:13.1999998092651px">Mirantis</span></p><p dir="ltr"><span style="font-size:13.1999998092651px">Fuel Community Ambassador</span></p><p dir="ltr"><span style="font-size:13.1999998092651px">Ceph Community</span></p>
</div>
</font></span><br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div></div></div>
</blockquote></div><br></div>