GitHub user andreaturli opened a pull request:

    https://github.com/apache/brooklyn-server/pull/949

    Improve JcloudsWinrmingLiveTest coverage

    - add Azure ARM
    - improve error message for DefaultConnectivityResolver
    
    ---
    
    When `DefaultConnectivityResolver.getReachableAddresses` now fails when no 
reachable address are found with this error message
    ```
    2018-02-21 11:47:15,502 ERROR None of the addresses [168.63.120.198, 
10.0.0.4] of node {id=brp4hwrr/brp4hwrr-9f7, 
providerId=/subscriptions/012e832d-7b27-4c30-9f21-22cdd9159d12/resourceGroups/brp4hwrr/providers/Microsoft.Compute/virtualMachines/brp4hwrr-9f7,
 name=brp4hwrr-9f7, location={scope=REGION, id=westeurope, description=West 
Europe, parent=azurecompute-arm, iso3166Codes=[NL]}, group=brp4hwrr, 
imageId=westeurope/MicrosoftWindowsServer/WindowsServer/2016-Datacenter, 
os={family=windows, version=2016-Datacenter, description=2016-Datacenter, 
is64Bit=true}, status=RUNNING[ProvisioningState/succeeded,PowerState/running], 
loginPort=5985, hostname=brp4hwrr-9f7, privateAddresses=[10.0.0.4], 
publicAddresses=[168.63.120.198], hardware={id=westeurope/Standard_B1s, 
providerId=Standard_B1s, name=Standard_B1s, location={scope=REGION, 
id=westeurope, description=West Europe, parent=azurecompute-arm, 
iso3166Codes=[NL]}, processors=[{cores=1.0, speed=2.0}], ram=1024, 
volumes=[{type=LOCAL, siz
 e=2048.0, bootDevice=false, durable=false}, {type=LOCAL, size=1047552.0, 
bootDevice=false, durable=false}], supportsImage=Predicates.alwaysTrue(), 
userMetadata={maxDataDiskCount=2}}, loginUser=jclouds, 
userMetadata={Name=brp4hwrr-idg4, brooklyn-user=andrea, 
jclouds_group=brp4hwrr}} are reachable in mode PREFER_PUBLIC
    ```
    instead of the IllegalStateException 
    ```
    jclouds did not return any IP addresses matching " + getNetworkMode() + " 
in " + location
    ```

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/andreaturli/brooklyn-server 
improvements/win-arm-messages

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/brooklyn-server/pull/949.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #949
    
----
commit e5b70df0185f685d5541dc7303742c85849cecd1
Author: andreaturli <andrea.turli@...>
Date:   2018-02-21T11:10:32Z

    Improve JcloudsWinrmingLiveTest coverage
    
    - add Azure ARM
    - improve error message for DefaultConnectivityResolver

----


---

Reply via email to