[glue-wg] GLUE2.1 draft: request for final discussion and approval

Stephen Burke - UKRI STFC stephen.burke at stfc.ac.uk
Sun Jun 3 10:28:51 EDT 2018


> I'll stop there and come back later ...

CloudComputingImage: conceptually this seems like another kind of Resource. I don't think we ever considered having two types of Resource in one Service and maybe it's better not to do it explicitly, but I'd expect it to be similar, and it is except for not having a relationship to Manager. Doesn't a Manager manage Images too? There's also no relationship to InstanceType - does that mean that any VM can run any OS? For the first two attributes I have the same comments as for InstanceType. DefaultPassword is a bit surprising, should that be published?

I find ImageAcceleratorInfo a bit surprising, are there OS images that require GPUs? And if so, it seems unlikely that there are images that require more than one type?

NetworkTraffic doesn't seem like the right name, maybe NetworkConfiguration? And is it really part of the Image rather than the Instance? Maybe I just don't understand what it's for, the explanation isn't very detailed.

CloudComputingInstance: again I don't especially like the ID names, maybe ExternalID and InternalID? It also seems like at least one of those should be mandatory so you can tell which job the object refers to. For Owner I'm not sure why you don't just make it an optional attribute and remove the CONFIDENTIAL value? You have two attributes with ComputeManager in the name, should be ComputingManager.

ToStorageService: again I'd call the IDForEndpoint attribute something different, maybe LocationID? 


...

StorageSharePath: it still isn't clear to me that multiplicity * makes sense. This is supposed to be a default path, if there are several you don't know which to use. Also if there is any software using this (possibly not) it would expect a unique result and probably break if it got more than one. If this is really wanted I would say add a new multivalued attribute Paths or OtherPaths. (AccessMode also has an underline, was there a change for that?)

Stephen




More information about the glue-wg mailing list