Libcloud
  1. Libcloud
  2. LIBCLOUD-18

New Libcloud Driver for DreamHost Private Servers

    Details

    • Type: New Feature New Feature
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.3.0
    • Fix Version/s: 0.3.0
    • Component/s: Core
    • Labels:
      None

      Description

      Driver to expose the DreamHost Private Server API through Libcloud.

      It currently implements all of the Libcloud interface except list_sizes and list_locations which don't have any meaningful equivalents with DreamHost's private servers. There are elements of the DreamHost API that are not currently exposed by this driver that may be exposed in the future.

      1. dreamhost_driver_squash.patch
        19 kB
        Kyle Marsh
      2. dreamhost_driver.patch
        91 kB
        Kyle Marsh

        Activity

        Hide
        Kyle Marsh added a comment -

        Patch containing new DreamHost driver and a minor change to the mockhttp code for compatibility with the DreamHost API

        Show
        Kyle Marsh added a comment - Patch containing new DreamHost driver and a minor change to the mockhttp code for compatibility with the DreamHost API
        Hide
        Jeremy Orem added a comment -

        The patch looks good, but a couple of issues need addressed:

        • ASF Granted License needs to be "Yes" on your file attachments
        • Convert tabs to 4 spaces
        • resize_node isn't part of the api right now, so it'd be best to remove it or make it private until a standardized extension format is agreed upon.
        • list_sizes: could hardcode some sizes. See the ec2 driver for an example.
        • create_node
          • should change kwargs['type'] to kwargs['image']
          • Maybe this could also accept a size and call the _resize_node method after creating
        • I wasn't able to locate dreamhost_ps-list_images in the docs. Does this method still exist?
        Show
        Jeremy Orem added a comment - The patch looks good, but a couple of issues need addressed: ASF Granted License needs to be "Yes" on your file attachments Convert tabs to 4 spaces resize_node isn't part of the api right now, so it'd be best to remove it or make it private until a standardized extension format is agreed upon. list_sizes: could hardcode some sizes. See the ec2 driver for an example. create_node should change kwargs ['type'] to kwargs ['image'] Maybe this could also accept a size and call the _resize_node method after creating I wasn't able to locate dreamhost_ps-list_images in the docs. Does this method still exist?
        Hide
        Kyle Marsh added a comment -

        Jeremy,

        Thanks for getting back to me so quickly; I'll make these changes next week and make sure to hit the ASF License button on the attachments...I remember seeing mention of it on the list but must have missed the button when attaching my files! As for the dreamhost_ps-list_images API command...I added it so that I could call it for the libcloud driver but I haven't updated the docs yet!

        I'll let you know when I get everything ready.

        Show
        Kyle Marsh added a comment - Jeremy, Thanks for getting back to me so quickly; I'll make these changes next week and make sure to hit the ASF License button on the attachments...I remember seeing mention of it on the list but must have missed the button when attaching my files! As for the dreamhost_ps-list_images API command...I added it so that I could call it for the libcloud driver but I haven't updated the docs yet! I'll let you know when I get everything ready.
        Hide
        Kyle Marsh added a comment -

        Jeremy,

        Thanks for getting back to me so quickly; I'll make these changes next week and make sure to hit the ASF License button on the attachments...I remember seeing mention of it on the list but must have missed the button when attaching my files! As for the dreamhost_ps-list_images API command...I added it so that I could call it for the libcloud driver but I haven't updated the docs yet!

        I'll let you know when I get everything ready.

        Show
        Kyle Marsh added a comment - Jeremy, Thanks for getting back to me so quickly; I'll make these changes next week and make sure to hit the ASF License button on the attachments...I remember seeing mention of it on the list but must have missed the button when attaching my files! As for the dreamhost_ps-list_images API command...I added it so that I could call it for the libcloud driver but I haven't updated the docs yet! I'll let you know when I get everything ready.
        Hide
        Kyle Marsh added a comment -

        Updated patches, same format as before.

        Show
        Kyle Marsh added a comment - Updated patches, same format as before.
        Hide
        Jeremy Orem added a comment -

        Before committing, I'd like to make "movedata" optional. Can we default to 'yes' or 'no' if it is not provided?

        Show
        Jeremy Orem added a comment - Before committing, I'd like to make "movedata" optional. Can we default to 'yes' or 'no' if it is not provided?
        Hide
        Kyle Marsh added a comment -

        Of course! The "movedata" argument should default to 'no' if it's not provided. Thanks!

        Show
        Kyle Marsh added a comment - Of course! The "movedata" argument should default to 'no' if it's not provided. Thanks!
        Hide
        Jeremy Orem added a comment -

        Committed in r941891.

        Show
        Jeremy Orem added a comment - Committed in r941891.

          People

          • Assignee:
            Jeremy Orem
            Reporter:
            Kyle Marsh
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

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

                Development