<div dir="ltr"><div><div>I think the metadata in server group is an important feature and it might be used by <a href="https://blueprints.launchpad.net/nova/+spec/soft-affinity-for-server-group">https://blueprints.launchpad.net/nova/+spec/soft-affinity-for-server-group</a><br>
<br></div>Actually, we are now doing an internal development for above bp and want to contribute this back to community later. We are now setting hard/soft flags in server group metadata to identify if the server group want hard/soft affinity.<br>
<br></div><div>I prefer Dan's first suggestion, what do you think?<br>=====================<br>If we care to have this functionality, then I propose we change the<br>
attribute on the object (we can handle this with versioning) and reflect<br>
it as "metadata" in the API.<br>=====================<br></div><div><br></div>Thanks!<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-08-12 0:50 GMT+08:00 Sylvain Bauza <span dir="ltr"><<a href="mailto:sbauza@redhat.com" target="_blank">sbauza@redhat.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Le 11/08/2014 18:03, Gary Kotton a écrit :<div><div class="h5"><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On 8/11/14, 6:06 PM, "Dan Smith" <<a href="mailto:dms@danplanet.com" target="_blank">dms@danplanet.com</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
As the person who -2'd the review, I'm thankful you raised this issue on<br>
the ML, Jay. Much appreciated.<br>
</blockquote>
The "metadetails" term isn't being invented in this patch, of course. I<br>
originally complained about the difference when this was being added:<br>
<br>
<a href="https://review.openstack.org/#/c/109505/1/nova/api/openstack/compute/contr" target="_blank">https://review.openstack.org/#<u></u>/c/109505/1/nova/api/<u></u>openstack/compute/contr</a><br>
ib/server_groups.py,cm<br>
<br>
As best I can tell, the response in that patch set about why it's being<br>
translated is wrong (backwards). I expect that the API extension at the<br>
time called it "metadetails" and they decided to make the object the<br>
same and do the translation there.<br>
<br>
</blockquote>
>From what I can tell, the actual server_group API extension that made it<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
into the tree never got the ability to set/change/etc the<br>
metadata/metadetails anyway, so there's no reason (AFAICT) to add it in<br>
wrongly.<br>
<br>
If we care to have this functionality, then I propose we change the<br>
attribute on the object (we can handle this with versioning) and reflect<br>
it as "metadata" in the API.<br>
<br>
However, I have to ask: do we really need another distinct metadata<br>
store attached to server_groups? If not, how about we just remove it<br>
</blockquote>
>from the database and the object, clean up the bit of residue that is<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
still in the API extension and be done with it?<br>
</blockquote>
The initial version of the feature did not make use of this. The reason<br>
was that we chose for a very<br>
Limited subset to be used, that is, affinity and anti affinity. Moving<br>
forwards we would like to implement<br>
A number of different policies with this. We can drop it at the moment due<br>
to the fact that it is not used.<br>
<br>
I think that Yathi may be using this for the constrain scheduler. But I am<br>
not 100% sure.<br>
</blockquote>
<br>
<br></div></div>
Unless I'm wrong, I can't see where this metadata is being used in the scheduler, either for filtering or for other reasons.<br>
<br>
So, please give us context why this is currently useful ?<br>
<br>
If this is something for the next future, I would love discussing it with regards to the current split.<br>
<br>
<br>
Thanks,<br>
-Sylvain<div class="HOEnZb"><div class="h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
--Dan<br>
<br>
</blockquote>
<br>
______________________________<u></u>_________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.<u></u>org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-dev</a><br>
</blockquote>
<br>
<br>
______________________________<u></u>_________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.<u></u>org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div dir="ltr"><div>Thanks,<br><br></div>Jay<br></div>
</div>