> + .add("portRangeMax", portRangeMax) > + .add("protocol", protocol) > + .add("remoteGroupId", remoteGroupId) > + .add("remoteIpPrefix", remoteIpPrefix) > + .toString(); > + } > + > + > + /* > + * Methods to get the Create and Update builders follow > + */ > + > + /** > + * @return the Builder for creating a new SecurityGroupRule > + */ > + public static CreateBuilder createOptions(RuleDirection direction, String > securityGroupId) {
> It's a bit confusing. createOptions returns a builder that allows you to > create a CreateOptions object. Thanks for explaining - became clearer as I saw more of the code ;-) I'm still uneasy about the use of `*Options` as a name for a domain object class, even though with the intended usage pattern I guess it's meant to be hidden. I just sent an email to the dev list - hopefully someone will have some bright suggestion... --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/132/files#r17028061