> + return this;
> + }
> +
> + public CreateStack build() {
> + String missing = "";
> +
> + if (name == null) {
> + missing += " name";
> + }
> +
> +
> + if (template == null && templateUrl == null) {
> + missing += " template and templateUrl";
> + }
> +
> + if (!missing.isEmpty()) {
@limorbortman Does heat fail properly with a decent descriptive fail message?
If so, I don't think we need this. It's basically a type of code/feature
duplication and the failing forward benefit does not seem that great to me in
this case.
However, if the heat validation message is something cryptic, I say keep this
code.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/188/files#r28357181
