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

Implement FileDeployment, which can be used by all drivers, and is a general replacement for create_node() ex_files param

    Details

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

      Description

      Not all drivers support the ex_files param, but this class allows any driver to install files on the remote node in a consistent way.

        Activity

        Hide
        kami Tomaz Muraus added a comment -

        I've committed your patch into trunk with two changes:

        • Your patch didn't close a file and was leaking an open file description so I changed it to use with
        • Changed perms to correctly reflect file permissions in octal notation. st_mode included more then just the file permissions it also includes sticky bit, set-group-id and set-user-id*
        Show
        kami Tomaz Muraus added a comment - I've committed your patch into trunk with two changes: Your patch didn't close a file and was leaking an open file description so I changed it to use with Changed perms to correctly reflect file permissions in octal notation. st_mode included more then just the file permissions it also includes sticky bit, set-group-id and set-user-id* http://docs.python.org/library/stat.html#stat.S_IMODE
        Hide
        jaydoane Jay Doane added a comment -

        I've tried to make this patch python 3 compatible, but was unable to fully test it during a deployment since paramiko doesn't seem to work correctly with python 3

        Show
        jaydoane Jay Doane added a comment - I've tried to make this patch python 3 compatible, but was unable to fully test it during a deployment since paramiko doesn't seem to work correctly with python 3
        Hide
        jaydoane Jay Doane added a comment -

        The previous version of this patch I uploaded had a bug in it, and didn't actually send the contents of the source file, but just the path.

        Show
        jaydoane Jay Doane added a comment - The previous version of this patch I uploaded had a bug in it, and didn't actually send the contents of the source file, but just the path.

          People

          • Assignee:
            Unassigned
            Reporter:
            jaydoane Jay Doane
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development