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

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            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.
            Hide
            Permalink
            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
            Permalink
            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

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development