Apache Whirr (retired)
  1. Apache Whirr (retired)
  2. WHIRR-569

create a default set of keys for itests so that we can login into instances when itests fail

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: 0.9.0
    • Component/s: core
    • Labels:
      None

      Description

      as discussed in IRC right now when an itest fails we need to edit .properties rerun without temporary keys to be able to login into the boxes, which is less than ideal.

      As per Andrei's suggestion while we should't make the itests depend on a user's keys we could make Cluster.withTemporaryKeys() generate the keypair @ ~/.whirr if not present already (and reuse if present), thus keeping the process secure but allowing users to use the itest keys to log into instances.

        Issue Links

          Activity

          Hide
          Tom White added a comment -

          When I run itests I use

          mvn verify -Pintegration -DargLine="-Dwhirr.test.provider=aws-ec2 -Dwhirr.test.identity=$AWS_ACCESS_KEY_ID -Dwhirr.test.credential=$AWS_SECRET_ACCESS_KEY -Dconfig=.whirr-test.properties"
          

          and put the following in .whirr-test.properties:

          whirr.private-key-file=${sys:user.home}/.ssh/id_rsa
          whirr.public-key-file=${sys:user.home}/.ssh/id_rsa.pub
          

          That way I can log in to the machines if something fails. Andrei's suggestion sounds good too.

          Show
          Tom White added a comment - When I run itests I use mvn verify -Pintegration -DargLine="-Dwhirr.test.provider=aws-ec2 -Dwhirr.test.identity=$AWS_ACCESS_KEY_ID -Dwhirr.test.credential=$AWS_SECRET_ACCESS_KEY -Dconfig=.whirr-test.properties" and put the following in .whirr-test.properties: whirr.private-key-file=${sys:user.home}/.ssh/id_rsa whirr.public-key-file=${sys:user.home}/.ssh/id_rsa.pub That way I can log in to the machines if something fails. Andrei's suggestion sounds good too.

            People

            • Assignee:
              David Alves
              Reporter:
              David Alves
            • Votes:
              1 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:

                Development