Whirr
  1. Whirr
  2. WHIRR-602

Cloud providers may only return "private" IPs

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.7.1, 0.8.0
    • Fix Version/s: 0.8.0
    • Component/s: core
    • Labels:
      None

      Description

      For example, CloudStack with Basic Networking only has one IP per instance, which is returned as the private address. In these sorts of situations, where only a "private" address exists in the NodeMetadata, we should fall back to that private address rather than giving up because there's no public address.

      1. WHIRR-602.patch
        2 kB
        Andrew Bayer

        Issue Links

          Activity

          Hide
          Andrei Savu added a comment -

          Committed to trunk. Thanks Andrew!

          Show
          Andrei Savu added a comment - Committed to trunk. Thanks Andrew!
          Hide
          Andrei Savu added a comment -

          +1 looks good to me. This is also addressing an issue I've seen some time ago while testing Whirr with CloudSigma.

          Show
          Andrei Savu added a comment - +1 looks good to me. This is also addressing an issue I've seen some time ago while testing Whirr with CloudSigma.
          Hide
          Andrew Bayer added a comment -

          I've been using this patch in my internal testing for jclouds 1.5 and other bugfixes against our CloudStack instance, and it hasn't caused any issues that I've seen with EC2 or Rackspace tests running against the same code.

          Show
          Andrew Bayer added a comment - I've been using this patch in my internal testing for jclouds 1.5 and other bugfixes against our CloudStack instance, and it hasn't caused any issues that I've seen with EC2 or Rackspace tests running against the same code.

            People

            • Assignee:
              Andrew Bayer
              Reporter:
              Andrew Bayer
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development