Re: [jclouds-labs] JCLOUDS-1014: Make the login port lookup function configurable (#209)

2015-10-09 Thread Ignasi Barrera
The build got stuck and I had to abort it. Let's try again: rebuild please! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/209#issuecomment-147047041

Re: [jclouds] JCLOUDS-1008: Add @Encoded annotation. (#861)

2015-10-09 Thread Ignasi Barrera
> Do you think this is something that should be added now or deferred until > there is a use case for it? I'd add it now, for consistency and completeness of the feature. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/861#issuecomment-147008484

Re: [jclouds] JCLOUDS-1008: Add @Encoded annotation. (#861)

2015-10-09 Thread Timur Alperovich
If I'm reading ```http/utils/Queries.java``` correctly, the parameters are individually encoded, since the ```UriBuilder``` class will call ```encodeQueryLine()```, which in turn encodes the string before appending it to the URI. In that sense, I think you're right that maybe adding ```@Encoded

[jira] [Commented] (JCLOUDS-786) Remove old swift api

2015-10-09 Thread Ignasi Barrera (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-786?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14951329#comment-14951329 ] Ignasi Barrera commented on JCLOUDS-786: AFAIK our SoftLayer provider only implem

Re: [jclouds] JCLOUDS-1008: Add @Encoded annotation. (#861)

2015-10-09 Thread Ignasi Barrera
Thanks @timuralp! This lgtm and is good to merge. One last question: should this also apply to query parameters? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/861#issuecomment-147004430

[jira] [Commented] (JCLOUDS-1014) Docker: Add possibility to change loginPort

2015-10-09 Thread Ignasi Barrera (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1014?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14951309#comment-14951309 ] Ignasi Barrera commented on JCLOUDS-1014: - Patch ready for review: https://githu

[jclouds-labs] JCLOUDS-1014: Make the login port lookup function configurable (#209)

2015-10-09 Thread Ignasi Barrera
https://issues.apache.org/jira/browse/JCLOUDS-1014 Some Docker images can have the SSH server running in a port different than 22, and that configuration might not be populated in the container's networking configuration section. This pull request introduces a pluggable mechanism to let users p

[jira] [Commented] (JCLOUDS-786) Remove old swift api

2015-10-09 Thread Zack Shoylev (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-786?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14950513#comment-14950513 ] Zack Shoylev commented on JCLOUDS-786: -- Is that something that can be set in the sof

[jira] [Assigned] (JCLOUDS-1014) Docker: Add possibility to change loginPort

2015-10-09 Thread Ignasi Barrera (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1014?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ignasi Barrera reassigned JCLOUDS-1014: --- Assignee: Ignasi Barrera > Docker: Add possibility to change loginPort > --

[jira] [Updated] (JCLOUDS-1014) Docker: Add possibility to change loginPort

2015-10-09 Thread Ignasi Barrera (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1014?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ignasi Barrera updated JCLOUDS-1014: Labels: docker (was: ) > Docker: Add possibility to change loginPort > --

[jira] [Created] (JCLOUDS-1014) Docker: Add possibility to change loginPort

2015-10-09 Thread Josef Cacek (JIRA)
Josef Cacek created JCLOUDS-1014: Summary: Docker: Add possibility to change loginPort Key: JCLOUDS-1014 URL: https://issues.apache.org/jira/browse/JCLOUDS-1014 Project: jclouds Issue Type: I