TODO: backport
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/745#issuecomment-101465971
Closed #745.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/745#event-303374279
merged.
Some conflicts backporting. @andrewgaul - did you backport your code?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/745#issuecomment-101465926
[
https://issues.apache.org/jira/browse/JCLOUDS-894?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14541081#comment-14541081
]
Zack Shoylev commented on JCLOUDS-894:
--
https://github.com/jclouds/jclouds/pull
> @@ -553,7 +553,7 @@ protected void addTenObjectsUnderPrefix(String
> containerName, String prefix) thr
>
> protected void awaitConsistency() {
>if (view.getConsistencyModel() == ConsistencyModel.EVENTUAL) {
> - Uninterruptibles.sleepUninterruptibly(10, TimeUnit.SECONDS);
>
> One spurious failure:
I have seen this twice. A bit hard to reproduce.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/745#issuecomment-101449608
> Can you tag these commits with JCLOUDS-894?
I will try to remember to do this when I rebase :)
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/745#issuecomment-101445023
> @@ -553,7 +553,7 @@ protected void addTenObjectsUnderPrefix(String
> containerName, String prefix) thr
>
> protected void awaitConsistency() {
>if (view.getConsistencyModel() == ConsistencyModel.EVENTUAL) {
> - Uninterruptibles.sleepUninterruptibly(10, TimeUnit.SECONDS);
>
[
https://issues.apache.org/jira/browse/JCLOUDS-615?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14540764#comment-14540764
]
Zack Shoylev commented on JCLOUDS-615:
--
Specifically https://github.com/jcl
@andrewgaul Let me know if this looks ok now.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/745#issuecomment-101421031
Closed #191.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/191#event-303233388
merged, backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/191#issuecomment-101417505
@andrewgaul
I got the tests to work with swift and in the process I think I improved a bit
on how the metadata and headers are processed. There is probably more work
left, though, and I would like to see an improvement in how the content-*
headers are handled.
---
Reply to this email directly
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/745
-- Commit Summary --
* Multipart upload code fixes for swift
-- File Changes --
A
apis/openstack-swift/src/main/java/org/jclouds/openstack/swift/v1/binders/BindManifestToJsonPa
> + public boolean createContainerInLocation(String name, Location location,
> + CreateContainerOptions createContainerOptions) {
> + logger.debug("Creating container %s", name);
> + jdbcContainerNameValidator.validate(name);
> + containerRepository.create(Container.builder
> + public boolean createContainerInLocation(String name, Location location,
> + CreateContainerOptions createContainerOptions) {
> + logger.debug("Creating container %s", name);
> + jdbcContainerNameValidator.validate(name);
> + containerRepository.create(Container.builder
> + public boolean createContainerInLocation(String name, Location location,
> + CreateContainerOptions createContainerOptions) {
> + logger.debug("Creating container %s", name);
> + jdbcContainerNameValidator.validate(name);
> + containerRepository.create(Container.builder
Closed #78.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/78#event-299481604
merged
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/78#issuecomment-99940104
Thanks!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/commit/e92fd921cfbaf10256126263e8e2814c24574b2e#commitcomment-11086689
Work in progress, investigating some problems with 1.9.0.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/78#issuecomment-99210727
zack-shoylev wants to merge 1 commit into jclouds:master from
rackerlabs:neutron-examples:
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-examples/pull/78
-- Commit Summary --
* Adds neutron examples and some cleanup
-- File Changes
@limorbortman : Will this work if you have the original code @Deprecated
instead of removed?
Then we would also be able to backport, and remove the @Deprecated
AttachInterfaceApi in a few releases.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/697
zack-shoylev wants to merge 1 commit into jclouds:master from
rackerlabs:remove-deprecated-neutron:
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-labs-openstack/pull/191
-- Commit Summary --
* Remove deprecated neutron code
-- File
[
https://issues.apache.org/jira/browse/JCLOUDS-894?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14526494#comment-14526494
]
Zack Shoylev commented on JCLOUDS-894:
--
[~argaul] I will be able to get to
We will changed with 1.9.0
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/76#issuecomment-97895807
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-examples/pull/76
-- Commit Summary --
* Adds CDN examples
-- File Changes --
M rackspace/src/main/java/org/jclouds/examples/rackspace/SmokeTest.java (35)
A rackspace/src/main/java/or
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-labs-openstack/pull/190
-- Commit Summary --
* Make the flavor nullable. Handles service edge cases.
-- File Changes --
M
openstack-poppy/src/main/java/org/jclouds/openstack/poppy/v1/do
@limorbortman : Looks good. I will run some tests and merge today. Thanks!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/188#issuecomment-97502203
Some test failures
https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1728/org.apache.jclouds$jclouds-blobstore/console
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/737#issuecomment-97243820
> + ImmutableList.Builder builder = ImmutableList.builder();
> + for (MultipartPart part : parts) {
> + String path = mpu.containerName() + "/" + mpu.blobName() + "-" +
> part.partNumber();
> +
> builder.add(Segment.builder().path(path).etag(part.partETag()).sizeBytes(pa
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/736
-- Commit Summary --
* Replace legacy cloudfiles with modern rackspace-cloudfiles
-- File Changes --
M allblobstore/pom.xml (9)
-- Patch Links --
https://github.com/jclouds/jc
merged and backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/733#issuecomment-97181104
About to rebase, merge, backport.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/733#issuecomment-97171506
[
https://issues.apache.org/jira/browse/JCLOUDS-760?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14515049#comment-14515049
]
Zack Shoylev commented on JCLOUDS-760:
--
About 30% on it. Hopefully fixed for 1
@limorbortman the comment is for this line:
https://github.com/limorbortman/jclouds-labs-openstack/blob/adding_heat/openstack-heat/src/main/java/org/jclouds/openstack/heat/v1/options/CreateStack.java#L101
I'm not sure why it's not showing up, but I can see it in
https://github.com/jclouds/jclouds
> +.template(template)
> +.templateUrl(templateUrl)
> +.parameters(parameters)
> +.disableRollback(disableRollback)
> +.files(files)
> +.environment(environment).build();
> + }
> +
> + public static final class Builder {
>
It should be easy to refactor.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/733#issuecomment-95685477
> - try {
> - permissions = getPosixFilePermissions(path);
> - if (access == ContainerAccess.PRIVATE) {
> -permissions.remove(PosixFilePermission.OTHERS_READ);
> - } else if (access == ContainerAccess.PUBLIC_READ) {
> -permissions.add(PosixFilePe
> +.template(template)
> +.templateUrl(templateUrl)
> +.parameters(parameters)
> +.disableRollback(disableRollback)
> +.files(files)
> +.environment(environment).build();
> + }
> +
> + public static final class Builder {
>
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/733
-- Commit Summary --
* Adds windows ACL code for file permissions, fixes build.
-- File Changes --
M
apis/filesystem/src/main/java/org/jclouds/filesystem/strategy/internal/File
[
https://issues.apache.org/jira/browse/JCLOUDS-877?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14506226#comment-14506226
]
Zack Shoylev commented on JCLOUDS-877:
--
https://github.com/jclouds/jclouds-
[
https://issues.apache.org/jira/browse/JCLOUDS-877?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Zack Shoylev resolved JCLOUDS-877.
--
Resolution: Fixed
> Creating a service without caching or restrictions lists throws
merged and backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/189#issuecomment-94995351
merged
not backported (not applicable)
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/726#issuecomment-94960702
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-labs-openstack/pull/189
-- Commit Summary --
* Fixes NPEs with CreateService
-- File Changes --
M
openstack-poppy/src/main/java/org/jclouds/openstack/poppy/v1/domain/CreateService.java
Alright. I think it's ready to merge. @nacx ?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/188#issuecomment-94950349
Having some issues testing the examples. I have some more commits to add.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/74#issuecomment-94599247
@andrewgaul is this good to merge? Or do we need to track it down.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/726#issuecomment-94528568
> @RequestFilters(AuthenticateRequest.class)
> @Consumes(MediaType.APPLICATION_JSON)
> @Path("/servers")
> -public interface AttachInterfaceApi {
> +public interface InterfaceApi {
But the problem might be that existing code will break. We might want to
consider deprecating the original code i
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-examples/pull/74
-- Commit Summary --
* Adds CDN examples
* Some import fixes
-- File Changes --
M rackspace/pom.xml (2)
A rackspace/src/main/java/org/jclouds/examples/rackspace/cd
> + * distributed under the License is distributed on an "AS IS" BASIS,
> + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
> + * See the License for the specific language governing permissions and
> + * limitations under the License.
> + */
> +
> +package org.jclouds.exa
There are some SNAPSHOT/1.9.x-only features that would make this look a bit
better.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/74#issuecomment-93958891
Add changes to smoke test
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/74#issuecomment-93958818
> + @Nullable public abstract String getSatusReason();
> +
> + /**
> +* @return the date this Stack was created.
> +*/
> + @Nullable public abstract Date getCreated();
> +
> + /**
> +* @return the date this Stack was last updated.
> +*/
> + @Nullable public abstract Date g
Zack Shoylev created JCLOUDS-888:
Summary: Add examples for rackspace cloudnetworks in
jclouds-examples
Key: JCLOUDS-888
URL: https://issues.apache.org/jira/browse/JCLOUDS-888
Project: jclouds
+1
Thanks @nacx
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/730#issuecomment-93467176
> @RequestFilters(AuthenticateRequest.class)
> @Consumes(MediaType.APPLICATION_JSON)
> @Path("/servers")
> -public interface AttachInterfaceApi {
> +public interface InterfaceApi {
Will this be backported? It seems like a breaking change for existing code.
---
Reply to this email directly or v
> + return this;
> + }
> +
> + public CreateStack build() {
> + String missing = "";
> +
> + if (name == null) {
> +missing += " name";
> + }
> +
> +
> + if (template == null && templateUrl == null) {
> +missing += " template
I really like how this is turning out. Great work @limorbortman!
My one regret is that the autovalue builders are not officially out yet. But
this will still be very easy to refactor once they are.
Thanks!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-l
> +import static org.testng.Assert.assertEquals;
> +import static org.testng.Assert.assertTrue;
> +
> +/**
> + * Tests annotation parsing of {@code StackApi}
> + */
> +@Test(groups = "unit", testName = "StackApiMockTest")
> +public class StackApiMockTest extends BaseHeatApiMockTest {
> +
> + publ
> + boolean delete(@PathParam("stack_name") String name,
> @PathParam("stack_id") String id);
> +
> + @Named("stack:update")
> + @PUT
> + @Path("/{stack_name}/{stack_id}")
> + @Fallback(NullOnNotFoundOr404.class)
> + @Nullable
> + boolean update(@PathParam("stack_name") String name,
> + return this;
> + }
> +
> + public CreateStack build() {
> + String missing = "";
> +
> + if (name == null) {
> +missing += " name";
> + }
> +
> +
> + if (template == null && templateUrl == null) {
> +missing += " template
> + assertThat(stackId).isNotEmpty();
> +
> + boolean success = retry(new Predicate() {
> +public boolean apply(String stackId) {
> + return stackApi.get(stackId).getStatus() ==
> StackStatus.CREATE_COMPLETE;
> +}
> +
> + }, 60, 1, SECO
> + boolean delete(@PathParam("stack_name") String name,
> @PathParam("stack_id") String id);
> +
> + @Named("stack:update")
> + @PUT
> + @Path("/{stack_name}/{stack_id}")
> + @Fallback(NullOnNotFoundOr404.class)
> + @Nullable
> + boolean update(@PathParam("stack_name") String name,
>provided
>
>
>com.google.auto.service
>auto-service
> + 1.0-rc2
Good point.
In any case, the right way to use this dependency, if I remember correctly,
would be this:
com.google.auto.service
auto-service
provided
---
Reply to
> @@ -0,0 +1,11 @@
> +{
> +"stack": {
> +"id": "3095aefc-09fb-4bc7-b1f0-f21a304e864c",
> +"links": [
> +{
> +"href":
> "http://192.168.123.200:8004/v1/eb1c63a4f77141548385f113a28f0f52/stacks/simple_stack/3095aefc-09fb-4bc7-b1f0-f21a304e864c";,
> +
> + /*
> + * Check request
> + */
> + assertEquals(server.getRequestCount(), 2);
> + assertAuthentication(server);
> + assertRequest(server.takeRequest(), "GET", BASE_URI + "/stacks/" +
> TEST_STACK_NAME + "/" + TEST_STACK_ID + "/resources/" + RESO
> + assertThat(stackFromList).isNotNull();
> + assertThat(stackFromList.getId()).isEqualTo(stack.getId());
> + for (String parmName : parameters.keySet()) {
> +
> assertThat(stackFromList.getParameters().containsKey(parmName)).isTrue();
> + }
> +
> +
> + @MapBinder(CreateStackOptions.class)
> + Stack create(@PayloadParam("stack_name") String name,
> CreateStackOptions... options);
> +
> + @Named("stack:delete")
> + @DELETE
> + @Path("/{stack_name}/{stack_id}")
> + @Fallback(NullOnNotFoundOr404.class)
> + @Nullable
> + boolean d
> + return physicalResourceId;
> + }
> +
> + public Set getRequiredBy() {
> + return requiredBy;
> + }
> +
> + public String getResourceType() {
> + return resourceType;
> + }
> +
> + public Date getUpdated() {
> + return updated;
> + }
> +
> + public enum Status
> +import org.jclouds.openstack.v2_0.domain.Link;
> +import org.jclouds.openstack.v2_0.domain.Resource;
> +
> +import javax.inject.Named;
> +import java.beans.ConstructorProperties;
> +import java.util.Date;
> +import java.util.List;
> +import java.util.Map;
> +import java.util.Set;
> +
> +import s
> + public String value() {
> + return CaseFormat.UPPER_UNDERSCORE.to(CaseFormat.LOWER_HYPHEN,
> name());
> + }
> +
> + @Override
> + public String toString() {
> + return value();
> + }
> +
> + /**
> + * This provides GSON enum support in jcloud
>provided
>
>
>com.google.auto.service
>auto-service
> + 1.0-rc2
1.0, I think, don't use the rc.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/188/files#r28096880
> @@ -98,13 +98,21 @@
>
>com.google.auto.value
>auto-value
> + 1.0-rc2
The latest version is 1.0
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/188/files#r28096847
Squashed, merged, and backported.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/184#issuecomment-91341360
Closed #184.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/184#event-277514904
backport?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/722#issuecomment-91339375
[
https://issues.apache.org/jira/browse/JCLOUDS-883?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Zack Shoylev reassigned JCLOUDS-883:
Assignee: Zack Shoylev
> Swift portable copy blob cannot replace metad
backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/187#issuecomment-91337785
merged and backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/725#issuecomment-91333866
@andrewgaul I'm ok with that. Figuring out the swift quirk might need some more
work.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/722#issuecomment-91101772
Both
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/725#issuecomment-91056170
That would be the plan going forward, though maybe not as part of this PR. I am
not sure what the right timing would be to do that, yet.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/725#issuecomment-90946669
This should probably be backported
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/187#issuecomment-90762488
merged
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/187#issuecomment-90762467
Closed #187.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/187#event-275789666
I would like to know that too! It would be better to exclude it, rather than do
it by extensions. Just unsure how to do that, though.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/726#issuecomment-90759796
Reopened #725.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/725#event-275781896
Rebuild please?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/725#issuecomment-90759616
Superceded by https://github.com/jclouds/jclouds/pull/720
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/704#issuecomment-90753719
Closed #704.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/704#event-275763785
Closed #720.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/720#event-275763644
For example:
[INFO] --- maven-checkstyle-plugin:2.15:check (default) @ jclouds-resources ---
[WARNING] target\maven-archiver\pom.properties[1] (regexp) RegexpMultiline: Do
not use Windows line endings
[WARNING] target\maven-archiver\pom.properties[2] (regexp) RegexpMultiline: Do
not use Windows
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/725
-- Commit Summary --
* Update cloudfiles poms
-- File Changes --
M apis/pom.xml (1)
M apis/rackspace-cloudfiles/pom.xml (20)
M providers/pom.xml (2)
M providers/rack
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-labs-openstack/pull/187
-- Commit Summary --
* Remove promoted cloudfiles api and providers
-- File Changes --
M README.md (3)
M pom.xml (5)
D rackspace-cloudfiles-uk/README.md (
> @@ -796,9 +808,8 @@ public void testCopyBlobReplaceMetadata() throws
> Exception {
> } finally {
> Closeables2.closeQuietly(is);
> }
> - // TODO: S3 overrideMetadataWith also overrides system metadata
> - // TODO: Swift does not preserve system me
> @@ -796,9 +808,8 @@ public void testCopyBlobReplaceMetadata() throws
> Exception {
> } finally {
> Closeables2.closeQuietly(is);
> }
> - // TODO: S3 overrideMetadataWith also overrides system metadata
> - // TODO: Swift does not preserve system me
@nacx do you mind testing merging this?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/720#issuecomment-90724644
@nacx Alright, will do; This also works on windows (I tested that yesterday),
so it's a nice bonus.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/720#issuecomment-90721967
701 - 800 of 1837 matches
Mail list logo
