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

fix ssh for servers with PubkeyAuthentication only

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      On libcloud/compute/ssh.py we specify password OR public key to ssh to a cloud server. This results in deploy_node to fail to ssh to a server where it allows PubkeyAuthentication only (example HPCloud servers).

      However there's no need to specify only one of the two, both can be specified. Paramiko's authentication is attempted in the following order of priority:

      (from paramiko/paramiko/client.py)

      public key(s) passed,
      any key found through an SSH agent,
      any "id_rsa" or "id_dsa" key discoverable in ~/.ssh/,
      plain username/password auth, if a password was given

      I've tried the patch on several backends (including HPCloud) and it worked without issues

        Activity

        Hide
        kami Tomaz Muraus added a comment -

        First to clarify this a bit - before your change, deploy_node method would only fail if a driver supplied both arguments - "password" and a "key". This would happen because "password" argument had precedence over "key" argument.

        In any case, as noted in the pull request, I've addressed some issues and merged patch into trunk.

        Thanks!

        Show
        kami Tomaz Muraus added a comment - First to clarify this a bit - before your change, deploy_node method would only fail if a driver supplied both arguments - "password" and a "key". This would happen because "password" argument had precedence over "key" argument. In any case, as noted in the pull request, I've addressed some issues and merged patch into trunk. Thanks!
        Hide
        kami Tomaz Muraus added a comment -

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

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

          People

          • Assignee:
            Unassigned
            Reporter:
            mgogoulos Markos Gogoulos
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development