Re: [jclouds] Split availability Zone API to return Basic Info and Detailed info (#815)

2015-07-16 Thread Zack Shoylev
Closed #815. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/815#event-357808807

Re: [jclouds] Split availability Zone API to return Basic Info and Detailed info (#815)

2015-07-16 Thread Zack Shoylev
merged, backported --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/815#issuecomment-122041635

Re: [jclouds] Split availability Zone API to return Basic Info and Detailed info (#815)

2015-07-15 Thread Zack Shoylev
Going to try rebuilding... --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/815#issuecomment-121796608

Re: [jclouds] Split availability Zone API to return Basic Info and Detailed info (#815)

2015-07-15 Thread Zack Shoylev
Closed #815. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/815#event-357070225

Re: [jclouds] Split availability Zone API to return Basic Info and Detailed info (#815)

2015-07-15 Thread Zack Shoylev
Reopened #815. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/815#event-357070227

Re: [jclouds] Split availability Zone API to return Basic Info and Detailed info (#815)

2015-07-15 Thread arvindn05
Thanks for the review. @zack-shoylev Added few more negetive test cases and comments. Leaving the host service as is for now. If there are other domain classes which need to use host service domain class, we can move it out. --- Reply to this email directly or view it on GitHub: https://github.

Re: [jclouds] Split availability Zone API to return Basic Info and Detailed info (#815)

2015-07-15 Thread Zack Shoylev
Can we add some of your first comment in this PR as in-code docs for the domain classes? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/815#issuecomment-121768474

Re: [jclouds] Split availability Zone API to return Basic Info and Detailed info (#815)

2015-07-15 Thread Zack Shoylev
> @@ -34,7 +36,7 @@ > @Test(groups = "unit", testName = "AvailabilityZoneApiExpectTest") > public class AvailabilityZoneApiExpectTest extends BaseNovaApiExpectTest { > > - public void testAvailabilityZonesList() throws Exception { > + public void testListAvailabilityZones() throws Exception

Re: [jclouds] Split availability Zone API to return Basic Info and Detailed info (#815)

2015-07-15 Thread Zack Shoylev
> + * See the License for the specific language governing permissions and > + * limitations under the License. > + */ > +package org.jclouds.openstack.nova.v2_0.domain.regionscoped; > + > +import com.google.common.base.Objects; > +import com.google.common.collect.ImmutableMap; > +import com.google.

Re: [jclouds] Split availability Zone API to return Basic Info and Detailed info (#815)

2015-07-15 Thread arvindn05
> + * See the License for the specific language governing permissions and > + * limitations under the License. > + */ > +package org.jclouds.openstack.nova.v2_0.domain.regionscoped; > + > +import com.google.common.base.Objects; > +import com.google.common.collect.ImmutableMap; > +import com.google.

Re: [jclouds] Split availability Zone API to return Basic Info and Detailed info (#815)

2015-07-15 Thread Zack Shoylev
> + * See the License for the specific language governing permissions and > + * limitations under the License. > + */ > +package org.jclouds.openstack.nova.v2_0.domain.regionscoped; > + > +import com.google.common.base.Objects; > +import com.google.common.collect.ImmutableMap; > +import com.google.

[jclouds] Split availability Zone API to return Basic Info and Detailed info (#815)

2015-07-15 Thread arvindn05
Split availability Zone API to return Basic Info and Detailed info. My previous checkin did not show detail for availability zone which is a different path than just plain availability zone info(name and state) Introduced AvailabilityZone class which just shows name and state Introduced Availabil