<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 9, 2015 at 12:51 PM, Przemyslaw Kaminski <span dir="ltr"><<a href="mailto:pkaminski@mirantis.com" target="_blank">pkaminski@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 id=":41y" class="a3s" style="overflow:hidden">Well, there are some problems with this solution:<br>
1. No 'pick latest one with filtering to network_verify' handler is<br>
available currently.</div></blockquote><div> </div><div>Well i think there should be finished_at field anyway, why not to add it for this purpose? </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":41y" class="a3s" style="overflow:hidden">
2. Tasks are ephemeral entities -- they get deleted here and there.<br>
Look at nailgun/task/manager.py for example -- lines 83-88 or lines<br>
108-120 and others<br></div></blockquote><div> </div><div>I dont actually recall what was the reason to delete them, but if it happens imo it is ok to show right now</div><div>that network verification wasnt performed.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":41y" class="a3s" style="overflow:hidden">
3. Just having network verification status as ready is NOT enough.<br>
>From the UI you can fire off network verification for unsaved changes.<br>
Some JSON request is made, network configuration validated by tasks<br>
and RPC call made returing that all is OK for example. But if you<br>
haven't saved your changes then in fact you haven't verified your<br>
current configuration, just some other one. So in this case task<br>
status 'ready' doesn't mean that current cluster config is valid. What<br>
do you propose in this case? Fail the task on purpose? I only see a</div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":41y" class="a3s" style="overflow:hidden">
solution to this by introducting a new flag and network_check_status<br>
seems to be an appropriate one.</div></blockquote><div> </div><div>My point that it has very limited UX. Right now network check is:</div><div>- l2 with vlans verication</div><div>- dhcp verification</div><div><br></div><div>When we will have time we will add:</div><div>- multicast routing verification</div><div>- public gateway</div><div>Also there is more stuff that different users was asking about.</div><div><br></div><div>Then i know that vmware team also wants to implement pre_deployment verifications.</div><div><br></div><div>So what this net_check_status will refer to at that point?</div><div><br></div><div><br></div></div><br><br></div></div>