On Mon, Mar 23, 2015 at 10:20:31AM +0100, Markus Armbruster wrote: > > +- "device": Device name (json-string) > > +- "node-name": Node name, if it's present (json-string) > > By convention, we mark optional members like this: > > - "node-name": Node name (json-string, optional)
You're right, thanks for pointing that out! This can be fixed when committing the patch, otherwise I can resend the series again. > Ignorant question: can "device" be ""? If yes, we should document > what that means, possibly in a separate patch. Yes it can be "", the purpose of the patch is precisely to expand the event with a new field that can identify the node in such cases. The only reason why it is not optional is to keep backward compatibility. But it's true that it can be clarified in the documentation, I can send a follow-up patch as soon as this one is merged. Berto
