You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jclouds.apache.org by Everett Toews <no...@github.com> on 2013/10/14 23:06:34 UTC

[jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Fixes to go along with the fixes to BaseOpenStackMockTest in https://github.com/jclouds/jclouds/pull/182

Better formatting for the labs READMEs.

You can merge this Pull Request by running:

  git pull https://github.com/rackerlabs/jclouds-labs-openstack fixes

Or you can view, comment on it, or merge it online at:

  https://github.com/jclouds/jclouds-labs-openstack/pull/41

-- Commit Summary --

  * Renamed BaseOpenStackMockTest variable and method more appropriately.
  * Better formatting of production ready providers/APIs.

-- File Changes --

    M README.md (12)
    M openstack-glance/README.md (3)
    M openstack-neutron/README.md (3)
    M openstack-swift/README.md (3)
    M openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/AuthenticationMockTest.java (4)
    M openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/TemporaryUrlSignerMockTest.java (8)
    M openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/features/AccountApiMockTest.java (16)
    M openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/features/BulkApiMockTest.java (4)
    M openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/features/ContainerApiMockTest.java (44)
    M openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/features/ObjectApiMockTest.java (36)
    M openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/features/StaticLargeObjectApiMockTest.java (12)
    M openstack-trove/README.md (3)
    M rackspace-autoscale-us/README.md (3)
    M rackspace-autoscale/README.md (3)
    M rackspace-clouddatabases-uk/README.md (3)
    M rackspace-clouddatabases-us/README.md (3)

-- Patch Links --

https://github.com/jclouds/jclouds-labs-openstack/pull/41.patch
https://github.com/jclouds/jclouds-labs-openstack/pull/41.diff

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-labs-openstack-pull-requests #76](https://jclouds.ci.cloudbees.com/job/jclouds-labs-openstack-pull-requests/76/) FAILURE
Looks like there's a problem with this pull request

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26355465

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds-labs-openstack #522](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-openstack/522/) FAILURE
Looks like there's a problem with this pull request
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26289350

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by Everett Toews <no...@github.com>.
Not sure why CloudBees is failing with errors like

    symbol  : method mockOpenStackServer()
    location: class org.jclouds.openstack.swift.v1.features.AccountApiMockTest
    [ERROR] /scratch/jenkins/workspace/jclouds-labs-openstack-pull-requests/openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/features/AccountApiMockTest.java:[44,48] cannot find symbol

    symbol  : variable accessRackspace
    location: class org.jclouds.openstack.swift.v1.features.AccountApiMockTest
    [ERROR] /scratch/jenkins/workspace/jclouds-labs-openstack-pull-requests/openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/features/AccountApiMockTest.java:[70,29] cannot find symbol

Clearly those changes are already in the repo, see https://github.com/jclouds/jclouds/blob/master/apis/openstack-keystone/src/test/java/org/jclouds/openstack/v2_0/internal/BaseOpenStackMockTest.java

I even decompiled the Keystone tests JAR at https://repository.apache.org/content/groups/snapshots/org/apache/jclouds/api/openstack-keystone/1.7.0-SNAPSHOT/openstack-keystone-1.7.0-20131015.164442-171-tests.jar to make sure they were in the snapshot too.

I do see CloudBees referring to it's own snapshot repo at https://repository-jclouds.forge.cloudbees.com/snapshot/org/apache/jclouds/api/openstack-keystone/1.7.0-SNAPSHOT/maven-metadata.xml but those files don't even seem to exist.

@demobox Do you have any idea why CloudBees might be failing? Do you think wiping the jclouds-labs-openstack-pull-requests workspace at https://jclouds.ci.cloudbees.com/job/jclouds-labs-openstack-pull-requests/ws/ would help?



---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26356788

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by Everett Toews <no...@github.com>.
Closed #41.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-labs-openstack-pull-requests #73](https://jclouds.ci.cloudbees.com/job/jclouds-labs-openstack-pull-requests/73/) FAILURE
Looks like there's a problem with this pull request

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26289069

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-labs-openstack-pull-requests #75](https://jclouds.ci.cloudbees.com/job/jclouds-labs-openstack-pull-requests/75/) FAILURE
Looks like there's a problem with this pull request

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26353210

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-labs-openstack-pull-requests #78](https://jclouds.ci.cloudbees.com/job/jclouds-labs-openstack-pull-requests/78/) SUCCESS
This pull request looks good

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26359156

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by Everett Toews <no...@github.com>.
Wiped workspace. Trying again.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26358863

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by Andrew Phillips <no...@github.com>.
> Yay. Merged.

Yay indeed. Thanks, @everett-toews!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26404710

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by Everett Toews <no...@github.com>.
Another try to see if I can get CloudBees to behave.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26355177

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds-labs-openstack #531](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-openstack/531/) SUCCESS
This pull request looks good
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26356052

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by Everett Toews <no...@github.com>.
Failures expected as https://github.com/jclouds/jclouds/pull/182 isn't merged yet.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26289634

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-labs-openstack-pull-requests #74](https://jclouds.ci.cloudbees.com/job/jclouds-labs-openstack-pull-requests/74/) FAILURE
Looks like there's a problem with this pull request

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26351210

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by Everett Toews <no...@github.com>.
Triggering build.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26350874

Re: [jclouds-labs-openstack] Fixes for BaseOpenStackMockTest and labs READMEs (#41)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds-labs-openstack #528](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-openstack/528/) FAILURE
Looks like there's a problem with this pull request
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/41#issuecomment-26351433