Build failed in Jenkins: jclouds-karaf #1886

See -- [...truncated 4161 lines...] Downloading: https://repository.apache.org/content/repositories/snapshots/org/apache/jclouds/provider/google-compute-engine/2.0.0-SNAPSHOT/maven-metadata.xml Down

Build failed in Jenkins: jclouds-labs-openstack » jclouds openstack-glance api #1101

See -- [...truncated 2672 lines...] Downloading: https://repository-jclouds.forge.cloudbees.com/release/com/google/code/findbugs/annotations/3.0.0/

[jclouds/jclouds] Do not try to connect to ssh if already connected (#995)

The `refreshAndRunAction` is called by the `future.cancel()` method when cancelling the future associated to a running script (typically obtained by calling the `compute.submitScriptOnNode` method. I've observed that when the ssh connection is already established, this method fails to connect a

[jira] [Commented] (JCLOUDS-1145) Cannot create images from powered off droplets

[ https://issues.apache.org/jira/browse/JCLOUDS-1145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15418070#comment-15418070 ] Ignasi Barrera commented on JCLOUDS-1145: - PR: https://github.com/jclouds/jcloud

[jclouds/jclouds] 1145: Fix DigitalOceanImageExtension and live tests (#994)

https://issues.apache.org/jira/browse/JCLOUDS-1145 I've changed the default template to Ubuntu 14, because there are no Ubuntu 15 in DigitalOcean and using Ubuntu 16 images, the most recent ones, causes issue [JCLOUDS-1133](https://issues.apache.org/jira/browse/JCLOUDS-1133). You can view, comme

Re: [jclouds/jclouds] Always take into account the configured template builder spec in live tests (#993)

Closed #993. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/993#event-753356726

Re: [jclouds/jclouds-labs] JCLOUDS-1153 Fix empty Docker repoTags field in ImageToImage function (#309)

> @@ -91,9 +91,15 @@ public OsFamily apply(final String description) { >}; > } > > - private String parseVersion(String description) { > - String version = get(Splitter.on(":").split(description), 1); > - logger.debug("os version for item: %s is %s", description, version);

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

> + > + public abstract Builder healthCheckInterval(int > healthCheckInterval); > + > + public abstract Builder healthCheckPath(String healthCheckPath); > + > + public abstract Builder healthCheckParse(String healthCheckParse); > + > + public abstract Builder persis

[jira] [Assigned] (JCLOUDS-1145) Cannot create images from powered off droplets

[ https://issues.apache.org/jira/browse/JCLOUDS-1145?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ignasi Barrera reassigned JCLOUDS-1145: --- Assignee: Ignasi Barrera > Cannot create images from powered off droplets > ---

[jira] [Commented] (JCLOUDS-1153) Docker ImageToImage class doesn't support empty repoTags list

[ https://issues.apache.org/jira/browse/JCLOUDS-1153?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15417564#comment-15417564 ] Josef Cacek commented on JCLOUDS-1153: -- PR sent: https://github.com/jclouds/jclouds

[jclouds/jclouds-labs] JCLOUDS-1153 Fix empty Docker repoTags field in ImageToImage function (#309)

JIRA: https://issues.apache.org/jira/browse/JCLOUDS-1153 This commit allows to work with empty `repoTags` field in Docker images. You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds-labs/pull/309 -- Commit Summary -- * JCLOUDS-1153 Fix empty Do

[jira] [Created] (JCLOUDS-1153) Docker ImageToImage class doesn't support empty repoTags list

Josef Cacek created JCLOUDS-1153: Summary: Docker ImageToImage class doesn't support empty repoTags list Key: JCLOUDS-1153 URL: https://issues.apache.org/jira/browse/JCLOUDS-1153 Project: jclouds

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

@alibazlamit pushed 1 commit. 0e2c5df minor fix -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds-labs/pull/307/files/3314d73a8de6255e81070c51c0fd7af4fb57e585..0e2c5df717e5493e0bade8dfe22569ee1b7bc56e

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

@alibazlamit pushed 1 commit. 3314d73 minor changes applied -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds-labs/pull/307/files/915aa395f4fe7805f8c6fe7f0e333184935eb43c..3314d73a8de6255e81070c51c0fd7af4fb57e585

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

> + > + public abstract Builder healthCheckInterval(int > healthCheckInterval); > + > + public abstract Builder healthCheckPath(String healthCheckPath); > + > + public abstract Builder healthCheckParse(String healthCheckParse); > + > + public abstract Builder persis

Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

> @@ -28,6 +30,7 @@ public static Properties defaultProperties(Properties > properties) { > properties.put("oauth.credential", "password"); > properties.put("oauth.endpoint", > "https://login.microsoftonline.com/oauth2/token";); > properties.put(CREDENTIAL_TYPE, CLIENT_CRE

Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

> @@ -28,6 +30,7 @@ public static Properties defaultProperties(Properties > properties) { > properties.put("oauth.credential", "password"); > properties.put("oauth.endpoint", > "https://login.microsoftonline.com/oauth2/token";); > properties.put(CREDENTIAL_TYPE, CLIENT_CRE

Re: [jclouds/jclouds] Fix for ssh reload for CentOS 7 (#992)

Thanks @iyovcheva! Unfortunately it looks like our CI system only runs the unit tests and not the integration ones. When running a build locally (with `mvn clean install`) I get these failures in jclouds-compute: Tests run: 40, Failures: 3, Errors: 0, Skipped: 11 testCreateAndRunAServic

Re: [jclouds/jclouds] Add support for organization internal projects. (#990)

Pushed to master as [52477d5e](http://git-wip-us.apache.org/repos/asf/jclouds/commit/52477d5e). Thanks @nelsonjr! -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/990#issuecomment-2391

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

> + assertNotNull(updateResult); > + assertEquals(updateResult.name(), updatedName); > + > + } > + > + @Test(dependsOnMethods = "testUpdate") > + public void testAssignServerIp() throws InterruptedException { > + assertNodeAvailable(currentServer); > + currentServer = upda

Re: [jclouds/jclouds] Add support for organization internal projects. (#990)

Closed #990. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/990#event-752697578

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

Thanks @alibazlamit! This looks great! :) All comments are minors. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/307#issuecomment-239168521

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

> @@ -38,7 +38,7 @@ > > > 2.0.0-SNAPSHOT > - > + Discard re-indentation -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/307/files/915aa395f4fe7805f8c6fe7f0e3331

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

> > - public abstract String location(); > - > - @SerializedNames({"id", "country_code", "location"}) > - public static DataCenter create(String id, String countryCode, String > location) { > - return new AutoValue_DataCenter(id, countryCode, location); > - } > +@SerializedNames

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

> > - public abstract String name(); > - > - @SerializedNames({"id", "name"}) > - public static Dvd create(String id, String name) { > - return new AutoValue_Dvd(id, name); > - } > +@SerializedNames({"id", "name"}) > +public static Dvd create(String id, String name) { > +

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

> + > + public abstract Builder healthCheckInterval(int > healthCheckInterval); > + > + public abstract Builder healthCheckPath(String healthCheckPath); > + > + public abstract Builder healthCheckParse(String healthCheckParse); > + > + public abstract Builder persis

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

> + @Nullable > + public abstract String creationDate(); > + > + @Nullable > + public abstract String ip(); > + > + @Nullable > + public abstract HealthCheckTestTypes healthCheckTest(); > + > + public abstract int healthCheckInterval(); > + > + @Nullable > + public abstract String

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

> + public void testListRules() { > + List servers = > loadBalancerApi().listRules(currentLoadBalancer.id()); > + > + assertNotNull(servers); > + assertFalse(servers.isEmpty()); > + Assert.assertTrue(servers.size() > 0); > + } > + > + @Test(dependsOnMethods = "testAddRule

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

> + @Test(dependsOnMethods = "testAssignServerIp") > + public void testServerIpGet() { > + LoadBalancer.ServerIp result = > loadBalancerApi().getServerIp(currentLoadBalancer.id(), > currentServer.ips().get(0).id()); > + > + assertNotNull(result); > + } > + > + @Test(dependsOnMet

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

> + LoadBalancer updateResult = > loadBalancerApi().update(currentLoadBalancer.id(), > LoadBalancer.UpdateLoadBalancer.builder() > + .name(updatedName) > + .description("description") > + .healthCheckInterval(40) > + .healthCheckTest(Types.

Re: [jclouds/jclouds-labs] JCLOUDS-1151 oneandone-loadbalancers-api (#307)

> + public void testListServerIps() { > + List servers = > loadBalancerApi().listServerIps(currentLoadBalancer.id()); > + > + assertNotNull(servers); > + assertFalse(servers.isEmpty()); > + Assert.assertTrue(servers.size() > 0); > + } > + > + @Test(dependsOnMethods = "tes

Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

Thanks @andreaturli! -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/308#issuecomment-239163128

Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

> @@ -28,6 +30,7 @@ public static Properties defaultProperties(Properties > properties) { > properties.put("oauth.credential", "password"); > properties.put("oauth.endpoint", > "https://login.microsoftonline.com/oauth2/token";); > properties.put(CREDENTIAL_TYPE, CLIENT_CRE

Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

> > import static com.google.common.base.Preconditions.checkNotNull; > -import org.jclouds.logging.slf4j.config.SLF4JLoggingModule; > -import org.jclouds.logging.config.LoggingModule; > - > +import static > org.jclouds.azurecompute.arm.config.AzureComputeProperties.RESOURCE_GROUP_NAME; > +impor

Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

> @@ -47,20 +47,14 @@ > @Override > public org.jclouds.domain.Location apply(final Location location) { >final LocationBuilder builder = new LocationBuilder(); > - String id = location.id(); > - int index = id.lastIndexOf('/'); > - if (index > 0 && (index + 1) < id.le

Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

> @Inject > AzureComputeServiceAdapter(final AzureComputeApi api, final > AzureComputeConstants azureComputeConstants, > - CleanupResources cleanupResources) { > + CleanupResources cleanupResources, > ProviderMetadata providerMetad

Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

@andreaturli pushed 1 commit. 961db5c address initial comments from @nacx -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds-labs/pull/308/files/bea9005a1dbad01499584a1932c4445b64a60659..961db5c145342971726063e25776e0011ab

[jclouds/jclouds-labs] add support for whitelisting locations (#308)

- change location scope to ZONE vs REGION - edit the README - fix Region.byName - add more Regions in Region class You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds-labs/pull/308 -- Commit Summary -- * add support for whitelisting locations -

Re: [jclouds/jclouds] Always take into account the configured template builder spec in live tests (#993)

> Yes. The base class for the compute service live tests was ignoring the > "template" test property in some tests, so even when configured in the > pom.xml or as system property, a default template was used instead of the > configured one. Ah, OK - thanks for explaining! -- You are receiving

Re: [jclouds/jclouds] Always take into account the configured template builder spec in live tests (#993)

Yes. The base class for the compute service live tests was ignoring the "template" test property in some tests, so even when configured in the pom.xml or as system property, a default template was used instead of the configured one. This lead to some tests failing to connect via SSH if the temp