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

Add EBS attributes into extra dictionary for _to_volume within EC2

    Details

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

      Description

      The current _to_volume method does not contain all of the EBS attributes, specifically the volume type (standard/io1) or the IOPS. This type of information is important and should be included. I'm going to update the method and store the additional data inside of the extra dict.

        Activity

        Hide
        jira-bot ASF subversion and git services added a comment -

        Commit 480b46543759411816988a4676eab8bdab27a12b in branch refs/heads/trunk from Chris DeRamus
        [ https://git-wip-us.apache.org/repos/asf?p=libcloud.git;h=480b465 ]

        Issue LIBCLOUD-473: Add more EBS attributes to the "extra" dictionary in the
        StorageVolume class.

        Also refactor code to use new "_get_resource_tags" method.

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

        Show
        jira-bot ASF subversion and git services added a comment - Commit 480b46543759411816988a4676eab8bdab27a12b in branch refs/heads/trunk from Chris DeRamus [ https://git-wip-us.apache.org/repos/asf?p=libcloud.git;h=480b465 ] Issue LIBCLOUD-473 : Add more EBS attributes to the "extra" dictionary in the StorageVolume class. Also refactor code to use new "_get_resource_tags" method. Signed-off-by: Tomaz Muraus <tomaz@apache.org>
        Hide
        kami Tomaz Muraus added a comment -

        Corresponding Github pull request: https://github.com/apache/libcloud/pull/210

        Show
        kami Tomaz Muraus added a comment - Corresponding Github pull request: https://github.com/apache/libcloud/pull/210
        Hide
        kami Tomaz Muraus added a comment -

        Patch merged into trunk. Thanks!

        Show
        kami Tomaz Muraus added a comment - Patch merged into trunk. Thanks!

          People

          • Assignee:
            Unassigned
            Reporter:
            cderamus Chris DeRamus
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 24h
              24h
              Remaining:
              Remaining Estimate - 24h
              24h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development