Sorry @nacx, my bad. Thanks @ilgrosso for your fix.
Please, consider that traffic manager operations are ready for a new PR.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/148#issuecomment-78217667
@nacx I have just removed the "offending" commit from master and created a
dedicated branch with it (for future PR)
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/148#issuecomment-78167128
> The PR looks good to me. But I'm missing the storage api mock tests.
The intent of this PR was to fix the live test execution: we can address this
need in a future PR.
> It can't be merged, though, because it contains the unrelated traffic manager
> operations commit
You are right, @fmartell
The PR looks good to me. But I'm missing the storage api mock tests.
It can't be merged, though, because it contains the unrelated traffic manager
operations commit. Please, develop each feature in its own topic branch instead
of working in master, so we can properly have one PR per feature. Can
using your last version, all live tests are now green!
Let's wait for @nacx's proper review to merge it
Thanks!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/148#issuecomment-78050039
@andreaturli, we normally get no errors at all; such failures look quite
strange, especially the first one which is very basic.
Could you please check if there is something strange with the checked sources
or your subscription data (a clean situation always helps).
---
Reply to this email direct
much better now,
`Failed tests:
OSImageApiLiveTest.testList:58->checkOSImage:78 Locations not in [North
Europe, West Europe, East US 2, Central US, South Central US, West US, East US,
Southeast Asia, East Asia, Japan West, Japan East] :[East Asia, Southeast Asia,
Australia East, Australia Sou
@andreaturli @nacx This commit introduces `A10` / `A11` in
`org.jclouds.azurecompute.domain.RoleSize.Type` and properly defaults to
`UNRECOGNIZED` for all enum types which provides that - by adding a static
`fromString` method.
---
Reply to this email directly or view it on GitHub:
https://gith
thanks @fmartelli and @ilgrosso
I thinks there was a problem with my test.
[Here](https://gist.githubusercontent.com/andreaturli/e17ee3f9b305096e4138/raw/live-tests)
the results using #148
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/148#i
Sorry, I've forgotten to say that recently MS changed something into its
platform. The following error comes from these.
java.lang.IllegalArgumentException: No enum constant
org.jclouds.azurecompute.domain.RoleSize.Type.A10
A10 role type has to be added and the role type reading mechanism has t
Hi Andrea, it seems that you have excuted the test without applying the pull
request before: actually, that was exactly the output before our changes.
Your output is not compliant with the work done till now. For example, I see
that you have several 409 errors returned, a lot of work has been do
Sure!
'mvn clean install -Plive
-Dtest.azurecompute.endpoint=https://management.core.windows.net/341751b0-f348-45ce--41cc68b4b45f
-Dtest.azurecompute.credential=password
-Dtest.azurecompute.identity=/Users/andrea/.jclouds/azure.p12'
I'm getting this
[output](https://gist.githubusercontent
Hi @andreaturli, could you please provide the whole live execution log
(including command-line) and not just the tail? Thanks.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/148#issuecomment-77755377
Thanks @ilgrosso
I've tried your PR using my azure account. Please find the result of my run
[here](https://gist.github.com/andreaturli/0a206fa810d749563bc8)
FYI I've started from a clean situation: no network, no cloudservice, no
storage account.
---
Reply to this email directly or view it on
Functionally equivalent to #147 but with only two commits as required by
JClouds policies.
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-labs/pull/148
-- Commit Summary --
* [JCLOUDS-664] Fixing live test execution
* White noise: format
15 matches
Mail list logo
