On 07/26/2012 04:33 PM, Adam Litke wrote:
On Thu, Jul 26, 2012 at 11:47:51AM +0300, Itamar Heim wrote:
On 07/17/2012 01:19 AM, Itamar Heim wrote:
On 07/09/2012 09:52 PM, Saggi Mizrahi wrote:
----- Original Message -----
From: "Itamar Heim" iheim@redhat.com To: "Saggi Mizrahi" smizrahi@redhat.com Cc: "Adam Litke" agl@us.ibm.com, vdsm-devel@lists.fedorahosted.org Sent: Monday, July 9, 2012 11:03:43 AM Subject: Re: [vdsm] [RFC] An alternative way to provide a supported interface -- libvdsm
On 07/09/2012 05:56 PM, Saggi Mizrahi wrote:
I don't think AMQP is a good low level supported protocol as it's a very complex protocol to set up and support. Also brokers are known to have their differences in standard implementation which means supporting them all is a mess.
It looks like the most accepted route is the libvirt route of having a c library abstracting away client server communication and having more advanced consumers build protocol specific bridges that may have different support standards.
On a more personal note, I think brokerless messaging is the way to go in ovirt because, unlike traditional clustering, worker nodes are not interchangeable so direct communication is the way to go, rendering brokers pretty much useless.
but brokerless doesn't let multiple consumers which a bus provides?
All consumers can connect to the host and *some* events can be broadcasted to all connected clients.
The real question is weather you want to depend on AMQP's routing \ message storing Also, if you find it preferable to have a centralized host (single point of failure) to get all events from all hosts for the price of some clients (I assume read only clients) not needing to know the locations of all worker nodes. But IMHO we already have something like that, it's called the ovirt-engine, and it could send aggregated events about the cluster (maybe with some extra enginy data).
The question is what does mandating a broker gives us something that an "AMQP bridge" wouldn't. The only thing I can think of is vdsm can assume unmoderated vdsm to vdsm communication bypassing the engine. This means that VDSM can have some clustered behavior that requires no engine intervention. Further more, the engine can send a request and let the nodes decide who is performing the operation among themselves.
Essentially:
[ engine ] [ engine ] | | VS | [vdsm][vdsm] [ broker ] | | [vdsm][vdsm]
*All links are two way links
This has dire consequences on API usability and supportability. So we need to converge on that.
There needs to be a good reason why the aforementioned logic code can't sit on a another ovirt specific entity (lets call it ovirt-dynamo) that uses VDSM's supported API but it's own APIs (or more likely messaging algorithms) are unsupported.
[ engine ] | | | | [ broker ] | | | | | [vdsm]-[dynamo] : [dynamo]-[vdsm] Host A : Host B
*All links are two way links
- we have engine today 'in the path' to the history db. but it makes no
sense for engine to be aware of each statistic we want to keep in the history db. same would be for an event/stats correlation service. they don't need to depend on each other for availability/redundancy.
- we are already looking at quantum integration, which is doing engine
to nodes communication via amqp.
- with somewhat of a forward looking - moving some scheduling logic
"down to vdsm" will probably mean we'll want one of the nodes to listen to statistics and state from the other nodes.
to all of these, setting up a bus which allows multiple peer listeners seems more robust
I'm still against developing a C level binding for amqp and rest support over a codebase which is in python. rest and amqp allow for both local and remote bindings in any language. C bindings should/could be a parallel implementation, but they seem like an unneeded overhead and complexity in the middle of the codebase.
Sure, it's probably possible to bind a REST or AMQP API in other languages but I don't think there is an automatic way of doing it. That means having to keep up with maintenance of each and every binding every time the API changes. If we look at libvirt, they will say this is a large source of pain that they have recommended we avoid.
we'd need this on top of the C api as well - but it would probably be simpler doing it over the python api, rather than the C one.
For the C/gobject approach, we write a single API schema file. From that, we automatically generate the C API and bindings. Sure, the generation could be a bit complex but much of it will be someone else's codebase (and one that is used by lots of Gnome projects).
that's a critical part of the question for the C api - what is needed for adding a verb / parameter.
I'm not against having c bindings. what i don't understand is why to put them in the 'middle', rather than as peers to the others on top of the python api classes.