Whirr
  1. Whirr
  2. WHIRR-417

Allow users to choose their own jclouds modules with properties

    Details

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

      Description

      Currently, we add jclouds modules in a hard-coded way. This implies that the patches to jclouds, which ssh driver we use, etc. cannot be changed without a code release or patching Whirr. jclouds has a property: "jclouds.modules" which when specified overrides the wiring that we would use.

      I believe that except in the case of dry run, we need the following property so that we can stop hard-coding the list:

      jclouds.modules=org.jclouds.logging.slf4j.config.SLF4JLoggingModule,org.jclouds.enterprise.config.EnterpriseConfigurationModule,org.jclouds.sshj.config.SshjSshClientModule,org.apache.whirr.service.jclouds.BindLoginCredentialsPatchForEC2

      With this in place, people can feel free to use jsch or sshj, for example, and also remove or update patches we make to jclouds.

        Activity

        Adrian Cole (Inactive) created issue -
        Adrian Cole (Inactive) made changes -
        Field Original Value New Value
        Assignee Adrian Cole [ adrian@jclouds.org ]
        Adrian Cole (Inactive) made changes -
        Attachment WHIRR-417.patch [ 12501503 ]
        Adrian Cole (Inactive) made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Adrian Cole (Inactive) added a comment -

        This patch allows people to change to jsch if they like

        Show
        Adrian Cole (Inactive) added a comment - This patch allows people to change to jsch if they like
        Adrian Cole (Inactive) made changes -
        Status In Progress [ 3 ] Patch Available [ 10002 ]
        Affects Version/s 0.6.0 [ 12316468 ]
        Fix Version/s 0.7.0 [ 12317571 ]
        Hide
        Adrian Cole (Inactive) added a comment - - edited

        if you'd like to try this wrt changing modules, add the jclouds maven dependency org.jclouds.driver/jclouds-jsch and set the following:
        jclouds.modules=org.jclouds.logging.slf4j.config.SLF4JLoggingModule,org.jclouds.enterprise.config.EnterpriseConfigurationModule,org.jclouds.ssh.jsch.config.JschSshClientModule,org.apache.whirr.service.jclouds.BindLoginCredentialsPatchForEC2

        Show
        Adrian Cole (Inactive) added a comment - - edited if you'd like to try this wrt changing modules, add the jclouds maven dependency org.jclouds.driver/jclouds-jsch and set the following: jclouds.modules=org.jclouds.logging.slf4j.config.SLF4JLoggingModule,org.jclouds.enterprise.config.EnterpriseConfigurationModule,org.jclouds.ssh.jsch.config.JschSshClientModule,org.apache.whirr.service.jclouds.BindLoginCredentialsPatchForEC2
        Hide
        Adrian Cole (Inactive) added a comment -

        tested several times with cloudservers-uk (as I'm testing from brussels)

        Show
        Adrian Cole (Inactive) added a comment - tested several times with cloudservers-uk (as I'm testing from brussels)
        Hide
        Andrei Savu added a comment -

        +1 tested on aws-ec2 using the hbase integration tests (fixed a typo)

        Show
        Andrei Savu added a comment - +1 tested on aws-ec2 using the hbase integration tests (fixed a typo)
        Hide
        Andrei Savu added a comment -

        Commited. Thanks Adrian!

        Show
        Andrei Savu added a comment - Commited. Thanks Adrian!
        Andrei Savu made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        1h 17m 1 Adrian Cole (Inactive) 30/Oct/11 09:25
        In Progress In Progress Patch Available Patch Available
        27s 1 Adrian Cole (Inactive) 30/Oct/11 09:26
        Patch Available Patch Available Resolved Resolved
        8d 11h 20m 1 Andrei Savu 07/Nov/11 20:46

          People

          • Assignee:
            Adrian Cole (Inactive)
            Reporter:
            Adrian Cole (Inactive)
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

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

                Development