<div dir="ltr"><div><div>Ruby,<br><br></div>What you say makes sense to me.  On keeping things consistent.  So sounds good to me to always use them and not have them be optional.<br><br></div>John<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 19, 2015 at 9:32 AM, Ruby Loo <span dir="ltr"><<a href="mailto:rlooyahoo@gmail.com" target="_blank">rlooyahoo@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">I think that if there is a use case for an *ED state, then we should have it. And if we have one *ED state, I think it makes sense (and is consistent) to have them for all the active states.<div><br></div><div>If we have *ED states, I would prefer that we add them in when the active state is added. So add <state>ING, <state>ED, <state>FAIL. If a particular driver has nothing it wants to do in an *ED state, it can cause a transition from the *ED state to the passive/stable state.</div><div><br></div><div>I don't want the *ED states to be optional because that puts the onus on the developer that needs the *ED state, to add it in (assuming they are aware that this is possible) and put in whatever plumbing might be needed. Which may mean that they'd have to modify code in another driver, that didn't need *ED in the first place. (If an *ED state is added, all drivers using that active state should handle the *ED state too because it is in the state machine and I'd rather not complicate things by having state-ING -> state-ED -> stable-state and state-ING -> stable-state.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>--ruby</div></font></span></div>
<br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><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>
<br></blockquote></div><br></div>