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 ]
        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 ]
        Andrei Savu made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]

          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