<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><br>[0] <a href="https://github.com/stackforge/fuel-web/blob/master/nailgun/nailgun/task/task.py#L1160-L1163">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 class="h5"><br>
<br>
On Mon, Aug 31, 2015 at 2:54 PM, Roman Prykhodchenko <<a href="mailto:me@romcheg.me">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 class="gmail_signature">Andrey Danin<br><a href="mailto:adanin@mirantis.com" target="_blank">adanin@mirantis.com</a><br>skype: gcon.monolake<br></div>
</div>