Uploaded image for project: 'Libcloud'
  1. Libcloud
  2. LIBCLOUD-363

ElasticHosts: Checking wrong HTTP Response status code

        Details

        • Type: Bug
        • Status: Resolved
        • Priority: Trivial
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: None
        • Component/s: None
        • Labels:
          None

          Description

          In the ElasticHosts create_node() method, we are checking for the wrong response status to determine drive imaging success/failure. It should check HTTP 200 instead of 204.
          Modify ElasticStack to check for both.

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            jira-bot ASF subversion and git services added a comment -

            Commit 73559d0b741e8f86dcb8c0075eac714c32466185 in branch refs/heads/trunk from Bob Thompson
            [ https://git-wip-us.apache.org/repos/asf?p=libcloud.git;h=73559d0 ]

            Issue LIBCLOUD-363: Fix ElasticHosts bug. Now checks for HTTP 204 and HTTP 200 to determine drive imaging status.

            Signed-off-by: Tomaz Muraus <tomaz@apache.org>

            Show
            jira-bot ASF subversion and git services added a comment - Commit 73559d0b741e8f86dcb8c0075eac714c32466185 in branch refs/heads/trunk from Bob Thompson [ https://git-wip-us.apache.org/repos/asf?p=libcloud.git;h=73559d0 ] Issue LIBCLOUD-363 : Fix ElasticHosts bug. Now checks for HTTP 204 and HTTP 200 to determine drive imaging status. Signed-off-by: Tomaz Muraus <tomaz@apache.org>
            Hide
            Permalink
            kami Tomaz Muraus added a comment -

            Sorry for the delay, I've totally missed out this issue.

            I've merged the patch into trunk now - thanks!

            Show
            kami Tomaz Muraus added a comment - Sorry for the delay, I've totally missed out this issue. I've merged the patch into trunk now - thanks!
            Hide
            Permalink
            bobt Bob Thompson added a comment -

            Thanks

            Show
            bobt Bob Thompson added a comment - Thanks

              People

              • Assignee:
                kami Tomaz Muraus
                Reporter:
                bobt Bob Thompson
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development