Details

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

      Description

      The idea is to provide a 'local' storage driver. This will provide developers with the ability to implement storage features with libcloud such that it can be deployed on a cloud platform or over local file-systems (for eg: over a local disk or a n/w mounted storage). This will allow for hybrid deployments where the storage layer can be altered via config changes.

      In addition, this also let's users play around with the library without the need to access a cloud system

      A sample implementation could

      • use the 'key' argument as the base file-system where the objects will be stored
      • use a file-system hierarchy to represent containers and objects
      1. libcloud-252.patch
        41 kB
        Mahendra M
      2. libcloud-252.patch
        32 kB
        Mahendra M

        Activity

        Hide
        mahendra.m Mahendra M added a comment -

        A sample implementation for this feature. Uses the key as the file-system path where the data must be stored

        Show
        mahendra.m Mahendra M added a comment - A sample implementation for this feature. Uses the key as the file-system path where the data must be stored
        Hide
        kami Tomaz Muraus added a comment - - edited

        Thanks, this patch should definitely decrease the barrier for people who just want to play with the library

        I'll review it today and post comments in-line on Github pull request.

        Show
        kami Tomaz Muraus added a comment - - edited Thanks, this patch should definitely decrease the barrier for people who just want to play with the library I'll review it today and post comments in-line on Github pull request.
        Hide
        mahendra.m Mahendra M added a comment -

        Hi,

        Thanks for the detailed review. I am travelling now. Will be back on Monday morning. Will address your comments first thing

        Regards,
        Mahendra

        Sent from my iPhone

        Show
        mahendra.m Mahendra M added a comment - Hi, Thanks for the detailed review. I am travelling now. Will be back on Monday morning. Will address your comments first thing Regards, Mahendra Sent from my iPhone
        Hide
        kami Tomaz Muraus added a comment -

        No problem

        I just finished the review and besides some minor issues, patch looks great.

        Show
        kami Tomaz Muraus added a comment - No problem I just finished the review and besides some minor issues, patch looks great.
        Hide
        mahendra.m Mahendra M added a comment -

        Updated patch based on review comments

        Show
        mahendra.m Mahendra M added a comment - Updated patch based on review comments
        Hide
        mahendra.m Mahendra M added a comment -

        Updated as per review

        Show
        mahendra.m Mahendra M added a comment - Updated as per review
        Hide
        kami Tomaz Muraus added a comment -

        For the reference, review was conducted at https://github.com/apache/libcloud/pull/74

        Show
        kami Tomaz Muraus added a comment - For the reference, review was conducted at https://github.com/apache/libcloud/pull/74
        Hide
        kami Tomaz Muraus added a comment -

        Encountered some more Python version compatibility issues.

        I've fixed most of them but one - lockfile doesn't work with Python 3.2. I will look into fixing it and submitting a patch upstream.

        Show
        kami Tomaz Muraus added a comment - Encountered some more Python version compatibility issues. I've fixed most of them but one - lockfile doesn't work with Python 3.2. I will look into fixing it and submitting a patch upstream.
        Hide
        kami Tomaz Muraus added a comment -

        I've made some minor changes to the patch and committed it into trunk - http://svn.apache.org/viewvc?view=revision&revision=1406447, thanks!

        I also need to say that this was one of the best patches (quality and test wise) we have received recently so I hope this is not your last contribution

        Show
        kami Tomaz Muraus added a comment - I've made some minor changes to the patch and committed it into trunk - http://svn.apache.org/viewvc?view=revision&revision=1406447 , thanks! I also need to say that this was one of the best patches (quality and test wise) we have received recently so I hope this is not your last contribution

          People

          • Assignee:
            Unassigned
            Reporter:
            mahendra.m Mahendra M
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development